General-purpose coding policy for Baruch's AI agents
95
91%
Does it follow best practices?
Impact
96%
1.31xAverage score across 10 eval scenarios
Advisory
Suggest reviewing before use
REQUEST_CHANGES — reviewers exit without reading the diff rather than silently reviewing as human, because an unannounced AI author is the exact case the rule exists to catch**Author-Model:** line to the PR body, near the top:
**Author-Model:** claude-opus-4-7Co-authored-by: git trailer on any commit in the PR (Claude Code emits this by default)**Author-Model:** human**Author-Model:** human claude-opus-4-7Author-Model: wins when both exist (explicit beats implicit)Claude Opus 4.7 → claude-opus-4-7); unknown display names are still accepted as ad-hoc model IDs so a trailer is never silently rejectedREQUEST_CHANGES before reading the diff (no degraded-fallback review), so the missing declaration blocks the PR until it's addedclaude-* → anthropicgpt-*, codex-* → openaigemini-* → google**Author-Model:** human claude-opus-4-7 is treated as claude-authored for family-mismatch purposes; the openai-family reviewer runs, the claude-family reviewer skipsgpt-5.4 claude-opus-4-7), no cross-family reviewer is available — both run as a degraded fallback rather than leaving the PR unreviewed. Author teams that want to preserve cross-family review should avoid co-authoring across both paired families on the same PRgemini-2.5, a custom model name, or human-only), both paired reviewers run. Both ARE cross-family relative to the author, so neither is biased; the duplicate review is accepted noise. Picking one reviewer arbitrarily would silently halve coverage on PRs from other-family or human-only authors, which is worse than the duplication. The neither-family case is therefore deliberate, not an oversight in the gate logic