From 1459946f8f4fd8e861ffb886f3ab8b6ad76543e2 Mon Sep 17 00:00:00 2001 From: Jeroen Gordijn Date: Wed, 4 Mar 2026 11:10:26 +0000 Subject: [PATCH] fix: prevent server-side Svelte code in client bundle The resolve.conditions config added for Vitest support set conditions to [] for production builds, which removed the 'browser' condition in Vite 7. This caused Svelte's server-side onDestroy (ssr_context.r.on_destroy) to be bundled into the client, crashing on page load with: TypeError: Cannot read properties of null (reading 'r') Fix: only set resolve.conditions when VITEST is active. --- ui/vite.config.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ui/vite.config.ts b/ui/vite.config.ts index 37148c4..ac9986e 100644 --- a/ui/vite.config.ts +++ b/ui/vite.config.ts @@ -10,7 +10,5 @@ export default defineConfig({ environment: 'jsdom', setupFiles: ['./src/test-setup.ts'], }, - resolve: { - conditions: process.env.VITEST ? ['browser'] : [] - } + ...(process.env.VITEST ? { resolve: { conditions: ['browser'] } } : {}) });