feat: enhance ReactHostHelper with UI thread management

fix: add missing import for RCTReloadCommand in ReactHostHelper.h

Author: YogendraShelke

ios/Modules/Helper/ReactHostHelper.h

@@ -6,13 +6,16 @@
 //
 
 #import <Foundation/Foundation.h>
+#import <React/RCTReloadCommand.h>
 
 NS_ASSUME_NONNULL_BEGIN
 
 @interface ReactHostHelper : NSObject
 
 - (nullable id) moduleForClass: (Class) clazz;
 - (void) reloadClientWithState;
+- (void) reload;
+- (void) updateBundleURL: (nonnull NSURL*) bundleURL;
 - (BOOL) isReactAppActive;
 - (void) emitEvent: (nonnull NSString*) eventName payload: (nullable id) payload;
 

ios/Modules/Helper/ReactHostHelper.mm

@@ -7,7 +7,7 @@
 
 #import "ReactHostHelper.h"
 #import <ReactCommon/RCTHost.h>
-#import <React/RCTReloadCommand.h>
+#import <ReactCommon/RCTHost+Internal.h>
 #import "RCTDefaultReactNativeFactoryDelegate.h"
 #import "RCTReactNativeFactory.h"
 #import "MendixNative-Swift.h"
@@ -17,18 +17,34 @@
 
 @implementation ReactHostHelper
 
-- (nullable id) moduleForClass: (Class) clazz {
+#pragma mark - Thread Helpers
+
+- (void) runOnUiThread:(dispatch_block_t)block {
     if ([NSThread isMainThread]) {
-        return [self getModuleForClass: clazz];
+        block();
     } else {
-        __block id result;
-        dispatch_sync(dispatch_get_main_queue(), ^{
-            result = [self getModuleForClass: clazz];
-        });
-        return result;
+        dispatch_async(dispatch_get_main_queue(), block);
     }
 }
 
+- (void) runOnUiThreadSync:(dispatch_block_t)block {
+    if ([NSThread isMainThread]) {
+        block();
+    } else {
+        dispatch_sync(dispatch_get_main_queue(), block);
+    }
+}
+
+#pragma mark - Module Access
+
+- (nullable id) moduleForClass: (Class) clazz {
+    __block id result;
+    [self runOnUiThreadSync:^{
+        result = [self getModuleForClass: clazz];
+    }];
+    return result;
+}
+
 - (nullable id) getModuleForClass: (Class) clazz {
     RCTHost *reactHost = [self currentHost];
     RCTModuleRegistry *moduleRegistry = [reactHost moduleRegistry];
@@ -50,16 +66,26 @@ - (void) reloadClientWithState {
     [mxReload emitOnReloadWithState];
 }
 
+- (void) reload {
+    [self runOnUiThread:^{
+        [[self currentHost] reload];
+    }];
+}
+
+- (void) updateBundleURL:(NSURL *)bundleURL {
+    [self runOnUiThread:^{
+        [[self currentHost] setBundleURLProvider:^NSURL * _Nullable{
+            return bundleURL;
+        }];
+    }];
+}
+
 - (BOOL)isReactAppActive {
-    if ([NSThread isMainThread]) {
-        return [self currentHost] != nil;
-    } else {
-        __block bool result;
-        dispatch_sync(dispatch_get_main_queue(), ^{
-            result = [self currentHost] != nil;
-        });
-        return result;
-    }
+    __block BOOL result;
+    [self runOnUiThreadSync:^{
+        result = [self currentHost] != nil;
+    }];
+    return result;
 }
 
 - (void)emitEvent:(nonnull NSString *)eventName payload:(nullable id)payload {

ios/Modules/ReactNative.swift

@@ -52,7 +52,9 @@ open class ReactNative: NSObject, RCTReloadListener {
         DevHelper.setDebugMode(enabled: AppPreferences.devModeEnabled && AppPreferences.remoteDebuggingEnabled)
 
         showSplashScreen()
-        
+
+        // Register reload listener to intercept ALL reload triggers (dev menu, programmatic, etc.)
+        // Only in developer apps or when three-finger gestures are enabled
         if mendixApp.isDeveloperApp || mendixApp.enableThreeFingerGestures {
             DispatchQueue.main.async {
                 RCTRegisterReloadCommandListener(self)
@@ -79,6 +81,10 @@ open class ReactNative: NSObject, RCTReloadListener {
     }
 
     // MARK: - Reload Methods
+
+    /// Reloads the React Native instance. Always allowed - used for OTA updates.
+    /// - Note: This is the main entry point for app-triggered reloads (OTA, programmatic).
+    ///   No permission check - OTA updates must work in production apps.
     public func reload() {
         guard let mendixApp = mendixApp else { return }
 
@@ -93,24 +99,38 @@ open class ReactNative: NSObject, RCTReloadListener {
                 if response.status == "SUCCESS", let version = response.runtimeInfo?.version {
                     MendixBackwardsCompatUtility.update(version)
                 }
-                self?.reloadWithBridge()
+                self?.triggerReload()
             }
         } else {
-            reloadWithBridge()
+            triggerReload()
         }
     }
-    
-    private func reloadWithBridge() {
-        RCTTriggerReloadCommandListeners("Reload command from app")
+
+    private func triggerReload() {
+        showSplashScreen()
+        // Note: This bypasses RCTReloadListener notifications (only dev menu Cmd+R triggers those)
+        ReactHostHelper().reload()
+    }
+
+    private func updateBundleURL(_ url: URL) {
+        self.bundleUrl = url
+        ReactHostHelper().updateBundleURL(url)
     }
 
+    /// Reloads the React Native instance while preserving state.
+    /// Guarded - only works in developer apps or when three-finger gestures are explicitly enabled.
+    /// - Note: This is used for gesture-triggered reloads, not OTA updates.
     public func reloadWithState() {
         ReactHostHelper().reloadClientWithState()
     }
 
     // MARK: - RCTReloadListener
+    /// Intercepts ALL reload triggers (dev menu Cmd+R, programmatic, DevTools, etc.)
+    /// Only called if listener was registered (i.e., isDeveloperApp || enableThreeFingerGestures)
     @objc public func didReceiveReloadCommand() {
         showSplashScreen()
+        // Note: Actual reload is handled by RCTHost automatically
+        // This callback is just for pre-reload side effects
     }
 
     // MARK: - Debugging Methods