release:notes: a gh reply over 1 MiB aborts the run with spawnSync ENOBUFS
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
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github, node.js, typescript
Línea de trabajo
Empieza por run() en scripts/release-notes.mjs e inspecciona sus opciones de spawnSync; luego revisa el helper gh() en scripts/lib/sweep.mjs, que también señala el issue. Añade pruebas que demuestren que una respuesta de más de 1 MiB se procesa correctamente y que se vuelve a lanzar un error real de spawn. El trabajo estará terminado cuando el comando de notas de versión gestione el tamaño de respuesta indicado y las pruebas cubran ambos casos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
npm run release:notes aborts with release-notes: spawnSync gh ENOBUFS when any gh reply is larger than 1 MiB. run() in scripts/release-notes.mjs calls spawnSync with no maxBuffer, so Node's default of 1 MiB applies, and an overflow throws before any notes are produced. The preview and --draft both fail.
It first happened on the v2.2.0 release (#5330). The GraphQL query that lists the tree under src/ at the merge commit (expr=<merge-sha>:src) returned 1,087,532 bytes for 60200eb7. v2.1.0's tree was under the limit. The tree only grows, so every later release fails the same way.
Reproduce
npm run --silent release:notes -- --milestone v2.2.0 \
--merge-sha 60200eb73634fd2c611f60ea5bc5801d19325c89 \
--merge-branch v2/chore/5330-release-v2.2.0 --ledger-url https://example.invalid
# release notes: v2.1.0 → v2.2.0
# release-notes: spawnSync gh ENOBUFS
Expected
The run completes. run() passes a maxBuffer large enough for any reply the script reads, for example 256 MiB. A test should show that a reply over 1 MiB gets through, and that a genuine spawn error is still rethrown.
Notes
- For v2.2.0, the maintainer authorized drafting through a driver that calls the script's own
main()with aspawnthat retries an overflowing call with a larger buffer, so this fix was not required for that release. scripts/lib/sweep.mjs'sgh()helper, used by the sweeps, also callsspawnSyncwith nomaxBuffer. Check whether any of its calls can grow past 1 MiB.
- Lenguaje dominante
- TypeScript
- Estrellas
- 91.6k
- Forks
- 11.9k
- Merge medio
- 4 h 38 min
- PR fusionados (30 d)
- 241
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 modelcontextprotocol/servers
-
enhancement server-everything v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
modelcontextprotocol/servers#5371 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
chore server-filesystem server-memory server-sequentialthinking v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
modelcontextprotocol/servers#5345 ·
Los mantenedores suelen responder en 1 día
-
documentation v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
modelcontextprotocol/servers#5343 ·
Los mantenedores suelen responder en 1 día
-
chore v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
modelcontextprotocol/servers#5337 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
release skill: link the Prepare Python Release PR to the release issue with addCloseIssueReferencesAbiertodocumentation v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
modelcontextprotocol/servers#5335 ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelcontextprotocol/servers
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4860 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
sveltejs/eslint-plugin-svelte#1615 ·
Los mantenedores suelen responder en 7 días
-
Parallel reticle_verify opens per-flow leases without the source lease's ignoreHTTPSErrorsPosiblemente ocupada @as992949791 la tomó hoy. Abiertogood first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
reticlehq/reticle#2033 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día