docs: normalize stuttered URLs and cut the op-alloy glossary stub - #22139
Queued
claude[bot] wants to merge 1 commit into
Queued
docs: normalize stuttered URLs and cut the op-alloy glossary stub#22139claude[bot] wants to merge 1 commit into
claude[bot] wants to merge 1 commit into
Conversation
…dup D13) U1: rename chain-operators/tutorials/create-l2-rollup/create-l2-rollup.mdx to index.mdx so the tutorial hub renders at /chain-operators/tutorials/create-l2-rollup; same-PR redirect from the old stuttered URL, 5 existing redirect entries retargeted, 13 inbound content links repointed. U2: rename node-operators/reference/architecture/architecture.mdx to index.mdx (same stutter); same-PR redirect, 1 existing redirect entry retargeted, 3 inbound links plus the op-geth-eol snippet consumer list updated. U3: cut the 6-line op-alloy glossary stub; redirect it and the legacy /rust/op-alloy/glossary entry to the canonical /op-stack/reference/glossary; nav entry removed. 3 new redirects (632 total), no chains, nav and redirect lints clean. Co-Authored-By: Claude
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
sbvegan
approved these changes
Jul 31, 2026
sbvegan
added this pull request to the merge queue
Jul 31, 2026
Any commits made after this event will not be merged.
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.
Requested by Matthew Cruz · Slack thread
Dedup slice D13 of docs-improver iteration 2 (sweep addendum clusters U1-U3, plan in
projects/docs-improver/plans/iteration-2/deduplication.mdon ethereum-optimism/solutions main). Tracking issue: https://github.com/ethereum-optimism/solutions/issues/1169Before / After
Before: two pages rendered at stuttered URLs whose last segment repeats the folder name: the create-l2-rollup tutorial hub at
/chain-operators/tutorials/create-l2-rollup/create-l2-rollupand the node architecture overview at/node-operators/reference/architecture/architecture. The op-alloy section also carried a 6-line glossary page at/app-developers/tools-sdks/op-alloy/glossarythat contained zero definitions.After: the tutorial hub renders at
/chain-operators/tutorials/create-l2-rollupand the architecture page at/node-operators/reference/architecture; the doubled URLs are gone. The empty op-alloy glossary is deleted and its URL lands on the full OP Stack glossary at/op-stack/reference/glossary. Every old link keeps working: the three old URLs redirect, and the seven existing redirect entries that pointed at them are retargeted so nothing chains.How
chain-operators/tutorials/create-l2-rollup/create-l2-rollup.mdxrenamed toindex.mdx(pure git rename, 100% similarity, no content change); nav entry updated; 13 inbound links across 10 files repointed, including the non-renderedcreate-l2-rollup-example/README.mdthat documents the tutorial URL.node-operators/reference/architecture/architecture.mdxrenamed toindex.mdx(pure git rename; content carries the earlier T1/D5 rewrites verbatim); nav entry updated inside the existing Architecture group (that group's restructure belongs to IA slice 3); 3 inbound links plus theop-geth-eolsnippet's consumer-list comment updated.app-developers/tools-sdks/op-alloy/glossary.mdxdeleted; nav entry removed from the op-alloy group; URL redirected to the canonical glossary (zero inbound content links existed).Redirect changes (3 new, 7 retargeted; 632 total, was 629):
/chain-operators/tutorials/create-l2-rollup/create-l2-rollup/chain-operators/tutorials/create-l2-rollup/node-operators/reference/architecture/architecture/node-operators/reference/architecture/app-developers/tools-sdks/op-alloy/glossary/op-stack/reference/glossary/operators/chain-operators/deploy/chain-operators/tutorials/create-l2-rollup/operators/chain-operators/deploy/overview/chain-operators/tutorials/create-l2-rollup/operators/chain-operators/tutorials/chain-operators/tutorials/create-l2-rollup/operators/chain-operators/tutorials/create-l2-rollup/chain-operators/tutorials/create-l2-rollup/chain-operators/guides/deployment/overview/chain-operators/tutorials/create-l2-rollup/operators/node-operators/architecture/node-operators/reference/architecture/architecture, now/node-operators/reference/architecture/rust/op-alloy/glossary/app-developers/tools-sdks/op-alloy/glossary, now/op-stack/reference/glossarydocs.json slot: this PR is the current holder of the serialized docs.json/redirects queue (follows D9 #22060, merged into base). No other docs-improver PR touches docs.json until this merges. It lands before IA slices 3 and 6 so these URLs are final before those nav re-parents (no page moves twice).
Road not taken (U3)
The plan default is implemented here: cut the empty glossary and redirect to the canonical one. The plan's flagged alternative is to fill
op-alloy/glossary.mdxfrom the upstream op-alloy book instead, keeping the op-alloy import self-contained; that path would cost zero redirects and would mean reverting the deletion, restoring the nav entry, dropping redirect row 3, and restoring/rust/op-alloy/glossary's old destination. Note the cut also closes IA slice 8's fill-or-delist hygiene item for the op-alloy glossary (D13 is the first lander per the merged program order); choosing the fill alternative hands that item back to IA slice 8.Verification (independent adversarial pass)
921ef195; the architecture page content is byte-identical to develop's, so the T1/D5 rewrites carry over verbatim.sourcelines in docs.json remain./op-stack/reference/glossaryverified as a live page on this branch (the file never moved; IA-2 moved only its nav entry), not itself a redirect source.index.mdxline 37); it is pre-existing list style shared by all sibling bullets in that section, and only the URL changed on that line. No other flagged patterns.Reviewer checklist (for Matthew)
/chain-operators/tutorials/create-l2-rolluprenders the tutorial hub index page, and the old/chain-operators/tutorials/create-l2-rollup/create-l2-rollupURL redirects to it./node-operators/reference/architecturerenders the architecture page, and the old/node-operators/reference/architecture/architectureURL redirects to it./app-developers/tools-sdks/op-alloy/glossaryredirects and lands on the full glossary at/op-stack/reference/glossary; the op-alloy nav group no longer lists Glossary.Never merge without human review; this PR is ready for review, not auto-merge.
🤖 Generated with Claude Code
https://claude.ai/code/session_01VMdppxjadjFC8zYhS6fjCD