fix: ensure the callback 'name' arg is the module name when matching the module main file, even when 'internals: true' option is used#241

Merged

timfish merged 14 commits intonodejs:mainfrom

trentm:trentm-issue238-ritm-compat

Jan 27, 2026

Commits on Jan 23, 2026

example that demonstrates the problem with 'experimentalPatchInternals'
trentm
committed
fix: clarify the matching and fix a couple issues with duplicate entries and specifier (submodule) matching
trentm
committed
remove play/dev files added earlier for a demo
trentm
committed
test fixes for running with older Node.js versions
trentm
committed
move the new tests to only test with Node.js v20 and later, for the improved import(cjs) export sniffing
trentm
committed
need at least v20.19 for these tests apparently
trentm
committed
fix lint
trentm
committed
only run these tests with Node.js v22 and later, ^20.19 is actually fine, but CI currently tests v21 which did NOT get sufficient backports to work for these test files
trentm
committed

Commits on Jan 24, 2026

since limiting to Node.js v22, can use top-level await in these tests
trentm
committed

Commits on Jan 25, 2026

Merge branch 'main' into trentm-clearer-match-block
trentm
committed

Commits on Jan 26, 2026

fix: ensure the callback 'name' arg is the module name when matching to module main file, even when 'internals: true' option is used
trentm
committed

Commits on Jan 27, 2026