Skip to content

fix: normalize workflow manifest schema#1071

Merged
bmadcode merged 4 commits intobmad-code-org:mainfrom
dickymoore:fix/workflow-manifest-schema
Dec 11, 2025
Merged

fix: normalize workflow manifest schema#1071
bmadcode merged 4 commits intobmad-code-org:mainfrom
dickymoore:fix/workflow-manifest-schema

Conversation

@dickymoore
Copy link
Contributor

What

Normalize workflow manifest generation so all rows conform to the 4-column schema expected by IDE config tooling.

Why

Installer preserved legacy rows that still contained a fifth standalone column, producing _cfg/workflow-manifest.csv files with mismatched column counts. IDE setup then failed with Invalid Record Length: columns length is 4, got 5.
Fixes #1070: #1070

How

  • Parse existing workflow-manifest rows into structured objects instead of keeping raw lines.
  • Escape/quote values and rewrite every row with exactly name,description,module,path columns.

Testing

  • Manual: reinstall BMAD locally after change; _cfg/workflow-manifest.csv now matches header and IDE setup proceeds without CSV parse errors.

@coderabbitai
Copy link

coderabbitai bot commented Dec 8, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

@dickymoore dickymoore marked this pull request as ready for review December 8, 2025 21:40
@bmadcode
Copy link
Collaborator

thanks @dickymoore !

@bmadcode bmadcode merged commit ed0defb into bmad-code-org:main Dec 11, 2025
12 checks passed
bidadh pushed a commit to nextgendev-solutions/BMAD-METHOD-nextgen that referenced this pull request Dec 15, 2025
* fix: normalize workflow manifest schema

* fix: escape workflow manifest values safely

---------

Co-authored-by: Brian <bmadcode@gmail.com>
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.

Installer writes invalid workflow-manifest.csv schema

2 participants