From 026e22bb8d7b38b3bd66ffcc7d4ee446adfee943 Mon Sep 17 00:00:00 2001 From: Nicola Corti Date: Thu, 7 Aug 2025 08:56:26 -0700 Subject: [PATCH] Deprecate the DefaultDevSupportManagerFactory.create() method used for Old Arch (#53137) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/53137 One of those 2 methods can be deprecated as it was used only for old architecture. We'll be removing it at some point in the future. Changelog: [Android] [Deprecated] - DefaultDevSupportManagerFactory.create() method used for Old Arch Reviewed By: rshest Differential Revision: D79806116 fbshipit-source-id: ad2d5515f93bb85e3b7c495b369078f4c66d143b --- .../main/java/com/facebook/react/ReactInstanceManager.java | 3 +++ .../react/devsupport/DefaultDevSupportManagerFactory.kt | 5 +++++ .../facebook/react/devsupport/DevSupportManagerFactory.kt | 6 ++++++ 3 files changed, 14 insertions(+) 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 560a6648ba6..f5bb7352ac9 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 @@ -229,6 +229,9 @@ public class ReactInstanceManager { return new ReactInstanceManagerBuilder(); } + /** + * @noinspection deprecation + */ /* package */ ReactInstanceManager( Context applicationContext, @Nullable Activity currentActivity, diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DefaultDevSupportManagerFactory.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DefaultDevSupportManagerFactory.kt index 1cca4bb0128..7cb88df90d2 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DefaultDevSupportManagerFactory.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DefaultDevSupportManagerFactory.kt @@ -25,6 +25,11 @@ import com.facebook.react.packagerconnection.RequestHandler */ internal class DefaultDevSupportManagerFactory : DevSupportManagerFactory { + @Deprecated( + "Use the other create() method with useDevSupport parameter for New Architecture. This method will be removed in a future release.", + replaceWith = + ReplaceWith( + "create(applicationContext, reactInstanceManagerHelper, packagerPathForJSBundleName, enableOnCreate, redBoxHandler, devBundleDownloadListener, minNumShakes, customPackagerCommandHandlers, surfaceDelegateFactory, devLoadingViewManager, pausedInDebuggerOverlayManager)")) override fun create( applicationContext: Context, reactInstanceManagerHelper: ReactInstanceDevHelper, diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerFactory.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerFactory.kt index c006de2d7ba..e002388a429 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerFactory.kt +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/DevSupportManagerFactory.kt @@ -22,6 +22,12 @@ public interface DevSupportManagerFactory { * Factory used by the Old Architecture flow to create a [DevSupportManager] and a * [BridgeDevSupportManager] */ + @Deprecated( + message = + "Use the other create() method with useDevSupport parameter for New Architecture. This method will be removed in a future release.", + replaceWith = + ReplaceWith( + "create(applicationContext, reactInstanceManagerHelper, packagerPathForJSBundleName, enableOnCreate, redBoxHandler, devBundleDownloadListener, minNumShakes, customPackagerCommandHandlers, surfaceDelegateFactory, devLoadingViewManager, pausedInDebuggerOverlayManager)")) public fun create( applicationContext: Context, reactInstanceManagerHelper: ReactInstanceDevHelper,