fixup! Replace require() with await import() in EXPORT_ES6 shell/runtime files

fixup! Replace require() with await import() in EXPORT_ES6 shell/runtime files

Revert the bundler-hint removal from 548e32d.  It was based on a
mistaken assumption that webpack auto-handles `node:`-prefixed dynamic
imports -- it does not.  CI run on 548e32d hit:

  test_webpack_esm_output_clean:
  ERROR in node:fs
  Module build failed: UnhandledSchemeError: Reading from "node:fs"
    is not handled by plugins (Unhandled scheme).
    Webpack supports "data:" and "file:" URIs by default.
    You may need an additional plugin to handle "node:" URIs.

Same error for node:path, node:url. All three show up because without
`/* webpackIgnore: true */`, webpack attempts to resolve the dynamic
import as a module and fails on the unknown scheme.

Vite (per test/vite/vite.config.js) does auto-externalize `node:*` for
browser -- emitting a warning that the config already whitelists --
so strictly speaking `/* @vite-ignore */` is optional.  But we keep
both for symmetry and to also silence vite's dynamic-import analysis
noise.

Updates the inline comment to explain the requirement so a future
reader does not attempt the same simplification again.

Addresses and supersedes the response to r3075818347 -- the hints
cannot be removed unless we add a webpack plugin or drop support for
webpack consumers of EXPORT_ES6 output.

Author: Bartek Wrona

src/parseTools.mjs

@@ -962,11 +962,12 @@ function makeNodeImport(module, guard = true) {
     // `await import` by `fix_js_mangling()` after Closure runs. See parseTools
     // preprocess() and tools/link.py fix_js_mangling().
     //
-    // Bundlers (webpack 5, vite, rollup) auto-externalize `node:`-prefixed
-    // dynamic imports for browser targets, so no per-call bundler hints are
-    // needed here.  test/vite/vite.config.js whitelists the resulting
-    // "externalized for browser compatibility" warning.
-    expr = `EMSCRIPTEN$AWAIT$IMPORT('${module}')`;
+    // The `/* webpackIgnore: true */` hint is required because webpack does
+    // NOT auto-handle `node:`-prefixed URIs for dynamic import; without it,
+    // webpack fails with `UnhandledSchemeError: Reading from "node:xxx" is
+    // not handled by plugins` (see test_webpack_esm_output_clean).
+    // `/* @vite-ignore */` similarly silences vite's dynamic-import analysis.
+    expr = `EMSCRIPTEN$AWAIT$IMPORT(/* webpackIgnore: true */ /* @vite-ignore */ '${module}')`;
   } else {
     expr = `require('${module}')`;
   }