ccantynz/Gluecron.com
CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
Understand
Explain codebaseAsk AIChat with repoClaude sessionsNL searchSemantic searchArchaeologyDebt mapDocs tracking
Build
Spec to PRWorkspaceGenerate testsPropose migrationDev environmentCodebase migrator
Ship
Preview URLsAI release notesAI changelog
More
Repository
CommitsBranchesTagsReleasesPackagesSymbolsStale branches
Planning
MilestonesProjectsDiscussionsWikiContributors
Delivery
GatesMerge queueMerge checksDeploymentsPipelineAgents
Activity
Health reportPulsePush log
Settings

fix(gates): an AI review that never ran no longer shows "approved · Passed" #5448

Merged⚡ AI-generatedXSccantynz wants to mergefix/honest-ai-review-gate→mainopened 24d ago
ccantynzcommented 24d ago

Owner spotted the contradiction on an imported PR: the bot comment said "AI review unavailable (credit balance too low)" while the gate panel directly below said "AI review approved · Passed".

  • aiReviewGateState(): true | false | "unavailable". Fail-open policy unchanged — an unavailable review never blocks a merge — but the gate row now reports Skipped with "AI review could not run (AI service unavailable) — not blocking the merge" instead of a green "approved".
  • Branch-protection semantics untouched (unavailable fails open explicitly at both evaluateProtection call sites).
  • The "AI review unavailable" bot comment now uses humanizeAiError() instead of dumping the raw Anthropic error envelope.

tsc clean; review/gate suites green (3 unrelated pre-existing failures: deploy-gate ×2, close-keyword sweep — fail identically without this change).

🤖 Generated with Claude Code

ccantynzAI Reviewcommented 24d ago

AI Triage

(no summary)

Priority: medium Risk area: mixed

Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)

Suggestions only — nothing has been applied. The PR author stays in control.

gluecron[bot]🤖 botAI Reviewcommented 24d ago

AI review unavailable

The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts