objc-js 1.3.0 → 1.3.1

package/dist/index.js

@@ -32,7 +32,12 @@ class NobjcLibrary {
                     LoadLibrary(library);
                     this.wasLoaded = true;
                 }
-                cls = new NobjcObject(GetClassObject(className));
+                const classObject = GetClassObject(className);
+                if (classObject === undefined) {
+                    // Class not found. Make sure the class exists before trying to access it.
+                    return undefined;
+                }
+                cls = new NobjcObject(classObject);
                 classCache.set(className, cls);
                 return cls;
             }
@@ -55,6 +60,9 @@ class NobjcObject {
                 // Return true for the special Symbol to enable unwrapping
                 if (p === NATIVE_OBJC_OBJECT)
                     return true;
+                // Return true for inspect symbols so console.log uses custom inspect
+                if (p === customInspectSymbol)
+                    return true;
                 // guard against other symbols
                 if (typeof p === "symbol")
                     return Reflect.has(target, p);
@@ -117,6 +125,10 @@ class NobjcObject {
                     // object (avoids triggering proxy 'has' trap which would be a second FFI call)
                     const selector = NobjcMethodNameToObjcSelector(methodName);
                     if (!target.$respondsToSelector(selector)) {
+                        // special case since JS checks for `.then` on Promise objects
+                        if (methodName === "then")
+                            return undefined;
+                        // Otherwise, throw an error
                         throw new Error(`Method ${methodName} not found on object`);
                     }
                     method = NobjcMethod(object, methodName);
@@ -127,6 +139,9 @@ class NobjcObject {
         };
         // Create the proxy
         const proxy = new Proxy(object, handler);
+        // Set custom inspect on the native object so console.log works through the Proxy.
+        // Runtimes (Node, Bun) bypass Proxy traps during inspect and read the target directly.
+        object[customInspectSymbol] = () => proxy.toString();
         // Store proxy → native mapping in WeakMap for O(1) unwrap (bypasses Proxy traps)
         nativeObjectMap.set(proxy, object);
         // Return the proxy

package/package.json

@@ -46,7 +46,7 @@
     "prebuild:x64": "prebuildify --napi --strip --arch x64 -r node",
     "prebuild:all": "bun run prebuild:arm64 && bun run prebuild:x64"
   },
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "Objective-C bridge for Node.js",
   "main": "dist/index.js",
   "dependencies": {

package/src/native/ObjcObject.h

@@ -4,11 +4,18 @@
 #include <napi.h>
 #include <objc/objc.h>
 #include <objc/runtime.h>
+#include <objc/message.h>
 #include <optional>
 #include <variant>
 #include <unordered_map>
 #include <vector>
 
+// objc_retain/objc_release are part of the stable ObjC ABI (macOS 10.12+)
+// but not declared in public headers. We use them for manual reference counting
+// since ARC is not enabled for .mm files in this project.
+extern "C" id objc_retain(id value);
+extern "C" void objc_release(id value);
+
 #ifdef __OBJC__
 @class NSMethodSignature;
 #else
@@ -51,6 +58,14 @@ public:
       // This better be an Napi::External<id>! We lost the type info at runtime.
       Napi::External<id> external = info[0].As<Napi::External<id>>();
       objcObject = *(external.Data());
+      // Retain the ObjC object so it stays alive as long as this JS wrapper
+      // exists. Without this, ARC/autorelease can deallocate the object while
+      // JS still holds a reference, causing Use-After-Free crashes (SIGTRAP)
+      // in completion handler callbacks and other async contexts.
+      // Note: ARC is not enabled for .mm files in this project (the -fobjc-arc
+      // flag is in OTHER_CFLAGS, not OTHER_CPLUSPLUSFLAGS), so __strong has
+      // no effect — we must manage retain/release manually.
+      if (objcObject) objc_retain(objcObject);
       return;
     }
     // If someone tries `new ObjcObject()` from JS, forbid it:
@@ -58,7 +73,12 @@ public:
         .ThrowAsJavaScriptException();
   }
   static Napi::Object NewInstance(Napi::Env env, id obj);
-  ~ObjcObject() = default;
+  ~ObjcObject() {
+    if (objcObject) {
+      objc_release(objcObject);
+      objcObject = nil;
+    }
+  }
 
 private:
   Napi::Value $MsgSend(const Napi::CallbackInfo &info);