[lxsmnsyc]

Fixes #369 on main (1.x). The fix for next is in a separate PR.

In jsdom tests, packages that import solid-js/web, such as @solidjs/testing-library, now get the browser build under Deno. Before, render threw Client-only API called on the server side.

Cause

• The client test setup kept solid-js external, along with the packages that use it.
• External packages are loaded by the host runtime, not by Vite.
• Deno does not pass Vitest's browser condition to worker threads (pool: 'threads'). Those threads resolved solid-js/web to the server build.

Changes

• With ssr off, test mode now inlines solid-js and every package that lists it as a dependency or peer. They resolve through Vite's browser condition.
• solid-js and the packages that use it are inlined together, so they share one copy of solid-js.
• A user server.deps.inline list is merged with the plugin's. inline: true and a user external entry for solid-js are left alone.
• With ssr: true, solid-js stays external as before.
• The second commit sets test.sharedViteServer = false when test.projects is set and the option is unset. This ports fix(vitest): resolve each inline project's own config under Vitest 5 #370 from next.
  • Vitest 5 shares the root Vite server with inline projects by default. Those projects then never get the plugin's test settings, including the inline list above.
  • Without this commit, the reporter's multi-project setup still fails on Deno.
  • It is a separate commit, so it can be dropped if that change should go in on its own.

Known limit

• The plugin finds a package that uses solid-js only if it depends on it directly. A package that reaches solid-js only through another such package is not inlined, and would still load its own copy.

Testing

• Added examples/vite-6/tests/jsdom/testing-library.test.tsx, with its own vitest.jsdom.config.ts and a test:jsdom script. It runs with a resolve hook that removes browser and adds deno, the same as Deno.
  • Without the fix, it fails with the issue's error.
  • With the fix, it passes.
• Added a pnpm run test:jsdom step to the Vitest workflow.
• Ran the reporter's multi-project setup on Deno 2.9.6 with Vitest 5.0.1 and solid-js 1.9.4.
  • It fails on 2.11.14 with the same error.
  • It passes 3 of 3 with both commits.
• tsc --noEmit passes.
• The browser-mode example suite was not run locally, because Playwright's browser is not installed. The browser-mode config now excludes tests/jsdom/**.

🤖 Generated with Claude Code

[changeset-bot]

🦋 Changeset detected

Latest commit: 5d6846c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
vite-plugin-solid	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/vite-plugin-solid@434

commit: 5d6846c