Compile out RCTCxxBridge when legacy arch is disabled (#56837)

packages/react-native/Libraries/AppDelegate/RCTAppDelegate.h

@@ -66,19 +66,20 @@ __attribute__((deprecated(
 /// The window object, used to render the UViewControllers
 @property (nonatomic, strong, nonnull) UIWindow *window;
 
+#if !defined(RCT_REMOVE_LEGACY_ARCH)
 @property (nonatomic, nullable) RCTBridge *bridge
     __attribute__((deprecated("The bridge is deprecated and will be removed when removing the legacy architecture.")));
+@property (nonatomic, nullable) RCTSurfacePresenterBridgeAdapter *bridgeAdapter __attribute__((
+    deprecated("The bridge adapter is deprecated and will be removed when removing the legacy architecture.")));
+#endif
+
 @property (nonatomic, strong, nullable) NSString *moduleName;
 @property (nonatomic, strong, nullable) NSDictionary *initialProps;
 @property (nonatomic, strong) RCTReactNativeFactory *reactNativeFactory;
 
 /// If `automaticallyLoadReactNativeWindow` is set to `true`, the React Native window will be loaded automatically.
 @property (nonatomic, assign) BOOL automaticallyLoadReactNativeWindow;
 
-@property (nonatomic, nullable) RCTSurfacePresenterBridgeAdapter *bridgeAdapter __attribute__((
-    deprecated("The bridge adapter is deprecated and will be removed when removing the legacy architecture.")));
-;
-
 - (RCTRootViewFactory *)rootViewFactory;
 
 @end

packages/react-native/Libraries/AppDelegate/RCTAppDelegate.mm

@@ -67,6 +67,7 @@ - (RCTRootViewFactory *)rootViewFactory
   return self.reactNativeFactory.rootViewFactory;
 }
 
+#if !defined(RCT_REMOVE_LEGACY_ARCH)
 - (RCTBridge *)bridge
 {
   return self.rootViewFactory.bridge;
@@ -86,5 +87,6 @@ - (void)setBridgeAdapter:(RCTSurfacePresenterBridgeAdapter *)bridgeAdapter
 {
   self.reactNativeFactory.rootViewFactory.bridgeAdapter = bridgeAdapter;
 }
+#endif
 
 @end

packages/react-native/Libraries/AppDelegate/RCTRootViewFactory.h

@@ -184,9 +184,12 @@ typedef void (^RCTLoadSourceForBridgeBlock)(RCTBridge *bridge, RCTSourceLoadBloc
  */
 @interface RCTRootViewFactory : NSObject
 
+#if !defined(RCT_REMOVE_LEGACY_ARCH)
 @property (nonatomic, strong, nullable) RCTBridge *bridge;
-@property (nonatomic, strong, nullable) RCTHost *reactHost;
 @property (nonatomic, strong, nullable) RCTSurfacePresenterBridgeAdapter *bridgeAdapter;
+#endif
+
+@property (nonatomic, strong, nullable) RCTHost *reactHost;
 
 - (instancetype)initWithConfiguration:(RCTRootViewFactoryConfiguration *)configuration
         andTurboModuleManagerDelegate:(id<RCTTurboModuleManagerDelegate> _Nullable)turboModuleManagerDelegate;

packages/react-native/Libraries/AppDelegate/RCTRootViewFactory.mm

@@ -6,11 +6,9 @@
  */
 
 #import "RCTRootViewFactory.h"
-#import <React/RCTCxxBridgeDelegate.h>
 #import <React/RCTDevMenu.h>
 #import <React/RCTLog.h>
 #import <React/RCTRootView.h>
-#import <React/RCTSurfacePresenterBridgeAdapter.h>
 #import <React/RCTUtils.h>
 #import <react/renderer/runtimescheduler/RuntimeScheduler.h>
 #import "RCTAppDelegate.h"
@@ -90,7 +88,7 @@ - (instancetype)initWithBundleURL:(NSURL *)bundleURL
 
 @end
 
-@interface RCTRootViewFactory () <RCTCxxBridgeDelegate> {
+@interface RCTRootViewFactory () {
   std::shared_ptr<const facebook::react::ContextContainer> _contextContainer;
   std::shared_ptr<facebook::react::RuntimeScheduler> _runtimeScheduler;
 }
@@ -201,63 +199,6 @@ - (UIView *)viewWithModuleName:(NSString *)moduleName
   return surfaceHostingProxyRootView;
 }
 
-- (RCTBridge *)createBridgeWithDelegate:(id<RCTBridgeDelegate>)delegate launchOptions:(NSDictionary *)launchOptions
-{
-  return [[RCTBridge alloc] initWithDelegate:delegate launchOptions:launchOptions];
-}
-
-- (UIView *)createRootViewWithBridge:(RCTBridge *)bridge
-                          moduleName:(NSString *)moduleName
-                           initProps:(NSDictionary *)initProps
-{
-  UIView *rootView = RCTAppSetupDefaultRootView(bridge, moduleName, initProps, YES);
-#if !TARGET_OS_TV
-  rootView.backgroundColor = [UIColor systemBackgroundColor];
-#else
-  rootView.backgroundColor = [UIColor blackColor];
-#endif
-  return rootView;
-}
-
-#pragma mark - RCTCxxBridgeDelegate
-- (std::unique_ptr<facebook::react::JSExecutorFactory>)jsExecutorFactoryForBridge:(RCTBridge *)bridge
-{
-  _runtimeScheduler = std::make_shared<facebook::react::RuntimeScheduler>(RCTRuntimeExecutorFromBridge(bridge));
-
-  std::shared_ptr<facebook::react::CallInvoker> callInvoker =
-      std::make_shared<facebook::react::RuntimeSchedulerCallInvoker>(_runtimeScheduler);
-  RCTTurboModuleManager *turboModuleManager = [[RCTTurboModuleManager alloc] initWithBridge:bridge
-                                                                                   delegate:_turboModuleManagerDelegate
-                                                                                  jsInvoker:callInvoker];
-  _contextContainer->erase(facebook::react::RuntimeSchedulerKey);
-  _contextContainer->insert(facebook::react::RuntimeSchedulerKey, _runtimeScheduler);
-  return RCTAppSetupDefaultJsExecutorFactory(bridge, turboModuleManager, _runtimeScheduler);
-}
-
-- (void)createBridgeIfNeeded:(NSDictionary *)launchOptions
-{
-  if (self.bridge != nil) {
-    return;
-  }
-
-  if (self->_configuration.createBridgeWithDelegate != nil) {
-    self.bridge = self->_configuration.createBridgeWithDelegate(self, launchOptions);
-  } else {
-    self.bridge = [self createBridgeWithDelegate:self launchOptions:launchOptions];
-  }
-}
-
-- (void)createBridgeAdapterIfNeeded
-{
-  if (self.bridgeAdapter != nullptr) {
-    return;
-  }
-
-  self.bridgeAdapter = [[RCTSurfacePresenterBridgeAdapter alloc] initWithBridge:self.bridge
-                                                               contextContainer:_contextContainer];
-  self.bridge.surfacePresenter = self.bridgeAdapter.surfacePresenter;
-}
-
 #pragma mark - New Arch Utilities
 
 - (void)createReactHostIfNeeded:(NSDictionary *)launchOptions
@@ -324,6 +265,7 @@ - (RCTHost *)createReactHost:(NSDictionary *)launchOptions
   return nil;
 }
 
+#ifndef RCT_REMOVE_LEGACY_ARCH
 - (NSDictionary<NSString *, Class> *)extraLazyModuleClassesForBridge:(RCTBridge *)bridge
 {
   if (_configuration.extraLazyModuleClassesForBridge != nil) {
@@ -363,6 +305,7 @@ - (void)loadSourceForBridge:(RCTBridge *)bridge
     _configuration.loadSourceForBridgeWithProgress(bridge, onProgress, loadCallback);
   }
 }
+#endif
 
 - (NSURL *)bundleURL
 {

packages/react-native/React/Base/RCTBridge+Private.h

@@ -145,14 +145,14 @@ RCT_EXTERN void RCTRegisterModule(Class);
 
 @end
 
+#if !defined(RCT_REMOVE_LEGACY_ARCH)
 @interface RCTCxxBridge : RCTBridge
 
 // TODO(cjhopman): this seems unsafe unless we require that it is only called on the main js queue.
 @property (nonatomic, readonly) void *runtime;
 
-#ifndef RCT_REMOVE_LEGACY_ARCH
 - (instancetype)initWithParentBridge:(RCTBridge *)bridge NS_DESIGNATED_INITIALIZER
     __deprecated_msg("This API will be removed along with the legacy architecture.");
-#endif // RCT_REMOVE_LEGACY_ARCH
 
 @end
+#endif // RCT_REMOVE_LEGACY_ARCH

packages/react-native/React/Base/RCTBridge.mm

@@ -550,7 +550,11 @@ - (void)requestReload
 
 - (Class)bridgeClass
 {
+#if !defined(RCT_REMOVE_LEGACY_ARCH)
   return [RCTCxxBridge class];
+#else
+  return nil;
+#endif
 }
 
 - (void)setUp

packages/react-native/React/Base/RCTBridgeProxy.mm

@@ -387,7 +387,11 @@ - (RCTBridgeProxy *)object
  */
 - (NSMethodSignature *)methodSignatureForSelector:(SEL)sel;
 {
+#ifndef RCT_REMOVE_LEGACY_ARCH
   return [RCTCxxBridge instanceMethodSignatureForSelector:sel];
+#else
+  return [RCTBridge instanceMethodSignatureForSelector:sel];
+#endif
 }
 
 - (void)forwardInvocation:(NSInvocation *)invocation

packages/react-native/React/CxxBridge/RCTCxxBridge.mm

@@ -1566,7 +1566,4 @@ - (RCTCallInvoker *)callInvokerForModuleData:(RCTModuleData *)moduleData
 
 @end
 
-#else // RCT_REMOVE_LEGACY_ARCH
-@implementation RCTCxxBridge
-@end
 #endif // RCT_REMOVE_LEGACY_ARCH

packages/react-native/React/Fabric/RCTSurfacePresenterBridgeAdapter.h

@@ -15,6 +15,7 @@ NS_ASSUME_NONNULL_BEGIN
 @class RCTSurfacePresenter;
 @class RCTBridge;
 
+#ifndef RCT_REMOVE_LEGACY_ARCH
 facebook::react::RuntimeExecutor RCTRuntimeExecutorFromBridge(RCTBridge *bridge);
 
 /*
@@ -39,5 +40,6 @@ facebook::react::RuntimeExecutor RCTRuntimeExecutorFromBridge(RCTBridge *bridge)
 @property (nonatomic, weak) RCTBridge *bridge;
 
 @end
+#endif
 
 NS_ASSUME_NONNULL_END

packages/react-native/React/Fabric/RCTSurfacePresenterBridgeAdapter.mm

@@ -24,8 +24,8 @@
 
 using namespace facebook::react;
 
+#ifndef RCT_REMOVE_LEGACY_ARCH
 @interface RCTBridge ()
-- (std::shared_ptr<facebook::react::MessageQueueThread>)jsMessageThread;
 - (void)invokeAsync:(std::function<void()> &&)func;
 @end
 
@@ -203,3 +203,4 @@ - (void)handleJavaScriptDidLoadNotification:(NSNotification *)notification
 }
 
 @end
+#endif

packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/ReactCommon/RCTTurboModuleManager.mm

@@ -7,6 +7,7 @@
 
 #import "RCTTurboModuleManager.h"
 #import "RCTInteropTurboModule.h"
+#import "RCTTurboModule.h"
 
 #import <atomic>
 #import <cassert>
@@ -165,23 +166,23 @@ bool isTurboModuleInstance(id module)
   return isTurboModuleClass([module class]);
 }
 
-} // namespace
+struct ModuleQueuePair {
+  id<RCTBridgeModule> module;
+  dispatch_queue_t methodQueue;
+};
 
 // Fallback lookup since RCT class prefix is sometimes stripped in the existing NativeModule system.
 // This will be removed in the future.
-static Class getFallbackClassFromName(const char *name)
+Class getFallbackClassFromName(const char *name)
 {
   Class moduleClass = NSClassFromString([NSString stringWithUTF8String:name]);
-  if (!moduleClass) {
+  if (moduleClass == nil) {
     moduleClass = NSClassFromString([NSString stringWithFormat:@"RCT%s", name]);
   }
   return moduleClass;
 }
 
-typedef struct {
-  id<RCTBridgeModule> module;
-  dispatch_queue_t methodQueue;
-} ModuleQueuePair;
+} // namespace
 
 @implementation RCTTurboModuleManager {
   std::shared_ptr<CallInvoker> _jsInvoker;