Skip to content

fix: extend iOS physical install timeout#964

Merged
thymikee merged 1 commit into
mainfrom
codex/ios-install-timeout
Jun 30, 2026
Merged

fix: extend iOS physical install timeout#964
thymikee merged 1 commit into
mainfrom
codex/ios-install-timeout

Conversation

@thymikee

Copy link
Copy Markdown
Member

Summary

Extend iOS physical-device install timeouts so larger .app bundles are not killed by the generic 20s devicectl limit.

Closes #963

Validation

Focused unit coverage verifies physical iOS install uses the extended devicectl timeout and install-like daemon requests get a longer client envelope. Ran pnpm format, pnpm check:quick, pnpm check:unit, pnpm build, and pnpm clean:daemon. Verified live on thymikee-iphone by installing AgentDeviceTester.app; the install returned success for com.callstack.agentdevicelab, and apps --all --json showed Agent Device Tester (com.callstack.agentdevicelab) in the device inventory.

Touched 6 files; scope stayed within iOS install timeout handling and regression tests.

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.4 MB 1.4 MB +402 B
JS gzip 450.0 kB 450.1 kB +91 B
npm tarball 549.0 kB 549.0 kB +97 B
npm unpacked 1.9 MB 1.9 MB +402 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 26.7 ms 26.5 ms -0.2 ms
CLI --help 47.4 ms 47.2 ms -0.3 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/9722.js +343 B +74 B
dist/src/495.js +42 B +10 B
dist/src/apps.js +17 B +7 B

@thymikee

Copy link
Copy Markdown
Member Author

Reviewed against #963 and the relevant iOS install/daemon-client timeout path. This targets the exact physical-device xcrun devicectl device install app failure without broadening every devicectl call: install gets IOS_DEVICE_INSTALL_TIMEOUT_MS, other device ops keep the short default, and the daemon client envelope is extended for install/reinstall/install_source so the client does not terminate the request first. The focused tests would fail if either timeout leg were reverted, and the PR includes live physical-device install evidence plus green checks. No blockers; marking ready-for-human.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Jun 30, 2026
@thymikee thymikee force-pushed the codex/ios-install-timeout branch from 773c543 to 7263166 Compare June 30, 2026 13:48
@thymikee thymikee force-pushed the codex/ios-install-timeout branch from 7263166 to b760436 Compare June 30, 2026 15:04
@thymikee thymikee merged commit 903a356 into main Jun 30, 2026
20 checks passed
@thymikee thymikee deleted the codex/ios-install-timeout branch June 30, 2026 15:05
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-30 15:05 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

iOS physical device install times out after 20s with no timeout option

1 participant