run with all passes failed still reports Approve with comments
- Lingua principale
- TypeScript
- Stelle
- 1
- Fork
- 0
- Merge medio
- 1g 21h
- PR unite (30g)
- 2
Descrizione
Seen on 0.9.2. A run where every review pass failed (subscription session limit) still prints the verdict "AI code review — Approve with comments":
```
terraform-safety: FAILED (You've hit your session limit · resets 1:50am)
cross-repo-boundary: FAILED (...)
cluster-scaling: FAILED (...)
kubernetes-manifests: FAILED (...)
infra-security: FAILED (...)
All review passes failed — reporting an incomplete review.
Token usage — input 0, output 0, cache read 0, cache write 0
AI code review — Approve with comments
no findings
⚠️ The AI review could not complete: every review pass failed or timed out, so these changes were effectively NOT reviewed.
```
The body does say "treat this as no review", but the headline is still an approve variant. In `ecr ci` that posts a PR comment that reads as a green light unless you read the fine print, and the check stays green. When all passes fail (or coverage drops below some threshold) the verdict should be a distinct failed/incomplete state, not approve, and `ecr ci` should exit non-zero so the CI check goes red.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia dal punto di ingresso di ecr ci e traccia il modo in cui i passaggi di revisione falliti vengono convertiti nel verdetto visualizzato, nel commento della pull request e nello stato di uscita del processo. Riproduci il caso in cui tutti i passaggi sono falliti descritto nell’issue, quindi verifica che una coverage incompleta o al di sotto della soglia produca uno stato di errore distinto, un’uscita diversa da zero e un check CI rosso invece di un’approvazione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- ci-cd, cli
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 48/100