gracefulKill is a no-op on Windows: node-pty kill(signal) throws and is swallowed
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 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時間 31分
- マージ済み PR（30日）
- 362
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
diffplug/dormouse のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
diffplug/dormouse#912 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
diffplug/dormouse#1007 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
diffplug/dormouse#968 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 42/100
diffplug/dormouse#910 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
review-runs-tracking
難易度 5/5 1週間以上 初心者へのやさしさ 10/100
diffplug/dormouse#881 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
diffplug/dormouse の issue をすべて見る
似ている issue
-
fix(data-lake): wizard source step still previews the local slug, not the server-disambiguated oneオープンdata-lake
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue priority: low size: XS
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
-
bug ios mobile priority:P1
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
streamplace/streamplace#1351 ·
メンテナーはふだん 2 日以内に返信