gracefulKill is a no-op on Windows: node-pty kill(signal) throws and is swallowed
維護者通常 1 天內回覆
已經有一個關聯 PR 被合併了。
- #985 來自 @dormouse-bot —— 已合併
評估
- 難度
- 3/5
- 預估耗時
- 1-2 天
- 新手友好度
- 62/100
- Issue 類型
- 缺陷
- 描述清晰度
- 基本清楚
- 活躍度
- 活躍
- 技術堆疊
- typescript, vscode
- 領域
- cli
研究方向
Start in standalone/sidecar/pty-core.js at pty:gracefulKill, then trace the VS Code deactivate path that calls gracefulKill. Reproduce the Windows behavior with a fake node-pty whose kill(signal) throws, and add coverage for both affected hosts. Done means Windows shutdown sends a graceful stop, avoids swallowing the failure, and does not wait the full two-second timeout before exit.
由索引模型根據 Issue 內容生成。
描述
Found during review of #983 (standalone/sidecar/pty-core.js gracefulKill).
On Windows, node-pty's kill(signal) throws for a signal argument. gracefulKill wraps it in a try/catch that swallows the error, so no PTY receives a graceful stop. The function then polls for the full timeout (2 s) before the hard kill. Both hosts are affected: standalone (pty:gracefulKill) and VS Code (gracefulKill on deactivate).
Effect: on Windows, quit and deactivate always wait out the grace period, and shells never get a chance to flush or save history.
Likely fix: on Windows, use the ConPTY close path or send Ctrl+C / Ctrl+Break through the console instead of kill(signal). Then shorten the wait to the actual exit. A test with a fake node-pty whose kill(signal) throws should go red on main.
🤖 Generated with Claude Code
- 主要語言
- TypeScript
- 星號
- 5
- 分支
- 1
- 平均合併
- 11 小時 45 分鐘
- 30 天內合併 PR
- 478
環境準備
這個專案沒有提供開發容器、Dockerfile 或貢獻指南，環境需要你自己搭建：先看它的 README，通用步驟見我們的新手貢獻指南。
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
diffplug/dormouse 的其他 Issue
-
難度 3/5 半天 新手友好度 25/100
diffplug/dormouse#1393 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 5/5 一週以上 新手友好度 1/100
diffplug/dormouse#1383 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 3/5 1-2 天 新手友好度 35/100
diffplug/dormouse#1373 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 4/5 1-2 天 新手友好度 25/100
diffplug/dormouse#1235 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 4/5 1-2 天 新手友好度 48/100
diffplug/dormouse#1234 · 1 則留言 ·
維護者通常 1 天內回覆
查看 diffplug/dormouse 的全部 Issue
相似的 Issue
-
難度 2/5 1-3 小時 新手友好度 76/100
marmelab/react-admin#11431 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 88/100
db-ux-design-system/core-web#8655 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 74/100
維護者通常 1 天內回覆
-
bug
難度 2/5 1-3 小時 新手友好度 76/100
stripe/react-stripe-js#728 ·
-
[Upload Grade C dataset] the pop-up changes it's size while uploading the document => Jumping effect未關閉bug
難度 2/5 1-3 小時 新手友好度 66/100
epam/statgpt-admin-frontend#305 ·
維護者通常 1 天內回覆