Skip to content

test: isolate HOME in context-mode-detector tests#57

Merged
Shahinyanm merged 1 commit into
masterfrom
fix/context-mode-test-isolation
Jun 24, 2026
Merged

test: isolate HOME in context-mode-detector tests#57
Shahinyanm merged 1 commit into
masterfrom
fix/context-mode-test-isolation

Conversation

@Shahinyanm

Copy link
Copy Markdown
Member

detectContextMode falls back to ~/.mcp.json; on a dev machine with a real one the 3 'no detection' cases failed locally (clean in CI). beforeEach now isolates HOME to a clean temp home + restores. Test-only, no shipped change. Full suite 1416/1416 locally.

🤖 Generated with Claude Code

detectContextMode falls back to ~/.mcp.json (HOME/USERPROFILE). On a dev
machine with a real ~/.mcp.json the three "no detection" cases reported
detected=true and failed locally (clean in CI, which has no ~/.mcp.json).
beforeEach now points HOME at a clean temp home with no .mcp.json and restores
it after, so the suite is deterministic everywhere. Test-only; no shipped change.

Full suite now 1416/1416 locally.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Shahinyanm
Shahinyanm merged commit 8cc295b into master Jun 24, 2026
3 checks passed
@Shahinyanm
Shahinyanm deleted the fix/context-mode-test-isolation branch June 24, 2026 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant