From f5030c70842cfd4cc34de581635582cc74e8ff62 Mon Sep 17 00:00:00 2001 From: Alex Verkhovsky Date: Fri, 10 Apr 2026 05:53:54 -0700 Subject: [PATCH] feat(review): enforce model parity for all review subagents (#2236) Prevent review subagents from being downgraded to cheaper models. Rare findings from the Acceptance Auditor tend to be high-severity, and research shows smaller models have worse recall on rare-event detection. Co-authored-by: Claude Opus 4.6 (1M context) --- .../4-implementation/bmad-code-review/steps/step-02-review.md | 1 + .../4-implementation/bmad-quick-dev/step-04-review.md | 1 + src/bmm-skills/4-implementation/bmad-quick-dev/step-oneshot.md | 2 +- 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/bmm-skills/4-implementation/bmad-code-review/steps/step-02-review.md b/src/bmm-skills/4-implementation/bmad-code-review/steps/step-02-review.md index c262a4971..bbc1f9a82 100644 --- a/src/bmm-skills/4-implementation/bmad-code-review/steps/step-02-review.md +++ b/src/bmm-skills/4-implementation/bmad-code-review/steps/step-02-review.md @@ -10,6 +10,7 @@ failed_layers: '' # set at runtime: comma-separated list of layers that failed o - The Blind Hunter subagent receives NO project context — diff only. - The Edge Case Hunter subagent receives diff and project read access. - The Acceptance Auditor subagent receives diff, spec, and context docs. +- All review subagents must run at the same model capability as the current session. ## INSTRUCTIONS diff --git a/src/bmm-skills/4-implementation/bmad-quick-dev/step-04-review.md b/src/bmm-skills/4-implementation/bmad-quick-dev/step-04-review.md index 2e4449733..2d96fd25d 100644 --- a/src/bmm-skills/4-implementation/bmad-quick-dev/step-04-review.md +++ b/src/bmm-skills/4-implementation/bmad-quick-dev/step-04-review.md @@ -9,6 +9,7 @@ specLoopIteration: 1 - YOU MUST ALWAYS SPEAK OUTPUT in your Agent communication style with the config `{communication_language}` - Review subagents get NO conversation context. +- All review subagents must run at the same model capability as the current session. ## INSTRUCTIONS diff --git a/src/bmm-skills/4-implementation/bmad-quick-dev/step-oneshot.md b/src/bmm-skills/4-implementation/bmad-quick-dev/step-oneshot.md index 0c52d4328..c9da6c288 100644 --- a/src/bmm-skills/4-implementation/bmad-quick-dev/step-oneshot.md +++ b/src/bmm-skills/4-implementation/bmad-quick-dev/step-oneshot.md @@ -17,7 +17,7 @@ Implement the clarified intent directly. ### Review -Invoke the `bmad-review-adversarial-general` skill in a subagent with the changed files. The subagent gets NO conversation context — to avoid anchoring bias. If no sub-agents are available, write the changed files to a review prompt file in `{implementation_artifacts}` and HALT. Ask the human to run the review in a separate session and paste back the findings. +Invoke the `bmad-review-adversarial-general` skill in a subagent with the changed files. The subagent gets NO conversation context — to avoid anchoring bias. Launch at the same model capability as the current session. If no sub-agents are available, write the changed files to a review prompt file in `{implementation_artifacts}` and HALT. Ask the human to run the review in a separate session and paste back the findings. ### Classify