Skip to content

Conversation

peppescg
Copy link

@peppescg peppescg commented Sep 10, 2025

Description

Create section for remote MCP in the UI (currently visible under feature flag but not released yet )

Related issues/PRs

Screenshots

Screenshot 2025-09-10 at 16 54 23

Merge checklist

Content

  • (N/A) New pages include a frontmatter section with title and description at a minimum
  • (N/A) Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files
  • (N/A) Redirects added to vercel.json for moved, renamed, or deleted pages (i.e., if the URL slug changed)

Reviews

  • Content has been reviewed for technical accuracy
  • Content has been reviewed for spelling, grammar, and style

@peppescg peppescg self-assigned this Sep 10, 2025
@Copilot Copilot AI review requested due to automatic review settings September 10, 2025 14:58
Copy link

vercel bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-website Ready Ready Preview Comment Sep 19, 2025 3:41pm

Copy link
Contributor

@Copilot Copilot AI left a 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 documentation for remote MCP server functionality to the ToolHive UI guide. The documentation explains how users can connect to remotely hosted MCP servers by providing a URL, in addition to the existing local custom server options.

Key changes:

  • Documents remote MCP server configuration including authentication options (OAuth2/OIDC)
  • Reorganizes the custom server section to distinguish between remote and local options
  • Updates navigation and terminology to reflect the new remote server capability

amirejaz
amirejaz previously approved these changes Sep 11, 2025
Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@peppescg Thanks for providing the examples and talking through the flow, I know this one has been a slog. I have some clarity/flow suggestions; can you make sure I didn't change any technical accuracy?

With this, I think we'd be good to merge once this comes out of feature flag.

@peppescg
Copy link
Author

peppescg commented Sep 19, 2025

@danbarr

Replace registry snapshot
Screenshot 2025-09-19 at 13 11 34

Wrap local/remote mcp for both registry and custom in tabs
To be consistent I was thinking to have tabs not only for registry section but also for custom one

Kapture.2025-09-19.at.12.20.26.mp4

@peppescg peppescg requested a review from danbarr September 19, 2025 12:49
Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nits and just renaming run-mcp-servers.md to .mdx (you'll need to find/replace guides-ui/run-mcp-servers.md in other pages too)

Thanks again for all the effort on this 💪

Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, let's plan to merge once the feature flag is lifted! 🚀

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.

3 participants