Skip to content

Split UI Run MCP servers guide into two pages#982

Merged
danbarr merged 2 commits into
mainfrom
split-ui-run-mcp-servers-guide
Jun 26, 2026
Merged

Split UI Run MCP servers guide into two pages#982
danbarr merged 2 commits into
mainfrom
split-ui-run-mcp-servers-guide

Conversation

@danbarr

@danbarr danbarr commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Description

Splits the ToolHive UI "Run MCP servers" guide into a how-to and a reference, mirroring the CLI split in #981.

The original page crossed two axes (registry vs. custom, local vs. remote) through nested tabs, which duplicated every field description up to four times and repeated the entire remote-auth block verbatim. This restructures it:

  • run-mcp-servers.mdx is now a linear how-to with no tabs: install from the registry, install a custom local server (Docker image or source package), install a custom remote server, then manage servers and upgrade image versions. Each install flow ends at the configuration form.
  • configure-mcp-servers.mdx (new) is a per-setting reference organized into options for all servers, local servers, and remote servers, with the four authentication examples expanded into a real section (previously collapsed in a <details>).

Supporting changes:

  • Updated sidebars.ts to add the configure page under the Run MCP servers category.
  • Fixed inbound anchor links in connect-clients.mdx and the filesystem MCP guide that pointed at moved anchors.
  • Inlined and deleted the now-single-use _remote-mcp-auth-examples.mdx partial, and removed two orphaned icon SVGs no longer referenced after the tabs were dropped.

While restructuring, corrected three accuracy issues carried over from the original:

  • PKCE / client secret: dropped the claim that the client secret is "optional if PKCE is enabled." GitHub added PKCE support in July 2025 but still requires client_secret in the web application flow, so the GitHub examples now state the secret is required, and the generic field reference notes the behavior is provider-dependent.
  • Fabricated Brave example: the Bearer Token example used https://mcp.bravesearch.com/sse (NXDOMAIN; Brave runs no hosted remote MCP server). Replaced with the https://api.example.com/mcp placeholder convention used by the CLI guide.
  • Deprecated SSE: that example also advertised SSE transport; switched to Streamable HTTP.

Note for reviewers: the run page no longer has the old tab query strings (?server-type=, ?custom-type=) or the #volumes / #remote-mcp-examples anchors. No page slug changed, so no vercel.json redirects are needed, but any external deep links to those in-page anchors will land on the page top.

Type of change

  • Navigation/structure change
  • Documentation update
  • Bug fix (typo, broken link, etc.)

Related issues/PRs

Screenshots

N/A

Submitter checklist

Content and formatting

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Navigation

  • New pages include a frontmatter section with title and description at a minimum
  • Sidebar navigation (sidebars.ts) updated for added, deleted, reordered, or renamed files

Reviewer checklist

Content

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Extract the per-field configuration reference out of the UI Run MCP
servers guide into a new Configure MCP servers page, mirroring the
CLI split in PR 981.

The run page becomes a linear how-to (registry, custom local, custom
remote) with no tabs, ending each flow at the configuration form. The
new configure page documents every setting once, organized by options
for all servers, local servers, and remote servers. This removes the
2x2 tab matrix and the duplicated remote authentication block.

Update the sidebar and fix inbound anchor links in connect-clients and
the filesystem guide.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 26, 2026 15:55
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs-website Ready Ready Preview, Comment Jun 26, 2026 4:00pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 restructures the ToolHive UI “Run MCP servers” documentation by splitting the previous combined how-to/reference content into a linear how-to page plus a dedicated per-setting reference page, reducing duplication and improving navigability.

Changes:

  • Refactors Run MCP servers into a linear how-to flow and links out to a new configuration reference.
  • Adds a new Configure MCP servers reference page and wires it into the UI sidebar nav.
  • Updates inbound links to moved anchors, removes a now-unused partial, and deletes unused icon assets.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
static/img/mcp-servers/remote-mcp.svg Removes an unused icon asset.
static/img/mcp-servers/local-mcp.svg Removes an unused icon asset.
sidebars.ts Adds the new configure page under the “Run MCP servers” UI category.
docs/toolhive/guides-ui/run-mcp-servers.mdx Converts the page to a linear how-to and links to the new configuration reference.
docs/toolhive/guides-ui/configure-mcp-servers.mdx New reference page documenting configuration form settings and remote auth examples.
docs/toolhive/guides-mcp/filesystem.mdx Repoints the storage volumes anchor link to the new configure page.
docs/toolhive/guides-k8s/connect-clients.mdx Updates the UI deep link to the new “Custom remote servers” anchor.
docs/toolhive/_partials/_remote-mcp-auth-examples.mdx Removes an inlined, now-unused partial.

Comment thread docs/toolhive/guides-ui/configure-mcp-servers.mdx Outdated
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@danbarr danbarr merged commit 9eef6f4 into main Jun 26, 2026
4 checks passed
@danbarr danbarr deleted the split-ui-run-mcp-servers-guide branch June 26, 2026 17:16
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