mirror of https://github.com/bmad-code-org/BMAD-METHOD.git synced 2026-08-29 03:44:18 +08:00

Files

T

Alex Verkhovsky 513f440a23 refactor(installer): restructure installer with clean separation of concerns (#2129 )

* refactor(installer): restructure installer with clean separation of concerns

Move tools/cli/ to tools/installer/ with major structural cleanup:

- InstallPaths async factory for path resolution and directory creation
- Config value object (frozen) replaces mutable config bag
- ExistingInstall value object replaces stateful Detector class
- OfficialModules + CustomModules + ExternalModuleManager replace monolithic ModuleManager
- install() is prompt-free; all user interaction in ui.js
- Update state returned explicitly instead of mutating customConfig
- Delete dead code: dependency-resolver, _base-ide, IdeConfigManager,
  platform-codes helpers, npx wrapper, xml-utils
- Flatten directory structure: custom/handler → custom-handler,
  tools/cli/ → tools/installer/, lib/ directories removed
- Update all path references in package.json, tests, CI, and docs

* fix(installer): guard ExistingInstall.version and surface module.yaml errors

Guard ExistingInstall.version access with .installed check in
uninstall.js, ui.js, and installer.js to prevent throwing on
empty/partial _bmad dirs. Surface invalid module.yaml parse errors
as warnings instead of silently returning empty results.

2026-03-27 06:50:07 -06:00

218 B

Raw Blame History

JavaScript Conventions

Function ordering

Define functions top-to-bottom in call order: callers above callees. If install() calls _initPaths(), then install appears first and _initPaths appears after it.

218 B Raw Blame History

JavaScript Conventions

Function ordering

218 B

Raw Blame History