feat: add module-help.csv and module.yaml to src/skills#38
Conversation
Add canonical module metadata files at src/skills/ level: - module-help.csv: module help catalog in new 13-column format - module.yaml: module configuration prompts and defaults
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
WalkthroughA new skill/module configuration file for "BMad Builder" has been added to define module metadata, greeting text, and two configurable parameters for managing output and report folder paths. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
✨ 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: Adds canonical skills-level metadata and help catalog files for the BMad Builder module. 🤖 Was this summary useful? React with 👍 or 👎 |
| @@ -0,0 +1,20 @@ | |||
| code: bmb | |||
There was a problem hiding this comment.
tools/validate-file-refs.mjs auto-detects the module code from src/module.yaml (not src/skills/module.yaml), so this addition likely won’t fix “Module: (unknown)” and external-ref skipping in that validator. Is the intent to also add/relocate a src/module.yaml, or update the validator to look under src/skills/?
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| default_selected: false | ||
| module_greeting: > | ||
| Enjoy making your dream creations with the BMad Builder Module! | ||
| Run this again at any time if you want to reconfigure a setting or have updated the module, (or optionally just update _bmad/config.yaml and config.user.yaml to change existing values) |
There was a problem hiding this comment.
In module_greeting, the text references config.user.yaml without the _bmad/ prefix (it says _bmad/config.yaml and config.user.yaml). If users follow this literally, they may look in the wrong location for {project-root}/_bmad/config.user.yaml.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Summary
src/skills/module-help.csv— canonical module help catalog at the skills level (new 13-column format)src/skills/module.yaml— module configuration prompts and defaultsThese were part of the module-builder branch work but missed the CSV format PR (#37).
Test plan
Summary by CodeRabbit