[Bug Fix] MCP: point install instructions at canonical www host - #492
Merged
Conversation
rubyui.com/mcp 301-redirects to www.rubyui.com/mcp (see #481). Every place we hand out the MCP endpoint — the docs page's copy-pasteable snippets, the get_project_registries tool response, and the README — still pointed at the bare domain, so copying the "installation" command sent clients through a mid-handshake redirect that trips some MCP implementations. Point all of them at the final destination directly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to #481 (second sub-problem, not the Host-header rejection itself — that's PR #491).
Problem
https://rubyui.com/mcp301-redirects tohttps://www.rubyui.com/mcp. A redirect mid-handshake trips some MCP clients. Every place we hand people (or agents) the endpoint to copy-paste pointed at the bare, redirecting domain:docs/app/views/docs/mcp.rb— the Claude Codeclaude mcp addsnippet, and the Cursor/Claude Desktop/Windsurf/Zed JSON config snippets.mcp/lib/ruby_ui/mcp/tools/get_project_registries.rb— theurltheget_project_registriesMCP tool itself hands back to a connected agent at runtime.mcp/README.md— the "Hosted endpoint" line.Fix
Point all of the above at
https://www.rubyui.com/mcpdirectly — the actual final destination — so copying any of these no longer round-trips through a redirect. Left alone the general "visit the site" links (e.g.mcp/README.md's intro link and its "see docs" link, rootCLAUDE.md/README.md), since those are normal browser navigations where a 301 is harmless.Test plan
cd mcp && bundle exec rake test— 24 runs, 59 assertions, 0 failures (no test asserts the literal URL string, so nothing needed updating there).bundle exec standardrbon the changed Ruby files — clean.mcp/data/registry.json(the static build artifact) doesn't embed this URL, so no rebuild needed.rubyui.com/mcp(bare, non-www) references — none left.Summary by cubic
Point MCP install and config instructions to the canonical host to avoid mid-handshake redirects that break some clients. All endpoints now use https://www.rubyui.com/mcp.
get_project_registriestool to return https://www.rubyui.com/mcp.mcp/README.mdhosted endpoint.Written for commit 69db642. Summary will update on new commits.