refactor: split out docs/manual routes from shared MdxRoute file#1221
refactor: split out docs/manual routes from shared MdxRoute file#1221jderochervlk wants to merge 3 commits intomasterfrom
Conversation
jderochervlk
changed the title
jderochervlk
force-pushed
the
vlk/split-out-docs-manual
branch
from
1ae7265 to
05ae835
Compare
There was a problem hiding this comment.
Pull request overview
Refactors routing so /docs/manual/* pages are served by a dedicated DocsManualRoute (with sidebar + TOC), and introduces shared helpers/utilities to support manual doc navigation.
Changes:
- Add
DocsManualRouteto load/compile manual MDX, build sidebar categories, and generate a per-page table of contents. - Introduce
SidebarHelpersandMdxFile.loadAllAttributesto generate sidebar nav data from MDX frontmatter. - Update route configuration to include manual docs routes and exclude them from the shared
mdxRoutes; document PR/commit conventions inAGENTS.md.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/SidebarHelpers.resi | Exposes sidebar helper function signatures for nav/category construction. |
| src/SidebarHelpers.res | Implements sidebar nav item conversion and grouped category helpers. |
| src/MdxFile.resi | Adds public API for loadAllAttributes to scan MDX frontmatter. |
| src/MdxFile.res | Implements loadAllAttributes and refactors resolveFilePath formatting. |
| app/routes/DocsManualRoute.resi | Declares loader data contract for manual docs route. |
| app/routes/DocsManualRoute.res | New manual-docs route: compiles MDX, builds sidebar + TOC, renders layout. |
| app/routes.res | Registers manual-docs routes and filters them out of mdxRoutes. |
| AGENTS.md | Adds guidance for commit/PR conventions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jderochervlk
force-pushed
the
vlk/split-out-docs-manual
branch
from
05ae835 to
1b8f46d
Compare
jderochervlk
force-pushed
the
vlk/split-out-docs-manual
branch
from
de2ead6 to
5d7a8f2
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Replace unsafe %identity cast with dedicated sidebarEntry type - Narrow directory scan from docs/ to docs/manual/ - Add module-level cache for manualTableOfContents - Remove dead manual-specific code from MdxRoute - Move shared sidebar helpers to SidebarHelpers module
Cloudflare deploymentDeployement ID: b523ee12-d738-478e-9cd8-fa1a8f9e1fa9 ⛅️ wrangler 4.63.0 (update available 4.81.1) ✨ Uploading _redirects |
Note
This work was done with AI assistance