Skip to content

fix: align iOS runner cache target metadata#682

Merged
thymikee merged 2 commits into
mainfrom
fix/ios-runner-cache-target
Jun 3, 2026
Merged

fix: align iOS runner cache target metadata#682
thymikee merged 2 commits into
mainfrom
fix/ios-runner-cache-target

Conversation

@thymikee
Copy link
Copy Markdown
Member

@thymikee thymikee commented Jun 3, 2026

Summary

Align iOS runner cache metadata with the canonical mobile device target so CI prebuilt XCTest runner artifacts are reused instead of rejected as cache_metadata_mismatch.

Adds a focused regression test that compares the setup metadata script output with runtime expected metadata, and keeps script stderr visible if that subprocess fails.

Runner artifacts cached with the old target: phone metadata will rebuild once after this lands, then reuse the corrected target: mobile metadata. Local dev caches already written by runtime prepare use mobile and are unaffected.

Touched files: 3. Scope stayed within iOS runner cache metadata.

Validation

Verified the focused iOS runner cache test passes with pnpm exec vitest run src/platforms/ios/tests/runner-xctestrun.test.ts. Also ran pnpm check:quick; lint and typecheck passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Size Report

Metric Base Current Diff
JS raw 1.1 MB 1.1 MB +1 B
JS gzip 367.5 kB 367.5 kB -2 B
npm tarball 473.5 kB 473.5 kB -2 B
npm unpacked 1.6 MB 1.6 MB +1 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 26.6 ms 26.0 ms -0.7 ms
CLI --help 41.9 ms 40.9 ms -1.0 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/2415.js +1 B -2 B

@thymikee thymikee merged commit 86db7e8 into main Jun 3, 2026
18 checks passed
@thymikee thymikee deleted the fix/ios-runner-cache-target branch June 3, 2026 16:35
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-06-03 16:35 UTC

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant