Skip to content

Commit f2bb627

Browse files
committed
refactor: remove jsdom dependency from vitest config
The jsdom environment was configured but none of the unit tests use DOM APIs — they all test pure logic (dependency graphs, content hashing, URL paths, entry path validation). Switch to the default node environment and remove the jsdom devDependency. Closes #88 https://claude.ai/code/session_01MXf6zdcermnEnEYtCNES6K
1 parent 49292da commit f2bb627

4 files changed

Lines changed: 56 additions & 25 deletions

File tree

packages/static/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
"@types/node": "catalog:",
6161
"@types/react": "^19.2.14",
6262
"@types/react-dom": "^19.2.3",
63-
"jsdom": "^29.0.1",
63+
6464
"react": "catalog:",
6565
"react-dom": "catalog:",
6666
"tsdown": "^0.21.4",

packages/static/src/__tests__/vitest.setup.ts

Lines changed: 0 additions & 1 deletion
This file was deleted.

packages/static/vitest.config.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@ import { defineConfig } from "vitest/config";
22

33
export default defineConfig({
44
test: {
5-
environment: "jsdom",
6-
setupFiles: ["./src/__tests__/vitest.setup.ts"],
75
globals: true,
86
exclude: ["**/node_modules/**", "**/e2e/**"],
97
},

pnpm-lock.yaml

Lines changed: 55 additions & 21 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)