[Bug] block() function leaves stdin in raw mode on Windows, breaking Ctrl+C signal handling
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 42/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- node.js, typescript
- Ambito
- cli
Direzione di ricerca
Inizia con il codice di cleanup di block() di @clack/prompts e il flusso di spinner.stop(), in particolare la condizione raw-mode specifica per Windows. Esegui la riproduzione minima collegata su Windows e verifica se Ctrl+C genera SIGINT dopo l’arresto dello spinner. Il lavoro è completo quando stdin viene ripristinato senza regredire il comportamento di Windows associato a issue #176.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Environment
- OS: Windows 11 (MSYS_NT-10.0-26100)
- Node Version: v22.20.0
- Package: @clack/prompts
- Package Version: v1.0.0-alpha.6
Describe the bug
After calling spinner.stop(), stdin remains in raw mode on Windows, preventing Ctrl+C from generating SIGINT signals. This breaks signal handling for any code that runs after the spinner stops.
The root cause is in the block() function's cleanup code, which intentionally skips restoring raw mode on Windows to prevent issue #176:
// From clack source - block() cleanup
if (input instanceof ReadStream && input.isTTY && !isWindows) {
input.setRawMode(false);
}
This leaves stdin stuck in raw mode on Windows. In raw mode, Ctrl+C sends raw keypress data instead of generating SIGINT signals, so signal handlers never fire.
To Reproduce
Link to minimal reproduction:
https://stackblitz.com/edit/node-3wkqdznz?file=index.js,package.json
Steps to reproduce:
- Run the code above on Windows
- Wait for spinner to stop
- Press Ctrl+C
- Bug: Process hangs - SIGINT handler never fires because stdin is in raw mode
Expected behavior
Pressing Ctrl+C should:
- Generate a SIGINT signal
- Trigger the signal handler
- Print "✓ SIGINT received!"
- Exit the process
This works correctly on macOS/Linux but fails on Windows due to the !isWindows condition.
Additional Information
Workaround: Manually restore stdin after stopping the spinner:
spinner.stop("Done");
if (process.stdin.isTTY) {
process.stdin.setRawMode(false);
}
If there's a Windows-specific reason to keep raw mode during spinner execution, I'm unsure of the reason. This is causing my issues on Windows.
- Lingua principale
- TypeScript
- Stelle
- 8.1k
- Fork
- 217
- Merge medio
- 1g 11h
- PR unite (30g)
- 5
Guida per i contributori
Apri la guida per i contributori
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 bombshell-dev/clack
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
bombshell-dev/clack#597 ·
-
rfc: accessible mode Apertaenhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
bombshell-dev/clack#585 · 1 commento · 1 reazione ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
bombshell-dev/clack#573 · 1 commento · 2 reazioni ·
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
bombshell-dev/clack#565 · 1 commento · 1 reazione ·
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
bombshell-dev/clack#564 · 1 reazione ·
Tutte le issue di bombshell-dev/clack
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
mksglu/context-mode#1200 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
jaegertracing/jaeger-ui#4506 ·
-
area:desktop area:ui bug platform:macos
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
anthropics/claude-code#96687 ·
-
good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
AOSSIE-Org/DebateAI#582 · 2 commenti ·