expo / expo/code-review-cli

run with all passes failed still reports Approve with comments

Aperta
#55 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.