fixup! esm: add experimental support for addon modules

Author: legendecas

doc/api/module.md

@@ -957,6 +957,7 @@ The final value of `format` must be one of the following:
 
 | `format`     | Description                    | Acceptable types for `source` returned by `load`                           |
 | ------------ | ------------------------------ | -------------------------------------------------------------------------- |
+| `'addon'`    | Load a Node.js addon           | { `null` }                                                                 |
 | `'builtin'`  | Load a Node.js builtin module  | Not applicable                                                             |
 | `'commonjs'` | Load a Node.js CommonJS module | { [`string`][], [`ArrayBuffer`][], [`TypedArray`][], `null`, `undefined` } |
 | `'json'`     | Load a JSON file               | { [`string`][], [`ArrayBuffer`][], [`TypedArray`][] }                      |

lib/internal/modules/esm/translators.js

@@ -245,13 +245,14 @@ function createCJSNoSourceModuleWrap(url, isMain) {
 
   // Addon export names are not known until the addon is loaded.
   const exportNames = ['default', 'module.exports'];
-  return new ModuleWrap(url, undefined, exportNames, function() {
+  return new ModuleWrap(url, undefined, exportNames, function evaluationCallback() {
     debug(`Loading CJSModule ${url}`);
 
     if (!module.loaded) {
       wrapModuleLoad(filename, null, isMain);
     }
 
+    /** @type {import('./loader').ModuleExports} */
     let exports;
     if (module[kModuleExport] !== undefined) {
       exports = module[kModuleExport];
@@ -323,18 +324,18 @@ translators.set('commonjs', function commonjsStrategy(url, source, isMain) {
  */
 function cjsEmplaceModuleCacheEntry(filename, exportNames) {
   // TODO: Do we want to keep hitting the user mutable CJS loader here?
-  let module = CJSModule._cache[filename];
-  if (module) {
-    return module;
+  let cjsMod = CJSModule._cache[filename];
+  if (cjsMod) {
+    return cjsMod;
   }
 
-  module = new CJSModule(filename);
-  module.filename = filename;
-  module.paths = CJSModule._nodeModulePaths(module.path);
-  module[kIsCachedByESMLoader] = true;
-  CJSModule._cache[filename] = module;
+  cjsMod = new CJSModule(filename);
+  cjsMod.filename = filename;
+  cjsMod.paths = CJSModule._nodeModulePaths(cjsMod.path);
+  cjsMod[kIsCachedByESMLoader] = true;
+  CJSModule._cache[filename] = cjsMod;
 
-  return module;
+  return cjsMod;
 }
 
 /**
@@ -510,7 +511,7 @@ translators.set('wasm', async function(url, source) {
 });
 
 // Strategy for loading a addon
-translators.set('addon', function(url, source, isMain) {
+translators.set('addon', function translateAddon(url, source, isMain) {
   emitExperimentalWarning('Importing addons');
 
   // The addon must be loaded from file system with dlopen. Assert

test/addons/esm-package-dependent/node_modules/esm-package

@@ -0,0 +1,15 @@
+// Flags: --experimental-addon-modules
+'use strict';
+const common = require('../../common');
+const assert = require('node:assert');
+
+/**
+ * Test that the export condition `node-addons` can be used
+ * with `*.node` files with the ESM loader.
+ */
+
+import(`esm-package/${common.buildType}`)
+  .then((mod) => {
+    assert.strictEqual(mod.default.hello(), 'world');
+  })
+  .then(common.mustCall());

test/addons/esm-package-dependent/test-import.js

@@ -0,0 +1,15 @@
+// Flags: --experimental-addon-modules
+'use strict';
+const common = require('../../common');
+const assert = require('node:assert');
+
+/**
+ * Test that the export condition `node-addons` can be used
+ * with `*.node` files with the CJS loader.
+ */
+
+require(`esm-package/${common.buildType}`)
+  .then((mod) => {
+    assert.strictEqual(mod.hello(), 'world');
+  })
+  .then(common.mustCall());

test/addons/esm-package-dependent/test-require.js

@@ -0,0 +1,11 @@
+{
+  "name": "esm-package",
+  "exports": {
+    "./Debug": {
+      "node-addons": "./build/Debug/binding.node"
+    },
+    "./Release": {
+      "node-addons": "./build/Release/binding.node"
+    }
+  }
+}