Skip to content

[v22.x] backport module loader & loader hook fixes #62029

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
joyeecheung wants to merge 7 commits into from
Closed

[v22.x] backport module loader & loader hook fixes #62029

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
7e19cd0
esm: populate separate cache for require(esm) in imported CJS
joyeecheung Sep 8, 2025
041e977
module: only put directly require-d ESM into require.cache
joyeecheung Sep 17, 2025
4aff83c
module: use sync cjs when importing cts
marco-ippolito Oct 2, 2025
63468af
module: handle null source from async loader hooks in sync hooks
joyeecheung Oct 14, 2025
cf1e37d
module: fix sync resolve hooks for require with node: prefixes
joyeecheung Dec 22, 2025
6df0cdf
module: do not wrap module._load when tracing is not enabled
joyeecheung Jan 26, 2026
7d1b217
module: do not invoke resolve hooks twice for imported cjs
joyeecheung Feb 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
293 changes: 184 additions & 109 deletions lib/internal/modules/cjs/loader.js
View file
Open in desktop

Large diffs are not rendered by default.