fix(snapshots): scope registry teardown signal by fengmk2 · Pull Request #2115 · voidzero-dev/vite-plus

fengmk2 · 2026-07-09T14:52:59Z

The local-registry teardown sent SIGTERM to both the server process and
its process group (kill -TERM -). On the GitHub-hosted
ubuntu-latest PTY runs that process-group signal leaked past the case
boundary and reached the Actions runner agent, which reported "runner
received a shutdown signal" and cancelled the job. Confirmed via a
bpftrace probe on signal:signal_generate: no suite process ever sends a
SIGINT, so the runner itself was doing the cancelling in response to the
stray group SIGTERM. Scope the SIGTERM to the direct child; the server's
own handler still removes its throwaway caches and the force-kill fallback
still guarantees teardown.

netlify · 2026-07-09T14:53:04Z

✅ Deploy Preview for viteplus-preview canceled.

Name	Link
🔨 Latest commit	`52ce2b9`
🔍 Latest deploy log	https://app.netlify.com/projects/viteplus-preview/deploys/6a504ff8990052000838b009

fengmk2 · 2026-07-09T14:53:17Z

fix(snapshots): scope registry teardown signal #2115 : 2 dependent PRs (#2107 , #2120 ) 👈 (View in Graphite)
main

How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

_{An organization admin has enabled the Graphite Merge Queue in this repository.} _{Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.}

This stack of pull requests is managed by Graphite. Learn more about stacking.

The local-registry teardown sent SIGTERM to both the server process and its process group (kill -TERM <pid> -<pid>). On the GitHub-hosted ubuntu-latest PTY runs that process-group signal leaked past the case boundary and reached the Actions runner agent, which reported "runner received a shutdown signal" and cancelled the job. Confirmed via a bpftrace probe on signal:signal_generate: no suite process ever sends a SIGINT, so the runner itself was doing the cancelling in response to the stray group SIGTERM. Scope the SIGTERM to the direct child; the server's own handler still removes its throwaway caches and the force-kill fallback still guarantees teardown.

#2126 (vite-task bump) resolved the ConPTY block on Namespace's Windows runners, so move cli-snapshot-test-windows from GitHub-hosted windows-latest to namespace-profile-windows-4c-8g. The comment keeps the prior Session-0 caveat as a revert path.

graphite-app · 2026-07-10T02:26:24Z

Merge activity

Jul 10, 2:26 AM UTC: fengmk2 added this pull request to the Graphite merge queue.

Migrates all 149 legacy `vp migrate` snap-test cases (144 from `snap-tests-global`, 5 from `snap-tests`) to the PTY snapshot suite under `crates/vite_cli_snapshots`, and removes the legacy dirs. Part of #2078. Adds `redact.rs` rules for the managed-toolchain versions that appear in `vp migrate` output and bump on release: the completion banner (`Migrated . to Vite+ <version>`), the version-change table's managed rows (vite-plus, vite, vitest, `@vitest/*`, keeping the installed-source column), and the bundled vitest pins written into catalogs/overrides. The `not-supported-vite6`/`vitest3` cases stub `node_modules/<pkg>/package.json` instead of running a network `vp install`, so migrate reads the unsupported version offline. Stacked on #2115 (local-registry teardown signal fix), which the Linux snapshot job needs.

@liangmiQwQ

) Release vite-plus v0.2.5: Cleaner environments and more reliable workflows. This release adds cleanup and package version commands, supports TypeScript 7 declaration generation, reduces the standalone binary size, and hardens task tracking, CLI output, package-manager routing, Docker workflows, and project scaffolding. ### Highlights - Add `vp env clean` to remove unused managed Node.js runtimes and package-manager caches ([#2003](#2003)), by @liangmiQwQ - Add `vp pm version`, forwarding native version commands to npm, pnpm, Yarn, and Bun ([#2127](#2127)), by @jong-kyung - Add TypeScript 7 support to declaration generation and generated library projects with tsdown `0.22.7` ([#2104](#2104), [#2168](#2168)), by @wan9chi - Improve automatic task input tracking for Bun on macOS, Linux containers with constrained `/dev/shm`, and Windows process images, reduce Windows backing-file allocation, and clarify task wait failures ([vite-task#515](voidzero-dev/vite-task#515), [vite-task#518](voidzero-dev/vite-task#518), [vite-task#523](voidzero-dev/vite-task#523), [vite-task#524](voidzero-dev/vite-task#524), [vite-task#542](voidzero-dev/vite-task#542), [#2126](#2126), [#2167](#2167)), by @wan9chi - Prevent CLI diagnostics from truncating or panicking when inherited standard output or error is nonblocking ([#2172](#2172), [#2173](#2173)), by @wan9chi ### Fixes & Enhancements - Reduce standalone `vp` binary sizes by about 5% on Linux and macOS and 19% on Windows ([#2043](#2043)), by @fengmk2 - Keep the Docker image non-root by default while allowing passwordless `sudo` for Playwright browser dependencies and system packages ([#2088](#2088)), by @fengmk2 - Ignore `.env` and `.env.*` in generated projects while keeping `.env.example` trackable ([#2095](#2095)), by @forehalo - Create `vp-use.cmd` after the Windows environment bin directory is ready and document its Command Prompt-only usage ([#2128](#2128)), by @liangmiQwQ - Render user-facing process statuses and paths without Rust debug wrappers ([#2130](#2130)), by @liangmiQwQ - Make `vp dedupe` fall back to `yarn install` with a warning on Yarn Classic ([#2139](#2139)), by @jong-kyung - Ensure package-manager child processes, including Bun, can always find the managed Node.js runtime ([#2158](#2158)), by @BlankParticle ### Docs - Update the release-manager post-release guidance ([#2096](#2096)), by @wan9chi - Add Windows to the Namespace sponsor credit ([#2108](#2108)), by @fengmk2 - Document installer and runtime environment variables, registry settings, TLS options, and precedence rules ([#2114](#2114)), by @yukinoshi - Mark shell examples in `CONTRIBUTING.md` as Bash code blocks ([#2159](#2159)), by @BlankParticle ### Chore - Assign prepared release PRs to the workflow actor ([#2094](#2094)), by @wan9chi - Complete the PTY snapshot migration and harden the runner, fixtures, release-version redaction, registry teardown, timeouts, and Linux parallelism ([#2071](#2071), [#2080](#2080), [#2084](#2084), [#2099](#2099), [#2102](#2102), [#2103](#2103), [#2107](#2107), [#2109](#2109), [#2115](#2115), [#2117](#2117), [#2118](#2118), [#2119](#2119), [#2120](#2120), [#2121](#2121), [#2131](#2131), [#2132](#2132), [#2134](#2134), [#2137](#2137), [#2138](#2138), [#2141](#2141), [#2143](#2143), [#2145](#2145), [#2182](#2182)), by @fengmk2 and @wan9chi - Expose Vite+ managed binaries consistently inside snapshot fixtures ([#2110](#2110)), by @liangmiQwQ - Exclude PTY snapshot configuration from Vitest discovery and remove the legacy snapshot infrastructure ([#2135](#2135), [#2146](#2146)), by @jong-kyung - Modernize Windows runners, registry bridge publishing, native size reporting, merge-base comparisons, and package-manager installation CI ([#2083](#2083), [#2100](#2100), [#2105](#2105), [#2106](#2106), [#2112](#2112), [#2133](#2133)), by @fengmk2 - Verify that created monorepo libraries emit declarations in CI ([#2171](#2171)), by @wan9chi ### Bundled Versions | Tool | Version | Source | | --------------- | ------- | ----------------------------------------------------------------------- | | vite | `8.1.4` | [`a477454`](vitejs/vite@a477454) | | rolldown | `1.1.5` | [`f09947a`](rolldown/rolldown@f09947a) | | tsdown | `0.22.7` | [npm](https://npmx.dev/package/tsdown/v/0.22.7) | | vitest | `4.1.10` | [npm](https://npmx.dev/package/vitest/v/4.1.10) | | oxlint | `1.73.0` | [npm](https://npmx.dev/package/oxlint/v/1.73.0) | | oxlint-tsgolint | `0.24.0` | [npm](https://npmx.dev/package/oxlint-tsgolint/v/0.24.0) | | oxfmt | `0.58.0` | [npm](https://npmx.dev/package/oxfmt/v/0.58.0) | ### Upgrade ```bash vp upgrade ``` ### New Contributors Welcome @yukinoshi and @BlankParticle. **Full Changelog**: v0.2.4...v0.2.5 --- Merging this PR will trigger the release workflow. --------- Co-authored-by: voidzero-guard[bot] <278573678+voidzero-guard[bot]@users.noreply.github.com> Co-authored-by: wan9chi <dk4rest@gmail.com>

fengmk2 mentioned this pull request Jul 9, 2026

test(snapshots): migrate vp migrate snap test cases #2107

Merged

fengmk2 mentioned this pull request Jul 9, 2026

test(snapshots): migrate shim snap test cases #2109

Merged

fengmk2 marked this pull request as ready for review July 9, 2026 14:56

fengmk2 self-assigned this Jul 9, 2026

fengmk2 requested a review from wan9chi July 9, 2026 14:56

fengmk2 force-pushed the fix-snap-registry-teardown branch from 9b411d2 to cfbc122 Compare July 9, 2026 14:57

This was referenced Jul 9, 2026

Tracking: migrate legacy snap tests to the PTY snapshot suite #2116

Closed

test(snapshots): migrate vp check cases to the PTY snapshot suite #2117

Merged

test(snapshots): migrate vite-* cases to the PTY snapshot suite #2118

Merged

fengmk2 force-pushed the fix-snap-registry-teardown branch from 8397ec3 to cfbc122 Compare July 9, 2026 16:49

This was referenced Jul 9, 2026

test(snapshots): migrate the local npm install case to the PTY snapshot suite #2119

Merged

test(snapshots): migrate vp-* cases to the PTY snapshot suite #2120

Merged

test(snapshots): migrate the misc tail cases to the PTY snapshot suite #2121

Merged

fengmk2 added 2 commits July 10, 2026 09:49

fengmk2 force-pushed the fix-snap-registry-teardown branch from cfbc122 to 52ce2b9 Compare July 10, 2026 01:50

wan9chi approved these changes Jul 10, 2026

View reviewed changes

fengmk2 mentioned this pull request Jul 10, 2026

fix(env): clarify the use case of vp-use.cmd #2128

Merged

fengmk2 merged commit 9247583 into main Jul 10, 2026
52 checks passed

fengmk2 deleted the fix-snap-registry-teardown branch July 10, 2026 02:26

wan9chi mentioned this pull request Jul 15, 2026

release: v0.2.5: Cleaner environments and more reliable workflows #2180

Merged

BrewTestBot mentioned this pull request Jul 17, 2026

vite-plus 0.2.5 Homebrew/homebrew-core#293663

Merged

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(snapshots): scope registry teardown signal#2115

fix(snapshots): scope registry teardown signal#2115
fengmk2 merged 2 commits into
mainfrom
fix-snap-registry-teardown

fengmk2 commented Jul 9, 2026

Uh oh!

netlify Bot commented Jul 9, 2026 •

edited

Loading

Uh oh!

fengmk2 commented Jul 9, 2026 •

edited

Loading

Uh oh!

graphite-app Bot commented Jul 10, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

fengmk2 commented Jul 9, 2026

Uh oh!

netlify Bot commented Jul 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for viteplus-preview canceled.

Uh oh!

fengmk2 commented Jul 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

graphite-app Bot commented Jul 10, 2026

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

netlify Bot commented Jul 9, 2026 •

edited

Loading

fengmk2 commented Jul 9, 2026 •

edited

Loading