fix(e2e): stabilize playwright against Vite dep-optimizer races (#38)

The desktop golden-path e2e intermittently failed in CI with "504
Outdated Optimize Dep" on the dynamic import of OSShell.tsx. Vite's
dep optimizer was re-bundling mid-flight as it discovered deps reached
only through lazy-loaded islands (zustand, zustand/middleware,
react-rnd), invalidating in-flight module URLs.

- astro.config.mjs: pre-bundle the offending deps via optimizeDeps.include
  and warm up the island entries with server.warmup.clientFiles so
  Vite's crawl completes before the first request.
- playwright.config.ts: in CI, serve the built site via astro preview —
  no dev optimizer at all, closest to prod. Local dev keeps astro dev
  for fast iteration.

Verified: 5/5 cold-cache runs pass in CI mode, 5/5 in local dev mode.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: schmug

astro.config.mjs

@@ -12,6 +12,23 @@ export default defineConfig({
   trailingSlash: 'ignore',
   vite: {
     plugins: [tailwindcss()],
+    // Pre-bundle deps reached only through lazy-loaded islands (OSShell, MobileShell).
+    // Without this, Vite discovers them on first page load and re-optimizes mid-flight,
+    // returning 504 "Outdated Optimize Dep" on in-flight dynamic imports and flaking e2e.
+    optimizeDeps: {
+      include: ['zustand', 'zustand/middleware', 'react-rnd'],
+    },
+    // Warm up the lazy island entries so Vite's dep crawl completes before the
+    // first browser request — complements optimizeDeps.include above.
+    server: {
+      warmup: {
+        clientFiles: [
+          './src/components/RootShell.tsx',
+          './src/components/os/OSShell.tsx',
+          './src/components/mobile/MobileShell.tsx',
+        ],
+      },
+    },
   },
 
   integrations: [

playwright.config.ts

@@ -12,10 +12,14 @@ export default defineConfig({
     screenshot: 'only-on-failure',
   },
   webServer: {
-    command: 'npm run dev',
+    // In CI, serve the built site via `astro preview` — no Vite dep optimizer,
+    // so no "504 Outdated Optimize Dep" races on in-flight dynamic islands.
+    // Locally, keep `astro dev` for fast iteration; `optimizeDeps.include` in
+    // astro.config.mjs softens the same race for dev runs.
+    command: process.env.CI ? 'npm run build && npm run preview' : 'npm run dev',
     url: 'http://localhost:4321',
     reuseExistingServer: !process.env.CI,
-    timeout: 60_000,
+    timeout: process.env.CI ? 120_000 : 60_000,
   },
   projects: [{ name: 'chromium', use: { ...devices['Desktop Chrome'] } }],
 });