-
Notifications
You must be signed in to change notification settings - Fork 870
docs: remove remaining mcp-remote command usage #7408
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: remove remaining mcp-remote command usage #7408
Conversation
WalkthroughDocumentation updates to MCP server integration examples: remote server configuration is simplified from explicit command/args format to a direct URL field, and a CLI setup instruction tab is removed from the integration guide. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Pre-merge checks✅ Passed checks (3 passed)
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 |
Dangerous URL checkNo absolute URLs to prisma.io/docs found. |
Redirect checkThis PR probably requires the following redirects to be added to static/_redirects:
|
🍈 Lychee Link Check Report
📊 Results Overview
Errors per inputErrors in 200-orm/050-overview/500-databases/840-cockroachdb.mdx
Errors in 200-orm/800-more/350-ai-tools/100-cursor.mdx
Errors in 800-guides/380-vercel-app-deployment.mdx
|
Deploying docs with
|
| Latest commit: |
523beb6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://34667cd1.docs-51g.pages.dev |
| Branch Preview URL: | https://dr-6657-approve-pr-and-creat.docs-51g.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
content/250-postgres/350-integrations/400-mcp-server.mdx (1)
23-25: Remove themcp-remotestartup command or clarify the preferred approach for remote MCP server configuration.Lines 23-25 show
npx -y mcp-remoteas the startup method, but this conflicts with the PR objective to remove remainingmcp-remoteusage. Additionally, the JSON configuration example immediately following (lines 48-56) doesn't referencemcp-remoteat all—it only specifies the URL. Ifmcp-remoteis being phased out, consider removing the startup command section entirely or replacing it with guidance on configuring the remote server directly via JSON in the MCP client, consistent with how the remote server is actually used.
🧹 Nitpick comments (1)
content/900-ai/index.mdx (1)
66-79: Consider removing the extra blank line.After removing the CLI TabItem, line 78 appears to be an unnecessary blank line before the closing
</TabbedContent>tag. Consider removing it for cleaner formatting.🔎 Proposed fix
</TabItem> - </TabbedContent>
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
content/250-postgres/350-integrations/400-mcp-server.mdxcontent/900-ai/index.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Check internal links
- GitHub Check: Cloudflare Pages
🔇 Additional comments (4)
content/250-postgres/350-integrations/400-mcp-server.mdx (4)
184-196: LGTM! Configuration simplified correctly.The update from command/args format to the URL field for the remote MCP server aligns with the PR objective and simplifies the configuration.
250-262: LGTM! Windsurf configuration updated correctly.The simplified URL format for the remote MCP server is consistent with the changes made in the Cursor section and aligns with the PR objective.
315-319: LGTM! Claude Code command updated appropriately.The update to use
--transport httpflag instead ofnpx mcp-remoteis correct and aligns with the PR objective to remove mcp-remote command usage.
378-388: The proposed fix is incorrect—MCPServerStdio does not support aurlparameter.The OpenAI Agents SDK's
MCPServerStdiois designed for stdio-based subprocess communication and usesparamswithcommandandargsonly. Theurlparameter exists in the MCP schema for HTTP/stream transports, not for stdio servers.If the goal is to remove
mcp-remoteusage from this example, the correct approach would need clarification—it may require usingMCPServerStreamableHttpfor remote servers instead, or a different connection method. The current code should remain as-is unless you can confirm a valid replacement for themcp-remotecommand within theMCPServerStdiocontext.Likely an incorrect or invalid review comment.
kristof-siket
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AmanVarshney01 thanks! 💯
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.