fix(cli): odd osc sequences after killing vp run with ctrl-c

[forehalo]

close#2036

the fix ignores the listening on ctrl-c by vp itself until child process exits.

[netlify]

✅ Deploy Preview for viteplus-preview canceled.

Name	Link
🔨 Latest commit	b9bf779
🔍 Latest deploy log	https://app.netlify.com/projects/viteplus-preview/deploys/6a4c8eff18eb0b000819ddb2

[Copilot]

Pull request overview

This PR addresses issue #2036 where terminating vp run with Ctrl+C can leave stray terminal escape sequences in fish by ensuring the global CLI doesn’t handle SIGINT itself while a delegated JS child is still shutting down, and by restoring terminal state around child execution. It also adds a CLI snapshot fixture and new vpt helpers to reproduce/assert Ctrl+C teardown behavior.

Changes:

• Use vite_command::execute_with_terminal_guard for JS CLI delegation so terminal state is restored and SIGINT is ignored in the parent while the child exits.
• Extend TerminalStateGuard to temporarily ignore SIGINT (Unix) and restore the prior signal disposition on drop.
• Add a new snapshot fixture plus vpt report-orphan-on-ctrlc and vpt wait-file to validate Ctrl+C teardown timing.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
crates/vite_global_cli/src/js_executor.rs	Switches JS delegation to execute_with_terminal_guard so Ctrl+C doesn’t tear down the parent before the JS child exits.
crates/vite_command/src/lib.rs	Enhances terminal guard to ignore SIGINT for the parent while waiting, restoring prior signal disposition afterward.
crates/vite_command/Cargo.toml	Enables nix signal feature for Unix builds.
crates/vite_cli_snapshots/tests/cli_snapshots/README.md	Documents new vpt subcommands used by snapshot fixtures.
crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/run_ctrlc_teardown/snapshots/run_ctrlc_exits_before_task_shutdown.md	Adds a snapshot case capturing Ctrl+C teardown behavior (expected output needs to reflect fixed behavior).
crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/run_ctrlc_teardown/snapshots.toml	Registers the new snapshot case and its steps.
crates/vite_cli_snapshots/tests/cli_snapshots/fixtures/run_ctrlc_teardown/package.json	Adds a fixture script used by vp run dev in the snapshot.
crates/vite_cli_snapshots/src/bin/vpt/wait_file.rs	Adds vpt wait-file utility to poll for and print async-written files.
crates/vite_cli_snapshots/src/bin/vpt/report_orphan_on_ctrlc.rs	Adds a Unix-only helper to detect whether graceful shutdown completed after Ctrl+C.
crates/vite_cli_snapshots/src/bin/vpt/main.rs	Wires new vpt subcommands into the multitool binary.
crates/vite_cli_snapshots/Cargo.toml	Adds Unix-only nix deps needed by the new vpt helper.
Cargo.lock	Records the new nix dependency usage for the snapshots crate.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.