Skip to content

Conversation

@ceIia
Copy link
Contributor

@ceIia ceIia commented Nov 15, 2025

this is a follow up to #191.

changes:

  • remove build time llms.txt and llms-full.txt generation in favor of route handlers. we make them fully static, and let Vercel handle CDN level caching on future deployments, that way we don't risk machines/browsers caching a stale version for too long (and we don't explode Rocicorp's Vercel bill).
  • properly have the .md suffix to documentation links inside llms.txt, llms-full.txt, and the markdown files themselves.
  • fix a bug where some links would be missing a space between the closing parenthesis and the word after
  • update the /docs/llms page

notes:

  1. i held off from adding the "Copy page" or "View as Markdown" buttons that most other doc sites/frameworks have been adding, but if you think it would be useful, let me know.
  2. @aboodman you mentionned a "few follow-up things I think we should do" in the other PR, let me know what you had in mind - it's fun working on a more "artisanal" docs site!

@vercel
Copy link

vercel bot commented Nov 15, 2025

@ceIia is attempting to deploy a commit to the Rocicorp Team on Vercel.

A member of the Team first needs to authorize it.

@ceIia ceIia marked this pull request as ready for review November 15, 2025 15:20
@0xcadams
Copy link
Member

Can you explain what this PR is about? Sorry it got missed. Why can't we keep the build time

@0xcadams 0xcadams self-requested a review November 21, 2025 19:07
@aboodman
Copy link
Contributor

I think @ceIia was probably feeling it was awkward/weird to generate /doc/foo-page.md statically for every single page. I have similar feeling. I prefer it being runtime generated too , for consistency with how rest of docs work.

@aboodman
Copy link
Contributor

Sorry for forgetting to review this @ceIia - I really appreciate the contribution.

it's fun working on a more "artisanal" docs site!

😂

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.

3 participants