Skip to content

fix(polygon-zkevm): pin endpoint slug so overview links resolve - #1387

Merged
dslovinsky merged 2 commits into
mainfrom
ds/fix-polygon-zkevm-endpoint-slug
Jun 15, 2026
Merged

fix(polygon-zkevm): pin endpoint slug so overview links resolve#1387
dslovinsky merged 2 commits into
mainfrom
ds/fix-polygon-zkevm-endpoint-slug

Conversation

@dslovinsky

@dslovinsky dslovinsky commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Problem

The weekly link check flagged 53 broken polygon-zkevm API endpoint links (all 404s). Slack thread
Root cause: the URL segment for the API section is derived two different ways that disagree on zkEVM:

  • Route / content-indexer (src/content-indexer/visitors/process-openrpc.ts, what's actually live): kebabCase("Polygon zkEVM API Endpoints")polygon-zk-evm-api-endpoints
  • Overview/FAQ link convention (and the Daikon overview-table generator): polygon-zkevm-api-endpoints

The links were correct until the Daikon spec update (#1377) regenerated the overview table and rewrote all 53 links from polygon-zk-evm-api-endpoints to polygon-zkevm-api-endpoints, which no longer matched the live route → 404s. Only zkEVM regressed; the other inner-cap chains' overview links already match their routes.

Fix (pin the slug — fix URL generation)

Rather than re-editing links (which Daikon would re-break), pin the section slug so the route matches the link convention and stays stable across future spec regenerations:

  • content/docs.yml — add slug: polygon-zkevm-api-endpoints to the Polygon zkEVM API Endpoints entry. The schema (docs-yml.schema.json) supports this; the indexer honors apiConfig.slug ?? kebabCase(...).
  • content/redirects.yml — the previously-live canonical was polygon-zk-evm-api-endpoints, so:
    • add /docs/chains/polygon-zkevm/polygon-zk-evm-api-endpoints/:method*…/polygon-zkevm-api-endpoints/:method* to preserve external/indexed links;
    • repoint the legacy /node/ redirect (which sent the simple form to the split form — now backwards) straight to the new /chains/ canonical, avoiding a self-loop/multi-hop chain.
  • FAQ (polygon-zkevm-and-ethereum-differences.mdx) — update its 16 internal links to the canonical form so they resolve directly instead of via a redirect.

Verification

  • docs.yml validates against docs-yml.schema.json.
  • No remaining polygon-zk-evm-api-endpoints references in content/ except the intentional redirect source.
  • Overview links were already in the canonical form, so they now match the route.

Note: pnpm run validate:docs-yml currently fails locally due to a pre-existing ajv import bug, unrelated to this change.

🤖 Generated with Claude Code

The content-indexer derives the API section slug via kebabCase("Polygon
zkEVM API Endpoints"), which splits zkEVM into "zk-evm" and produces
polygon-zk-evm-api-endpoints. The overview/FAQ link convention uses
polygon-zkevm-api-endpoints, so all 53 overview links 404'd after the
Daikon spec update (#1377) rewrote them to the simple form.

Pin slug: polygon-zkevm-api-endpoints on the api entry so endpoint URLs
match the link convention and stay stable across future spec regens.
Add redirects from the previously-live split form (and legacy /node
path) to the pinned slug, and update FAQ links to the canonical form.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown

🔗 Preview Mode

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Jun 15, 2026, 9:29 PM

The polygon-zkevm-api-overview.mdx file existed but was never referenced
in docs.yml, so it wasn't part of the sidebar navigation. Add the page
entry after FAQ and before the API Endpoints, matching the ordering used
by Polygon PoS and the other chain sections.

Co-Authored-By: Claude <noreply@anthropic.com>
@dslovinsky
dslovinsky marked this pull request as ready for review June 15, 2026 21:46
@dslovinsky
dslovinsky requested a review from a team as a code owner June 15, 2026 21:46
@dslovinsky
dslovinsky merged commit 4af493c into main Jun 15, 2026
10 checks passed
@dslovinsky
dslovinsky deleted the ds/fix-polygon-zkevm-endpoint-slug branch June 15, 2026 21:48
dslovinsky pushed a commit that referenced this pull request Jun 15, 2026
…block) (#1385)

https://bitsofco.de/solidity-function-visibility-explained/ was flagged by
the weekly link check with 'Network error: Connection failed', but the page
loads fine from this session (3 consecutive curl 200s in ~100ms with both
default and lychee user-agents). Treat as intermittent crawler-block,
matching the existing geth.ethereum.org / portal.zksync.io pattern.

Originally part of DOCS-118 alongside polygon-zkevm slug + katana
quickstart fixes; those have been handled separately in #1386 and #1387, so
this PR is now scoped to the one remaining lychee config change.

Refs DOCS-118
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants