esm: fix loading of CJS modules from ESM

PR-URL: #49500
Fixes: #49497
Reviewed-By: Stephen Belanger <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>

Author: aduh95

lib/internal/modules/esm/translators.js

@@ -279,7 +279,8 @@ translators.set('commonjs', async function commonjsStrategy(url, source,
   // obtained by calling the monkey-patchable CJS loader.
   const cjsLoader = source == null ? (module, source, url, filename) => {
     try {
-      module.load(filename);
+      assert(module === CJSModule._cache[filename]);
+      CJSModule._load(filename);
     } catch (err) {
       enrichCJSError(err, source, url);
       throw err;

test/es-module/test-esm-recursive-cjs-dependencies.mjs

@@ -0,0 +1,7 @@
+import '../common/index.mjs';
+import { strictEqual } from 'node:assert';
+
+import '../fixtures/recursive-a.cjs';
+
+strictEqual(global.counter, 1);
+delete global.counter;

test/fixtures/recursive-a.cjs

@@ -0,0 +1,6 @@
+'use strict';
+
+global.counter ??= 0;
+global.counter++;
+
+require('./recursive-b.cjs');

test/fixtures/recursive-b.cjs

@@ -0,0 +1,3 @@
+'use strict';
+
+require('./recursive-a.cjs');