refactor: remove downloads page and bundle generation#1577
refactor: remove downloads page and bundle generation#1577bmadcode merged 2 commits intobmad-code-org:mainfrom
Conversation
The downloads page offered bmad-sources.zip and bmad-prompts.zip, both redundant: GitHub provides source archives for every tag natively, and npx bmad-method install is the supported path for compiled prompts. Remove the downloads page, all bundle generation code, the archiver dependency, and nav links. The llms.txt and llms-full.txt files (the genuinely useful artifacts) continue to be generated as before.
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis pull request removes downloadable bundle functionality from the codebase. It deletes the downloads documentation page, removes bundle generation code from the build system, eliminates the archiver dependency, and removes Downloads navigation links from the UI components. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🤖 Augment PR SummarySummary: Removes the documentation Downloads page and eliminates ZIP bundle generation from the docs build pipeline.
llms.txt and llms-full.txt are still generated and copied into the built site; the build no longer produces a downloads/ directory, simplifying CI/release coupling.
🤖 Was this summary useful? React with 👍 or 👎 |
| "@astrojs/starlight": "^0.37.5", | ||
| "@eslint/js": "^9.33.0", | ||
| "archiver": "^7.0.1", | ||
| "astro": "^5.16.0", |
There was a problem hiding this comment.
package-lock.json in this branch still lists archiver, so npm ci will likely fail due to the lockfile being out of sync with package.json. Consider updating the lockfile as part of this PR to fully remove archiver from the dependency tree.
Severity: high
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| @@ -176,8 +172,6 @@ function generateLlmsTxt(outputDir) { | |||
| '## Quick Links', | |||
| '', | |||
| `- [Full Documentation (llms-full.txt)](${siteUrl}/llms-full.txt) - Complete docs for AI context`, | |||
There was a problem hiding this comment.
After removing the ZIP bundle feature, CHANGELOG.md still advertises “Downloadable Source Bundles” (e.g., bmad-sources.zip/bmad-prompts.zip). Consider updating that entry so release notes don’t point users at artifacts that no longer exist.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Removes leftover archiver dependencies from the lock file following bmad-code-org#1577. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Removes leftover archiver dependencies from the lock file following #1577. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Summary
Why
The downloads page was a unique artifact in the docs scheme of things -- too much hassle for what it is worth:
npx bmad-method installdoes this and is the supported path. A stale zip is strictly worse.The genuinely useful artifacts (llms.txt, llms-full.txt) continue to be generated as before.
Test plan
npm testpasses (lint, format, schemas, install tests)npm run docs:buildsucceeds -- no zip generation, no downloads/ in outputnpm run docs:validate-links-- all links valid