fix: tolerate delayed App Store Connect processing

[azooz2003-bit]

Summary

• Increase App Store Connect build ingestion waits from 15 minutes to 30 minutes for internal assignment and TestFlight notes.
• Keep the notes job alive for the full wait window.
• Add regression coverage for the shared processing timeout.

Changelog

Fixed official cmux.app TestFlight uploads timing out while Apple is still processing an accepted build.

Validation

• PYTHONPATH=tests python3 - <<'PY' ... test_processing_wait_defaults_cover_apple_ingest() ... PY
• bash -n ios/scripts/set-testflight-notes.sh
• python3 -m py_compile ios/scripts/asc_assign_internal_testflight_group.py ios/scripts/asc_set_testflight_notes.py
• git diff --check

---

^{Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.}

---

Summary by cubic

Fixes official cmux.app TestFlight uploads timing out while Apple is still processing an accepted build.

• Increases the default App Store Connect ingestion wait from 15 to 30 minutes for internal assignment and TestFlight notes.
• Keeps the notes job alive for the full wait window and adds regression coverage.
• Makes the shell notes timeout overridable via CMUX_TESTFLIGHT_PROCESSING_TIMEOUT_SECONDS.

^{Written for commit 55159a2. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 83469e69-be33-409a-9b2e-815cc5f156b4

📥 Commits

Reviewing files that changed from the base of the PR and between fcbf13c and 55159a2.

📒 Files selected for processing (5)

• .github/workflows/ios-appstore-upload.yml
• ios/scripts/asc_assign_internal_testflight_group.py
• ios/scripts/asc_set_testflight_notes.py
• ios/scripts/set-testflight-notes.sh
• tests/test_ios_testflight_notes.py

 _________________________________________________________________________________________________
< Question: How does a large software project get to be one year late? Answer: One day at a time! >
 -------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Merge receipt for 55159a2de6, merged 2026-10-01 22:39:33 UTC

• Not verified at merge: ci-status (not reported), CI fast guards (in progress), detect-ios-changes (in progress), Fast static checks (in progress), GhosttyKit release check (in progress), guards (4) (in progress), Web complexity (in progress)
• Verified: runner, web-validation
• Skipped by policy: browser, Dogfood build #​${{ github.event.pull_request.number }}, full-suite-coverage, suite-coverage, web, web-build, web-database-tests, web-tests
• Full suite: runs on main after merge.

Labeled merged-unverified: if main breaks near this merge, look here first.

[github-actions]

main no longer compiles after this merge

@azooz2003-bit: after 79febcf342 landed on main, the app-host test product (the app and cmuxTests, build-for-testing) stops compiling. These errors first show up in a range of 1 merges (fcbf13cda8..79febcf342), and this pull request's diff is the one that reaches them. The other merges in that range () are being compiled on their own to confirm.

Evidence: https://github.com/manaflow-ai/cmux/actions/runs/36936384119/job/110617739862

Sources/TerminalSSHSessionDetector.swift:208: error: 'shellSingleQuoted' is inaccessible due to 'internal' protection level
Sources/TerminalSSHSessionDetector.swift:212: error: cannot infer contextual base in reference to member 'whitespacesAndNewlines'

Nothing blocks merging meanwhile. A fix-forward (or, failing that, a revert) is attempted automatically unless an open pull request already fixes this.

_{main_compile_attribution.py: post-merge, nothing here gates a merge.}

[teamleaderleo]

Correction to the compile note above: this PR didn't break main. It only touches the iOS TestFlight scripts. The break came from #16523 (SSH image paste), which used a string helper that's internal to another package. #16531 fixed it, and #16533 fixed the attribution so it no longer blames the first PR in the red range. Sorry for the noise.