Skip to content

fix: show sidebar on documentation index#14

Merged
smiggleworth merged 3 commits into
mainfrom
fix/website-docs-index-sidebar
Jul 26, 2026
Merged

fix: show sidebar on documentation index#14
smiggleworth merged 3 commits into
mainfrom
fix/website-docs-index-sidebar

Conversation

@smiggleworth

Copy link
Copy Markdown
Contributor

Closes #13

Summary

  • Route /docs through DocsLayout so the documentation entry point includes the sidebar.
  • Restore docs-layout expectations for /docs in static verification.
  • Add missing keys to migrated docs navigation and heading lists to remove dev warnings.

Verification

  • npm run check
  • /docs generated output contains docs layout, sidebar, navigation, and TOC markers ✅
  • 338 routes and 32,442 API anchors verified ✅

Copilot AI review requested due to automatic review settings July 26, 2026 10:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Routes the /docs entry point through the docs shell so the documentation index consistently renders with the docs sidebar/navigation/TOC, updates static-output verification expectations accordingly, and eliminates React missing-key warnings in docs lists.

Changes:

  • Register /docs under DocsLayout (instead of the marketing layout) in both runtime and static route registries.
  • Update verify-static-output.ts so /docs is treated as a docs route for layout/marker verification.
  • Add missing key props to mapped navigation and TOC elements in docs UI.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/pages/docs/page.tsx Adds a key to headings rendering to remove missing-key warnings.
src/pages/docs/_layout.tsx Adds keys to docs sidebar and TOC list rendering to remove missing-key warnings.
src/pages/_routes.tsx Routes /docs through DocsLayout by registering all docsCatalog entries (including /docs) under the docs layout.
scripts/verify-static-output.ts Reclassifies /docs as a docs route in static verification expectations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/pages/docs/page.tsx
@smiggleworth
smiggleworth merged commit 94dbcef into main Jul 26, 2026
1 check passed
@smiggleworth
smiggleworth deleted the fix/website-docs-index-sidebar branch July 26, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: show sidebar on documentation index

2 participants