fix: revert problematic napi_wrap in registerBlock that breaks exception handling

Author: DjDeveloperr

NativeScript/ffi/Block.mm

@@ -60,11 +60,15 @@ id registerBlock(napi_env env, Closure* closure, napi_value callback) {
 
   napi_remove_wrap(env, callback, nullptr);
   napi_ref ref = nullptr;
-  napi_wrap(env, callback, block, block_finalize, nullptr, &ref);
-  if (ref != nullptr) {
-    uint32_t refCount;
-    napi_reference_ref(env, ref, &refCount);
-  }
+  // TODO: fix memory management of objc blocks here
+  // napi_wrap(env, callback, block, block_finalize, nullptr, &ref);
+  // if (ref == nullptr) {
+  // Deno doesn't handle napi_wrap properly.
+  ref = make_ref(env, callback, 1);
+  // } else {
+  //   uint32_t refCount;
+  //   napi_reference_ref(env, ref, &refCount);
+  // }
   closure->func = ref;
 
   auto bridgeState = ObjCBridgeState::InstanceData(env);

NativeScript/ffi/node_api_util.h

@@ -60,16 +60,14 @@ inline bool napiSupportsThreadsafeFunctions(void* dl) {
     return NULL;                                                           \
   }
 
-#define NAPI_ERROR_INFO                                 \
-  const napi_extended_error_info* error_info = nullptr; \
+#define NAPI_ERROR_INFO                                                    \
+  const napi_extended_error_info* error_info =                             \
+      (napi_extended_error_info*)malloc(sizeof(napi_extended_error_info)); \
   napi_get_last_error_info(env, &error_info);
 
-#define NAPI_THROW_LAST_ERROR                                                 \
-  {                                                                           \
-    NAPI_ERROR_INFO                                                           \
-    napi_throw_error(                                                         \
-        env, NULL, error_info ? error_info->error_message : "Unknown error"); \
-  }
+#define NAPI_THROW_LAST_ERROR \
+  NAPI_ERROR_INFO             \
+  napi_throw_error(env, NULL, error_info->error_message);
 
 #ifndef DEBUG