Skip to content

fix: resolve "vite" relative to the project root instead of using a dependency, if possible#9586

Closed
sheremet-va wants to merge 9 commits intovitest-dev:mainfrom
sheremet-va:fix/import-vite-dynamically
Closed

fix: resolve "vite" relative to the project root instead of using a dependency, if possible#9586
sheremet-va wants to merge 9 commits intovitest-dev:mainfrom
sheremet-va:fix/import-vite-dynamically

Conversation

@sheremet-va
Copy link
Copy Markdown
Member

@sheremet-va sheremet-va commented Feb 4, 2026

Description

The goal is to avoid any breaking changes even with the most unhinged solutions

Turns out we cannot use TLA because it taints the whole module graph, meaning user dependencies in the config can't be loaded, for example (even require(vite)). I assumed that it doesn't affect siblings imports previously.

Testing simple use cases with vite 6, vite 7 and vite 8 seems to work (including Node 20 without require(esm) support).

The exports tests should be fixed with antfu/vitest-package-exports#5 because require(esm) cannot be imported in Promise.all() 🤷

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 4, 2026

@vitest/browser

npm i https://pkg.pr.new/@vitest/browser@9586

@vitest/browser-playwright

npm i https://pkg.pr.new/@vitest/browser-playwright@9586

@vitest/browser-preview

npm i https://pkg.pr.new/@vitest/browser-preview@9586

@vitest/browser-webdriverio

npm i https://pkg.pr.new/@vitest/browser-webdriverio@9586

@vitest/coverage-istanbul

npm i https://pkg.pr.new/@vitest/coverage-istanbul@9586

@vitest/coverage-v8

npm i https://pkg.pr.new/@vitest/coverage-v8@9586

@vitest/expect

npm i https://pkg.pr.new/@vitest/expect@9586

@vitest/mocker

npm i https://pkg.pr.new/@vitest/mocker@9586

@vitest/pretty-format

npm i https://pkg.pr.new/@vitest/pretty-format@9586

@vitest/runner

npm i https://pkg.pr.new/@vitest/runner@9586

@vitest/snapshot

npm i https://pkg.pr.new/@vitest/snapshot@9586

@vitest/spy

npm i https://pkg.pr.new/@vitest/spy@9586

@vitest/ui

npm i https://pkg.pr.new/@vitest/ui@9586

@vitest/utils

npm i https://pkg.pr.new/@vitest/utils@9586

vitest

npm i https://pkg.pr.new/vitest@9586

@vitest/web-worker

npm i https://pkg.pr.new/@vitest/web-worker@9586

@vitest/ws-client

npm i https://pkg.pr.new/@vitest/ws-client@9586

commit: 63fa6f3

@sheremet-va sheremet-va moved this to P2 - 5 in Team Board Feb 4, 2026
@sheremet-va
Copy link
Copy Markdown
Member Author

Superseded by a much simpler fix at #9587

@sheremet-va sheremet-va closed this Feb 4, 2026
@sheremet-va sheremet-va removed this from Team Board Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant