module: derive builtinModules from enabled builtin set · nodejs/node@54a5e25 · GitHub
Skip to content

Commit 54a5e25

Browse files
sjungwon03aduh95
authored andcommitted
module: derive builtinModules from enabled builtin set
Signed-off-by: sjungwon03 <sjungwon03@gmail.com> PR-URL: #65418 Refs: #56185 Refs: #56870 Refs: #63158 Reviewed-By: Jordan Harband <ljharb@gmail.com>
1 parent b9ee2e6 commit 54a5e25

3 files changed

Lines changed: 46 additions & 15 deletions

File tree

‎lib/internal/bootstrap/realm.js‎

Lines changed: 10 additions & 1 deletion

‎lib/internal/modules/cjs/loader.js‎

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -492,20 +492,7 @@ Module.isBuiltin = BuiltinModule.isBuiltin;
492492
function initializeCJS() {
493493
// This need to be done at runtime in case --expose-internals is set.
494494

495-
let modules = Module.builtinModules = BuiltinModule.getAllBuiltinModuleIds();
496-
if (!getOptionValue('--experimental-dtls')) {
497-
modules = modules.filter((i) => i !== 'node:dtls');
498-
}
499-
if (!getOptionValue('--experimental-quic')) {
500-
modules = modules.filter((i) => i !== 'node:quic');
501-
}
502-
if (!getOptionValue('--experimental-ffi')) {
503-
modules = modules.filter((i) => i !== 'node:ffi');
504-
}
505-
if (!getOptionValue('--experimental-vfs')) {
506-
modules = modules.filter((i) => i !== 'node:vfs');
507-
}
508-
Module.builtinModules = ObjectFreeze(modules);
495+
Module.builtinModules = ObjectFreeze(BuiltinModule.getAllBuiltinModuleIds());
509496

510497
initializeCjsConditions();
511498

Lines changed: 35 additions & 0 deletions

0 commit comments

Comments
 (0)