Skip to content

fix: quick-dev self-review verifies AC coverage against parent requirements #2168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
gabadi wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: quick-dev self-review verifies AC coverage against parent requirements #2168

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
398860f
fix: quick-dev self-review verifies AC coverage against parent artifa…
gabadi Mar 29, 2026
d80d05a
fix: remove redundant editorial clause from self-review instruction
gabadi Mar 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion src/bmm-skills/4-implementation/bmad-quick-dev/spec-template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,8 @@ context: [] # optional: max 3 project-wide standards/docs. NO source code files.

<!-- Tasks: backtick-quoted file path -- action -- rationale. Prefer one task per file; group tightly-coupled changes when splitting would be artificial. -->
<!-- If an I/O Matrix is present, include a task to unit-test its edge cases. -->
<!-- AC covers system-level behaviors not captured by the I/O Matrix. Do not duplicate I/O scenarios here. -->
<!-- AC covers system-level behaviors not captured by the I/O Matrix. Do not duplicate I/O scenarios here.
If requirements were assigned from a parent artifact, every assigned requirement must have an AC here. -->

**Execution:**
- [ ] `FILE` -- ACTION -- RATIONALE
Expand Down
2 changes: 1 addition & 1 deletion src/bmm-skills/4-implementation/bmad-quick-dev/step-02-plan.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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.
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.
Expand Down
Loading