fix(node): require esm should prefer module.exports export (#28376)

Author: dsherret

ext/node/polyfills/01_require.js

@@ -1101,8 +1101,11 @@ function loadESMFromCJS(module, filename, code) {
     url.pathToFileURL(filename).toString(),
     code,
   );
-
-  module.exports = namespace;
+  if (ObjectHasOwn(namespace, "module.exports")) {
+    module.exports = namespace["module.exports"];
+  } else {
+    module.exports = namespace;
+  }
 }
 
 function stripBOM(content) {

tests/specs/node/require_esm_module_exports/__test__.jsonc

@@ -0,0 +1,4 @@
+{
+  "args": "run -A main.cjs",
+  "output": "main.out"
+}

tests/specs/node/require_esm_module_exports/main.cjs

@@ -0,0 +1,2 @@
+console.log(require("./undefined.mjs"));
+console.log(require("./string.mjs"));

tests/specs/node/require_esm_module_exports/main.out

@@ -0,0 +1,2 @@
+undefined
+value

tests/specs/node/require_esm_module_exports/string.mjs

@@ -0,0 +1,2 @@
+const value = "value";
+export { value as "module.exports" };

tests/specs/node/require_esm_module_exports/undefined.mjs

@@ -0,0 +1,2 @@
+const value = undefined;
+export { value as "module.exports" };