fix: add the 8 Reference pages missing from the website sidebar - #429
fix: add the 8 Reference pages missing from the website sidebar#429d-morrison wants to merge 3 commits into
Conversation
website/_quarto.yml's Reference section omitted 8 existing reference pages, so they were reachable only by direct URL or via workflows.qmd's catalog links. #273 originally listed 4 (check-equation-renders, lint-yaml, lint-markdown, request-dependabot-review); re-deriving the set against current main found 8, as more capabilities were added since the issue was filed without sidebar entries: also check-new-line-breaks, lint-changed-lines, lint-qmd, and altdoc-multiversion-docs. Each is inserted at its canonical position, so the sidebar order stays byte-identical to workflows.qmd's catalog order (verified). No existing entries were reordered. Verified: all reference/*.qmd are now listed, every sidebar entry resolves to a real file, and YAML parses. Closes #273 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Matches the repo precedent (document-update-snapshots.added.md) of recording website reference/sidebar doc changes in changelog.d/. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Built to branch |
|
Claude finished review — View run Code Review: #429PR: "fix: add the 8 Reference pages missing from the website sidebar" (closes #273) Summary: This PR adds 8 missing Review process: Four independent agents (two for CLAUDE.md compliance, two for bugs/logic issues) checked out the PR's actual merged state and empirically verified every claim rather than trusting the PR description, per this repo's own "never just theorize" and "completeness claim ... needs the grep before it ships" guidance:
No CLAUDE.md violations and no bugs were found by any of the four review passes. No issues survived to the validation stage (step 5), since none were raised in step 4. One non-blocking observation (not a compliance finding): issue #273 itself suggested adding this grep as a VerdictReady for merge. |
ARD summaryVerdict Ready for merge on the first round -- all completeness/ordering/YAML claims independently verified by the review, no threads, no blocking findings. One non-blocking observation (the |
Closes #273
The bug
website/_quarto.yml's "Reference" sidebar omitted several existing reference pages, so they were reachable only by direct URL or viawebsite/workflows.qmd's catalog links.Scope re-derived (8, not the issue's 4)
#273 (2026-07-18) listed 4 missing pages. Re-running its own verification against current
mainfound 8 -- more capabilities were added since without sidebar entries:Missing:
lint-yaml,lint-markdown,check-new-line-breaks,lint-qmd,lint-changed-lines,check-equation-renders,request-dependabot-review,altdoc-multiversion-docs.The fix
Each entry is inserted at its canonical position, so the sidebar order stays byte-identical to
workflows.qmd's catalog order -- no existing entries reordered.Verified:
website/reference/*.qmdis now listed in the sidebar (the check above prints nothing).reference/*.qmdentry resolves to a real file.website/_quarto.ymlparses as YAML.diffof the sidebar order againstworkflows.qmd's catalog order is empty (identical).Adds a
changelog.d/fragment, matching the repo's precedent (document-update-snapshots.added.md) for website reference/sidebar doc changes.