bare-module 3.1.12 → 3.1.14

package/binding.c

@@ -442,13 +442,13 @@ static js_value_t *
 bare_module_run_module (js_env_t *env, js_callback_info_t *info) {
   int err;
 
-  size_t argc = 2;
-  js_value_t *argv[2];
+  size_t argc = 3;
+  js_value_t *argv[3];
 
   err = js_get_callback_info(env, info, &argc, argv, NULL, NULL);
   assert(err == 0);
 
-  assert(argc == 2);
+  assert(argc == 3);
 
   js_module_t *module;
   err = js_get_value_external(env, argv[0], (void **) &module);
@@ -461,30 +461,41 @@ bare_module_run_module (js_env_t *env, js_callback_info_t *info) {
   err = js_instantiate_module(env, module, on_static_import, (void *) context);
   if (err < 0) return NULL;
 
-  js_value_t *result;
-  err = js_run_module(env, module, &result);
+  js_value_t *promise;
+  err = js_run_module(env, module, &promise);
   if (err < 0) return NULL;
 
   bool is_promise;
-  err = js_is_promise(env, result, &is_promise);
+  err = js_is_promise(env, promise, &is_promise);
   assert(err == 0);
 
   if (is_promise) {
     js_promise_state_t state;
-    err = js_get_promise_state(env, result, &state);
+    err = js_get_promise_state(env, promise, &state);
     assert(err == 0);
 
     if (state == js_promise_rejected) {
-      js_value_t *error;
-      err = js_get_promise_result(env, result, &error);
+      js_value_t *reason;
+      err = js_get_promise_result(env, promise, &reason);
       if (err < 0) return NULL;
 
-      err = js_set_named_property(env, result, "error", error);
-      if (err < 0) return NULL;
+      js_value_t *exception;
+      err = js_get_and_clear_last_exception(env, &exception);
+      assert(err == 0);
+
+      js_value_t *ctx;
+      err = js_get_reference_value(env, context->ctx, &ctx);
+      assert(err == 0);
+
+      js_value_t *args[3] = {reason, promise, exception};
+
+      js_call_function(env, ctx, argv[2], 3, args, NULL);
+
+      return NULL;
     }
   }
 
-  return result;
+  return promise;
 }
 
 static js_value_t *

package/index.js

@@ -179,13 +179,15 @@ const Module = module.exports = exports = class Module {
     this._handle = binding.createSyntheticModule(this._url.href, this._names, Module._handle)
   }
 
+  _run () {
+    binding.runModule(this._handle, Module._handle, Module._onrejection)
+  }
+
   _evaluate (eagerRun = false) {
     if ((this._state & constants.states.EVALUATED) !== 0) return
 
     this._state |= constants.states.EVALUATED
 
-    let result
-
     if (this._type === constants.types.SCRIPT) {
       const require = createRequire(this._url, { module: this })
 
@@ -201,23 +203,17 @@ const Module = module.exports = exports = class Module {
         urlToDirname(this._url)
       )
 
-      if (eagerRun) result = binding.runModule(this._handle, Module._handle)
+      if (eagerRun) this._run()
     }
 
     if (this._type === constants.types.MODULE) {
-      result = binding.runModule(this._handle, Module._handle)
+      this._run()
 
       this._exports = binding.getNamespace(this._handle)
     }
 
     if (this._type === constants.types.ADDON) {
-      if (eagerRun) result = binding.runModule(this._handle, Module._handle)
-    }
-
-    if (result && result.error) {
-      result.catch(() => {}) // Handle the promise rejection
-
-      throw result.error
+      if (eagerRun) this._run()
     }
   }
 
@@ -347,6 +343,12 @@ const Module = module.exports = exports = class Module {
     }
   }
 
+  static _onrejection (reason, promise, err = reason) {
+    promise.catch(() => {}) // Don't leak the rejection
+
+    throw err
+  }
+
   static Protocol = Protocol
   static Bundle = Bundle
   static constants = constants

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-module",
-  "version": "3.1.12",
+  "version": "3.1.14",
   "description": "Module support for JavaScript",
   "main": "index.js",
   "files": [