From a632dbc406c72b9dc79b3db5331f747c5b14ee4e Mon Sep 17 00:00:00 2001
From: PJ <ipriyanshujain@gmail.com>
Date: Thu, 13 Aug 2026 02:04:49 +0530
Subject: [PATCH] ci: pin calibrated seeds, skip the flaky ios reinstall, bound
 every job

---
 .github/scripts/folio-run.sh    |  7 ++++++-
 .github/workflows/folio.yml     |  3 +++
 .github/workflows/replay-ui.yml |  1 +
 docs/development/ci.md          | 20 +++++++++++++++-----
 4 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/.github/scripts/folio-run.sh b/.github/scripts/folio-run.sh
index 576f21f..0c4cea8 100755
--- a/.github/scripts/folio-run.sh
+++ b/.github/scripts/folio-run.sh
@@ -27,8 +27,13 @@ case "$platform" in
       examples/folio/app/androidApp/build/outputs/apk/debug/androidApp-debug.apk)
     ;;
   ios)
+    # --clear-data=false because the caller has just installed a fresh build (a
+    # freshly installed app IS clear state). The in-run reinstall path is worth
+    # avoiding here: `simctl uninstall` + `install` immediately followed by the
+    # XCTest runner's own launch hits "app.folio is unknown to FrontBoard"
+    # perhaps half the time, and the run then hangs rather than failing.
     folio_args+=(--platform ios
-      --ios-app-path examples/folio/app/iosApp/build/Build/Products/Debug-iphonesimulator/iosApp.app
+      --clear-data=false
       --ios-device "${IOS_DEVICE:-iPhone 16 Pro}")
     ;;
   web)
diff --git a/.github/workflows/folio.yml b/.github/workflows/folio.yml
index dc46bfb..dfa02cc 100644
--- a/.github/workflows/folio.yml
+++ b/.github/workflows/folio.yml
@@ -37,6 +37,7 @@ on:
 
 jobs:
   android:
+    timeout-minutes: 90
     if: ${{ inputs.platforms == 'all' || inputs.platforms == 'android' }}
     runs-on: ubuntu-latest
     steps:
@@ -111,6 +112,7 @@ jobs:
           retention-days: 14
 
   ios:
+    timeout-minutes: 90
     if: ${{ inputs.platforms == 'all' || inputs.platforms == 'ios' }}
     runs-on: macos-15
     steps:
@@ -192,6 +194,7 @@ jobs:
           retention-days: 14
 
   web:
+    timeout-minutes: 60
     if: ${{ inputs.platforms == 'all' || inputs.platforms == 'web' }}
     runs-on: ubuntu-latest
     steps:
diff --git a/.github/workflows/replay-ui.yml b/.github/workflows/replay-ui.yml
index 1efd5f4..bbd70e9 100644
--- a/.github/workflows/replay-ui.yml
+++ b/.github/workflows/replay-ui.yml
@@ -20,6 +20,7 @@ on:
 
 jobs:
   dogfood:
+    timeout-minutes: 45
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
diff --git a/docs/development/ci.md b/docs/development/ci.md
index eb11e0d..c0dc867 100644
--- a/docs/development/ci.md
+++ b/docs/development/ci.md
@@ -42,18 +42,28 @@ knew "non-zero" could not tell a working fuzzer from a broken emulator.
 
 **web is a health gate**, not an expect-the-bug leg. The same spec logs into the
 wasmJs build and drives it to the transaction screen (the job asserts the trace
-reached `AddTransactionScreen`), but the submit property cannot fire there. It
+reached `AddTransactionScreen`), but the submit property cannot fire there: it
 keys off `state.lastAction`, which the web runtime reports as `null`, and off the
-action's selector, which the web picker does not carry - it emits coordinates,
-and element identity never crosses the V8 boundary. Both are fixable; neither is
-a small fix, and until they are, asserting exit 2 on web would be asserting
-something the spec cannot observe.
+action's selector, which the web picker does not carry, since it emits
+coordinates and element identity never crosses the V8 boundary.
 
 The wasmJs app is served with `Cross-Origin-Opener-Policy` and
 `Cross-Origin-Embedder-Policy` headers, because its sqlite worker needs
 cross-origin isolation. Served without them the app loads a blank canvas and
 every step observes an empty accessibility tree.
 
+The seeds in the workflow are calibrated, not guessed. On an M-series mac,
+android seed 3 finds the bug at step 110-116 (seeds 1, 2 and 4 run 120 steps
+clean) and ios seed 1 finds it at step 129-134, both against a 240-step budget.
+The web leg ran five seeds x 200 steps clean.
+
+The ios leg passes `--clear-data=false`, because the job installs a fresh build
+immediately before the run and a freshly installed app is already clear state.
+The in-run reinstall is worth avoiding: `simctl uninstall` + `install` followed
+straight away by the XCTest runner's own launch fails with `app.folio is unknown
+to FrontBoard` maybe half the time, and the run then hangs rather than failing.
+The job timeouts are the backstop if it happens anyway.
+
 ## replay-ui
 
 Actions -> replay-ui -> Run workflow. This one is dogfooding: it records a trace