pr-merge: enable-auto-merge drops the verdict reader's babysit-listing-miss stderr
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 70/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions, javascript, python
Línea de trabajo
Lee .github/scripts/pr-merge/enable-auto-merge.mjs en torno a pythonVerify y a la llamada ghCallsLine(done.stderr), y luego inspecciona los casos correspondientes en enable-auto-merge.test.mjs. Sigue cómo se gestiona stderr para los veredictos de rechazo y añade cobertura para reenviar las líneas relevantes del lector; el trabajo estará terminado cuando la línea que indica que falta un listado en un rechazo aparezca en el registro del trabajo y la prueba cubra ese comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 3 h 38 min
- PR fusionados (30 d)
- 959
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de melodic-software/claude-code-plugins
-
agent-ready priority: medium status: ready work-class: mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#10804 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
agent-ready priority: medium status: ready work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
melodic-software/claude-code-plugins#10227 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Converge the awk and mjs YAML grammars before setup-apply uses yaml-subset.mjs - part 1 of 4Abiertoadmission-deferred priority: medium status: ready work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#10128 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is openPosiblemente ocupada @melodic-automation-lanes la tomó hoy. Abiertopriority: medium status: needs-decision work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
melodic-software/claude-code-plugins#10074 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
-
needs-human priority: medium status: ready work-class: scoped
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
melodic-software/claude-code-plugins#9667 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
allows image
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
DaoCloud/public-image-mirror#46272 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
kubernetes-csi/csi-driver-iscsi#485 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Euro-Office/DocumentServer#485 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
verify: just verify runs tools/verify_agents_imports.py (the recipe line was only read, not run)Abiertoneeds-verification
Dificultad 1/5 Menos de una hora Aptitud para principiantes 76/100
EchoTools/nevr-runtime#1322 ·
Los mantenedores suelen responder en 1 día