BMAD-METHOD/src/modules/bmb/workflows/module/steps-v
cx-noam-brendel 7851fd8b80 fix: address PR review comments
Changes per reviewer feedback:
- Remove ⚠️ CRITICAL warnings from documentation (per @alexeyv)
- Replace placeholder YAML examples with concrete values
- Simplify validation checklist text

Technical fixes per CodeRabbit:
- Remove .strict() that contradicted .passthrough()
- Add isCoreModule parameter for conditional default_selected validation
- Enforce mutual exclusivity: inherit/prompt and single-select/multi-select
- Use project_root for consistent path reporting
- Harden test runner with guarded access for error.issues
- Move parseTestMetadata inside try/catch block

All tests pass (52 agent + 17 module + 13 installation)
2026-01-15 07:23:52 +02:00
..
step-01-load-target.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-02-file-structure.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-03-module-yaml.md fix: address PR review comments 2026-01-15 07:23:52 +02:00
step-04-agent-specs.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-05-workflow-specs.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-06-documentation.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-07-installation.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00
step-08-report.md quadrivariate module workflow 2026-01-03 17:12:00 +08:00