[reference][native WIP] ci(native): macOS native build smoke lane (ERY-99)#598
[reference][native WIP] ci(native): macOS native build smoke lane (ERY-99)#598SheetMetalConnect wants to merge 2 commits into
Conversation
…ERY-99) Adds a repo-owned GitHub Actions lane (macos-15) that runs ios:init, builds the iOS simulator target, and assembles Android debug+release APKs so the ERY-71 native packaging changes get real build evidence. Hardens scripts/ios-init.sh to restore the darwin-arm64 Rollup/SWC bindings npm ci skips on Apple Silicon. Validation doc updated with the lane decision and local host evidence. Co-Authored-By: Paperclip <noreply@paperclip.ing>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
eryxon-flow-website | 66dcefb | Commit Preview URL Branch Preview URL |
May 24 2026, 07:31 PM |
Run 26370608416 (macos-15): toolchain provisioned OK, cap add ios + web build OK, pod install FAILED on GoogleMLKit/BarcodeScanning 7.0.0 deployment-target mismatch; Android steps skipped (job halts on first failure). Captures root cause, proposed iOS fix, and a workflow split follow-up so Android evidence is no longer masked by the iOS step. Co-Authored-By: Paperclip <noreply@paperclip.ing>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: adc5b4d6de
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| Android SDK, or a CI macOS runner) so iOS bootstrap and the Android build smoke | ||
| can produce pass/fail evidence. Tracked as the ERY-71 build-validation follow-up. | ||
| - At the v0.6 release cut, perform the coordinated `0.6.0` version bump atomically. | ||
| - The pre-TestFlight items remain explicitly tracked in [docs/IOS.md](/Users/vanenkhuizen/Documents/GitHub/products/eryxon-flow/docs/IOS.md): APNs wiring, Associated Domains / Universal Links, splash/icon assets, and App Store metadata polish. |
There was a problem hiding this comment.
Replace this absolute macOS filesystem path with a repo-relative link (for example docs/IOS.md or ./IOS.md from this folder). As written, /Users/vanenkhuizen/... only works on one machine and resolves to a broken path in GitHub, so readers cannot reach the referenced iOS blocker list during validation review.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 66dcefbbf7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| CODE_SIGNING_ALLOWED=NO \ | ||
| build 2>&1 | tee artifacts/ios-build.log | xcbeautify | ||
|
|
||
| - name: Assemble Android debug APK |
There was a problem hiding this comment.
Keep Android validation independent from the iOS bootstrap in this workflow: as written, a non-zero exit from npm run ios:init stops the entire native-build-smoke job before either Android assemble step runs, so Android regressions are never exercised whenever iOS is broken. In practice this defeats the lane’s stated goal of collecting smoke evidence for both platforms in a single run.
Useful? React with 👍 / 👎.
SheetMetalConnect
changed the title
|
Native macOS build smoke lane — intentionally NOT part of stable v0.6 (#603). Native stays roadmap / WIP. The two ERY-99 commits on |
|
Native mobile deferred per board directive. Closing. |
1 participant
Lands the macOS native build smoke lane for ERY-81/ERY-99 against the ERY-71 native packaging tree.
What this runs
npm run ios:init(cap add ios + pod install) on macos-15ios/App/App.xcworkspacenpm run android:assemble:debugand:release(validates ML Kitmlkit.vision.DEPENDENCIESmeta-data + rewrittennetwork_security_config.xml)Why a PR
The
pull_requestpath filter on the workflow triggers the lane against a tree that contains the ERY-71 native changes, producing the first real pass/fail evidence. Base isengineer/ery-71-native-packagingso the diff stays scoped to the workflow + Apple-Silicon ios-init repair + validation doc.Not for merge yet — this PR exists to execute the smoke lane and capture evidence.
🤖 Generated with Claude Code