Skip to content

fix(site): llms.txt and llms-full.txt not now working#7765

Merged
aidankmcalister merged 1 commit intomainfrom
fix/llms-stuff
Apr 2, 2026
Merged

fix(site): llms.txt and llms-full.txt not now working#7765
aidankmcalister merged 1 commit intomainfrom
fix/llms-stuff

Conversation

@aidankmcalister
Copy link
Copy Markdown
Member

@aidankmcalister aidankmcalister commented Apr 2, 2026

Summary by CodeRabbit

Release Notes

  • New Features
    • Introduced new LLM-friendly content endpoints providing structured site information, including navigation, product overviews, and pricing details for AI model consumption.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 2, 2026 4:29pm
docs Ready Ready Preview, Comment Apr 2, 2026 4:29pm
eclipse Ready Ready Preview, Comment Apr 2, 2026 4:29pm
site Ready Ready Preview, Comment Apr 2, 2026 4:29pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Added a new LLM content generation module (llms-content.ts) that builds index and full-site content payloads, along with two new route handlers at /llms.txt and /llms-full.txt to serve this content as plain text responses. The full-content route was also updated to dynamically generate content instead of returning empty content.

Changes

Cohort / File(s) Summary
LLM Content Generation Module
apps/site/src/app/llms-content.ts
New module that generates two text payloads for LLM consumption. Exports buildLlmsIndexContent() (builds index with key pages and link to full content) and buildLlmsFullContent() (builds comprehensive site content including product overviews, pricing, and sections). Includes helper functions for URL absolutization and yearly pricing calculation.
LLM Route Handlers
apps/site/src/app/llms.txt/route.ts, apps/site/src/app/llms-full.txt/route.ts
New and updated route handlers serving LLM-formatted content as plain text. Both export revalidate = false to disable ISR. The index route calls buildLlmsIndexContent(), and the full-content route now calls buildLlmsFullContent() instead of returning empty content.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title contains a grammatical error ('not now working' instead of 'now working') making it vague and potentially misleading about what's actually being fixed. Clarify the title to 'fix(site): llms.txt and llms-full.txt now working' or similar to accurately describe that these endpoints are being fixed/enabled.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 2, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 2, 2026, 4:36 PM

@aidankmcalister aidankmcalister merged commit 4473606 into main Apr 2, 2026
15 of 16 checks passed
@aidankmcalister aidankmcalister deleted the fix/llms-stuff branch April 2, 2026 16:31
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.

2 participants