You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
💡 AI Code Review Risk-Stratified Human Escalation & Confidence Scoring Standard
#1182
Define a standard for classifying PR changes by security/business risk category and requiring explicit human review attestation when changes fall in categories where AI code review tools have demonstrated blind spots — business logic, auth flows, race conditions, and financial calculations. Includes confidence scoring guidelines for interpreting AI review findings and preventing rubber-stamp bias.
Market Signal
Independent 2026 benchmarks of 309 pull requests show dramatic variability in AI code review bug detection: CodeRabbit 44%, GitHub Copilot 54%, Greptile 82%. All tools consistently miss business logic errors, authorization flow flaws, and race conditions. The OWASP Top 10 for Agentic Applications explicitly identifies "Human-Agent Trust" (ASI09) as a top-10 risk — teams over-trusting AI review output and reducing their own scrutiny. A separate analysis notes AI code review creates cognitive bias: "A green AI review creates cognitive bias where developers skip their own scrutiny."
The org uses both CodeRabbit and Copilot code review (AGENTS.md CI Quality Gates section) and requires addressing their comments. The Dependabot auto-merge workflow auto-resolves AI reviewer threads on patch/minor bumps — appropriate for dependency updates but establishing a pattern of AI-review-as-sufficient. The pr-quality ruleset requires ALL review threads resolved before merge but doesn't distinguish AI confidence levels. No existing standard classifies which change types require human-attentive review vs. AI-sufficient review.
Existing ideas #976 (Multi-Tool Arbitration) and #1147 (Resolution Feedback Loop) address routing between tools and feedback quality, but neither addresses risk-based escalation — requiring different review rigor for different change categories.
Technical Opportunity
The org's pr-auto-review system and pr-quality ruleset provide the enforcement mechanism. Implementation path:
Risk taxonomy — Classify file paths and change patterns: auth/, permissions/, .github/workflows/, **/middleware/**, financial logic → high-risk (explicit human attestation required). Infrastructure, tests, docs → standard (AI review sufficient with normal 1-reviewer gate).
CI labeling step — A lightweight step that tags PRs by risk category based on files changed, using the same changed-files pattern the org already employs.
CODEOWNERS routing — High-risk categories require security-team or domain-expert approval beyond the standard 1-reviewer gate.
Review attestation — For high-risk PRs, require a reviewer to explicitly acknowledge they reviewed the flagged areas (not just approved the PR overall).
Assessment
Dimension
Score
Rationale
Feasibility
med
Requires defining the risk taxonomy and wiring it into the existing pr-auto-review/pr-quality infrastructure. No new external tools needed.
Impact
high
Directly addresses the #1 cause of missed bugs in AI-reviewed PRs: human reviewers trusting AI output and reducing their own scrutiny on security-critical changes.
Urgency
med
The risk grows as AI review adoption increases. The org already auto-resolves AI threads for Dependabot — the pattern could spread to other PR types without guardrails.
Adversarial Review
Strongest objection: The org already requires 1 human reviewer via branch protection. Adding risk-stratified escalation creates process overhead and slows velocity. AI review accuracy will improve over time, making this standard obsolete.
Rebuttal: The 1-reviewer requirement doesn't account for rubber-stamp bias — research shows reviewers spend less time reviewing when AI says "looks good." Risk stratification is targeted, not blanket: only auth, permissions, and financial logic changes trigger escalation, which represent a small fraction of PRs. Even if AI accuracy improves from 44% to 80%, business logic and security-critical changes should always have attentive human review — this is a well-established security principle (defense-in-depth), not a bet on AI staying weak. The standard's risk categories can evolve as AI capabilities improve — it's a framework, not a frozen rule.
Suggested Next Step
Define the initial risk category taxonomy (file paths and change patterns that trigger escalation), prototype a CI labeling step that classifies PRs by risk level, and validate against the last 30 merged PRs to confirm the high-risk category captures the right subset without over-triggering.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
Summary
Define a standard for classifying PR changes by security/business risk category and requiring explicit human review attestation when changes fall in categories where AI code review tools have demonstrated blind spots — business logic, auth flows, race conditions, and financial calculations. Includes confidence scoring guidelines for interpreting AI review findings and preventing rubber-stamp bias.
Market Signal
Independent 2026 benchmarks of 309 pull requests show dramatic variability in AI code review bug detection: CodeRabbit 44%, GitHub Copilot 54%, Greptile 82%. All tools consistently miss business logic errors, authorization flow flaws, and race conditions. The OWASP Top 10 for Agentic Applications explicitly identifies "Human-Agent Trust" (ASI09) as a top-10 risk — teams over-trusting AI review output and reducing their own scrutiny. A separate analysis notes AI code review creates cognitive bias: "A green AI review creates cognitive bias where developers skip their own scrutiny."
Sources: Independent 2026 AI Code Review Benchmark, CodeRabbit vs Copilot Comparison, OWASP Agentic AI Top 10 — ASI09, Best AI Code Review Tools 2026
User Signal
The org uses both CodeRabbit and Copilot code review (AGENTS.md CI Quality Gates section) and requires addressing their comments. The Dependabot auto-merge workflow auto-resolves AI reviewer threads on patch/minor bumps — appropriate for dependency updates but establishing a pattern of AI-review-as-sufficient. The
pr-qualityruleset requires ALL review threads resolved before merge but doesn't distinguish AI confidence levels. No existing standard classifies which change types require human-attentive review vs. AI-sufficient review.Existing ideas #976 (Multi-Tool Arbitration) and #1147 (Resolution Feedback Loop) address routing between tools and feedback quality, but neither addresses risk-based escalation — requiring different review rigor for different change categories.
Technical Opportunity
The org's pr-auto-review system and pr-quality ruleset provide the enforcement mechanism. Implementation path:
auth/,permissions/,.github/workflows/,**/middleware/**, financial logic → high-risk (explicit human attestation required). Infrastructure, tests, docs → standard (AI review sufficient with normal 1-reviewer gate).changed-filespattern the org already employs.Assessment
Adversarial Review
Strongest objection: The org already requires 1 human reviewer via branch protection. Adding risk-stratified escalation creates process overhead and slows velocity. AI review accuracy will improve over time, making this standard obsolete.
Rebuttal: The 1-reviewer requirement doesn't account for rubber-stamp bias — research shows reviewers spend less time reviewing when AI says "looks good." Risk stratification is targeted, not blanket: only auth, permissions, and financial logic changes trigger escalation, which represent a small fraction of PRs. Even if AI accuracy improves from 44% to 80%, business logic and security-critical changes should always have attentive human review — this is a well-established security principle (defense-in-depth), not a bet on AI staying weak. The standard's risk categories can evolve as AI capabilities improve — it's a framework, not a frozen rule.
Suggested Next Step
Define the initial risk category taxonomy (file paths and change patterns that trigger escalation), prototype a CI labeling step that classifies PRs by risk level, and validate against the last 30 merged PRs to confirm the high-risk category captures the right subset without over-triggering.
All reactions