Skip to content

fix: llms.txt#1273

Merged
mbifulco merged 3 commits into
mainfrom
feat/llms-txt
Jun 23, 2026
Merged

fix: llms.txt#1273
mbifulco merged 3 commits into
mainfrom
feat/llms-txt

Conversation

@mbifulco

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 23, 2026 20:04
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
mikebifulco-com-bnbu Ready Ready Preview, Comment Jun 23, 2026 8:08pm

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 fixes the site’s llms.txt endpoint by ensuring the generated (dynamic) src/app/llms.txt/route.ts content is what’s actually served, improving the topics section to use tag popularity, and enhancing the output formatting and optional resources.

Changes:

  • Switch tag sourcing from “all tags, then slice” to “top tags by usage count” via getTopTags(15).
  • Improve list rendering so excerpts are only appended when present (avoids dangling :).
  • Remove the public/llms.txt static file to prevent it from shadowing the app route, and add an AT Protocol publication reference to the generated content.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/app/llms.txt/route.ts Generates llms.txt dynamically; now uses top tags by usage and improves formatting; adds AT Protocol publication link.
public/llms.txt Removes the static placeholder file so /llms.txt is served by the route handler.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mbifulco mbifulco merged commit 3562a1c into main Jun 23, 2026
9 checks passed
@mbifulco mbifulco deleted the feat/llms-txt branch June 23, 2026 20:15
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