docs: move facilitator route to /products/payments-facilitator (with redirects) - #388
Conversation
…edirects Completes the route rename deferred from #385. The product is now "Payments Facilitator" (x402 + MPP), so the URL follows the name. - Move products/x402-facilitator/{overview,how-it-works,mpp-seller}.mdx to products/payments-facilitator/ (git rename; content unchanged beyond links). - docs.json: repoint the nav group pages to the new paths; add Mintlify redirects from every old path so no inbound link 404s: /products/x402-facilitator -> .../payments-facilitator/overview /products/x402-facilitator/ -> .../payments-facilitator/overview /products/x402-facilitator/overview -> .../payments-facilitator/overview /products/x402-facilitator/how-it-works -> .../payments-facilitator/how-it-works /products/x402-facilitator/mpp-seller -> .../payments-facilitator/mpp-seller plus the new bare/trailing-slash root -> overview (repo convention). - Rewrite every internal /products/x402-facilitator/ link to the new path (nevermined-x402, catalog/overview, catalog/router/overview, api-providers, and cross-links inside the moved pages). - Drop the now-obsolete maintainer note that said not to rename the route. mintlify validate + broken-links pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PWktbVkhw2RC6LPiUVjFpH
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
There was a problem hiding this comment.
🔵 Needs a closer look
The new MPP seller page includes copy/paste curl examples with an invalid Authorization header (placeholder and missing Bearer), which will cause immediate authentication failures for readers.
Pull request overview
Moves the Payments Facilitator docs from the legacy /products/x402-facilitator/* route to /products/payments-facilitator/* and updates navigation/cross-links, while preserving link stability via Mintlify redirects.
Changes:
- Repoints
docs.jsonnavigation toproducts/payments-facilitator/*and adds redirects from all legacy/products/x402-facilitator/*paths (plus the new bare/trailing-slash root). - Updates internal links across solution/product/dev-guide pages to the new Payments Facilitator routes.
- Introduces the facilitator pages under
products/payments-facilitator/with internal cross-links updated.
File summaries
| File | Description |
|---|---|
| solutions/api-providers.mdx | Updates Payments Facilitator links to the new route. |
| products/payments-facilitator/overview.mdx | Updates cross-links/cards to point at the new facilitator subpages. |
| products/payments-facilitator/how-it-works.mdx | Updates the “Accepting MPP payments” cross-link to the new route. |
| products/payments-facilitator/mpp-seller.mdx | Adds/relocates the MPP seller guide under the new route (contains curl examples). |
| products/catalog/router/overview.mdx | Updates Payments Facilitator link in the Router comparison table. |
| products/catalog/overview.mdx | Updates Payments Facilitator cross-link from Catalog overview. |
| development-guide/nevermined-x402.mdx | Updates buyer-side note linking to the MPP seller page under the new route. |
| docs.json | Updates nav entries and adds redirects for old and new facilitator roots/paths. |
Review details
- Files reviewed: 7/8 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Re the Copilot review — this is a false positive, no change made: The This PR only moves mpp-seller.mdx to the new route; its content shipped unchanged in #385. Leaving the examples as-is. |
|
👀 Reviewing |
r-marques
left a comment
There was a problem hiding this comment.
🤖 Automated PR review — ✅ Ready to merge
Moves the facilitator route from products/x402-facilitator/* to products/payments-facilitator/*, so the URL matches the product name that #385 established. This is the one change in a docs repo that silently breaks every inbound link if done carelessly — and it is done carefully: all five old URL shapes redirect, every internal link is updated so nothing relies on the hop, and the now-obsolete "don't rename without redirects" note is removed rather than left to rot. Nothing to address.
What I reviewed
- Diff range
origin/main...HEAD— 8 files, +36/−18, headfde357f0. - Panel: none — no
pr-review-toolkitagents were spawned in this session, so this is my own verification only, not multi-agent coverage. - Redirect coverage, enumerated from the parsed
docs.jsonrather than the diff. Five sources cover every shape an existing link can take:Both bare forms and all three pages — the bare-path pair is the one usually forgotten, and it is the shape a human types./products/x402-facilitator -> /products/payments-facilitator/overview /products/x402-facilitator/ -> /products/payments-facilitator/overview /products/x402-facilitator/overview -> /products/payments-facilitator/overview /products/x402-facilitator/how-it-works -> /products/payments-facilitator/how-it-works /products/x402-facilitator/mpp-seller -> /products/payments-facilitator/mpp-seller - No internal link still points at the old path — grepped every
.mdxin the repo, not just the changed ones. So the redirects are a safety net for external inbound links, not load-bearing for the site's own navigation, which is the right way round. docs.jsonnav entries all moved (parsed the JSON: zero remainingproducts/x402-facilitator/*page entries) and the file is still a text edit, not ajson.load/dumpround-trip.- Every internal link in the changed files resolves to a real
.mdx, checked by script rather than by eye — including the updated anchor…/mpp-seller#protocol-isolation, whose## Protocol isolationheading is atmpp-seller.mdx:139. mintlify broken-linkswill not abort. git sees the renamed files as wholly new, so every line reads as added; of the three<…>placeholders that surfaced inmpp-seller.mdx, all three are inside code fences (lines 20, 26, 32) — verified with fence tracking, not just a backtick check.- The rename is faithful:
how-it-worksandmpp-sellerare line-for-line identical;overviewis −2, and those two lines are exactly the MDX note that should go (below). - CI: Mintlify Deployment pass, link-rot pass. (
vale-spellcheckreportsskipping.)
One thing worth recording — my review on #385 said the paths were deliberately left alone and that renaming them would break inbound links. #385 merged at 14:55 having added a note in overview.mdx saying precisely that. This PR then does the rename and satisfies the condition the note set, so removing it is correct: the instruction was followed, not overridden. That is the loop closing properly rather than a comment being deleted because it was in the way.
Verdict: ✅ Ready to merge — 0 blockers, 0 should-fix, 0 nits.
🧭 Inline comments (2)
Each is posted on its line in Files changed.
- ✅ NICE — Every old URL shape redirects, including the two bare forms —
docs.json:506 - ✅ NICE — The note that forbade this rename is removed because its condition was met —
products/payments-facilitator/overview.mdx:10
r-marques
left a comment
There was a problem hiding this comment.
✅ Approved — no blockers, nothing to address.
Inline review: #388 (review)
Why this matters
In #385 we renamed the product from "x402 Facilitator" to Payments Facilitator (it verifies and settles both x402 and MPP), but kept the old URL
/products/x402-facilitator/*to avoid breaking inbound links. That left the page's name and its address out of step — a reader lands on "Payments Facilitator" at anx402-facilitatorURL. This moves the pages to/products/payments-facilitator/*so the address matches the name, and adds redirects from every old path so nothing that already links to the old URLs — other docs, search results, external posts — 404s. No link breaks; the canonical URL just finally reads right.What changed
products/x402-facilitator/{overview,how-it-works,mpp-seller}.mdx→products/payments-facilitator/(git renames; page content unchanged apart from internal links).docs.json:Nav group pages repointed to the new paths.
Redirects from every old path (Mintlify
source→destination):/products/x402-facilitator/products/payments-facilitator/overview/products/x402-facilitator//products/payments-facilitator/overview/products/x402-facilitator/overview/products/payments-facilitator/overview/products/x402-facilitator/how-it-works/products/payments-facilitator/how-it-works/products/x402-facilitator/mpp-seller/products/payments-facilitator/mpp-sellerPlus the new bare/trailing-slash root →
overview, matching the convention used for Router, NVM Pay, etc./products/x402-facilitator/link to the new path:development-guide/nevermined-x402.mdx,products/catalog/overview.mdx,products/catalog/router/overview.mdx,solutions/api-providers.mdx, and the cross-links inside the moved pages.grep "/products/x402-facilitator/"is now empty across.mdx.This is the clean follow-up r-marques and I flagged on #385 ("renaming them would break every inbound link … unless redirects are added"). Redirects are added, so the concern is resolved rather than traded.
Test plan
mintlify validate(strict) passes — only the pre-existingreactimport warning in/snippets/hero-banner.jsx, untouched.mintlify broken-linksclean; no residual/products/x402-facilitator/link targets in.mdx.docs.jsonvalid JSON.main(includes docs: document MPP as the plan-model sibling of x402 #385)./products/payments-facilitator/*, and an old/products/x402-facilitator/overviewURL redirects.🤖 Generated with Claude Code
https://claude.ai/code/session_01PWktbVkhw2RC6LPiUVjFpH