Skip to content

process: clarify the pre- and post-condition of esm setup

Rodrigo Muino Tomonari requested to merge github/fork/joyeecheung/move-esm into master

This patch:

  • Clarifies the dependency of the ESM loader initialization (process.cwd() and the value of --loader) in node.js.
  • Moves the initialization of the per-isolate importModuleDynamically and initializeImportMetaObject callbacks into node.js
  • Moves the initialization of the ESM loader into prepareUserCodeExecution() since it potentially involves execution of user code (similar to --require for CJS modules).
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports