Conversation
There was a problem hiding this comment.
Pull request overview
Integrates vitepress-plugin-llms into the docs build to generate LLM-friendly outputs, and refactors the VitePress sidebar into a reusable constant to improve navigation/config maintainability.
Changes:
- Add
vitepress-plugin-llmsas a docs dev dependency (with lockfile updates). - Refactor sidebar definition into a typed constant and reuse it in the VitePress theme config.
- Register the
vitepress-plugin-llmsVite plugin in the VitePress config.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/package.json | Adds vitepress-plugin-llms to devDependencies. |
| docs/package-lock.json | Updates lockfile to include vitepress-plugin-llms and its dependency graph. |
| docs/.vitepress/config.mts | Extracts sidebar into a shared constant and configures the vitepress-plugin-llms plugin. |
Files not reviewed (1)
- docs/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "devDependencies": { | ||
| "vitepress": "^2.0.0-alpha.17" | ||
| "vitepress": "^2.0.0-alpha.17", | ||
| "vitepress-plugin-llms": "^1.12.0" | ||
| } |
There was a problem hiding this comment.
vitepress-plugin-llms pulls in transitive deps (e.g., pretty-bytes@7.1.0) that declare node >=20, so installs can fail or behave unexpectedly on Node 18/19 even though the plugin itself advertises >=18. Consider adding an explicit engines.node constraint (and optionally documenting it) in docs/package.json so local dev/CI environments are aligned.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Integrate vitepress-plugin-llms to enhance documentation capabilities and update the sidebar configuration for improved navigation.