docs(router): link the agent skill from the Router section - #290
Merged
Conversation
Adds .claude-plugin/marketplace.json so the two published skills install with '/plugin marketplace add nevermined-io/docs' plus one '/plugin install' each, and auto-update thereafter — replacing a six-line sparse-checkout incantation. The marketplace is named 'nevermined', so users see and type '@nevermined' rather than the repository name. Entries use source './skills' with an explicit per-plugin skills array (the shape Anthropic's own amd-skills entry uses): 'source: "./"' also works but copies the entire 33 MB docs repo into the plugin cache for a 64 KB skill. Verified with 'claude plugin validate', then by adding the marketplace and installing both plugins locally. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…files (#2595) .github/copilot-instructions.md and AGENTS.md are single files with no per-skill segment, so until now they carried the payments half only — an assistant reading either could help you charge for an agent but not buy from one. Each now gains a Nevermined Router section: the buy loop, the two money traps (requestId is an idempotency key not a counter; targetUrl is a complete URL not a base), whole-cent rounding, and the full BCK.ROUTER.* guardrail table with 'never widen a Delegation to get past a refusal'. Condensed rather than complete — these files are always-on context, so every line competes with the payments content. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…ent ClawHub (#2595) The install page told Claude Code users to sparse-checkout git, and never mentioned ClawHub at all — despite a CI job publishing both skills there on every push to skills/**. The registry we pay to publish to was invisible to readers. - Claude Code tab now leads with the plugin; the file copy stays as the pin-a-version fallback. - New ClawHub tab. Notes that --dir is a global flag (it must precede the subcommand) and that the payments skill's slug is 'nevermined', not 'nevermined-payments'. - 'Supported Tools at a Glance' no longer says Router is unavailable on Copilot and Codex, and gains a ClawHub row. - CLAUDE.md records both distribution channels and why source must stay './skills'. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The Router docs never mentioned the skill. Grepping products/router/ for 'skill' returned exactly one hit — a note pointing at the *internal* demo skill — so the skill linked out to these pages and the pages never linked back. #2595's 'stays in sync with the docs section' criterion was one-way. - overview.mdx: a section with the plugin and ClawHub one-liners, plus which of the two skills to install for which direction. - quickstart.mdx: a tip up top, since the five steps here are exactly what the skill automates. - agent-autonomy.mdx: points at the public skill for readers who want their own agent to do the run, next to the existing internal-demo note. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
# Conflicts: # .claude-plugin/marketplace.json # .github/copilot-instructions.md # AGENTS.md # CLAUDE.md # development-guide/build-using-nvm-skill.mdx
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
aaitor
approved these changes
Aug 12, 2026
aaitor
left a comment
aaitor
left a comment
Member
There was a problem hiding this comment.
Clean, fully additive docs change that closes the discovery half of nvm-monorepo#2595 the right way: three Router pages now link out to the nevermined-router skill, each placed where the reader is about to do by hand exactly what the skill automates. Every command string, link, and cross-skill claim in the diff verifies against the merged marketplace manifest and the canonical skill page. No blocking issues — approving.
Verification I ran
- Read all three changed files in post-PR state (head
docs/router-docs-link-skill, HEAD7162123); confirmed the stacked parent #289 is MERGED (11:01 UTC) and the base correctly retargeted tomain. - Cross-checked every plugin / ClawHub command against the merged
.claude-plugin/marketplace.json(marketplacename: nevermined; pluginsnevermined-router+nevermined-payments) and against the canonicaldevelopment-guide/build-using-nvm-skill.mdx— all copies identical (/plugin marketplace add nevermined-io/docs,/plugin install nevermined-router@nevermined,npx clawhub install nevermined-router). - Resolved the internal link
/development-guide/build-using-nvm-skill(file exists on head, present indocs.json) and the external linkgithub.com/nevermined-io/docs/tree/main/skills/nevermined-router(dir exists onmain→ 200). - Validated the diff's two substantive claims against
skills/nevermined-router/SKILL.md: it is a six-step buy loop (L55) and carriesBCK.ROUTER.0001–0011(a superset of the codes on these pages) with a full Guardrails section — so "the same six steps," "the same error codes," and "the guardrails an autonomous buyer must respect" all hold. Sibling split (nevermined-router= spending /nevermined-payments= receiving) matches the manifest. - MDX tag balance (
<CodeGroup>,<Note>,<Tip>) checked on all three pages. Environment naming compliant — onlysandbox/liveappear, nostaging_*leak. Step counts match each page's own content (quickstart "five", agent-autonomy "six"). - Over-engineering / leanness pass (
/ponytail-review): lean already, ship. Net removable lines: 0. The install snippet repeats across the three pages by design — point-of-need placement so a reader never has to navigate away to act — and each copy links back to the canonical skill page as the single source of truth. That is correct docs practice, not debt. - CI: 2/2 checks passing.
Out of scope (already noted by the author): mintlify broken-links can't run site-wide because of a pre-existing bare <token> in generated api-errors/codes.mdx (source fix nvm-monorepo#2805) — unrelated to this PR; links verified directly instead.
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
2 participants
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.
Closes the discovery half of nvm-monorepo#2595. Part of epic #2268.
Why
The Router documentation never mentioned the skill. Grepping
products/router/for "skill" returned one hit — a note pointing at the internalnvm-router-demo, not the public one. So the skill linked out to these pages, and the pages never linked back: a reader landing on the Router section had no idea a machine-readable twin existed, and #2595's "stays in sync with the docs section" criterion held in one direction only.What changed
Three pages, three placements chosen by what the reader is doing at that point:
overview.mdx— a "Let your coding assistant drive it" section with the Claude Code and ClawHub one-liners, and a note on which of the two skills goes with which direction of payment.quickstart.mdx— a tip at the top. The five steps on that page are exactly what the skill automates, so it's the highest-value moment to offer it.agent-autonomy.mdx— for readers who'd rather have their agent do the run than drive it by hand, next to the existing Nevermined-staff note about the internal demo skill. Says plainly that the skill teaches the loop but does not lift the cap.Verification
/development-guide/build-using-nvm-skillis present indocs.jsonand the file exists.mintlify broken-linkscan't run — a bare<token>in the generateddevelopment-guide/api-errors/codes.mdxaborts the parse for the whole site before it checks anything. Pre-existing onmain, untouched here, source fix is nvm-monorepo#2805. Links verified directly instead, as above.