-
Notifications
You must be signed in to change notification settings - Fork 46
Support explicit markdown links in llms.txt #3153
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
|
Important Review skippedAuto 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 WalkthroughRemoved language-related validation and handling from llmstxt generation. Simplified the GraphQL query by eliminating language fields and extraction logic. Updated base URL generation to explicitly include Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
GregHolmes
left a 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.
LGTM
- Markdown link returns content for all code snippets - Hence, having lang specific links is unncessary and increases llms.txt size
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
Updates the llms.txt generation script to emit explicit Markdown (.md) links for documentation pages and removes language-specific link variants to reduce file size.
Changes:
- Removes language detection and language-specific URL generation for
llms.txtentries. - Simplifies the GraphQL query and page model to only rely on MDX frontmatter + file path info.
- Updates serialization to link to
/docs/<slug>.mdfor each docs page.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
I know I approved beforehand but there's some things that might need changing while we're updating LLMs.txt. The generation script was before we had AI Transport as a product in the docs. It currently messes up how that's shown. So instead of being shown as a product first you see the guides in Platform: And last at the end all of these are AIT except FAQs: |
|
Thanks for the feedback, will take a look 👍 |
|
Okay, added separate category for |
Markdownlink returns content for all code snippets. So having lang specific links is unncessary and increases llms.txt sizeSummary by CodeRabbit
.mdfile extensions in output.✏️ Tip: You can customize this high-level summary in your review settings.