From 398860fb07caf59415399ca33f7bbd2d4396b874 Mon Sep 17 00:00:00 2001 From: 2-gabadi Date: Sun, 29 Mar 2026 18:40:57 -0300 Subject: [PATCH 1/2] fix: quick-dev self-review verifies AC coverage against parent artifact requirements Co-Authored-By: Claude Opus 4.6 (1M context) --- .../4-implementation/bmad-quick-dev/spec-template.md | 3 ++- src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/bmm-skills/4-implementation/bmad-quick-dev/spec-template.md b/src/bmm-skills/4-implementation/bmad-quick-dev/spec-template.md index 3f70a51340..637e3b67f2 100644 --- a/src/bmm-skills/4-implementation/bmad-quick-dev/spec-template.md +++ b/src/bmm-skills/4-implementation/bmad-quick-dev/spec-template.md @@ -54,7 +54,8 @@ context: [] # optional: max 3 project-wide standards/docs. NO source code files. - + **Execution:** - [ ] `FILE` -- ACTION -- RATIONALE diff --git a/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md b/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md index 361d4c5665..48e32a5a34 100644 --- a/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md +++ b/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md @@ -14,7 +14,7 @@ deferred_work_file: '{implementation_artifacts}/deferred-work.md' 1. Investigate codebase. _Isolate deep exploration in sub-agents/tasks where available. To prevent context snowballing, instruct subagents to give you distilled summaries only._ 2. Read `./spec-template.md` fully. Fill it out based on the intent and investigation, and write the result to `{wipFile}`. -3. Self-review against READY FOR DEVELOPMENT standard. +3. Self-review against READY FOR DEVELOPMENT standard. If the intent derives from a parent artifact (epic plan, PRD, story brief), verify that every requirement assigned to this scope has a corresponding AC — architectural mechanisms do not substitute for testable acceptance criteria. 4. If intent gaps exist, do not fantasize, do not leave open questions, HALT and ask the human. 5. Token count check (see SCOPE STANDARD). If spec exceeds 1600 tokens: - Show user the token count. From d80d05ad3019c6b1da9f9d2eec4f052e499a8860 Mon Sep 17 00:00:00 2001 From: 2-gabadi Date: Sun, 29 Mar 2026 18:47:46 -0300 Subject: [PATCH 2/2] fix: remove redundant editorial clause from self-review instruction Co-Authored-By: Claude Opus 4.6 (1M context) --- src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md b/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md index 48e32a5a34..a55ae66e5a 100644 --- a/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md +++ b/src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md @@ -14,7 +14,7 @@ deferred_work_file: '{implementation_artifacts}/deferred-work.md' 1. Investigate codebase. _Isolate deep exploration in sub-agents/tasks where available. To prevent context snowballing, instruct subagents to give you distilled summaries only._ 2. Read `./spec-template.md` fully. Fill it out based on the intent and investigation, and write the result to `{wipFile}`. -3. Self-review against READY FOR DEVELOPMENT standard. If the intent derives from a parent artifact (epic plan, PRD, story brief), verify that every requirement assigned to this scope has a corresponding AC — architectural mechanisms do not substitute for testable acceptance criteria. +3. Self-review against READY FOR DEVELOPMENT standard. If the intent derives from a parent artifact (epic plan, PRD, story brief), verify that every requirement assigned to this scope has a corresponding AC. 4. If intent gaps exist, do not fantasize, do not leave open questions, HALT and ask the human. 5. Token count check (see SCOPE STANDARD). If spec exceeds 1600 tokens: - Show user the token count.