Fix visual regression and performance server issue

.gitea/workflows/ci.yaml

@@ -132,57 +132,27 @@ jobs:
           ls -la .next || true
           test -f .next/BUILD_ID || (echo "No Next build output (.next) – did build fail?" && exit 1)
 
-      # 2) Start the right kind of server and capture logs
-      - name: Start app (auto-detect start vs export)
+      - name: Start app (background) + healthcheck
         run: |
           set -euxo pipefail
-
-          # prefer production server if Next build exists
-          if [ -f .next/BUILD_ID ]; then
-            # run Next in the foreground for 3s to flush any crash to logs, then background it
-            (NODE_ENV=production NEXT_TELEMETRY_DISABLED=1 \
-             node --trace-uncaught node_modules/next/dist/bin/next start -p 3000 -H 127.0.0.1 \
-            ) > server.log 2>&1 &
-
-          # fall back to static export (if your project uses output: 'export')
-          elif [ -d out ]; then
-            npx --yes http-server out -p 3000 --silent > server.log 2>&1 &
-          else
-            echo "Neither .next nor out/ present – nothing to serve"; cat server.log || true; exit 1
-          fi
-
-          echo $! > server.pid
-          sleep 2
-          echo "----- first server log lines -----"
-          head -n 200 server.log || true
-          echo "----------------------------------"
-
-      # 3) Fail fast if the process died
-      - name: Check server process
+          export PORT="${PORT:-3000}"
+          export HOST="127.0.0.1"
+          test -d .next || { echo "❌ Missing .next build output"; exit 1; }
+          mkdir -p .next
+          nohup npm run start -- -p "$PORT" -H "$HOST" > .next/runner.log 2>&1 &
+          echo $! > .next/runner.pid
+          echo "🌐 PID $(cat .next/runner.pid) listening on http://$HOST:$PORT"
+          npx wait-on -t 120000 "tcp:$HOST:$PORT"
+          curl -fsS "http://$HOST:$PORT" >/dev/null
+          echo "✅ App is responding"
+        env:
+          NEXT_TELEMETRY_DISABLED: "1"
+          NODE_ENV: production
+      - name: Show last 200 lines of server log on failure
+        if: failure()
         run: |
-          set -e
-          PID=$(cat server.pid)
-          if ! ps -p "$PID" > /dev/null; then
-            echo "Server crashed during startup:"
-            cat server.log || true
-            exit 1
-          fi
-
-      # 4) Wait for readiness with helpful diagnostics
-      - name: Wait for http://127.0.0.1:3000
-        run: |
-          set -euxo pipefail
-          for i in $(seq 1 60); do
-            if curl -sf http://127.0.0.1:3000 >/dev/null; then
-              echo "App is up ✅"; exit 0
-            fi
-            if [ $i -eq 1 ] || [ $((i%10)) -eq 0 ]; then
-              echo "Still waiting… attempt $i"
-              tail -n 50 server.log || true
-            fi
-            sleep 2
-          done
-          echo "Timed out waiting for app"; tail -n +1 server.log || true; exit 1
+          echo "––– .next/runner.log (tail) –––"
+          tail -n 200 .next/runner.log || true
       # Seed snapshots on main branch only (one-time setup)
       - name: Seed snapshots (main only)
         if: github.ref == 'refs/heads/main'
@@ -207,6 +177,13 @@ jobs:
           path: visual-regression.tgz
           retention-days: 30
 
+      - name: Stop app
+        if: always()
+        run: |
+          if [ -f .next/runner.pid ]; then
+            kill $(cat .next/runner.pid) 2>/dev/null || true
+          fi
+
   performance:
     runs-on: [self-hosted, macos-latest]
     steps:
@@ -234,57 +211,27 @@ jobs:
           ls -la .next || true
           test -f .next/BUILD_ID || (echo "No Next build output (.next) – did build fail?" && exit 1)
 
-      # 2) Start the right kind of server and capture logs
-      - name: Start app (auto-detect start vs export)
+      - name: Start app (background) + healthcheck
         run: |
           set -euxo pipefail
-
-          # prefer production server if Next build exists
-          if [ -f .next/BUILD_ID ]; then
-            # run Next in the foreground for 3s to flush any crash to logs, then background it
-            (NODE_ENV=production NEXT_TELEMETRY_DISABLED=1 \
-             node --trace-uncaught node_modules/next/dist/bin/next start -p 3000 -H 127.0.0.1 \
-            ) > server.log 2>&1 &
-
-          # fall back to static export (if your project uses output: 'export')
-          elif [ -d out ]; then
-            npx --yes http-server out -p 3000 --silent > server.log 2>&1 &
-          else
-            echo "Neither .next nor out/ present – nothing to serve"; cat server.log || true; exit 1
-          fi
-
-          echo $! > server.pid
-          sleep 2
-          echo "----- first server log lines -----"
-          head -n 200 server.log || true
-          echo "----------------------------------"
-
-      # 3) Fail fast if the process died
-      - name: Check server process
+          export PORT="${PORT:-3000}"
+          export HOST="127.0.0.1"
+          test -d .next || { echo "❌ Missing .next build output"; exit 1; }
+          mkdir -p .next
+          nohup npm run start -- -p "$PORT" -H "$HOST" > .next/runner.log 2>&1 &
+          echo $! > .next/runner.pid
+          echo "🌐 PID $(cat .next/runner.pid) listening on http://$HOST:$PORT"
+          npx wait-on -t 120000 "tcp:$HOST:$PORT"
+          curl -fsS "http://$HOST:$PORT" >/dev/null
+          echo "✅ App is responding"
+        env:
+          NEXT_TELEMETRY_DISABLED: "1"
+          NODE_ENV: production
+      - name: Show last 200 lines of server log on failure
+        if: failure()
         run: |
-          set -e
-          PID=$(cat server.pid)
-          if ! ps -p "$PID" > /dev/null; then
-            echo "Server crashed during startup:"
-            cat server.log || true
-            exit 1
-          fi
-
-      # 4) Wait for readiness with helpful diagnostics
-      - name: Wait for http://127.0.0.1:3000
-        run: |
-          set -euxo pipefail
-          for i in $(seq 1 60); do
-            if curl -sf http://127.0.0.1:3000 >/dev/null; then
-              echo "App is up ✅"; exit 0
-            fi
-            if [ $i -eq 1 ] || [ $((i%10)) -eq 0 ]; then
-              echo "Still waiting… attempt $i"
-              tail -n 50 server.log || true
-            fi
-            sleep 2
-          done
-          echo "Timed out waiting for app"; tail -n +1 server.log || true; exit 1
+          echo "––– .next/runner.log (tail) –––"
+          tail -n 200 .next/runner.log || true
 
       - name: Run Lighthouse CI
         run: npx lhci autorun --chrome-path="$CHROME_PATH"
@@ -297,6 +244,13 @@ jobs:
           name: lhci-results
           path: lhci-results
 
+      - name: Stop app
+        if: always()
+        run: |
+          if [ -f .next/runner.pid ]; then
+            kill $(cat .next/runner.pid) 2>/dev/null || true
+          fi
+
   storybook:
     runs-on: [self-hosted, macos-latest]
     steps: