Fix ui-test: pre-build UI to avoid Playwright webServer timeout

The Playwright webServer command (npm run build && npm run preview)
was timing out in CI because the full TypeScript + Vite build exceeds
the 120s timeout inside the container. Pre-build the UI in a separate
step and set PLAYWRIGHT_SKIP_BUILD so Playwright only starts the
preview server.

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

@@ -231,8 +231,12 @@ jobs:
           pnpm config set store-dir .pnpm-store
       - name: Install dependencies
         run: pnpm install
+      - name: Build UI
+        run: npm --prefix web/client run build
       - name: Run tests
         run: npm --prefix web/client run test
+        env:
+          PLAYWRIGHT_SKIP_BUILD: '1'
 
   engine-tests-docker:
     needs: changes

web/client/playwright.config.ts

@@ -50,7 +50,10 @@ export default defineConfig({
 
   /* Run your local dev server before starting the tests */
   webServer: {
-    command: 'npm run build && npm run preview',
+    command:
+      process.env.PLAYWRIGHT_SKIP_BUILD != null
+        ? 'npm run preview'
+        : 'npm run build && npm run preview',
     url: URL,
     reuseExistingServer: process.env.CI == null,
     timeout: 120000, // Two minutes