Skip to content

Add check that _toc.json entries resolve to real pages - #5779

Open
abbycross wants to merge 2 commits into
mainfrom
ajc/toc-check
Open

abbycross wants to merge 2 commits into
mainfrom
ajc/toc-check

Conversation

@abbycross

Copy link
Copy Markdown
Collaborator

Adds a check:toc command (and a --toc flag on check:internal-links) that validates every url in each _toc.json points to an existing page, so stale nav entries can't 404 in production. Shares resolution logic with the existing internal-link checker and the orphan-pages check via a new scripts/js/lib/tocFiles.ts module.

Adds a check:toc command (and a --toc flag on check:internal-links) that
validates every url in each _toc.json points to an existing page, so stale
nav entries can't 404 in production. Shares resolution logic with the
existing internal-link checker and the orphan-pages check via a new
scripts/js/lib/tocFiles.ts module.

Also removes two pre-existing broken TOC entries this check surfaced:
post-selection (page deleted + redirected) and samplomatic/release-notes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@abbycross
abbycross requested a review from a team October 9, 2026 14:26

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@abbycross
abbycross added this pull request to the merge queue Oct 15, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 15, 2026
@abbycross
abbycross added this pull request to the merge queue Oct 20, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 20, 2026
@abbycross abbycross closed this Oct 20, 2026
@abbycross abbycross reopened this Oct 20, 2026
@github-project-automation github-project-automation Bot moved this from Done to Todo in Docs Planning Oct 20, 2026
@abbycross
abbycross enabled auto-merge October 20, 2026 20:17

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants