feat: adaptive gating + cross-review dedup for review army (v0.15.2.0)#760
Open
feat: adaptive gating + cross-review dedup for review army (v0.15.2.0)#760
Conversation
All specialist prompts now document test_stub as an optional output field, enabling specialists to suggest test code alongside findings. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Adds gstack-specialist-stats binary for tracking specialist hit rates. Resolver now detects test framework for test_stub generation, applies adaptive gating to skip silent specialists, and compiles per-specialist stats for the review-log entry. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ew-log Step 5.0 suppresses findings previously skipped by the user when the relevant code hasn't changed. Test stub findings force ASK classification so users approve test creation. Review-log now includes quality_score, per-specialist stats, and per-finding action records. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
[ -f a ] || [ -f b ] && X="y" evaluates as A || (B && C), so the
assignment only runs when the second test passes. Wrap the OR group
in braces: { [ -f a ] || [ -f b ]; } && X="y".
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
E2E Evals: ✅ PASS5/5 tests passed | $.47 total cost | 12 parallel runners
12x ubicloud-standard-2 (Docker: pre-baked toolchain + deps) | wall clock ≈ slowest suite |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Reviews now learn from your decisions and get smarter over time.
--security,--performance, etc.Files Changed
review/specialists/*.md— addtest_stuboptional field to all specialist schemasreview/design-checklist.md— documenttest_stubfieldscripts/resolvers/review-army.ts— test framework detection, adaptive gating logic, per-specialist statsreview/SKILL.md.tmpl— Step 5.0 cross-review dedup, test stub override in Step 5a, enriched review-logreview/SKILL.md— regeneratedbin/gstack-specialist-stats— new binary for specialist hit rate trackingTest Coverage
All new code paths are prompt template logic (natural language instructions to Claude) and a shell script. No app-level code paths to unit test. Existing test suite passes (all assertions green).
Pre-Landing Review
No issues found. Changes are prompt templates + infrastructure only. No SQL, auth, or trust boundary changes.
Test plan
🤖 Generated with Claude Code