expo-modules-core 57.0.20 → 57.0.21

package/CHANGELOG.md

@@ -10,6 +10,14 @@
 
 ### 💡 Others
 
+## 57.0.21 — 2026-10-06
+
+### 🐛 Bug fixes
+
+- Fixed listeners removed with `subscription.remove()` never being garbage-collected, together with their emitter and everything their closure references, when the closure can reach the subscription (as in `useEvent`, `useEventListener` or a `useEffect` that returns `() => subscription.remove()`). ([#50603](https://github.com/expo/expo/pull/50603) by [@gkueny](https://github.com/gkueny))
+- [iOS] Fixed a crash (`The app context has been lost`) when an Expo view mounts during a reload. ([#50721](https://github.com/expo/expo/pull/50721) by [@tsapeta](https://github.com/tsapeta))
+- [Android] Fixed Expo views using Android layout not completing system-forced layout passes. ([#48679](https://github.com/expo/expo/pull/48679) by [@lujjjh](https://github.com/lujjjh))
+
 ## 57.0.20 — 2026-09-29
 
 ### 🐛 Bug fixes

package/android/build.gradle

@@ -27,7 +27,7 @@ if (shouldIncludeCompose) {
 }
 
 group = 'host.exp.exponent'
-version = '57.0.20'
+version = '57.0.21'
 
 def isExpoModulesCoreTests = {
   Gradle gradle = getGradle()
@@ -94,7 +94,7 @@ android {
   defaultConfig {
     consumerProguardFiles 'proguard-rules.pro'
     versionCode 1
-    versionName "57.0.20"
+    versionName "57.0.21"
     buildConfigField "String", "EXPO_MODULES_CORE_VERSION", "\"${versionName}\""
     buildConfigField "boolean", "IS_NEW_ARCHITECTURE_ENABLED", "true"
 

package/android/src/compose/expo/modules/kotlin/views/ExpoComposeView.kt

@@ -2,7 +2,6 @@ package expo.modules.kotlin.views
 
 import android.annotation.SuppressLint
 import android.content.Context
-import android.content.res.Configuration
 import android.util.Log
 import android.view.View
 import android.view.ViewGroup
@@ -114,15 +113,6 @@ abstract class ExpoComposeView<T : ComposeProps>(
     }
   }
 
-  override fun dispatchConfigurationChanged(newConfig: Configuration) {
-    super.dispatchConfigurationChanged(newConfig)
-    // React Native owns this view's bounds and may not schedule another Android layout pass
-    // when a configuration change leaves its Yoga layout unchanged.
-    if (withHostingView && isAttachedToWindow && isLaidOut) {
-      requestLayout()
-    }
-  }
-
   /**
    * Validates that this non-hosting Compose view has a valid Compose parent.
    *

package/android/src/main/java/expo/modules/kotlin/views/ExpoView.kt

@@ -35,7 +35,8 @@ abstract class ExpoView(
 
   /**
    * Manually trigger measure and layout.
-   * If [shouldUseAndroidLayout] is set to `true`, this method will be called automatically after [requestLayout].
+   * If [shouldUseAndroidLayout] is set to `true`, this method will be called automatically after
+   * [requestLayout] or [forceLayout].
    */
   @UiThread
   fun measureAndLayout() {
@@ -54,6 +55,19 @@ abstract class ExpoView(
     }
   }
 
+  override fun forceLayout() {
+    super.forceLayout()
+    if (shouldUseAndroidLayout && isAttachedToWindow && isLaidOut) {
+      // ViewRootImpl can force-layout the hierarchy without React Native laying out its
+      // managed children. Preserve the Android-layout contract at that boundary.
+      post {
+        if (isAttachedToWindow && isLayoutRequested) {
+          measureAndLayout()
+        }
+      }
+    }
+  }
+
   open fun clipToPaddingBox(canvas: Canvas) {
     // When the border radius is set, we need to clip the content to the padding box.
     // This is because the border radius is applied to the background drawable, not the view itself.

package/common/cpp/EventEmitter.cpp

@@ -188,10 +188,20 @@ jsi::Value createEventSubscription(jsi::Runtime &runtime, const std::string &eve
   std::shared_ptr<jsi::Value> listenerValue = std::make_shared<jsi::Value>(runtime, listener);
 
   jsi::HostFunctionType removeSubscription = [eventName, emitterValue, listenerValue](jsi::Runtime &runtime, const jsi::Value &thisValue, const jsi::Value *args, size_t count) -> jsi::Value {
+    if (!listenerValue->isObject()) {
+      // The subscription has already been removed.
+      return jsi::Value::undefined();
+    }
     jsi::Object emitter = emitterValue->getObject(runtime);
     jsi::Function listener = listenerValue->getObject(runtime).getFunction(runtime);
 
     removeListener(runtime, emitter, eventName, listener);
+
+    // Values held by this host function are GC roots. The listener's closure often references the subscription
+    // (e.g. a React effect returning `() => subscription.remove()`), so keeping them after the removal would form
+    // a cycle through a root that the garbage collector can never break, leaking the listener and the emitter.
+    *emitterValue = jsi::Value::undefined();
+    *listenerValue = jsi::Value::undefined();
     return jsi::Value::undefined();
   };
 

package/ios/Core/AppContext.swift

@@ -120,17 +120,30 @@ public final class AppContext: NSObject, EXAppContextProtocol, @unchecked Sendab
     }
   }
 
-  /**
-   The application identifier that is used to distinguish between different `RCTHost`.
-   It might be equal to `nil`, meaning we couldn't obtain the Id for the current app.
-   It shouldn't be used on the old architecture.
-   */
+  /// The number of app contexts created so far in this process.
+  private static let appContextsCount = Mutex<Int>(0)
+
+  /// The position of this app context in creation order, starting at 0.
+  internal let appIndex: Int = AppContext.appContextsCount.withLock { count in
+    defer {
+      count += 1
+    }
+    return count
+  }
+
+  /// The application identifier that distinguishes app contexts that are alive at the same time,
+  /// for example during a reload or when more than one `RCTHost` is running. It's `nil` for the first
+  /// app context, so its view names have no suffix.
   @objc
   public var appIdentifier: String? {
-    guard let moduleRegistry = reactBridge?.moduleRegistry else {
+    return AppContext.appIdentifier(forIndex: appIndex)
+  }
+
+  internal static func appIdentifier(forIndex index: Int) -> String? {
+    if index == 0 {
       return nil
     }
-    return "\(abs(ObjectIdentifier(moduleRegistry).hashValue))"
+    return String(index)
   }
 
   /**

package/ios/Platform/Platform.h

@@ -18,7 +18,12 @@
 @compatibility_alias UIImage NSImage;
 @compatibility_alias UIImageView NSImageView;
 
-#ifndef UIApplication
+// `@compatibility_alias` doesn't define a preprocessor macro, so `#ifndef
+// UIApplication` never detected that `React/RCTUIKit.h` (imported above)
+// already declares this same alias. Under Swift Explicit Modules, that
+// duplicate declaration becomes a real Clang conflict. Guard on the
+// header's presence instead.
+#if !__has_include(<React/RCTUIKit.h>)
 @compatibility_alias UIApplication NSApplication;
 #endif
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-modules-core",
-  "version": "57.0.20",
+  "version": "57.0.21",
   "description": "The core of Expo Modules architecture",
   "main": "src/index.ts",
   "types": "build/index.d.ts",
@@ -66,7 +66,7 @@
     "@types/invariant": "^2.2.33",
     "expo-module-scripts": "56.0.3"
   },
-  "gitHead": "ad1efa3fa5802156660198ba5d787f91014d8df3",
+  "gitHead": "50ac74f1ab5682c8e32b24da6e5e97dd4dbbb80f",
   "scripts": {
     "build": "expo-module build",
     "clean": "expo-module clean",