-
-
Notifications
You must be signed in to change notification settings - Fork 940
feat(website): Add llms.txt support #442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #442 +/- ##
=======================================
Coverage 90.32% 90.32%
=======================================
Files 96 96
Lines 4962 4962
Branches 1042 1042
=======================================
Hits 4482 4482
Misses 480 480 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
96cb428 to
575717f
Compare
289e365 to
cbf10c7
Compare
6f5a2e3 to
54b41e0
Compare
8608485 to
de11c88
Compare
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughA new plugin, Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant VitePress
participant vitepress-plugin-llms
User->>VitePress: Start build or serve
VitePress->>vitepress-plugin-llms: Initialize plugin (excludeIndexPage: true)
vitepress-plugin-llms-->>VitePress: Plugin hooks and modifies build as configured
VitePress-->>User: Serve or build site with plugin effects
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for llms.txt to the website. It introduces a new dependency and integrates a corresponding plugin in the VitePress configuration.
- Added "vitepress-plugin-llms" as a dependency.
- Imported and configured the new llms.txt plugin in the VitePress configuration.
Reviewed Changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| website/client/package.json | Added a new dependency for the llms.txt support. |
| website/client/.vitepress/config/configShard.ts | Imported and configured the llms.txt plugin. |
Comments suppressed due to low confidence (1)
website/client/.vitepress/config/configShard.ts:4
- [nitpick] Consider renaming the import 'llmstxt' to something more descriptive (e.g., 'llmsPlugin') to clearly reflect its functionality.
import llmstxt from 'vitepress-plugin-llms';
website/client/package.json
Outdated
| "rollup-plugin-visualizer": "^6.0.1", | ||
| "vitepress": "^1.6.3" | ||
| "vitepress": "^1.6.3", | ||
| "vitepress-plugin-llms": "https://pkg.pr.new/vitepress-plugin-llms@d35010c" |
Copilot
AI
Jun 24, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider using a versioned package release instead of a URL dependency to ensure stability and easier maintenance in production environments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
website/client/.vitepress/config/configShard.ts (1)
4-4: Consider using a more descriptive import alias.The import alias
llmstxtdoesn't clearly match the package namevitepress-plugin-llms, which could cause confusion for future maintainers.Consider using a more descriptive alias:
-import llmstxt from 'vitepress-plugin-llms'; +import { default as vitePressLlmsPlugin } from 'vitepress-plugin-llms';Or simply use the default import without alias:
-import llmstxt from 'vitepress-plugin-llms'; +import vitePressLlms from 'vitepress-plugin-llms';
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
website/client/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (2)
website/client/.vitepress/config/configShard.ts(2 hunks)website/client/package.json(1 hunks)
🔇 Additional comments (1)
website/client/.vitepress/config/configShard.ts (1)
149-151: Plugin configuration looks correct.The plugin is properly integrated into the Vite plugins array with appropriate configuration. The
excludeIndexPage: trueoption suggests intentional behavior to exclude the index page from llms.txt generation.Please verify that the
excludeIndexPageoption aligns with your intended llms.txt generation behavior:What does the excludeIndexPage option do in vitepress-plugin-llms?
website/client/package.json
Outdated
| "rollup-plugin-visualizer": "^6.0.1", | ||
| "vitepress": "^1.6.3" | ||
| "vitepress": "^1.6.3", | ||
| "vitepress-plugin-llms": "https://pkg.pr.new/vitepress-plugin-llms@d35010c" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
🧩 Verification correct
Consider using a published npm package version instead of URL-based dependency.
Using a URL-based package reference (https://pkg.pr.new/vitepress-plugin-llms@d35010c) introduces security and stability concerns:
- External URL dependencies can pose security risks
- Commit-hash based versioning makes it unclear what functionality is being included
- This appears to be a preview/development version, which may not be suitable for production
- Dependency management becomes more complex
Please verify if there's a published npm version available:
🏁 Script executed:
#!/bin/bash
# Check if vitepress-plugin-llms is available on npm
npm view vitepress-plugin-llms versions --json 2>/dev/null || echo "Package not found on npm registry"
# Check the specific commit referenced
curl -s "https://api.github.com/repos/okineadev/vitepress-plugin-llms/commits/d35010c" | jq -r '.commit.message // "Commit not found"'Length of output: 719
Consider using a published npm package version instead of URL-based dependency.
Using a URL-based package reference (https://pkg.pr.new/vitepress-plugin-llms@d35010c) introduces security and stability concerns:
- External URL dependencies can pose security risks
- Commit-hash based versioning makes it unclear what functionality is being included
- This appears to be a preview/development version, which may not be suitable for production
- Dependency management becomes more complex
Please verify if there's a published npm version available:
#!/bin/bash
# Check if vitepress-plugin-llms is available on npm
npm view vitepress-plugin-llms versions --json 2>/dev/null || echo "Package not found on npm registry"
# Check the specific commit referenced
curl -s "https://api.github.com/repos/okineadev/vitepress-plugin-llms/commits/d35010c" | jq -r '.commit.message // "Commit not found"'🤖 Prompt for AI Agents
In website/client/package.json at line 21, the dependency
"vitepress-plugin-llms" is specified using a URL with a commit hash, which can
cause security and stability issues. To fix this, check if a published npm
version of "vitepress-plugin-llms" exists by running npm view commands as
suggested. If available, replace the URL-based dependency with the latest stable
npm version number to ensure better security, clarity, and dependency
management.
16493ac to
66f6eee
Compare
PR Review: Add llms.txt SupportThank you for adding llms.txt support to the Repomix documentation site! This is a valuable addition that will help LLMs better understand and navigate the documentation. Here's my comprehensive review: ✅ Strengths
🔍 Code Quality & Best PracticesDetailsImport Organization (website/client/.vitepress/config/configShard.ts:4)The import is well-placed and follows the existing import structure. Good alphabetical ordering. import llmstxt from 'vitepress-plugin-llms';Plugin Configuration (website/client/.vitepress/config/configShard.ts:158-160)The plugin is correctly configured within the Vite plugins array. The excludeIndexPage: true option makes sense for this documentation site. llmstxt({
excludeIndexPage: true,
}),
|
Checklist
npm run testnpm run lint