Add MCP integrations for Claude Desktop, Cursor and Copilot#44
Add MCP integrations for Claude Desktop, Cursor and Copilot#44skools-here wants to merge 1 commit intoluarss:mainfrom
Conversation
…ndard configuration
📝 WalkthroughWalkthroughThree new MCP server configuration files are added for different AI coding assistants (Claude, GitHub Copilot, and Cursor), each defining identical settings that reference the "openroad-mcp" project via a GitHub repository URL with the uvx command. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.cursor/settings.json:
- Around line 6-8: Replace the floating git dependency
"git+https://github.com/luarss/openroad-mcp" in .cursor/settings.json with a
pinned ref (tag or commit SHA) to ensure reproducible installs, and add the
required environment variable keys OPENROAD_EXE and OPENROAD_FLOW into the same
settings file so the runtime can discover OpenROAD paths (mirror the vars used
in .mcp.json). Also update README.md's "Standard Configuration" example to show
the pinned dependency and document the OPENROAD_EXE/OPENROAD_FLOW environment
variable pattern and expected values.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: c70cce3f-86af-4b74-bce4-1e26d32b0141
📒 Files selected for processing (3)
.claude/settings.json.copilot/settings.json.cursor/settings.json
luarss
left a comment
There was a problem hiding this comment.
Have you tested on these clients? I would love to see some screenshots of them working
|
Or - even better if there is a way to test these client's compatibility with |
Adds MCP integration configs for:
Uses the standard configuration defined in the README.
Closes #8
Summary by CodeRabbit