Skip to content

[v20.x] backport require(esm) bugfixes and improvements #59504

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 9 commits into from
Closed

[v20.x] backport require(esm) bugfixes and improvements #59504

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
9b72f5f
module: handle .mjs in .js handler in CommonJS
joyeecheung Oct 31, 2024
a26abb6
src: do not format single string argument for THROW_ERR_*
joyeecheung Feb 18, 2025
914cc71
module: improve error message from asynchronicity in require(esm)
joyeecheung Feb 17, 2025
d52043a
module: handle cached linked async jobs in require(esm)
joyeecheung Mar 4, 2025
feb6167
module: fix incorrect formatting in require(esm) cycle error message
haykam821 Apr 9, 2025
08ff294
module: handle instantiated async module jobs in require(esm)
joyeecheung May 8, 2025
10c4efd
module: clarify cjs global-like error on ModuleJobSync
Ceres6 May 16, 2025
b29b874
module: allow cycles in require() in the CJS handling in ESM loader
joyeecheung Jun 11, 2025
bd6691a
module: throw error when re-runing errored module jobs
joyeecheung Jul 9, 2025
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
Prev Previous commit
Next Next commit
module: fix incorrect formatting in require(esm) cycle error message 
Fixes: #57451
PR-URL: #57453
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
@haykam821 @marco-ippolito
haykam821 authored and marco-ippolito committed Aug 20, 2025
commit feb6167ded4e8d55369ff66d4a41a03800631726
4 changes: 2 additions & 2 deletions lib/internal/modules/esm/loader.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -366,8 +366,8 @@ class ModuleLoader {
if (parentFilename) {
message += ` (from ${parentFilename})`;
}
message += 'A cycle involving require(esm) is disallowed to maintain ';
message += 'invariants madated by the ECMAScript specification';
message += ' A cycle involving require(esm) is not allowed to maintain ';
message += 'invariants mandated by the ECMAScript specification. ';
message += 'Try making at least part of the dependency in the graph lazily loaded.';
throw new ERR_REQUIRE_CYCLE_MODULE(message);

Expand Down