Fix Cursor install badge URL; trim host list in top-level README#33
Merged
Conversation
GitHub's markdown sanitizer strips the cursor:// scheme from rendered README links, so the badge merged in #32 fell through to the camo image URL when clicked instead of opening Cursor. The /install-mcp web URL (without the locale prefix) now returns 404; Cursor moved the endpoint to /en/install-mcp, which 307-redirects to /en-US/install-mcp and serves the install handoff page. Same fix github/github-mcp-server applied in their installation guide. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The README enumerated every editor/host with setup instructions in the sandbox README. Linking to the sandbox README is enough; the list adds noise and goes stale every time we add or drop a host. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
phoenixy1
changed the title
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
1 participant
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.
Two README touch-ups.
1. Cursor install badge URL (
sandbox/README.md); third time's the charm.Follow-up to #31 and #32. Both prior fixes left the badge non-functional:
https://cursor.com/install-mcp?..., which now returns 404.cursor://anysphere.cursor-deeplink/mcp/installdeeplink. GitHub's markdown sanitizer strips non-http(s) schemes from rendered link hrefs, so clicking the badge falls through to the camo image URL instead of opening Cursor.The fix is the locale-prefixed path Cursor migrated to:
https://cursor.com/en/install-mcp?.... It 307-redirects to/en-US/install-mcpand serves the install handoff page that triggers thecursor://deeplink. github/github-mcp-server applied the same fix in their Cursor installation guide (see issue #860).2. Trim host list from top-level README
The sandbox setup pointer used to enumerate every host. Dropped the list — the link is enough, and it goes stale whenever the host set changes.
Test plan
cursor.com/en-US/install-mcpand hands off to Cursorcommand=uvx,args=["mcp-server-plaid"], and the two env placeholdersADD_YOUR_*placeholders and verify the server starts🤖 Generated with Claude Code