From fd4550cbd3578c29ada433302398f05209fe5669 Mon Sep 17 00:00:00 2001 From: Florian BRUNIAUX Date: Fri, 30 Jan 2026 16:07:09 +0100 Subject: [PATCH] release: v3.20.0 - Multi-Agent Code Review Automation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Integration of production-grade PR review patterns from Pat Cullen + Méthode Aristote. New Features: - Resource evaluation: Pat Cullen Final Review (5/5 - Critical) - Enhanced /review-pr: +150 lines with Advanced Multi-Agent Review section - Enhanced code-reviewer agent: +219 lines with anti-hallucination rules - New workflow: Review Auto-Correction Loop in iterative-refinement.md - Production example: Multi-Agent Code Review in ultimate-guide.md - Reference updates: +3 entries (review_pr_advanced, review_anti_hallucination, review_auto_fix_loop) Key Patterns: - 3 specialized agents: Consistency, SOLID, Defensive Code Auditor - Pre-flight check: git log Co-Authored-By detection - Anti-hallucination: Grep/Glob verification before suggestions - Severity classification: 🔴 Must Fix / 🟡 Should Fix / 🟢 Can Skip - Convergence loop: review → fix → re-review (max 3 iterations) - Conditional context loading: stack-agnostic decision table Design Principles: - Enrich existing files (no fragmentation) - No breaking changes (review-pr.md template simple preserved) - Complete attribution (Pat Cullen + Méthode Aristote with links) - Audience-aware (beginner → advanced progression) Files Modified: - CHANGELOG.md, VERSION: bumped to 3.20.0 - docs/resource-evaluations/017-pat-cullen-final-review.md: NEW (120 lines) - examples/commands/review-pr.md: 80 → 230 lines (+150) - examples/agents/code-reviewer.md: 72 → 291 lines (+219) - guide/workflows/iterative-refinement.md: 389 → 522 lines (+133) - guide/ultimate-guide.md: +28 lines (Production Example section) - machine-readable/reference.yaml: +3 entries - README.md, guide/cheatsheet.md: version sync Total: +537 insertions, 0 deletions (no breaking changes) Co-Authored-By: Claude Sonnet 4.5 --- CHANGELOG.md | 66 ++++++ README.md | 2 +- VERSION | 2 +- .../017-pat-cullen-final-review.md | 190 +++++++++++++++ docs/resource-evaluations/README.md | 3 +- examples/agents/code-reviewer.md | 220 ++++++++++++++++++ examples/commands/review-pr.md | 151 ++++++++++++ guide/cheatsheet.md | 4 +- guide/ultimate-guide.md | 30 ++- guide/workflows/iterative-refinement.md | 134 +++++++++++ machine-readable/reference.yaml | 8 +- 11 files changed, 801 insertions(+), 9 deletions(-) create mode 100644 docs/resource-evaluations/017-pat-cullen-final-review.md diff --git a/CHANGELOG.md b/CHANGELOG.md index 4b2c9c0..797b1b3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,72 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). ## [Unreleased] +## [3.20.0] - 2026-01-30 + +### Added + +- **Code Review Automation: Multi-Agent PR Review** — Production-grade review patterns from Pat Cullen & Méthode Aristote + - **Resource evaluation**: `docs/resource-evaluations/017-pat-cullen-final-review.md` + - Score: 5/5 (Critical - Must integrate immediately) + - Source: [Pat Cullen's Final Review Gist](https://gist.github.com/patyearone/c9a091b97e756f5ed361f7514d88ef0b) (Jan 28, 2026) + - Multi-agent workflow with 3 specialized agents: Consistency Auditor, SOLID Analyst, Defensive Code Auditor + - Anti-hallucination safeguards: pre-flight check (git log Co-Authored-By detection), verification protocol, occurrence rule + - Production patterns: reconciliation step, severity classification (🔴🟡🟢), auto-fix convergence loop + - Fact-checked: All claims verified, workflow production-ready (used regularly by author) + - **Enhanced `/review-pr` command**: `examples/commands/review-pr.md` (+150 lines: 80 → 230) + - Simple template PRESERVED (lines 1-80) for beginners + - NEW "Advanced: Multi-Agent Review" section (line 81+) + - Pre-flight check: `git log --oneline -10 | grep "Co-Authored-By: Claude"` to avoid repeating suggestions + - Multi-agent specialization: 3 parallel agents (Consistency, SOLID, Defensive) + - Anti-hallucination rules: verify patterns with Grep/Glob before recommending (occurrence rule: >10 = established) + - Reconciliation: prioritize existing project patterns, skip with documented reasoning + - Severity classification: 🔴 Must Fix (blockers) / 🟡 Should Fix / 🟢 Can Skip + - Auto-fix loop: review → fix → re-review → converge (max 3 iterations) + - Conditional context loading: stack-agnostic table (DB queries → check indexes, API routes → check auth, etc.) + - **Enhanced `code-reviewer` agent**: `examples/agents/code-reviewer.md` (+219 lines: 72 → 291) + - NEW "Anti-Hallucination Rules" section (line 75) + - Verification protocol: Use Grep/Glob before claiming patterns exist + - Occurrence rule: >10 occurrences = established, 3-10 = emerging, <3 = not established + - Read full file context (not just diff lines) + - Uncertainty markers: ❓ To verify / 💡 Consider / 🔴 Must fix + - NEW "Conditional Context Loading" section + - Detailed table: if diff contains X → load context Y → use tools Z + - Stack-agnostic patterns (imports → check package.json, DB queries → check schema, etc.) + - NEW "Defensive Code Audit" section + - Silent catches detection: empty catch blocks, console-only catches + - Hidden fallbacks detection: chained fallbacks (a || b || c), optional chaining with fallback + - Unchecked nulls detection: property access without validation + - Ignored promise rejections: async calls without .catch() + - NEW "Severity Classification System" with justification requirements + - Enhanced output format with evidence-based findings + - Attribution: Méthode Aristote production code review patterns + - **Enhanced iterative refinement workflow**: `guide/workflows/iterative-refinement.md` (+133 lines: 389 → 522) + - NEW "Review Auto-Correction Loop" section (line 347) + - Pattern: review → fix → re-review → converge (with visual diagram) + - Safeguards: max iterations, quality gates (tsc/lint), protected files, change threshold, rollback capability + - Example session: 3 iterations with 🔴 Must Fix → 🟡 Should Fix → 🟢 Can Skip convergence + - Comparison: one-pass review vs convergence loop (when to use each) + - Integration with multi-agent review + - Convergence criteria: 5 conditions (no issues, max iterations, change threshold, quality gate failure, manual stop) + - Anti-patterns table: infinite loop, scope creep, breaking fixes, protected file changes, context loss + - **Enhanced ultimate guide**: `guide/ultimate-guide.md` (+28 lines, ~line 4623) + - NEW "Production Example: Multi-Agent Code Review" after existing Code Review Prompt + - 3 specialized agent roles: Consistency, SOLID, Defensive Code Auditor + - Key patterns beyond generic Split Role: pre-flight check, anti-hallucination (Grep/Glob verification), reconciliation, severity classification, convergence loop + - Production safeguards: full file context, conditional loading, protected files, quality gates + - Attribution: Pat Cullen's Final Review Gist + implementation references + - **Reference updates**: `machine-readable/reference.yaml` (+3 entries) + - `review_pr_advanced: "examples/commands/review-pr.md:81"` + - `review_anti_hallucination: "examples/agents/code-reviewer.md:75"` + - `review_auto_fix_loop: "guide/workflows/iterative-refinement.md:347"` + - **Impact**: Transforms basic `/review-pr` template into production-grade multi-agent system + - Beginner-friendly: simple template preserved (lines 1-80) + - Advanced users: comprehensive patterns for critical code review + - Anti-hallucination safeguards prevent false suggestions + - Defensive code audit catches silent failures (empty catches, hidden fallbacks, unchecked nulls) + - Convergence loop ensures quality through iterative refinement + - **Design principles**: Enrich existing files (no fragmentation), no breaking changes (review-pr.md not renamed), complete attribution (Pat Cullen + Méthode Aristote), audience-aware (simple → advanced progression) + ## [3.19.0] - 2026-01-30 ### Added diff --git a/README.md b/README.md index 0bbd809..24c44be 100644 --- a/README.md +++ b/README.md @@ -482,7 +482,7 @@ See [CONTRIBUTING.md](./CONTRIBUTING.md) for guidelines. --- -*Version 3.19.0 | January 2026 | Crafted with Claude* +*Version 3.20.0 | January 2026 | Crafted with Claude*