mirror of
https://github.com/bmad-code-org/BMAD-METHOD.git
synced 2026-09-01 15:09:07 +08:00
50b3238abb
* feat(review): add verification-gap reviewer as a third review layer Add bmad-review-verification-gap, a code-review layer orthogonal to the adversarial bug-hunter and edge-case hunter. It asks whether a regression in the changed behavior would be caught by a test where that behavior is actually exercised — never whether the code is wrong — and reports verification gaps with no severity (the triage step owns that). Wire it into the review steps of code-review, quick-dev, and dev-auto, and register it in the plugin marketplace. The code-review triage folds its findings into the unified normalization (source vgap); the triage step's per-layer format enumeration is dropped, since the triage agent sees the actual layer outputs and a description of them adds nothing. Per external review feedback: absence claims are gated on a repo-wide symbol and import-reference search, and the report's evidence field must show the searches run when it claims no test exists. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * docs(review): tighten verification-gap reviewer prompt * docs(review): apply decisions-log refinements to verification-gap reviewer Fold two decisions from docs/test-gap-hunter/decisions-log.md and follow-up review into the reviewer prompt: - Missing-adoption now qualifies on a supersession signal plus a shared observable contract, not the adoption failure alone; missing coverage of the non-adoption is the gap itself, not a disqualifier. Step 4 routes it as its own outcome so it is not collapsed into the regression-style gaps. - Findings report what was actually checked ("none of the tests I read cover this") and how far, and claim a test exists nowhere only when the symbol/import search establishes it. The gap definitions and Consequence stay conceptual; the how-far-verified qualification lives in the report fields. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>