[tomekzaw]

Summary:

In bridgeless mode, eager modules are created on the native modules thread while the JS bundle loads (ReactHostImpl). FrescoModule is one of them, and it initializes Fresco. If the first render preallocates an Image before that has finished, ReactImageManager.createViewInstance calls Fresco.newDraweeControllerBuilder() too early and the app crashes on startup:

java.lang.NullPointerException
  at com.facebook.common.internal.Preconditions.checkNotNull(Preconditions.java:213)
  at com.facebook.drawee.backends.pipeline.Fresco.newDraweeControllerBuilder(Fresco.java:139)
  at com.facebook.react.views.image.ReactImageManager.createViewInstance(ReactImageManager.kt:95)
  at com.facebook.react.fabric.mounting.SurfaceMountingManager.preallocateView(SurfaceMountingManager.kt:1123)

When FrescoModule hasn't finished initializing Fresco, createViewInstance now gets the module first. TurboModuleManager then either waits for the other thread to finish creating it or creates it on the spot. This uses FrescoModule.hasBeenInitialized() rather than Fresco.hasBeenInitialized(), because Fresco sets its flag when initialization starts, not when it ends. Once Fresco is initialized, the only extra work is reading that flag.

Changelog:

[ANDROID] [FIXED] - Fix a startup crash when an Image is created before Fresco finishes initializing

Test Plan:

Release RNTester (./gradlew :packages:rn-tester:android:app:assembleRelease -PreactNativeArchitectures=arm64-v8a), cold-started from the launcher repeatedly on a Galaxy A16 (Android 16). A run counts as a crash when the log has the stack trace above.

Feature flags	main	This PR
Default	5 of 40 cold starts crashed	0 of 40
Experimental release level	13 of 40	0 of 40

After the change, the Image examples load their network images as before.

🤖 Generated with Claude Code

[Abbondanzo]

Fresco initialization can be pretty costly, and the createViewInstance method is called on the main thread. Since the builder isn't called until the image mounts and has dimensions, could we instead make the constructor argument a Provider<T> for a drawee controller builder?