chore: Update main-dev for expo and release build configurations#29572
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
| else | ||
| fi | ||
|
|
||
| if [ "$IS_DEVICE_BUILD" = "true" ] || [ -z "$IS_SIM_BUILD" ]; then |
There was a problem hiding this comment.
Device build fallback condition narrower than original
Low Severity
The device build fallback changed from an else (triggered for any IS_SIM_BUILD value that isn't "true") to [ -z "$IS_SIM_BUILD" ] (triggered only when IS_SIM_BUILD is empty/unset). If IS_SIM_BUILD is ever set to a non-empty value other than "true" (e.g. "false"), and IS_DEVICE_BUILD is also not "true", the new code produces no iOS build at all — neither simulator nor device. The old code would correctly fall through to a device build via the else branch. Using [ "$IS_SIM_BUILD" != "true" ] instead of [ -z "$IS_SIM_BUILD" ] would preserve the original semantics.
Reviewed by Cursor Bugbot for commit 8690d5d. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
None of these changes affect:
Since these are purely build/CI infrastructure changes that don't affect the app's behavior or test infrastructure, no E2E test tags need to be run. Performance Test Selection: |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 3 total unresolved issues (including 2 from previous reviews).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit dc86270. Configure here.
|





Description
This PR updates build variants:
main-dev-expofor producing Expo debug dev builds (.app, .ipa, .apk, test .apk)main-devfor producing release dev builds for installation via side loadingSlack thread - https://consensys.slack.com/archives/C02U025CVU4/p1776795191936139
Changelog
CHANGELOG entry:
Related issues
Fixes: https://consensyssoftware.atlassian.net/browse/MCWP-561
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Medium Risk
Moderate risk because it changes CI build selection and iOS signing/export behavior (including overriding signing settings for Release archives), which could break artifact generation or produce incorrectly signed builds.
Overview
Separates dev build configurations by introducing
main-dev-expo(Debug Expo dev builds that produce simulator + device iOS artifacts and Android APK + test APK) while repurposingmain-devas a dev release build intended for sideloading.Updates the reusable
build.ymlworkflow to support an optionalscript_nameoverride frombuilds.yml, so different build configs can reuse the same underlying yarn build script. Adjustsscripts/build.shiOS packaging to use a newPROFILEflag for selecting export options and to allow development signing overrides when creating a Release archive.Reviewed by Cursor Bugbot for commit dc86270. Bugbot is set up for automated code reviews on this repo. Configure here.