esm: detect ESM syntax in ambiguous JavaScrip

nodejs/node#50096

Author: codebytere

patches/node/fix_do_not_resolve_electron_entrypoints.patch

@@ -5,16 +5,38 @@ Subject: fix: do not resolve electron entrypoints
 
 This wastes fs cycles and can result in strange behavior if this path actually exists on disk
 
+diff --git a/lib/internal/modules/esm/load.js b/lib/internal/modules/esm/load.js
+index 8406e8ca319533d6186c5d457e951b7a845b6b72..caa34187e17d9a0addcc8c94e46823ea4e05c590 100644
+--- a/lib/internal/modules/esm/load.js
++++ b/lib/internal/modules/esm/load.js
+@@ -131,7 +131,7 @@ async function defaultLoad(url, context = kEmptyObject) {
+     format ??= 'builtin';
+   } else {
+     let contextToPass = context;
+-    if (source == null) {
++    if (format !== 'electron' && source == null) {
+       ({ responseURL, source } = await getSource(urlInstance, context));
+       contextToPass = { __proto__: context, source };
+     }
+@@ -139,7 +139,7 @@ async function defaultLoad(url, context = kEmptyObject) {
+     // Now that we have the source for the module, run `defaultGetFormat` again in case we detect ESM syntax.
+     format ??= await defaultGetFormat(urlInstance, contextToPass);
+ 
+-    if ((format === 'commonjs' || format === 'electron') && contextToPass !== context) {
++    if (format === 'electron' || format === 'commonjs' && contextToPass !== context) {
+       // For backward compatibility reasons, we need to discard the source in
+       // order for the CJS loader to re-fetch it.
+       source = null;
 diff --git a/lib/internal/modules/esm/translators.js b/lib/internal/modules/esm/translators.js
-index dbbe37474c70addfbf8cf805c29732dcd38c74f6..f422a97b77f633020aed830a87a1c51bbb898acf 100644
+index dbbe37474c70addfbf8cf805c29732dcd38c74f6..96ffb2b0b051163d8a777ed3d1b5bc2c83fc37c5 100644
 --- a/lib/internal/modules/esm/translators.js
 +++ b/lib/internal/modules/esm/translators.js
 @@ -376,6 +376,9 @@ function cjsPreparseModuleExports(filename, source) {
      if (cached) {
        return { module, exportNames: cached.exportNames };
      }
 +    if (filename === 'electron') {
-+      return { module };
++      return { module, exportNames: new SafeSet(['default', ...Object.keys(module.exports)]) };
 +    }
    }
    const loaded = Boolean(module);