Skip to content

Commit d4fa7c8

Browse files
jonphippsclaude
andcommitted
fix: use more conservative CI test configuration
Changes for better CI stability: - Use forks instead of threads in CI for better process isolation - Reduce timeouts to prevent hanging processes - Reduce retry count to fail faster - Remove bail setting to see all test results - Keep single process/fork in CI for memory efficiency 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 21b0849 commit d4fa7c8

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

vite.config.ts

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,23 +31,26 @@ export default defineConfig({
3131
include: ['**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
3232
exclude: ['**/node_modules/**', '**/dist/**', '**/build/**', '**/e2e/**', '**/tests/visual-regression.spec.ts'],
3333
// Enhanced CI stability and performance
34-
testTimeout: process.env.CI ? 120000 : 30000, // Increased CI timeout to 2 minutes
35-
hookTimeout: process.env.CI ? 60000 : 10000, // Increased CI hook timeout to 1 minute
34+
testTimeout: process.env.CI ? 90000 : 30000, // Reduced CI timeout to 90 seconds
35+
hookTimeout: process.env.CI ? 30000 : 10000, // Reduced CI hook timeout to 30 seconds
3636
maxConcurrency: process.env.CI ? 1 : 5,
37-
pool: 'threads', // Better than forks for CI performance
37+
pool: process.env.CI ? 'forks' : 'threads', // Use forks in CI for better isolation
3838
poolOptions: {
3939
threads: {
4040
singleThread: !!process.env.CI,
4141
isolate: true,
42-
// Add memory management for CI
4342
maxThreads: process.env.CI ? 1 : undefined,
4443
minThreads: process.env.CI ? 1 : undefined
44+
},
45+
forks: {
46+
singleFork: !!process.env.CI,
47+
isolate: true
4548
}
4649
},
47-
retry: process.env.CI ? 3 : 0, // Increased retry count for CI
50+
retry: process.env.CI ? 2 : 0, // Reduced retry count for CI
4851
logHeapUsage: !!process.env.CI,
49-
// Add memory and resource management for CI
50-
bail: process.env.CI ? 1 : 0, // Stop on first failure in CI to prevent resource exhaustion
52+
// Disable bail in CI to see all test results
53+
bail: false,
5154
forceRerunTriggers: process.env.CI ? [] : [
5255
'**/vite.config.ts',
5356
'**/vitest.config.ts',

0 commit comments

Comments
 (0)