ci: correct prebuild-r2 workflow — drop missing tunnel step, output .ipa#15
Merged
Conversation
prebuild-r2.yml (merged in #13) runs on master, which has no SOCKS5 tunnel: no hev-socks5-tunnel submodule, no WebDriverAgentTunnel target, no Scripts/build-hev-socks5-tunnel.sh (those live on the unmerged timo/socks5-vpn branch). The "Build SOCKS5 tunnel xcframework" steps reference a script that does not exist and would fail the build-real / build-sim jobs on the first run. The same fix merged in #14, but that PR targeted the feature branch instead of master, so it never reached master. Reapply it here. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Timo972
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two corrections to the
prebuild-r2.ymlworkflow that #13 merged tomaster.1. Drop the SOCKS5 tunnel xcframework build steps
#13 added "Build SOCKS5 tunnel xcframework" steps that run
Scripts/build-hev-socks5-tunnel.sh. That script — plus thehev-socks5-tunnelsubmodule, theWebDriverAgentTunneltarget, and.gitmodules— only exists on the unmergedtimo/socks5-vpnbranch, not onmaster, so the steps would failbuild-real/build-simon the first run. (#14 fixed this but was merged into the feature branch instead ofmaster, so it never reachedmaster.)2. Publish
.ipainstead of.zipThe publish job now repackages each
<Scheme>-Runner.appinto a standard.ipa(the app under a top-levelPayload/directory) before checksumming and uploading. R2 objects becomeWebDriverAgentRunner-Runner.ipaetc. The sharedScripts/ci/build-real.sh/build-sim.share left untouched becausepublish.js.ymlalso depends on their.zipoutput — the conversion happens entirely inside this workflow.All six matrix variants (iOS + tvOS, real device + simulator arm64/x86_64) are converted. Verified locally that the resulting archive contains
Payload/<App>.app/….🤖 Generated with Claude Code