fix: RCTRedBox not appearing in Bridgeless when metro is not running

facebook · Feb 22, 2024 · d32f1a9 · d32f1a9
1 parent 4def40e
commit d32f1a9
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 4 deletions.
diff --git a/packages/react-native/React/CoreModules/RCTRedBox.mm b/packages/react-native/React/CoreModules/RCTRedBox.mm
@@ -274,7 +274,13 @@ - (void)dismiss
 
 - (void)reload
 {
-  [_actionDelegate reloadFromRedBoxController:self];
+  if (_actionDelegate != nil) {
+   [_actionDelegate reloadFromRedBoxController:self];
+  } else {
+    // In bridgeless mode `RCTRedBox` gets deallocted, need to notify listeners anyway.
+    RCTTriggerReloadCommandListeners(@"Redbox");
+    [self dismiss];
+  }
 }
 
 - (void)showExtraDataViewController

diff --git a/packages/react-native/ReactCommon/react/runtime/platform/ios/ReactCommon/RCTInstance.mm b/packages/react-native/ReactCommon/react/runtime/platform/ios/ReactCommon/RCTInstance.mm
@@ -24,7 +24,9 @@
 #import <React/RCTEventDispatcherProtocol.h>
 #import <React/RCTFollyConvert.h>
 #import <React/RCTJavaScriptLoader.h>
+#import <React/RCTReloadCommand.h>
 #import <React/RCTLog.h>
+#import <React/RCTRedBox.h>
 #import <React/RCTLogBox.h>
 #import <React/RCTModuleData.h>
 #import <React/RCTPerformanceLogger.h>
@@ -64,7 +66,7 @@ void RCTInstanceSetRuntimeDiagnosticFlags(NSString *flags)
   sRuntimeDiagnosticFlags = [flags copy];
 }
 
-@interface RCTInstance () <RCTTurboModuleManagerDelegate, RCTTurboModuleManagerRuntimeHandler>
+@interface RCTInstance () <RCTTurboModuleManagerDelegate, RCTTurboModuleManagerRuntimeHandler, RCTReloadListener>
 @end
 
 @implementation RCTInstance {
@@ -339,6 +341,10 @@ - (void)_start
     if (RCTGetUseNativeViewConfigsInBridgelessMode()) {
       installLegacyUIManagerConstantsProviderBinding(runtime);
     }
+
+    RCTExecuteOnMainQueue(^{
+      RCTRegisterReloadCommandListener(self);
+    });
 
     [strongSelf->_delegate instance:strongSelf didInitializeRuntime:runtime];
 
@@ -420,8 +426,13 @@ - (void)_loadJSBundle:(NSURL *)sourceURL
         }
 
         if (error) {
-          // TODO(T91461138): Properly address bundle loading errors.
-          RCTLogError(@"RCTInstance: Error while loading bundle: %@", error);
+          RCTRedBox *redBox = [strongSelf->_turboModuleManager moduleForName:"RedBox"];
+          RCTExecuteOnMainQueue(^{
+            [[NSNotificationCenter defaultCenter] postNotificationName:RCTJavaScriptDidFailToLoadNotification
+                                                                 object:self
+                                                               userInfo:@{@"error" : error}];
+            [redBox showErrorMessage:[error localizedDescription]];
+          });
           [strongSelf invalidate];
           return;
         }
@@ -490,4 +501,8 @@ - (void)_handleJSErrorMap:(facebook::react::MapBuffer)errorMap
                      isFatal:errorMap.getBool(JSErrorHandlerKey::kIsFatal)];
 }
 
+- (void)didReceiveReloadCommand { 
+  [self _loadJSBundle:[self->_bridgeModuleDecorator.bundleManager bundleURL]];
+}
+
 @end