2 Commits

Author	SHA1	Message	Date
Garry Tan	9ca8f1d7a9	feat: adaptive gating + cross-review dedup for review army (v0.15.2.0) (#760) ... * feat: add test_stub optional field to specialist finding schema 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> * feat: adaptive gating + test framework detection for review army 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> * feat: cross-review finding dedup + test stub override + enriched review-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> * chore: bump version and changelog (v0.15.2.0) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: bash operator precedence in test framework detection [ -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> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2026-04-04 22:46:21 -07:00
Garry Tan	a4a181ca92	feat: Review Army — parallel specialist reviewers for /review (v0.14.3.0) (#692) ... * feat: extend gstack-diff-scope with SCOPE_MIGRATIONS, SCOPE_API, SCOPE_AUTH Three new scope signals for Review Army specialist activation: - SCOPE_MIGRATIONS: db/migrate/, prisma/migrations/, alembic/, *.sql - SCOPE_API: *controller*, *route*, *endpoint*, *.graphql, openapi.* - SCOPE_AUTH: *auth*, *session*, *jwt*, *oauth*, *permission*, *role* * feat: add 7 specialist checklist files for Review Army - testing.md (always-on): coverage gaps, flaky patterns, security enforcement - maintainability.md (always-on): dead code, DRY, stale comments - security.md (conditional): OWASP deep analysis, auth bypass, injection - performance.md (conditional): N+1 queries, bundle impact, complexity - data-migration.md (conditional): reversibility, lock duration, backfill - api-contract.md (conditional): breaking changes, versioning, error format - red-team.md (conditional): adversarial analysis, cross-cutting concerns All use standard header with JSON output schema and NO FINDINGS fallback. * feat: Review Army resolver — parallel specialist dispatch + merge New resolver in review-army.ts generates template prose for: - Stack detection and specialist selection - Parallel Agent tool dispatch with learning-informed prompts - JSON finding collection, fingerprint dedup, consensus highlighting - PR quality score computation - Red Team conditional dispatch Registered as REVIEW_ARMY in resolvers/index.ts. * refactor: restructure /review template for Review Army - Replace Steps 4-4.75 with CRITICAL pass + {{REVIEW_ARMY}} - Remove {{DESIGN_REVIEW_LITE}} and {{TEST_COVERAGE_AUDIT_REVIEW}} (subsumed into Design and Testing specialists respectively) - Extract specialist-covered categories from checklist.md - Keep CRITICAL + uncovered INFORMATIONAL in main agent pass * test: Review Army — 14 diff-scope tests + 7 E2E tests - test/diff-scope.test.ts: 14 tests for all 9 scope signals - test/skill-e2e-review-army.test.ts: 7 E2E tests Gate: migration safety, N+1 detection, delivery audit, quality score, JSON findings Periodic: red team, consensus - Updated gen-skill-docs tests for new review structure - Added touchfile entries and tier classifications * docs: update SELF_LEARNING_V0.md with Release 2 status + Release 2.5 Mark Release 2 (Review Army) as in-progress. Add Release 2.5 for deferred expansions (E1 adaptive gating, E3 test stubs, E5 cross-review dedup, E7 specialist tracking). * chore: bump version and changelog (v0.14.3.0) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-30 22:07:50 -06:00