@novastera-oss/nitro-metamask 0.7.5 → 0.7.8

package/android/src/main/cpp/cpp-adapter.cpp

@@ -1,8 +1,6 @@
-#include <fbjni/fbjni.h>
 #include <jni.h>
-
 #include "NitroMetamaskOnLoad.hpp"
 
 JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM* vm, void*) {
-  return facebook::jni::initialize(vm, []() { margelo::nitro::nitrometamask::registerAllNatives(); });
+  return margelo::nitro::nitrometamask::initialize(vm);
 }

package/android/src/main/java/com/margelo/nitro/nitrometamask/NitroMetamaskPackage.kt

@@ -37,9 +37,10 @@ class NitroMetamaskPackage : BaseReactPackage() {
                     "NitroMetamaskPackage",
                     false, // canOverrideExistingModule
                     true, // needsEagerInit
+                    true, // hasConstants
                     false, // isCxxModule
-                    true, // isTurboModule
-                ),
+                    true // isTurboModule
+                )
             )
         }
     }

package/app.plugin.js

@@ -6,13 +6,16 @@ const withMetamaskAppDelegate = (config) => {
     
     // Check if AppDelegate is Swift
     if (modResults.language === 'swift') {
-      // Check if the method already exists
-      if (modResults.contents.includes('MetaMaskSDK.sharedInstance?.handleUrl')) {
-        return config;
+      // Migrate stale/incorrect import if present
+      if (modResults.contents.includes('import metamask_ios_sdk')) {
+        modResults.contents = modResults.contents.replace(
+          /import\s+metamask_ios_sdk/g,
+          'import NitroMetamask'
+        );
       }
-      
-      // Add import if not present
-      if (!modResults.contents.includes('import metamask_ios_sdk')) {
+
+      // Add NitroMetamask import if not present
+      if (!modResults.contents.includes('import NitroMetamask')) {
         // Find the last import statement and add after it
         const importRegex = /^import\s+.*$/gm;
         const imports = modResults.contents.match(importRegex);
@@ -21,17 +24,22 @@ const withMetamaskAppDelegate = (config) => {
           const lastImportIndex = modResults.contents.lastIndexOf(lastImport);
           modResults.contents = 
             modResults.contents.slice(0, lastImportIndex + lastImport.length) +
-            '\nimport metamask_ios_sdk' +
+            '\nimport NitroMetamask' +
             modResults.contents.slice(lastImportIndex + lastImport.length);
         } else {
           // No imports found, add at the top after the first line
           const firstLineIndex = modResults.contents.indexOf('\n');
           modResults.contents = 
             modResults.contents.slice(0, firstLineIndex + 1) +
-            'import metamask_ios_sdk\n' +
+            'import NitroMetamask\n' +
             modResults.contents.slice(firstLineIndex + 1);
         }
       }
+
+      // Check if the method already exists
+      if (modResults.contents.includes('MetaMaskSDK.sharedInstance?.handleUrl')) {
+        return config;
+      }
       
       // Add the deep link handler method
       const deepLinkHandler = `
@@ -76,22 +84,30 @@ const withMetamaskAppDelegate = (config) => {
       }
     } else if (modResults.language === 'objc') {
       // Handle Objective-C AppDelegate
-      if (modResults.contents.includes('MetaMaskSDK')) {
-        return config;
+      // Remove stale direct SDK import if present. Importing MetaMaskSDK headers
+      // in Objective-C (.m) can pull C++ headers and fail non-ObjC++ builds.
+      modResults.contents = modResults.contents.replace(
+        /^\s*#import\s+<MetaMaskSDK\/MetaMaskSDK\.h>\s*\n?/gm,
+        ''
+      );
+
+      // Migrate legacy direct call to a runtime-safe invocation in Objective-C.
+      if (modResults.contents.includes('[[MetaMaskSDK sharedInstance] handleUrl:url];')) {
+        modResults.contents = modResults.contents.replace(
+          /\[\[MetaMaskSDK sharedInstance\] handleUrl:url\];/g,
+          `Class metaMaskSDKClass = NSClassFromString(@"MetaMaskSDK");
+    id sharedInstance = [metaMaskSDKClass respondsToSelector:@selector(sharedInstance)]
+      ? [metaMaskSDKClass performSelector:@selector(sharedInstance)]
+      : nil;
+    if (sharedInstance && [sharedInstance respondsToSelector:@selector(handleUrl:)]) {
+      [sharedInstance performSelector:@selector(handleUrl:) withObject:url];
+    }`
+        );
       }
-      
-      // Add import
-      if (!modResults.contents.includes('#import <MetaMaskSDK/MetaMaskSDK.h>')) {
-        const importRegex = /^#import\s+.*$/gm;
-        const imports = modResults.contents.match(importRegex);
-        if (imports && imports.length > 0) {
-          const lastImport = imports[imports.length - 1];
-          const lastImportIndex = modResults.contents.lastIndexOf(lastImport);
-          modResults.contents = 
-            modResults.contents.slice(0, lastImportIndex + lastImport.length) +
-            '\n#import <MetaMaskSDK/MetaMaskSDK.h>' +
-            modResults.contents.slice(lastImportIndex + lastImport.length);
-        }
+
+      // Skip injection if handler already exists after migration.
+      if (modResults.contents.includes('[components.host isEqualToString:@"mmsdk"]')) {
+        return config;
       }
       
       const objcHandler = `
@@ -100,7 +116,13 @@ const withMetamaskAppDelegate = (config) => {
   NSURLComponents *components = [NSURLComponents componentsWithURL:url resolvingAgainstBaseURL:YES];
   if ([components.host isEqualToString:@"mmsdk"]) {
     // Handle MetaMask deep link return
-    [[MetaMaskSDK sharedInstance] handleUrl:url];
+    Class metaMaskSDKClass = NSClassFromString(@"MetaMaskSDK");
+    id sharedInstance = [metaMaskSDKClass respondsToSelector:@selector(sharedInstance)]
+      ? [metaMaskSDKClass performSelector:@selector(sharedInstance)]
+      : nil;
+    if (sharedInstance && [sharedInstance respondsToSelector:@selector(handleUrl:)]) {
+      [sharedInstance performSelector:@selector(handleUrl:) withObject:url];
+    }
     return YES;
   }
   

package/lib/typescript/src/__tests__/appPlugin.test.d.ts

@@ -0,0 +1,2 @@
+declare const withMetamaskAppDelegate: any;
+//# sourceMappingURL=appPlugin.test.d.ts.map

package/lib/typescript/src/__tests__/appPlugin.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"appPlugin.test.d.ts","sourceRoot":"","sources":["../../../../src/__tests__/appPlugin.test.ts"],"names":[],"mappings":"AAIA,QAAA,MAAM,uBAAuB,KAAiC,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@novastera-oss/nitro-metamask",
-  "version": "0.7.5",
+  "version": "0.7.8",
   "description": "Native mobile MetaMask wallet integration for React Native. Part of Novastera CRM/ERP platform ecosystem. Provides secure authentication and message signing for Web3 mobile applications.",
   "main": "./lib/commonjs/index.js",
   "module": "./lib/module/index.js",

package/src/__tests__/appPlugin.test.ts

@@ -0,0 +1,85 @@
+jest.mock('@expo/config-plugins', () => ({
+  withAppDelegate: (config: any, action: (cfg: any) => any) => action(config),
+}))
+
+const withMetamaskAppDelegate = require('../../app.plugin.js')
+
+describe('app.plugin.js', () => {
+  it('replaces stale Swift import even if handler already exists', () => {
+    const config = {
+      modResults: {
+        language: 'swift',
+        contents: `import UIKit
+import metamask_ios_sdk
+
+class AppDelegate: NSObject {
+  func application(
+    _ app: UIApplication,
+    open url: URL,
+    options: [UIApplication.OpenURLOptionsKey: Any] = [:]
+  ) -> Bool {
+    MetaMaskSDK.sharedInstance?.handleUrl(url)
+    return true
+  }
+}
+`,
+      },
+    }
+
+    const result = withMetamaskAppDelegate(config)
+    const contents: string = result.modResults.contents
+
+    expect(contents).toContain('import NitroMetamask')
+    expect(contents).not.toContain('import metamask_ios_sdk')
+    expect((contents.match(/MetaMaskSDK\.sharedInstance\?\.handleUrl\(url\)/g) || []).length).toBe(1)
+  })
+
+  it('injects NitroMetamask Swift import when missing', () => {
+    const config = {
+      modResults: {
+        language: 'swift',
+        contents: `import UIKit
+
+class AppDelegate: NSObject {
+}
+`,
+      },
+    }
+
+    const result = withMetamaskAppDelegate(config)
+    const contents: string = result.modResults.contents
+
+    expect(contents).toContain('import NitroMetamask')
+    expect(contents).toContain('MetaMaskSDK.sharedInstance?.handleUrl(url)')
+  })
+
+  it('migrates Objective-C direct SDK usage to runtime-safe calls', () => {
+    const config = {
+      modResults: {
+        language: 'objc',
+        contents: `#import "AppDelegate.h"
+#import <MetaMaskSDK/MetaMaskSDK.h>
+
+@implementation AppDelegate
+- (BOOL)application:(UIApplication *)app openURL:(NSURL *)url options:(NSDictionary<UIApplicationOpenURLOptionsKey,id> *)options {
+  NSURLComponents *components = [NSURLComponents componentsWithURL:url resolvingAgainstBaseURL:YES];
+  if ([components.host isEqualToString:@"mmsdk"]) {
+    [[MetaMaskSDK sharedInstance] handleUrl:url];
+    return YES;
+  }
+  return NO;
+}
+@end
+`,
+      },
+    }
+
+    const result = withMetamaskAppDelegate(config)
+    const contents: string = result.modResults.contents
+
+    expect(contents).not.toContain('#import <MetaMaskSDK/MetaMaskSDK.h>')
+    expect(contents).toContain('NSClassFromString(@"MetaMaskSDK")')
+    expect(contents).toContain('[sharedInstance performSelector:@selector(handleUrl:) withObject:url]')
+    expect(contents).not.toContain('[[MetaMaskSDK sharedInstance] handleUrl:url];')
+  })
+})