From c941ef077ca05b558339aa0c96b27e226918e2ff Mon Sep 17 00:00:00 2001
From: Waleed Latif <walif6@gmail.com>
Date: Mon, 12 Oct 2026 12:50:25 -0700
Subject: [PATCH 1/2] fix(e2e): connect the mobile suite to realtime and give
 dev-server round trips the server budget

- Start the realtime server for the mobile group. Without it every workspace page raised a
  persistent "Reconnecting..." toast ~2s after load; on the locked-workflow reload it raced the
  lock toast and, when it arrived second, sat in front of the Unlock Workflow action, so the tap
  timed out on "subtree intercepts pointer events"
- Wait 120s, the suite's budget for a next dev round trip, for the URL after a sidebar link tap,
  the Files list after it, and the table cell after a reload. next dev spent 16.4s (application
  code 70ms) serving the /home RSC payload and left the rows request unanswered past 20s
- Install the browsers while the app boots
---
 .github/scripts/http-e2e.sh         | 41 ++++++++++++++++++++++++-----
 apps/sim/scripts/test-mobile-e2e.ts | 12 ++++++---
 2 files changed, 43 insertions(+), 10 deletions(-)

diff --git a/.github/scripts/http-e2e.sh b/.github/scripts/http-e2e.sh
index ef9c500bd1d..eb824af15e9 100755
--- a/.github/scripts/http-e2e.sh
+++ b/.github/scripts/http-e2e.sh
@@ -26,6 +26,8 @@ server_pid=''
 app_tag=''
 server_log=''
 status_log=''
+realtime_pid=''
+realtime_tag=''
 
 finish() {
   local status=$?
@@ -39,6 +41,10 @@ finish() {
       tail -n 200 "$server_log"
     fi
   fi
+  if [ -n "$realtime_pid" ]; then
+    bash "$GITHUB_WORKSPACE/.github/scripts/stop-session.sh" "$realtime_pid" "$realtime_tag" || status=1
+    wait "$realtime_pid" 2>/dev/null || true
+  fi
   exit "$status"
 }
 trap finish EXIT
@@ -58,20 +64,38 @@ start_app() {
   E2E_APP="$app_tag" setsid node ../../node_modules/next/dist/bin/next dev --hostname 127.0.0.1 \
     --port "$port" > "$server_log" 2>&1 &
   server_pid=$!
+  wait_ready "$server_pid" "$NEXT_PUBLIC_APP_URL/api/health" "$label app"
+}
+
+# start_realtime <name> <port>: the Socket.IO server the app's browser clients connect to, so pages
+# render connected rather than under a "Reconnecting..." notification. Start after start_app, which
+# exports the app URL the server allows as an origin; NEXT_PUBLIC_SOCKET_URL must already be set
+# when the app starts.
+start_realtime() {
+  local name=$1 port=$2
+  realtime_tag="$name-realtime-$GITHUB_RUN_ID-$GITHUB_RUN_ATTEMPT-$$"
+  (cd ../realtime && PORT="$port" E2E_APP="$realtime_tag" exec setsid bun run start \
+    > "$report_dir/$name-realtime.log" 2>&1) &
+  realtime_pid=$!
+  wait_ready "$realtime_pid" "http://127.0.0.1:$port/health" "$name realtime server"
+}
 
+# wait_ready <pid> <health-url> <label>
+wait_ready() {
+  local pid=$1 url=$2 label=$3
   local started=$SECONDS
-  until curl --fail --silent --max-time 10 "$NEXT_PUBLIC_APP_URL/api/health" > /dev/null; do
-    if ! kill -0 "$server_pid" 2>/dev/null; then
-      echo "::error::Local $label app exited during startup."
+  until curl --fail --silent --max-time 10 "$url" > /dev/null; do
+    if ! kill -0 "$pid" 2>/dev/null; then
+      echo "::error::Local $label exited during startup."
       exit 1
     fi
     if [ $((SECONDS - started)) -ge "$ready_timeout_seconds" ]; then
-      echo "::error::Local $label app did not become ready within $ready_timeout_seconds seconds."
+      echo "::error::Local $label did not become ready within $ready_timeout_seconds seconds."
       exit 1
     fi
     sleep 2
   done
-  echo "Local $label app ready after $((SECONDS - started))s"
+  echo "Local $label ready after $((SECONDS - started))s"
 }
 
 case "$group" in
@@ -159,8 +183,13 @@ case "$group" in
     export NEXT_PUBLIC_FORCE_HOSTED=false
     export INTERNAL_API_SECRET=mobile-http-ci-local-secret-at-least-32-characters
     export NODE_OPTIONS="${NODE_OPTIONS:+$NODE_OPTIONS }--max-old-space-size=12288"
-    bunx --no-install playwright install --with-deps chromium webkit
+    export NEXT_PUBLIC_SOCKET_URL=http://127.0.0.1:3025 SOCKET_SERVER_URL=http://127.0.0.1:3025
+    # The browsers install while the app boots.
+    bunx --no-install playwright install --with-deps chromium webkit &
+    browsers_pid=$!
     start_app mobile 3024 'mobile browser' record-http-status
+    start_realtime mobile 3025
+    wait "$browsers_pid"
     MOBILE_E2E_BASE_URL="$NEXT_PUBLIC_APP_URL" \
     MOBILE_E2E_DATABASE_URL="$DATABASE_URL" \
     MOBILE_E2E_AUTH_SECRET="$BETTER_AUTH_SECRET" \
diff --git a/apps/sim/scripts/test-mobile-e2e.ts b/apps/sim/scripts/test-mobile-e2e.ts
index 73e54fc9733..bc596fcdd6d 100644
--- a/apps/sim/scripts/test-mobile-e2e.ts
+++ b/apps/sim/scripts/test-mobile-e2e.ts
@@ -684,7 +684,9 @@ async function exerciseViewport(
           await expectContained(page, sidebar)
           if (viewport.name === 'phone') await capture(page, `${browserName}-phone-navigation`)
           await sidebar.getByRole('link', { name: 'New chat', exact: true }).tap()
-          await expect(page).toHaveURL(`${baseUrl.origin}/workspace/${workspaceId}/home`)
+          await expect(page).toHaveURL(`${baseUrl.origin}/workspace/${workspaceId}/home`, {
+            timeout: 120_000,
+          })
           await expect(page.getByRole('textbox', { name: 'Message', exact: true })).toBeVisible()
           await expect(sidebar).toBeHidden()
           await page.getByRole('button', { name: 'Open navigation', exact: true }).tap()
@@ -692,12 +694,14 @@ async function exerciseViewport(
           await expect(sidebar).toBeHidden()
           await page.getByRole('button', { name: 'Open navigation', exact: true }).tap()
           await sidebar.getByRole('link', { name: 'Files', exact: true }).tap()
-          await expect(page).toHaveURL(new RegExp(`/workspace/${workspaceId}/files`))
+          await expect(page).toHaveURL(new RegExp(`/workspace/${workspaceId}/files`), {
+            timeout: 120_000,
+          })
           await expect(sidebar).toBeHidden()
           await expect(page.locator('.workspace-content-shell')).toBeVisible()
           await expect(
             page.getByRole('button', { name: `Actions for ${folderName}`, exact: true })
-          ).toBeVisible()
+          ).toBeVisible({ timeout: 120_000 })
           await expectDesktopPreference(context, page)
         }
       )
@@ -977,7 +981,7 @@ async function exerciseViewport(
           await withDocumentNavigation(page, page.url(), () =>
             page.reload({ waitUntil: 'commit', timeout: 360_000 })
           )
-          await expect(cell).toHaveText(value)
+          await expect(cell).toHaveText(value, { timeout: 120_000 })
           await page.getByRole('button', { name: 'Column options', exact: true }).first().tap()
           await expectContained(page, page.getByRole('menu'))
           await page.getByRole('menuitem', { name: 'Edit column', exact: true }).tap()

From f0a501ce9d9ac85556030881676915035e70df58 Mon Sep 17 00:00:00 2001
From: Waleed Latif <walif6@gmail.com>
Date: Mon, 12 Oct 2026 13:02:48 -0700
Subject: [PATCH 2/2] fix(e2e): stop the background browser install on exit and
 surface install and realtime logs on failure

---
 .github/scripts/http-e2e.sh | 25 +++++++++++++++++++++----
 1 file changed, 21 insertions(+), 4 deletions(-)

diff --git a/.github/scripts/http-e2e.sh b/.github/scripts/http-e2e.sh
index eb824af15e9..feb8925d30d 100755
--- a/.github/scripts/http-e2e.sh
+++ b/.github/scripts/http-e2e.sh
@@ -28,6 +28,8 @@ server_log=''
 status_log=''
 realtime_pid=''
 realtime_tag=''
+realtime_log=''
+browsers_pid=''
 
 finish() {
   local status=$?
@@ -44,6 +46,13 @@ finish() {
   if [ -n "$realtime_pid" ]; then
     bash "$GITHUB_WORKSPACE/.github/scripts/stop-session.sh" "$realtime_pid" "$realtime_tag" || status=1
     wait "$realtime_pid" 2>/dev/null || true
+    if [ "$status" -ne 0 ]; then
+      tail -n 100 "$realtime_log"
+    fi
+  fi
+  if [ -n "$browsers_pid" ]; then
+    kill -- "-$browsers_pid" 2>/dev/null || true
+    wait "$browsers_pid" 2>/dev/null || true
   fi
   exit "$status"
 }
@@ -74,8 +83,9 @@ start_app() {
 start_realtime() {
   local name=$1 port=$2
   realtime_tag="$name-realtime-$GITHUB_RUN_ID-$GITHUB_RUN_ATTEMPT-$$"
+  realtime_log="$report_dir/$name-realtime.log"
   (cd ../realtime && PORT="$port" E2E_APP="$realtime_tag" exec setsid bun run start \
-    > "$report_dir/$name-realtime.log" 2>&1) &
+    > "$realtime_log" 2>&1) &
   realtime_pid=$!
   wait_ready "$realtime_pid" "http://127.0.0.1:$port/health" "$name realtime server"
 }
@@ -184,12 +194,19 @@ case "$group" in
     export INTERNAL_API_SECRET=mobile-http-ci-local-secret-at-least-32-characters
     export NODE_OPTIONS="${NODE_OPTIONS:+$NODE_OPTIONS }--max-old-space-size=12288"
     export NEXT_PUBLIC_SOCKET_URL=http://127.0.0.1:3025 SOCKET_SERVER_URL=http://127.0.0.1:3025
-    # The browsers install while the app boots.
-    bunx --no-install playwright install --with-deps chromium webkit &
+    # The browsers install while the app boots, in their own process group so finish() can stop them.
+    browsers_log="$report_dir/mobile-playwright-install.log"
+    setsid --wait bunx --no-install playwright install --with-deps chromium webkit > "$browsers_log" 2>&1 &
     browsers_pid=$!
     start_app mobile 3024 'mobile browser' record-http-status
     start_realtime mobile 3025
-    wait "$browsers_pid"
+    if ! wait "$browsers_pid"; then
+      browsers_pid=''
+      tail -n 100 "$browsers_log"
+      echo "::error::Playwright browser install failed."
+      exit 1
+    fi
+    browsers_pid=''
     MOBILE_E2E_BASE_URL="$NEXT_PUBLIC_APP_URL" \
     MOBILE_E2E_DATABASE_URL="$DATABASE_URL" \
     MOBILE_E2E_AUTH_SECRET="$BETTER_AUTH_SECRET" \