Skip to content

Commit ffe3325

Browse files
committed
chore: disable vitest interopDefault, remove dotenvx
Set deps.interopDefault: false in all vitest configs for native CJS resolution matching Node runtime behavior. Remove dotenvx from pre-commit hook, workspace catalog, and devDependencies. Inline SOCKET_CLI_NO_API_TOKEN=1 directly.
1 parent 8f2b03b commit ffe3325

File tree

10 files changed

+25
-113
lines changed

10 files changed

+25
-113
lines changed

.config/vitest.config.base.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,9 @@ export default defineConfig({
6868
useAtomics: true,
6969
},
7070
},
71+
deps: {
72+
interopDefault: false,
73+
},
7174
testTimeout: 30_000,
7275
hookTimeout: 30_000,
7376
coverage: {

.config/vitest.config.isolated.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,9 @@ export default defineConfig({
3636
minForks: isCoverageEnabled ? 1 : 2,
3737
},
3838
},
39+
deps: {
40+
interopDefault: false,
41+
},
3942
testTimeout: 30_000,
4043
hookTimeout: 10_000,
4144
coverage: {

.husky/pre-commit

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,7 @@ else
1515
fi
1616

1717
if [ -z "${DISABLE_PRECOMMIT_TEST}" ]; then
18-
if ! command -v dotenvx >/dev/null 2>&1; then
19-
printf "Error: dotenvx not found in PATH\n" >&2
20-
printf "Install with: pnpm i\n" >&2
21-
exit 1
22-
fi
23-
# Note: .env.precommit is optional and not tracked in git (contains local test config).
24-
# If missing, dotenvx will continue without it. Create .env.precommit with test
25-
# environment variables to optimize pre-commit test performance.
26-
dotenvx -q run -f .env.precommit -- pnpm test --staged
18+
SOCKET_CLI_NO_API_TOKEN=1 pnpm test --staged
2719
else
2820
printf "Skipping testing due to DISABLE_PRECOMMIT_TEST env var\n"
2921
fi

package.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@
5252
"pretest": "pnpm run build:cli"
5353
},
5454
"devDependencies": {
55-
"@typescript/native-preview": "7.0.0-dev.20260415.1",
5655
"@anthropic-ai/claude-code": "catalog:",
5756
"@babel/core": "catalog:",
5857
"@babel/parser": "catalog:",
@@ -63,7 +62,6 @@
6362
"@babel/preset-typescript": "catalog:",
6463
"@babel/runtime": "catalog:",
6564
"@babel/traverse": "catalog:",
66-
"@dotenvx/dotenvx": "catalog:",
6765
"@npmcli/arborist": "catalog:",
6866
"@npmcli/config": "catalog:",
6967
"@octokit/graphql": "catalog:",
@@ -96,6 +94,7 @@
9694
"@types/semver": "catalog:",
9795
"@types/which": "catalog:",
9896
"@types/yargs-parser": "catalog:",
97+
"@typescript/native-preview": "7.0.0-dev.20260415.1",
9998
"@vitest/coverage-v8": "catalog:",
10099
"@yao-pkg/pkg": "catalog:",
101100
"browserslist": "catalog:",

packages/cli/vitest.config.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,9 @@ export default defineConfig({
7979
//
8080
// Performance impact is acceptable for reliability.
8181
isolate: true,
82+
deps: {
83+
interopDefault: false,
84+
},
8285
testTimeout: 30_000,
8386
hookTimeout: 30_000,
8487
// Enable file-level parallelization for better performance.

packages/cli/vitest.e2e.config.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@ export default defineConfig({
2323
maxWorkers: os.cpus().length,
2424
// E2E tests need full isolation for clean execution.
2525
isolate: true,
26+
deps: {
27+
interopDefault: false,
28+
},
2629
// E2E tests need longer timeouts for spawning processes.
2730
testTimeout: 60_000,
2831
hookTimeout: 60_000,

packages/cli/vitest.integration.config.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ export default defineConfig({
2121
pool: 'threads',
2222
maxWorkers: os.cpus().length,
2323
isolate: true,
24+
deps: {
25+
interopDefault: false,
26+
},
2427
testTimeout: 60_000, // Integration tests may take longer.
2528
hookTimeout: 30_000,
2629
fileParallelism: true,

pnpm-lock.yaml

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

pnpm-workspace.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ catalog:
2525
'@babel/runtime': 7.28.4
2626
'@babel/traverse': 7.28.4
2727
'@babel/types': 7.28.5
28-
'@dotenvx/dotenvx': 1.49.0
2928
'@gitbeaker/rest': 43.7.0
3029
'@iarna/toml': 2.2.5
3130
'@npmcli/arborist': 9.4.2

vitest.config.mts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,9 @@ export default defineConfig({
5555
useAtomics: true,
5656
},
5757
},
58+
deps: {
59+
interopDefault: false,
60+
},
5861
testTimeout: 30_000,
5962
hookTimeout: 30_000,
6063
bail: process.env.CI ? 1 : 0, // Exit on first failure in CI for faster feedback.

0 commit comments

Comments
 (0)