diff --git a/src/bmm/workflows/4-implementation/create-story/template.md b/src/bmm/workflows/4-implementation/create-story/template.md index c4e129f5f..b15c82dad 100644 --- a/src/bmm/workflows/4-implementation/create-story/template.md +++ b/src/bmm/workflows/4-implementation/create-story/template.md @@ -47,3 +47,14 @@ so that {{benefit}}. ### Completion Notes List ### File List + + + +- (list files here, one per line) diff --git a/src/bmm/workflows/4-implementation/dev-story/instructions.xml b/src/bmm/workflows/4-implementation/dev-story/instructions.xml index 3c4989f39..d2ec0cb65 100644 --- a/src/bmm/workflows/4-implementation/dev-story/instructions.xml +++ b/src/bmm/workflows/4-implementation/dev-story/instructions.xml @@ -296,8 +296,12 @@ ONLY THEN mark the task (and subtasks) checkbox with [x] - Update File List section with ALL new, modified, or deleted files (paths relative to repo root) Add completion notes to Dev Agent Record summarizing what was ACTUALLY implemented and tested + Save the story file NOW so its own changes appear in the working tree + Run `git diff --name-only`, `git diff --cached --name-only`, and `git status --porcelain` to discover ALL changed files + (including infrastructure side-effects like lock files, the story file itself, and sprint-status.yaml) + Filter out paths under `_bmad/` and `_bmad-output/` directories from the git output + REPLACE the File List section entirely with the filtered results: ALL new, modified, or deleted files (paths relative to repo root, one per line) @@ -323,7 +327,9 @@ Verify ALL tasks and subtasks are marked [x] (re-scan the story document now) Run the full regression suite (do not skip) - Confirm File List includes every changed file + Save the story file before inspecting the working tree + Run `git diff --name-only`, `git diff --cached --name-only`, and `git status --porcelain` to get actual changed files + (excluding paths under `_bmad/` and `_bmad-output/`); verify File List contains ALL reported paths and auto-append any missing entries Execute enhanced definition-of-done validation Update the story Status to: "review" @@ -336,7 +342,7 @@ - End-to-end tests for critical flows added when story demands them - All tests pass (no regressions, new tests successful) - Code quality checks pass (linting, static analysis if configured) - - File List includes every new/modified/deleted file (relative paths) + - File List matches all git-reported changes - run `git diff --name-only`, `git diff --cached --name-only`, and `git status --porcelain` to verify (exclude `_bmad/`, `_bmad-output/`) - Dev Agent Record contains implementation notes - Change Log includes summary of changes - Only permitted story sections were modified