fix(catalog): normalize module-help.csv to documented 13-column schema#25
fix(catalog): normalize module-help.csv to documented 13-column schema#25
Conversation
Thirty-four rows were missing one column between description and phase, leaving them at 12 fields. CSV consumers reading by header position were silently mapping data into the wrong columns (phase into args, required into before, etc). Inserted an empty cell at column index 5 to restore alignment with the header (module,skill,display-name,menu-code,description,action,args, phase,after,before,required,output-location,outputs).
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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: This PR normalizes Changes:
Technical Notes: This is a data-shape fix for positional CSV consumers; the change should reduce downstream parsing errors and improve install-time help/config generation consistency. 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
#25) Thirty-four rows were missing one column between description and phase, leaving them at 12 fields. CSV consumers reading by header position were silently mapping data into the wrong columns (phase into args, required into before, etc). Inserted an empty cell at column index 5 to restore alignment with the header (module,skill,display-name,menu-code,description,action,args, phase,after,before,required,output-location,outputs).
1 participant
Summary
Why
CSV consumers that read by header position were silently mapping data into the wrong columns (`phase` value into `args`, `required` into `before`, etc). Inserting one empty cell at index 5 restores correct alignment with the documented header (`module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs`).
This is part of a coordinated cleanup also fixing the installer in BMAD-METHOD (PR #2349 there) and the same defect across bmad-module-creative-intelligence-suite, bmad-method-test-architecture-enterprise, bmad-method-wds-expansion.
Test plan