refactor: update DevHelper and ReactHostHelper for improved module access and remove unused code

Author: YogendraShelke

ios/Modules/Helper/DevHelper.swift

@@ -7,6 +7,17 @@ public class DevHelper {
     }
 
     public static func setShakeToShowDevMenuEnabled(enabled: Bool) {
-        ReactHostHelper().emitEvent("mendixSetShakeToShowDevMenu", payload: enabled)
+        getModule(type: RCTDevSettings.self)?.isShakeToShowDevMenuEnabled = enabled
+        
+        // This event can be triggered to facilitate communication with the DevSettings JS module. Please refer to dev-settings.ts for further details.
+        // ReactHostHelper().emitEvent("mendixSetShakeToShowDevMenu", payload: enabled)
+    }
+    
+    public static func hideDevLoadingView() {
+        getModule(type: RCTDevLoadingView.self)?.hide()
+    }
+    
+    public static func getModule<T: NSObject>(type: T.Type) -> T? {
+        return ReactHostHelper().module(for: T.self) as? T
     }
 }

ios/Modules/Helper/ReactAppProvider.swift

@@ -70,10 +70,6 @@ open class ReactAppProvider: UIResponder, UIApplicationDelegate {
     public static func isReactAppActive() -> Bool {
         return ReactHostHelper().isReactAppActive()
     }
-
-    public static func getModule<T: NSObject>(name:String, type: T.Type) -> T? {
-        return ReactHostHelper().module(forName: name) as? T
-    }
 }
 
 

ios/Modules/Helper/ReactHostHelper.h

@@ -11,7 +11,7 @@ NS_ASSUME_NONNULL_BEGIN
 
 @interface ReactHostHelper : NSObject
 
-- (nullable id) moduleForName: (NSString *) name;
+- (nullable id) moduleForClass: (Class) clazz;
 - (void) reloadClientWithState;
 - (BOOL) isReactAppActive;
 - (void) emitEvent: (nonnull NSString*) eventName payload: (nullable id) payload;

ios/Modules/Helper/ReactHostHelper.mm

@@ -17,22 +17,22 @@
 
 @implementation ReactHostHelper
 
-- (nullable id)moduleForName:(nonnull NSString*)name {
+- (nullable id) moduleForClass: (Class) clazz {
     if ([NSThread isMainThread]) {
-        return [self getModuleForName: name];
+        return [self getModuleForClass: clazz];
     } else {
         __block id result;
         dispatch_sync(dispatch_get_main_queue(), ^{
-            result = [self getModuleForName: name];
+            result = [self getModuleForClass: clazz];
         });
         return result;
     }
 }
 
-- (nullable id) getModuleForName:(nonnull NSString*)name {
+- (nullable id) getModuleForClass: (Class) clazz {
     RCTHost *reactHost = [self currentHost];
     RCTModuleRegistry *moduleRegistry = [reactHost moduleRegistry];
-    id nativeModule = [moduleRegistry moduleForName: name.UTF8String];
+    id nativeModule = [moduleRegistry moduleForClass: clazz];
     return nativeModule;
 }
 
@@ -46,7 +46,7 @@ - (RCTHost *) currentHost {
 
 
 - (void) reloadClientWithState {
-    MxReload *mxReload = [self moduleForName: MxReload.moduleName];
+    MxReload *mxReload = [self moduleForClass: MxReload.class];
     [mxReload emitOnReloadWithState];
 }
 

ios/Modules/NativeFsModule/NativeFsModule.swift

@@ -25,7 +25,7 @@ public class NativeFsModule: NSObject {
     }
 
     private func getBlobManager() -> RCTBlobManager? {
-        guard let blobManager: RCTBlobManager = ReactAppProvider.getModule(name: RCTBlobManager.moduleName(), type: RCTBlobManager.self) else {
+        guard let blobManager: RCTBlobManager = DevHelper.getModule(type: RCTBlobManager.self) else {
             NSLog("NativeFsModule: Failed to get RCTBlobManager")
             return nil
         }

ios/Modules/ReactNative.swift

@@ -75,6 +75,7 @@ open class ReactNative: NSObject, RCTReloadListener {
 
     public func hideSplashScreen() {
         mendixApp?.splashScreenPresenter?.hide()
+        DevHelper.hideDevLoadingView()
     }
 
     // MARK: - Reload Methods

src/dev-settings.ts

@@ -1,67 +1,12 @@
-import { DeviceEventEmitter, NativeModules, Platform } from 'react-native';
-
-const NativeDevSettings = NativeModules.DevSettings;
-
-// Listen for native-to-JS calls to control shake gesture.
-// In RN 0.84+ bridgeless mode, native cannot directly set this on RCTDevSettings
-// because the TurboModule instance isn't accessible via moduleForName. Instead,
-// native uses RCTHost.callFunctionOnJSModule to emit this event.
-if (__DEV__ && Platform.OS === 'ios') {
-  DeviceEventEmitter.addListener(
-    'mendixSetShakeToShowDevMenu',
-    (enabled: boolean) => {
-      NativeDevSettings?.setIsShakeToShowDevMenuEnabled?.(enabled);
-    }
-  );
-}
-
-export const DevSettings = {
-  openDebugger(): void {
-    if (!__DEV__) return;
-    NativeDevSettings?.openDebugger?.();
-  },
-
-  toggleElementInspector(): void {
-    if (!__DEV__) return;
-
-    NativeDevSettings?.toggleElementInspector?.();
-  },
-
-  reload(reason?: string): void {
-    if (!__DEV__) return;
-    if (NativeDevSettings?.reloadWithReason) {
-      NativeDevSettings.reloadWithReason(reason ?? 'Manual reload from JS');
-    } else if (NativeDevSettings?.reload) {
-      NativeDevSettings.reload();
-    }
-  },
-
-  setHotLoadingEnabled(enabled: boolean): void {
-    if (__DEV__ && NativeDevSettings?.setHotLoadingEnabled) {
-      NativeDevSettings.setHotLoadingEnabled(enabled);
-    }
-  },
-
-  setProfilingEnabled(enabled: boolean): void {
-    if (__DEV__ && NativeDevSettings?.setProfilingEnabled) {
-      NativeDevSettings.setProfilingEnabled(enabled);
-    }
-  },
-
-  setShakeToShowDevMenuEnabled(enabled: boolean): void {
-    if (
-      __DEV__ &&
-      Platform.OS === 'ios' &&
-      NativeDevSettings?.setIsShakeToShowDevMenuEnabled
-    ) {
-      NativeDevSettings.setIsShakeToShowDevMenuEnabled(enabled);
-    }
-  },
-
-  addMenuItem(title: string, _handler: () => void): void {
-    if (__DEV__ && NativeDevSettings?.addMenuItem) {
-      NativeDevSettings.addMenuItem(title);
-      // Note: Event listener setup would need NativeEventEmitter
-    }
-  },
-};
+// Listen for native-to-JS calls to control the shake gesture.
+// If the native layer cannot update RCTDevSettings directly,
+// enable a listener and use RCTHost.callFunctionOnJSModule to emit the event.
+// This pattern can also apply to other methods.
+// if (__DEV__ && Platform.OS === 'ios') {
+//   DeviceEventEmitter.addListener(
+//     'mendixSetShakeToShowDevMenu',
+//     (enabled: boolean) => {
+//       NativeDevSettings?.setIsShakeToShowDevMenuEnabled?.(enabled);
+//     }
+//   );
+// }

src/index.ts

@@ -10,4 +10,3 @@ export * from './encrypted-storage';
 export * from './error';
 export * from './file-system';
 export * from './navigation-mode';
-export * from './dev-settings';