Fix Expo widget asset wiring order

[juliusmarminge]

Summary

• Reordered the mobile Expo plugins so withWidgetLogoAsset registers before expo-widgets, ensuring its Xcode mods run after the widget target is created.
• Made widget asset wiring fail loudly when the target is missing instead of silently skipping, which prevents shipping a widget without its logo assets.
• Updated the widget asset-catalog script and plugin comments to reflect the new ordering and failure behavior.

Testing

• Not run (PR description only).
• Existing project checks should still be run before merge: vp check and vp run typecheck.
• For mobile changes, also run vp run lint:mobile.

---

Note

Medium Risk
Changes only mobile iOS prebuild/plugin wiring, but incorrect ordering or a thrown error could block releases until fixed; the fix directly affects widget extension assets in production builds.

Overview
Fixes iOS widget branding by reordering Expo config plugins so withWidgetLogoAsset is registered before expo-widgets. Same-type mods run last-registered-first, so this makes the logo plugin run after expo-widgets creates ExpoWidgetsTarget and after its dangerous mod clears ios/ExpoWidgetsTarget/—the previous order caused a fresh prebuild to no-op and ship widgets without the T3 mark (prod build 8).

Fails loudly instead of silently skipping: addWidgetAssetCatalog now throws if the widget Xcode target is missing, and the actool shell phase errors if Assets.xcassets is absent. Comments and the one-off wire-widget-asset-catalog script message were updated to match.

^{Reviewed by Cursor Bugbot for commit 89ae300. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Fix Expo widget asset wiring by registering withWidgetLogoAsset before expo-widgets

• Reorders plugins in app.config.ts so withWidgetLogoAsset is listed before expo-widgets, ensuring its mods run after expo-widgets' mods (same-type mods execute in reverse registration order). This prevents expo-widgets from wiping the asset catalog and ensures the widget target exists before asset wiring.
• Changes addWidgetAssetCatalog in addWidgetAssetCatalog.cjs to throw an error when the PBXNativeTarget is missing instead of silently returning false, causing prebuild to fail fast on misconfiguration.
• The Xcode shell-script build phase now exits with code 1 (error) instead of 0 (warning) when the widget asset catalog is missing at build time.

^{Macroscope summarized 89ae300.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 78e68d09-c9fa-4024-b337-40395ba2d95f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch t3code/fix-widget-embed

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved

Build configuration fix that corrects Expo plugin ordering for widget asset wiring. Changes are limited to build-time behavior, making error handling stricter rather than silently failing. Self-contained bug fix with clear intent.

^{You can customize Macroscope's approvability policy. Learn more.}