Skip to content

fix: remove .mcp.json from repo#84

Merged
jcanizalez merged 1 commit intomainfrom
fix/remove-mcp-json
Mar 24, 2026
Merged

fix: remove .mcp.json from repo#84
jcanizalez merged 1 commit intomainfrom
fix/remove-mcp-json

Conversation

@jcanizalez
Copy link
Copy Markdown
Owner

Summary

  • Remove tracked .mcp.json (HTTP transport) that breaks MCP in worktrees
  • Add .mcp.json to .gitignore — MCP config lives in ~/.claude.json (stdio)

Test plan

  • MCP connects in main repo via ~/.claude.json stdio config
  • New worktrees no longer get a broken .mcp.json

The tracked .mcp.json used HTTP transport which fails in worktrees
where the Electron app isn't running. MCP config lives in
~/.claude.json (stdio) instead.
Copilot AI review requested due to automatic review settings March 24, 2026 22:17
@jcanizalez jcanizalez merged commit ab06393 into main Mar 24, 2026
3 checks passed
@jcanizalez jcanizalez deleted the fix/remove-mcp-json branch March 24, 2026 22:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes a repository-tracked MCP configuration file that can break MCP usage in git worktrees, and ensures future local MCP configs aren’t accidentally committed by ignoring them.

Changes:

  • Delete the tracked root .mcp.json (previously configured for HTTP transport).
  • Add .mcp.json to .gitignore to keep MCP configuration local to each developer environment.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
.mcp.json Removes the tracked MCP config that was causing worktree-related MCP breakage.
.gitignore Prevents re-adding .mcp.json to the repo in the future.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants