Repository navigation
Promote dev → main: verified user-facing commands, plugin 0.5.1 - #41
Merged
Merged
Conversation
The plugin README's install block offered three commands and none were valid syntax: `claude plugin install` resolves `plugin@marketplace` from a registered marketplace, so it rejects a local path, a `github:owner/repo?path=` URL, and an npm package name alike — and `@opentrace/claude-code-plugin` was never published, so that line was wrong twice over. Replaced with a single line verified from a clean HOME, plus the `otx login` route and `--plugin-dir` for local work. No "once we're listed" command this time: that is what produced the npm line. Sweeping the rest of the user-facing docs for the same class of defect found two more. All 13 endpoints in docs/mcp-installs.md carried a trailing slash, and none of those instructions pass a header, so they authenticate over OAuth — where the slash yields an RFC 8707 `resource` the authorization server does not recognise. That is the bug fixed in the plugin in 39b6838, still live in the docs that tell people to wire the server up by hand. And README's claim about the plugin's default endpoint still said `/mcp/v1/`, which that same commit made untrue. Left README's `--url` line alone: it describes normalizeMcpUrl, which really does still canonicalise to a trailing slash, and the key-auth examples where the slash is inert. Also grew the tools table from 12 rows to the 19 the server advertises, matching what plugin.json already claims. Plugin patch-bumped so installed copies do not keep the stale README. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
fix(docs): make every user-facing command one that actually runs
![opentrace[bot]](https://avatars.githubusercontent.com/u/160156268?s=60&v=4){kind=small}
💰 Agent economics50 agent sessions reported in the last 30 days, but none matched this PR's commits. A session matches on the author email it reports and a commit landing within ~2h of it — so check the agent's email matches the commit author, and that the work was committed in the same sitting. Sessions → Full economics → · by CodeHub |
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.
Strip trailing slash from all user-facing MCP URLs
📝 Docs · 🔧 Chore
Removes the trailing slash from every user-facing
https://api.opentrace.ai/mcp/v1/URL across docs and install examples, so they match themcp_urldefault already stored inplugin.json. Also rewrites the plugin README install section to reflect the real published distribution paths, and bumps the plugin to0.5.1.Complexity
🟢 Trivial ·
5 files changed, 43 insertions(+), 23 deletions(-)Docs and version-bump only — no functional code is touched. The runtime (
src/util/constants.ts) still normalises all URLs to the trailing-slash form internally; this PR only aligns what users see.Review focus
Pay particular attention to the following areas:
README.mdlines 192 and 198 (the CLI-key /mcp-remoteadvanced examples) still show/mcp/v1/with a trailing slash and were not updated; confirm this is intentional or flag as missed.