diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java index 04affe8015d..090e806461c 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/ReactInstanceManager.java @@ -328,6 +328,15 @@ public class ReactInstanceManager { } registerCxxErrorHandlerFunc(); + + // Using `if (true)` just to prevent tests / lint errors. + if (true) { + // Legacy architecture of React Native is deprecated and can't be initialized anymore. + // More details on: + // https://github.com/react-native-community/discussions-and-proposals/blob/nc/legacy-arch-removal/proposals/0929-legacy-architecture-removal.md + throw new UnsupportedOperationException( + "ReactInstanceManager.createReactContext is unsupported."); + } } private ReactInstanceDevHelper createDevHelperInterface() { @@ -1446,6 +1455,7 @@ public class ReactInstanceManager { */ private ReactApplicationContext createReactContext( JavaScriptExecutor jsExecutor, JSBundleLoader jsBundleLoader) { + FLog.d(ReactConstants.TAG, "ReactInstanceManager.createReactContext()"); ReactMarker.logMarker(CREATE_REACT_CONTEXT_START, jsExecutor.getName()); diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPoint.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPoint.kt index ca81f28d83b..cb9cdb1749a 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPoint.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPoint.kt @@ -125,6 +125,16 @@ public object DefaultNewArchitectureEntryPoint { privateConcurrentReactEnabled = featureFlags.enableFabricRenderer() privateBridgelessEnabled = featureFlags.enableBridgelessArchitecture() + val (isValid, errorMessage) = + isConfigurationValid( + privateTurboModulesEnabled, + privateFabricEnabled, + privateBridgelessEnabled, + ) + if (!isValid) { + error(errorMessage) + } + DefaultSoLoader.maybeLoadSoLibrary() } @@ -158,13 +168,13 @@ public object DefaultNewArchitectureEntryPoint { fabricEnabled: Boolean, bridgelessEnabled: Boolean, ): Pair = - when { - fabricEnabled && !turboModulesEnabled -> - false to - "fabricEnabled=true requires turboModulesEnabled=true (is now false) - Please update your DefaultNewArchitectureEntryPoint.load() parameters." - bridgelessEnabled && (!turboModulesEnabled || !fabricEnabled) -> - false to - "bridgelessEnabled=true requires (turboModulesEnabled=true AND fabricEnabled=true) - Please update your DefaultNewArchitectureEntryPoint.load() parameters." - else -> true to "" + if (!turboModulesEnabled || !fabricEnabled || !bridgelessEnabled) { + false to + "You cannot load React Native with the New Architecture disabled. " + + "Please use DefaultNewArchitectureEntryPoint.load() instead of " + + "DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=$turboModulesEnabled, " + + "fabricEnabled=$fabricEnabled, bridgelessEnabled=$bridgelessEnabled)" + } else { + true to "" } } diff --git a/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPointTest.kt b/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPointTest.kt index 68a1d720572..5727d2fc310 100644 --- a/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPointTest.kt +++ b/packages/react-native/ReactAndroid/src/test/java/com/facebook/react/defaults/DefaultNewArchitectureEntryPointTest.kt @@ -13,36 +13,48 @@ import org.junit.Test class DefaultNewArchitectureEntryPointTest { @Test - fun isConfigurationValid_withEverythingOff_returnsTrue() { - val (isValid, _) = + fun isConfigurationValid_withEverythingOff_returnsFalse() { + val (isValid, errorMessage) = DefaultNewArchitectureEntryPoint.isConfigurationValid( turboModulesEnabled = false, fabricEnabled = false, bridgelessEnabled = false, ) - assertThat(isValid).isTrue() + assertThat(isValid).isFalse() + assertThat(errorMessage) + .isEqualTo( + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=false, fabricEnabled=false, bridgelessEnabled=false)" + ) } @Test - fun isConfigurationValid_withNewArchOn_returnsTrue() { - val (isValid, _) = + fun isConfigurationValid_withNewArchOnlyOn_returnsFalse() { + val (isValid, errorMessage) = DefaultNewArchitectureEntryPoint.isConfigurationValid( turboModulesEnabled = true, fabricEnabled = true, bridgelessEnabled = false, ) - assertThat(isValid).isTrue() + assertThat(isValid).isFalse() + assertThat(errorMessage) + .isEqualTo( + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=true, fabricEnabled=true, bridgelessEnabled=false)" + ) } @Test - fun isConfigurationValid_withTurboModulesOnlyOn_returnsTrue() { - val (isValid, _) = + fun isConfigurationValid_withTurboModulesOnlyOn_returnsFalse() { + val (isValid, errorMessage) = DefaultNewArchitectureEntryPoint.isConfigurationValid( turboModulesEnabled = true, fabricEnabled = false, bridgelessEnabled = false, ) - assertThat(isValid).isTrue() + assertThat(isValid).isFalse() + assertThat(errorMessage) + .isEqualTo( + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=true, fabricEnabled=false, bridgelessEnabled=false)" + ) } @Test @@ -67,7 +79,7 @@ class DefaultNewArchitectureEntryPointTest { assertThat(isValid).isFalse() assertThat(errorMessage) .isEqualTo( - "fabricEnabled=true requires turboModulesEnabled=true (is now false) - Please update your DefaultNewArchitectureEntryPoint.load() parameters." + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=false, fabricEnabled=true, bridgelessEnabled=false)" ) } @@ -82,7 +94,7 @@ class DefaultNewArchitectureEntryPointTest { assertThat(isValid).isFalse() assertThat(errorMessage) .isEqualTo( - "fabricEnabled=true requires turboModulesEnabled=true (is now false) - Please update your DefaultNewArchitectureEntryPoint.load() parameters." + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=false, fabricEnabled=true, bridgelessEnabled=true)" ) } @@ -97,7 +109,7 @@ class DefaultNewArchitectureEntryPointTest { assertThat(isValid).isFalse() assertThat(errorMessage) .isEqualTo( - "bridgelessEnabled=true requires (turboModulesEnabled=true AND fabricEnabled=true) - Please update your DefaultNewArchitectureEntryPoint.load() parameters." + "You cannot load React Native with the New Architecture disabled. Please use DefaultNewArchitectureEntryPoint.load() instead of DefaultNewArchitectureEntryPoint.load(turboModulesEnabled=true, fabricEnabled=false, bridgelessEnabled=true)" ) } }