Skip to content

code-review workflow 2 x BUG - Missing checklist.md file and Step 5 (marks a story as "done") doesn't update sprint-status.yaml (V6 alpha.13) #1015

Description

@radaco99

Describe the bug

Two issues found in the bmm module's code-review workflow:

Bug 1: Missing checklist.md file
The workflow.yaml references validation: "{installed_path}/checklist.md" but no such file exists in the bmm
module.

Bug 2: Step 5 doesn't update sprint-status.yaml
When code-review marks a story as "done", it only updates the story file - not the sprint-status.yaml tracking
file. This causes sprint tracking to become out of sync with actual story status.

Steps to Reproduce

Bug 1:

  1. Install BMAD v6 alpha.13: npx bmad-method@alpha install
  2. Select bmm module
  3. Run code-review workflow
  4. Workflow fails reading .bmad/bmm/workflows/4-implementation/code-review/checklist.md

Bug 2:

  1. Complete a story via dev-story workflow
  2. Run code-review workflow and approve (mark story as "done")
  3. Check sprint-status.yaml → still shows old status (e.g., "review" instead of "done")

PR

Not currently working on a fix, but happy to contribute. Proposed fixes:

Bug 1: Copy checklist.md from bmgd module to bmm module:

  • Source: src/modules/bmgd/workflows/4-production/code-review/checklist.md
  • Target: src/modules/bmm/workflows/4-implementation/code-review/checklist.md

Bug 2: Update instructions.xml Step 5 to include sprint-status.yaml sync:

...existing actions...

<!-- CRITICAL: Keep sprint tracking in sync with story status -->
<action>Update {sprint_artifacts}/sprint-status.yaml to match new story status</action>
<action>Update the story line with:
  - New status (done, in-progress, etc.)
  - Code review approval note with date
  - Summary of fixes applied (if any)
</action>

Expected behavior

Bug 1: Workflow loads checklist.md successfully for validation.

Bug 2: When code-review marks a story "done", both the story file AND sprint-status.yaml are updated together.

Please be Specific if relevant

  • Model(s) Used: Claude Sonnet 4 / Claude Opus 4
  • Agentic IDE Used: Claude Code CLI
  • WebSite Used: N/A
  • Project Language: TypeScript (React + Express monorepo)
  • BMad Method version: v6.0.0-alpha.13

Screenshots or Links

Missing checklist.md in bmm (only 2 files exist):
https://github.com/bmad-code-org/BMAD-METHOD/tree/main/src/modules/bmm/workflows/4-implementation/code-review

workflow.yaml referencing missing file (line 21):
validation: "{installed_path}/checklist.md"

Existing checklist in bmgd that could be copied:
https://github.com/bmad-code-org/BMAD-METHOD/blob/main/src/modules/bmgd/workflows/4-production/code-review/checklist.md

Additional context

Workarounds applied:

  1. Manually created checklist.md by copying content from bmgd module
  2. Updated instructions.xml Step 5 to sync sprint-status.yaml

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions