Skip to content

Make clean MDs being a post build artifact#830

Closed
jottakka wants to merge 6 commits intomainfrom
feat/clean-md-vercel
Closed

Make clean MDs being a post build artifact#830
jottakka wants to merge 6 commits intomainfrom
feat/clean-md-vercel

Conversation

@jottakka
Copy link
Contributor

@jottakka jottakka commented Feb 26, 2026

Note

Medium Risk
Changes CI/build pipeline around postbuild and markdown generation/verification; failures could break deploys or LLMs.txt automation if expected markdown files aren’t produced.

Overview
Moves public/_markdown (clean markdown) from a committed artifact to a build-time artifact: deletes the dedicated generate-markdown workflow, removes existing generated files from the repo, and adds public/_markdown/ to .gitignore.

Updates postbuild to generate clean markdown and (on Vercel) run a new verify:markdown script that fails the build if any app/**/page.mdx pages are missing corresponding generated .md files.

Enhances the llmstxt workflow to build docs with SKIP_POSTBUILD, force a full markdown rebuild, verify completeness, and only commit changes when public/llms.txt actually differs.

Written by Cursor Bugbot for commit 3aacda8. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Feb 26, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 27, 2026 0:46am

Request Review

@jottakka jottakka requested a review from evantahler February 26, 2026 23:29
Copy link
Contributor

@evantahler evantahler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that resolveCleanMarkdownBuildMode might not be needed, and that a build should create all these markdown files, at least until we have Cloudflare do this for us.

Comment on lines 46 to 47
env:
SKIP_POSTBUILD: "true"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably want to ensure this works on CI, no?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did llms.txt loose pages?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we are still building the markdown, we should still test it I think

Comment on lines +32 to +33
# Generated clean markdown (built at build time, not committed)
public/_markdown/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

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