Skip to content

refactor: remove downloads page and bundle generation#1577

Merged
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:refactor/remove-downloads-page
Feb 7, 2026
Merged

refactor: remove downloads page and bundle generation#1577
bmadcode merged 2 commits intobmad-code-org:mainfrom
alexeyv:refactor/remove-downloads-page

Conversation

@alexeyv
Copy link
Copy Markdown
Collaborator

@alexeyv alexeyv commented Feb 7, 2026

Summary

  • Remove the downloads page and all ZIP bundle generation code
  • Remove archiver dependency, nav links, and stale references across 8 files

Why

The downloads page was a unique artifact in the docs scheme of things -- too much hassle for what it is worth:

  • bmad-sources.zip was a zip of the repo. GitHub already provides source archives for every tag. Anyone can clone, fork, or click the green button. Redundant.
  • bmad-prompts.zip was a pre-compiled _bmad/ directory. npx bmad-method install does this and is the supported path. A stale zip is strictly worse.
  • The page itself -- after removing bundles, everything remaining (LLM files, install command, API URLs) was duplicated on other pages.
  • The build pipeline had to jump through hoops (gh release, git archive, fallback chains) to produce these zips during docs builds that have nothing to do with releases. This broke CI (refactor: rebuild download bundles from releases and compiled prompts #1571) and added fragile logic for zero user value.

The genuinely useful artifacts (llms.txt, llms-full.txt) continue to be generated as before.

Test plan

  • npm test passes (lint, format, schemas, install tests)
  • npm run docs:build succeeds -- no zip generation, no downloads/ in output
  • npm run docs:validate-links -- all links valid
  • llms.txt has no references to zip files
  • Site navigation has no Downloads link

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.
@alexeyv alexeyv marked this pull request as ready for review February 7, 2026 05:24
@bmadcode bmadcode merged commit a8cda7c into bmad-code-org:main Feb 7, 2026
5 of 6 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 7, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Documentation Cleanup
docs/downloads.md, docs/how-to/get-answers-about-bmad.md, website/README.md, website/src/rehype-base-paths.js
Removed downloads page, reference to downloads page in guide, build output documentation, and associated comment.
Build System
tools/build-docs.mjs
Removed downloadable bundle generation functions (generateDownloadBundles, generateSourcesBundle, generatePromptsBundle), archiver import, and downloads-related LLM content entries. Public API surface reduced by three exported functions.
Dependencies
package.json
Removed archiver (^7.0.1) devDependency.
UI Navigation
website/src/components/Header.astro, website/src/components/MobileMenuFooter.astro
Removed Downloads navigation links and associated styling from header and mobile footer components.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • bmadcode
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Feb 7, 2026

🤖 Augment PR Summary

Summary: Removes the documentation Downloads page and eliminates ZIP bundle generation from the docs build pipeline.
Changes:

  • Deleted docs/downloads.md and removed in-doc references to that page
  • Removed ZIP bundle creation (bmad-sources.zip, bmad-prompts.zip) from tools/build-docs.mjs
  • Dropped the archiver dependency and associated artifact-copy logic for downloads/
  • Updated llms.txt generation to stop advertising bundle links
  • Removed “Downloads” navigation links from desktop and mobile header components
  • Updated website README/comments to reflect the new (LLM-only) artifacts output
Technical Notes: 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 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

"@astrojs/starlight": "^0.37.5",
"@eslint/js": "^9.33.0",
"archiver": "^7.0.1",
"astro": "^5.16.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Fix This in Augment

🤖 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`,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

alexeyv added a commit to alexeyv/BMAD-METHOD that referenced this pull request Feb 7, 2026
Removes leftover archiver dependencies from the lock file
following bmad-code-org#1577.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
bmadcode pushed a commit that referenced this pull request Feb 7, 2026
Removes leftover archiver dependencies from the lock file
following #1577.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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