6bf6a2b6d2
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)
|
||
|---|---|---|
| .. | ||
| cli | ||
| docs | ||
| flattener | ||
| lib | ||
| maintainer | ||
| schema | ||
| bmad-npx-wrapper.js | ||
| build-docs.js | ||
| fix-doc-links.js | ||
| format-workflow-md.js | ||
| migrate-custom-module-paths.js | ||
| platform-codes.yaml | ||
| validate-agent-schema.js | ||
| validate-doc-links.js | ||
| validate-module-schema.js | ||
| validate-svg-changes.sh | ||