pr-merge: enable-auto-merge drops the verdict reader's babysit-listing-miss stderr
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 70/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- github-actions, javascript, python
Direzione di ricerca
Leggi .github/scripts/pr-merge/enable-auto-merge.mjs intorno a pythonVerify e alla chiamata ghCallsLine(done.stderr), poi esamina i casi corrispondenti in enable-auto-merge.test.mjs. Segui come viene gestito stderr per i verdetti di rifiuto e aggiungi test per inoltrare le righe pertinenti del reader; il lavoro è completato quando la riga che segnala l’assenza di un elenco in un rifiuto compare nel log del job e il test copre questo comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
enable-auto-merge runs babysit_verify_evidence.py once per candidate and keeps only one line of its stderr, the babysit-gh-calls line. The reader prints a babysit-listing-miss {...} line on stderr for every incremental read that misses GitHub's live artifact count (Reader._incremental, LISTING_MISS_PREFIX). Those lines never reach the job log. When a candidate falls back to the full repository listing, the log cannot show why.
Where the line is dropped
.github/scripts/pr-merge/enable-auto-merge.mjs at 82432cf, in pythonVerify:
- L1007:
spawn("python3", ...)withencoding: "utf8"captures the reader's stderr intodone.stderr. - L1022-L1025:
ghCallsLine(done.stderr)prints only thebabysit-gh-callsline. Every other stderr line is discarded. - A refusal such as
verify:unreadableexits 0, so the error branch, which would print the last stderr line, does not run either.
Evidence
In enable-auto-merge run 40817220695, candidate #10867 logged:
enable-auto-merge: #10867 calls=128 github-token=124 paginated=0 ms=107111 cached=0
babysit-gh-calls {"calls":{..."override GET repos/{}/{}/actions/artifacts":7,"override GET {}/{}/actions/artifacts":117},...}
enable-auto-merge: #10867 left: pr-merge's gate refuses its verifier verdict (verify:unreadable)
That is 7 incremental page calls, then 117 full-listing pages over 107 s, ending in a refusal. No babysit-listing-miss line appears anywhere in the job log. Three possible causes remain open: the incremental reads missed the live count, a page failed, or the base was rejected.
Fix
Print each babysit-listing-miss line from done.stderr beside the babysit-gh-calls line. Better: print every stderr line except the gh-calls line when the reader's verdict is a refusal. enable-auto-merge.test.mjs would need a case for it. Note that #10743 also edits that test file.
Refs: #10880
- Lingua principale
- Shell
- Stelle
- 22
- Fork
- 2
- Merge medio
- 3h 38m
- PR unite (30g)
- 959
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di melodic-software/claude-code-plugins
-
agent-ready priority: medium status: ready work-class: mechanical
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
melodic-software/claude-code-plugins#10804 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
agent-ready priority: medium status: ready work-class: scoped
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
melodic-software/claude-code-plugins#10227 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
admission-deferred priority: medium status: ready work-class: scoped
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
melodic-software/claude-code-plugins#10128 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is openForse già presa @melodic-automation-lanes l’ha presa oggi. Apertapriority: medium status: needs-decision work-class: scoped
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
melodic-software/claude-code-plugins#10074 · 4 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
needs-human priority: medium status: ready work-class: scoped
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
melodic-software/claude-code-plugins#9667 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di melodic-software/claude-code-plugins
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
pypa/advisory-database#412 ·
I maintainer di solito rispondono entro 1 giorno
-
GEFSv12
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100