Skip to content

ci: make TestFlight cache optional - #223

Merged
0xSero merged 1 commit into
mainfrom
codex/optional-ios-release-cache
Aug 4, 2026
Merged

ci: make TestFlight cache optional#223
0xSero merged 1 commit into
mainfrom
codex/optional-ios-release-cache

Conversation

@0xSero

@0xSero 0xSero commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Purpose

Keep the standalone TestFlight release runnable when the optional remote sccache credentials are not installed.

Root cause

Run 30951375473 passed the live App Store Connect preflight, then failed before compilation because SCCACHE_R2_ENDPOINT and the R2 access secrets were empty while the workflow still forced RUSTC_WRAPPER=sccache and unconditionally started the remote cache.

Changes

  • configure sccache only when the endpoint and both credentials are present
  • skip setup/priming when the cache is unavailable
  • let Cargo run directly instead of forcing an unusable wrapper

This matches the existing optional-cache behavior in the merged Mobile Release lane.

Verification

  • parsed the workflow as YAML
  • exercised both cache-enabled and cache-disabled configuration paths from the workflow source
  • Actionlint passed, ignoring only the pre-existing SC2129 style finding elsewhere in the workflow
  • git diff --check

@0xSero 0xSero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-review complete: no blocking findings.

I traced both environment branches from the workflow source:

  • with endpoint + access key + secret key, the workflow sets RUSTC_WRAPPER=sccache and runs setup/priming;
  • if any value is absent, it clears the remote-cache environment, skips setup/priming, and leaves Cargo without a forced wrapper.

The change does not log secret values, does not affect signing or App Store credentials, and is limited to the standalone TestFlight preparation job. YAML parsing, both executable configuration-path assertions, Actionlint (apart from the pre-existing SC2129 style finding), and git diff --check pass.

@0xSero
0xSero merged commit 29dd2b1 into main Aug 4, 2026
7 checks passed
@0xSero
0xSero deleted the codex/optional-ios-release-cache branch August 4, 2026 21:20
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