fix(ai-review): mode-aware inconclusive copy + structural classification for claude-code auth errors

[JSONbored]

Summary

The other half of the inconclusive-verdict incident cleanup (sibling of #8794), closing the two alignment gaps found in the 2026-07-26 trace.

Changes

• Mode-aware copy — runLoopOverAiReview's ok-result gains plannedReviewerCount: 1 | 2 (the resolved plan's independent reviewer slots; distinct from reviewerCount, which counts produced output and is 0 on a fully-failed run). The ai_review_inconclusive finding's detail now says "dual-model" only when a genuine two-reviewer plan ran, and "one reviewer, including its provider fallback chain" otherwise. The gate-level summaries (rules/advisory.ts + engine parity twin) were already mode-neutral and are untouched.
• Structural classification — isStructuralProviderConfigError now matches claude_code_no_oauth_token / claude_code_error_401 / claude_code_error_403 alongside the codex shapes (fix(selfhost): validate AI reviewer-provider configuration and add a failure circuit breaker #2626 precedent): an expired/missing Claude Code credential is exactly as deterministic as codex_no_auth, so it earns the long structural circuit-breaker cooldown instead of burning the full retry budget on every review. claude_code_error_429 (rate limit — clears on its own, has its own break) and exit/stall shapes stay transient.

Test plan

• npm run typecheck — clean
• New tests: single-plan copy (via aiReviewCombine: "single"), dual-plan copy pinned, and 7 classifier cases (3 structural claude shapes bare + suffixed, 429/exit/stall non-matches)
• 458 tests green across ai-review/advisory/selfhost-ai suites
• Changed hunks: 0 uncovered statements/branches

Closes #8791

[superagent-security]

Superagent didn't find any vulnerabilities or security issues in this PR.

[github-actions]

Logic backtest

Replayed 0 historical case(s) for linked_issue_scope_mismatch through the base (7544cd3) and head (d2a14b8) versions of its detection logic (corpus checksum 4f53cda18c2b).

Backtest comparison: linked_issue_scope_mismatch

Verdict: unchanged — no comparable axis moved.

Advisory only — this check never blocks merge (#8105).

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	loopover-ui	2fa6c25	Commit Preview URL Branch Preview URL	Jul 26 2026, 06:26 AM

[loopover-orb]

Important

🟪🟪🟪🟪🟪🟪🟪🟪🟪🟪🟪🟪

🔍 LoopOver is reviewing…

AI analysis is in progress. This comment will update when the review is complete.

_{🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed · 🟪 Reviewing}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.26%. Comparing base (7544cd3) to head (d2a14b8).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8795      +/-   ##
==========================================
- Coverage   93.81%   92.26%   -1.55%     
==========================================
  Files         797      797              
  Lines       79606    79608       +2     
  Branches    24125    24127       +2     
==========================================
- Hits        74684    73453    -1231     
- Misses       3556     5090    +1534     
+ Partials     1366     1065     -301     

Flag	Coverage Δ
backend	92.94% <100.00%> (-2.16%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
src/queue/ai-review-orchestration.ts	100.00% <100.00%> (ø)
src/services/ai-review.ts	97.15% <100.00%> (+<0.01%)	⬆️

... and 3 files with indirect coverage changes