Commit 683c93f
module: refator ESM loader for adding future synchronous hooks
This lays the foundation for supporting synchronous hooks proposed
in nodejs/loaders#198 for ESM.
- Corrects and adds several JSDoc comments for internal functions
of the ESM loader, as well as explaining how require() for
import CJS work in the special resolve/load paths. This doesn't
consolidate it with import in require(esm) yet due to caching
differences, which is left as a TODO.
- The moduleProvider passed into ModuleJob is replaced as
moduleOrModulePromise, we call the translators directly in the
ESM loader and verify it right after loading for clarity.
- Reuse a few refactored out helpers for require(esm) in
getModuleJobForRequire().
PR-URL: #54769
Backport-PR-URL: #56927
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Refs: #526971 parent df8a045 commit 683c93fCopy full SHA for 683c93f
File tree
Expand file treeCollapse file tree
3 files changed
+264
-151
lines changedOpen diff view settings
Filter options
- lib/internal/modules/esm
Expand file treeCollapse file tree
3 files changed
+264
-151
lines changedOpen diff view settings
0 commit comments