Skip to content

fix(catalog): normalize module-help.csv to documented 13-column schema#28

Merged
bmadcode merged 1 commit intomainfrom
fix-module-help-schema
Apr 28, 2026
Merged

fix(catalog): normalize module-help.csv to documented 13-column schema#28
bmadcode merged 1 commit intomainfrom
fix-module-help-schema

Conversation

@bmadcode
Copy link
Copy Markdown
Contributor

Summary

  • Eighteen rows in `src/module-help.csv` were missing one column between `description` and `phase`, leaving them at 12 fields instead of 13.

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-module-game-dev-studio, bmad-method-test-architecture-enterprise.

Test plan

  • All rows re-audited with csv-parse — every data row now has exactly 13 fields
  • Run a fresh install and confirm `_bmad/_config/bmad-help.csv` data lands in the right columns

Eighteen 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).
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • src/module-help.csv is excluded by !**/*.csv

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a978c2fa-7805-477c-9fd5-ff05b2a9fd1f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-module-help-schema

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bmadcode bmadcode merged commit 5031275 into main Apr 28, 2026
20 checks passed
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Apr 28, 2026

🤖 Augment PR Summary

Summary: This PR normalizes src/module-help.csv to the documented 13-column schema.

Changes:

  • Fixes 18 rows that were missing a column between description and phase
  • Inserts an empty cell at the expected index so values align with the header
  • Prevents downstream CSV consumers from silently shifting fields into the wrong columns

Technical Notes: Intended to restore correct positional parsing for module,skill,display-name,menu-code,description,action,args,phase,after,before,required,output-location,outputs and keep installs emitting correctly-aligned bmad-help.csv.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant