Skip to content

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

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#95
bmadcode merged 1 commit intomainfrom
fix-module-help-schema

Conversation

@bmadcode
Copy link
Copy Markdown
Contributor

Summary

  • Nine 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-wds-expansion.

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

Nine 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).
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Apr 28, 2026

🤖 Augment PR Summary

Summary: Normalizes src/module-help.csv rows to match the documented 13-column schema.

Changes:

  • Fixes nine misaligned rows by inserting the missing empty field between description and phase so header-position CSV consumers map values into the correct columns.

🤖 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.

@bmadcode bmadcode merged commit b0ee2a5 into main Apr 28, 2026
5 checks passed
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