fix(mcp): use installed server commands - #99
Conversation
- replace npx/uvx launchers with direct command names - use the xcodebuildmcp mcp subcommand required by the current CLI
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
Updates MCP server launcher configuration to use locally installed CLI commands instead of npx/uvx, reducing session startup overhead and aligning with local tool installs.
Changes:
- Switch
.ruler/mcp.jsonserver commands fromnpx/uvxwrappers to direct installed executables. - Update XcodeBuildMCP to start via
xcodebuildmcp mcp. - Adjust arguments for ChromeDevtools / Context7 to match installed command invocation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "command": "mcp-remote", | ||
| "args": ["https://api.githubcopilot.com/mcp/"] |
There was a problem hiding this comment.
Changing from npx/uvx to installed command names makes startup faster, but also shifts responsibility to the environment: these executables must exist on PATH (and be the correct versions) for the config to work. To avoid breakage/repro issues across machines/CI, consider adding a repo-managed installation/pinning mechanism (e.g., a setup script/Make target, a tool version file like mise.toml/.tool-versions, or explicit dev docs in-repo) that ensures these commands are installed at known versions.
| "command": "mcp-remote", | |
| "args": ["https://api.githubcopilot.com/mcp/"] | |
| "command": "npx", | |
| "args": ["mcp-remote", "https://api.githubcopilot.com/mcp/"] |
| "command": "context7-mcp", | ||
| "args": [] |
There was a problem hiding this comment.
Changing from npx/uvx to installed command names makes startup faster, but also shifts responsibility to the environment: these executables must exist on PATH (and be the correct versions) for the config to work. To avoid breakage/repro issues across machines/CI, consider adding a repo-managed installation/pinning mechanism (e.g., a setup script/Make target, a tool version file like mise.toml/.tool-versions, or explicit dev docs in-repo) that ensures these commands are installed at known versions.
| "command": "context7-mcp", | |
| "args": [] | |
| "command": "uvx", | |
| "args": ["context7-mcp"] |
| "command": "context7-mcp", | ||
| "args": [] |
There was a problem hiding this comment.
Using an explicit empty args array is potentially unnecessary noise and can be less compatible if the consumer expects args to be omitted when empty. If the MCP config loader treats missing/empty identically, consider removing \"args\": [] here for consistency/clarity (or, alternatively, keep empty args arrays consistently for all servers).
| "command": "context7-mcp", | |
| "args": [] | |
| "command": "context7-mcp" |
Switch the MCP server entries in .ruler/mcp.json from npx / uvx launchers to the installed command names instead.
The previous setup re-resolved packages on session startup and kept Serena tied to a git-backed uvx --from invocation. Using the installed commands removes that startup overhead and keeps the config aligned with the local tool installs.
This also updates the Xcode MCP entry to call xcodebuildmcp mcp, which is what the current CLI expects when starting the server.
No related GitHub issue was found.
Summary by cubic
Switch MCP server launchers in
.ruler/mcp.jsonfromnpx/uvxto installed command names to speed up startup and match local installs. Also update Xcode to usexcodebuildmcpwith themcpsubcommand.xcodebuildmcp,serena,mcp-remote,codex,chrome-devtools-mcp,context7-mcp.Written for commit a7a6b3c. Summary will update on new commits.