Fix ui-test: bind preview to 127.0.0.1 and split test steps

The Playwright webServer was timing out because vite preview defaulted
to binding on localhost which may resolve to IPv6 ::1 in the container,
while Playwright checks 127.0.0.1:8005. Explicitly bind to 127.0.0.1
and split unit/e2e test steps for better isolation of the env var.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: eakmanrq <6326532+eakmanrq@users.noreply.github.com>

Author: eakmanrq

.github/workflows/pr.yaml

@@ -233,8 +233,10 @@ jobs:
         run: pnpm install
       - name: Build UI
         run: npm --prefix web/client run build
-      - name: Run tests
-        run: npm --prefix web/client run test
+      - name: Run unit tests
+        run: npm --prefix web/client run test:unit
+      - name: Run e2e tests
+        run: npm --prefix web/client run test:e2e
         env:
           PLAYWRIGHT_SKIP_BUILD: '1'
 

web/client/vite.config.ts

@@ -68,5 +68,6 @@ export default defineConfig({
         },
   preview: {
     port: 8005,
+    host: '127.0.0.1',
   },
 })