Skip to content

refactor: remove redundant generated files from repo#50

Merged
marc0olo merged 1 commit intomainfrom
marc0olo/remove-redundant-generated-files
Feb 27, 2026
Merged

refactor: remove redundant generated files from repo#50
marc0olo merged 1 commit intomainfrom
marc0olo/remove-redundant-generated-files

Conversation

@marc0olo
Copy link
Copy Markdown
Member

@marc0olo marc0olo commented Feb 27, 2026

Summary

The Astro migration (#39) introduced page routes that generate llms.txt, llms-full.txt, agent.json, and ai-plugin.json at build time. This made the previously committed copies in public/ and their standalone generation scripts redundant. Every PR touching a SKILL.md had to regenerate and commit these files, causing merge conflicts.

  • Delete 4 redundant scripts: generate-skills.js, generate-llms.js, generate-llms-full.js, generate-agent-json.js — all replaced by Astro pages in src/pages/ and the build-time loader in src/data/skills.ts
  • Remove generated files from git: public/llms.txt, public/llms-full.txt, public/.well-known/agent.json, public/.well-known/ai-plugin.json
  • Simplify npm run generate to only run generate-readme-table.js (the only remaining script that writes to a committed file)
  • Update CI to check README.md freshness instead of public/
  • Update documentation (CLAUDE.md, CONTRIBUTING.md) to reflect the simplified workflow

Zero impact on the deployed site — all URLs (/llms.txt, /.well-known/agent.json, etc.) continue to work via Astro page routes.

After the Astro migration, llms.txt, llms-full.txt, agent.json, and
ai-plugin.json are generated by Astro page routes at build time. The
committed copies in public/ were duplicates that caused merge conflicts
on every PR touching a SKILL.md.

- Delete scripts/generate-{skills,llms,llms-full,agent-json}.js
  (replaced by Astro pages and src/data/skills.ts)
- Remove public/llms.txt, llms-full.txt, .well-known/agent.json,
  .well-known/ai-plugin.json from git
- Add public/ generated paths to .gitignore
- Simplify npm run generate to only update README skills table
- Update CI to check README.md freshness instead of public/
- Update CLAUDE.md and CONTRIBUTING.md
@marc0olo marc0olo merged commit b88a360 into main Feb 27, 2026
6 checks passed
@marc0olo marc0olo deleted the marc0olo/remove-redundant-generated-files branch February 27, 2026 21:34
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