release:notes: a gh reply over 1 MiB aborts the run with spawnSync ENOBUFS
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 82/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- github, node.js, typescript
调研方向
从 scripts/release-notes.mjs 中的 run() 开始,检查它的 spawnSync 选项;然后查看 scripts/lib/sweep.mjs 中的 gh() 辅助函数,issue 也指出了它。添加覆盖,证明超过 1 MiB 的回复能够成功处理,并且真实的 spawn 错误会被重新抛出。当发行说明命令能够处理报告中的回复大小,且测试覆盖这两种情况时,工作即告完成。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- TypeScript
- 星标
- 91.6k
- 派生
- 11.9k
- 平均合并
- 4 小时 38 分钟
- 30 天内合并 PR
- 241
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
modelcontextprotocol/servers 的其他 Issue
-
enhancement server-everything v2
难度 2/5 1-3 小时 新手友好度 72/100
modelcontextprotocol/servers#5371 · 1 条评论 ·
维护者通常 1 天内回复
-
chore server-filesystem server-memory server-sequentialthinking v2
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5345 ·
维护者通常 1 天内回复
-
documentation v2
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5343 ·
维护者通常 1 天内回复
-
chore v2
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5337 · 1 条评论 ·
维护者通常 1 天内回复
-
release skill: link the Prepare Python Release PR to the release issue with addCloseIssueReferences未关闭documentation v2
难度 2/5 1-3 小时 新手友好度 78/100
modelcontextprotocol/servers#5335 ·
维护者通常 1 天内回复
查看 modelcontextprotocol/servers 的全部 Issue
相似的 Issue
-
fix(worker): keep normal prepare waiting states failure-free可能已有人在做 @LichKing-2234 今天认领。 未关闭agent:codex priority:P1
难度 2/5 1-3 小时 新手友好度 72/100
AgoraIO-Extensions/agent-infra#3745 ·
维护者通常 1 天内回复
-
The five-line work window can drop its oldest line: the per-line byte share ignores the JSON framing可能已有人在做 @Eapp1e 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 84/100
makecindy/cindy#7973 · 1 条评论 ·
维护者通常 1 天内回复
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
难度 1/5 1 小时以内 新手友好度 78/100
lingdojo/kana-dojo#33337 · 1 条评论 · 5 个 reaction ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
code-yeongyu/senpi#4917 · 1 条评论 ·
维护者通常 1 天内回复