-
Notifications
You must be signed in to change notification settings - Fork 2
CLI: Add remote server docs #183
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Pull Request Overview
This PR adds comprehensive documentation for remote MCP servers, expanding the existing guides to cover both local containerized and remote server functionality. The documentation explains how ToolHive's transparent HTTP proxy system handles remote servers differently from local containers.
Key changes:
- Added remote server configuration and authentication documentation
- Expanded troubleshooting section with remote-specific scenarios
- Updated management operations to cover remote server behavior
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/toolhive/guides-cli/run-mcp-servers.mdx | Added comprehensive remote server documentation including authentication, OAuth flows, and configuration options |
| docs/toolhive/guides-cli/manage-mcp-servers.md | Updated server management operations to explain remote server behavior and authentication handling |
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.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
danbarr
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.
Hey @amirejaz this looks pretty good, I have just a few minor nits/fixes and questions.
Description
Added the support docs for remote mcp servers in run-mcp-servers.md and manage-mcp-servers.md
Related issues/PRs
Screenshots
Merge checklist
Content
sidebars.ts) updated for added, deleted, reordered, or renamed filesvercel.jsonfor moved, renamed, or deleted pages (i.e., if the URL slug changed)Reviews