gracefulKill is a no-op on Windows: node-pty kill(signal) throws and is swallowed
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 62/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript, vscode
- Ambito
- cli
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- TypeScript
- Stelle
- 5
- Fork
- 1
- Merge medio
- 11h 51m
- PR unite (30g)
- 341
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di diffplug/dormouse
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
diffplug/dormouse#912 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
diffplug/dormouse#1007 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
diffplug/dormouse#968 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 42/100
diffplug/dormouse#910 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
review-runs-tracking
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
diffplug/dormouse#881 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di diffplug/dormouse
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Comfy-Org/ComfyUI_frontend#20346 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
decentralized-identity/didwebvh-ts#203 ·
I maintainer di solito rispondono entro 1 giorno
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
lingdojo/kana-dojo#31791 · 1 commento · 5 reazioni ·
I maintainer di solito rispondono entro 1 giorno
-
Telegram webhook: line breaks lost since switch to rich messagesForse già presa @Kshot3000 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
github_actions security
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno