Skip to content

fix(mcp): drop failing and unauthenticated servers - #214

Merged
shunkakinoki merged 1 commit into
mainfrom
fix/disable-broken-mcp
Oct 15, 2026
Merged

shunkakinoki merged 1 commit into
mainfrom
fix/disable-broken-mcp

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

Removes Codex (codex mcp-server no longer starts) and GitHub, OpenRouter, Paybox, PlanetScale, Sentry, Stripe (never authenticated) from .ruler/mcp.json. mcp-sync writes every key into enabledMcpjsonServers, which overrode the disabled list in dotfiles.

🤖 Generated with Claude Code


Summary by cubic

Removes broken and unauthenticated MCP servers from .ruler/mcp.json. Codex's mcp-server no longer starts, and GitHub, OpenRouter, Paybox, PlanetScale, Sentry, and Stripe never authenticated.

mcp-sync was writing every key into enabledMcpjsonServers, which overrode the disabled list, so these entries previously got re-enabled despite being marked as disabled.

Written for commit 4b2a8ed. Summary will update on new commits.

View guided diff Turn on auto-fix

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@shunkakinoki
shunkakinoki merged commit 1564f25 into main Oct 15, 2026
2 of 3 checks passed
@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cb9cc553-1c13-4084-8b4e-510159287e8a
📥 Commits

Reviewing files that changed from the base of the PR and between 7045ad6 and 4b2a8ed.

📒 Files selected for processing: 1
  • .ruler/mcp.json
+----------------------------------------------------------+
| The happy path is delighted. The sad path is litigating. |
+----------------------------------------------------------+
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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