fix(vitest): resolve each inline project's own config under Vitest 5

[brenelz]

Fixes #369.

Under Vitest 5, inline projects in a test.projects workspace lost everything the plugin injects in test mode. A test.environment: 'node' project kept the root's browser condition, so isServer was false. A jsdom project lost the @testing-library/jest-dom setup file and failed with Invalid Chai property: toBeInTheDocument. It also lost the solid-js server.deps handling.

Root cause

Vitest 5 defaults test.sharedViteServer to true. An inline project that only changes test options reuses the root Vite server, and its options are resolved from the root test block that vitest:capture-raw-test-config captures before any plugin config hook runs. The plugin picks the client or server posture and adds its test.* values in that hook, so none of it reaches a shared project. Vitest 4.1.11 resolves a Vite config per project and is not affected.

The issue describes this as externalized dependencies missing the browser condition. I could not reproduce that exact symptom, but the workspace shape from the issue reproduces the shared-server failure above, and sharedViteServer: false alone fixes it.

Change

When the root config declares test.projects and leaves test.sharedViteServer unset, the config hook sets it to false, so every project resolves its own Vite config as it did under Vitest 4. An explicit user value is respected. Vitest reads the option from the resolved root config (getOwnServerReason), so setting it from a plugin hook is enough.

A narrower condition, such as disabling sharing only when project postures differ, would not work: a shared project with the same posture still loses the jest-dom setup file and the server.deps values. Conditions are Vite-level, so they cannot vary per project on one server either.

The README's vitest section documents the behavior, and a changeset is included.

Verification

Scratch project with two extends: true projects (jsdom and node), vitest@5.0.1, vite@8, solid-js/@solidjs/web 2.0.0-rc.9, @solidjs/testing-library@1.0.0-beta.3, and the plugin installed from pnpm pack. The tests assert isServer in each project, a testing-library render, and a jest-dom matcher.

• Before, npx vitest run on 5.0.1: 2 failed, 2 passed. After: 4 passed.
• 4 passed also with pool: 'threads' plus isolate: false, and with projects that omit extends.
• Vitest 4.1.11: 4 passed before and after.
• npx tsc --noEmit -p . is clean, and the examples/start-ssr VITEST_PROJECTS=1 suite passes (6 tests, Vitest 4.1.11).

Left open

No in-repo test runs on Vitest 5, because the examples pin vitest ^4.1.11. Covering this needs an example and lockfile bump.

🤖 Generated with Claude Code

[changeset-bot]

🦋 Changeset detected

Latest commit: ec6c4dc

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

This PR includes changesets to release 1 package

Name	Type
@solidjs/vite-plugin	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/@solidjs/vite-plugin@370

commit: ec6c4dc

[ryansolid]

Reproduced and verified — the code change is correct and I'd merge it, but the issue link needs changing.

What this fixes: under Vitest 5, inline test.projects share the root Vite server by default and take their test options from vitest:capture-raw-test-config, which snapshots the root test block before any plugin config hook runs. So a shared project never sees this plugin's posture, jest-dom setup file, or server.deps. getOwnServerReason reads sharedViteServer from the resolved root config, so returning false from the hook is honored. Confirmed with a two-project repro on vitest@5.0.2 (2 failed → 4 passed; explicit true respected; Vitest 4.1.11 unchanged). environment isn't in Vitest's VITE_AFFECTING_TEST_OPTIONS, so there's no narrower condition available — agreed with the PR's reasoning.

What it doesn't fix: #369 as reported. The reporter's follow-up says the symptom is Deno-only — Deno resolves the externalized testing-library's @solidjs/web import with deno/node conditions, both of which map to dist/server.js. On Node the testing-library render already passes on next. Please change "Fixes #369" → "Refs #369" (PR body, commit, changeset first sentence); we can decide separately whether #369 stays open as a Deno item.

Small asks:

• Add a resolveConfig-level assertion in examples/start-ssr/test/run.mjs beside the projects-env check: sharedViteServer is false when projects is declared, undefined otherwise, preserved when explicit. Version-independent, so it covers this without a Vitest 5 example.
• Expand the code comment at src/index.ts:1195 to name the raw-capture mechanism, so the condition isn't narrowed later.
• README: mention that true also drops the server.deps handling, and that this trades Vitest 5's shared-server optimization for Vitest 4 behavior.
• Changeset could be a couple of sentences rather than the full PR body.

No overlap with #371 — hunks are disjoint and a trial merge is clean.

— Claude via Cursor