Skip to content

fix(vite-dev-server): sync remaining Vite fixes - #52

Merged
kazupon merged 3 commits into
mainfrom
fix/sync-vite-remaining-updates
Sep 30, 2026
Merged

kazupon merged 3 commits into
mainfrom
fix/sync-vite-remaining-updates

Conversation

@kazupon

@kazupon kazupon commented Sep 30, 2026

Copy link
Copy Markdown
Owner

Sync the remaining upstream Vite fixes that the previous syncs left out:

The upstream tests rely on config loading, chokidar and SSR, so unit tests cover getFileStartIndex() with each line terminator, makeWatcherCloseFinal() and the sourceURL helpers instead.

Validated with typecheck, lint, vp run test:unit:vite:node (663 → 674 tests) and the @vrowzer/vite-dev-server build. Integration and E2E tests were not run locally.

…ators

Port vitejs/vite#23038 (17f7b2f19). getFileStartIndex() is used by the dev import analysis. The config bundling part only updates the commented-out upstream code, and the css.ts and importAnalysisBuild.ts parts are not ported.
…fter server close

Port vitejs/vite#23572 (6f831f9ad). makeWatcherCloseFinal() is ported to watch.ts, but it is not applied to the watchers from watcherFactory: the virtual watcher is frozen and its add() never opens file system handles.
Port vitejs/vite#23513 (bbc8812be). The module runner sourcemap interceptor and ssrStacktrace parts are not ported, so decodeSourceURL() is not used yet.
@kazupon
kazupon merged commit f890748 into main Sep 30, 2026
4 checks passed
@kazupon
kazupon deleted the fix/sync-vite-remaining-updates branch September 30, 2026 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant