lib,repl: ignore non-canBeRequiredByUsers built-in

e.g. `wasi` under no `--experimental-wasi-unstable-preview1` flag
shouldn't be pre-required.
PR-URL: #39942
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>

Author: XadillaX

lib/internal/main/eval_string.js

@@ -16,7 +16,7 @@ const { addBuiltinLibsToObject } = require('internal/modules/cjs/helpers');
 const{ getOptionValue }=require('internal/options');
 
 prepareMainThreadExecution();
-addBuiltinLibsToObject(globalThis);
+addBuiltinLibsToObject(globalThis,'<eval>');
 markBootstrapComplete();
 
 constsource=getOptionValue('--eval');

lib/internal/modules/cjs/helpers.js

@@ -131,9 +131,16 @@ function stripBOM(content) {
 returncontent;
 }
 
-functionaddBuiltinLibsToObject(object){
+functionaddBuiltinLibsToObject(object,dummyModuleName){
 // Make built-in modules available directly (loaded lazily).
-const{ builtinModules }=require('internal/modules/cjs/loader').Module;
+constModule=require('internal/modules/cjs/loader').Module;
+const{ builtinModules }=Module;
+
+// To require built-in modules in user-land and ignore modules whose
+// `canBeRequiredByUsers` is false. So we create a dummy module object and not
+// use `require()` directly.
+constdummyModule=newModule(dummyModuleName);
+
 ArrayPrototypeForEach(builtinModules,(name)=>{
 // Neither add underscored modules, nor ones that contain slashes (e.g.,
 // 'fs/promises') or ones that are already defined.
@@ -157,7 +164,7 @@ function addBuiltinLibsToObject(object) {
 
 ObjectDefineProperty(object,name,{
 get: ()=>{
-constlib=require(name);
+constlib=dummyModule.require(name);
 
 // Disable the current getter/setter and set up a new
 // non-enumerable property.

lib/repl.js

@@ -1098,7 +1098,7 @@ REPLServer.prototype.createContext = function() {
 value: makeRequireFunction(replModule)
 });
 
-addBuiltinLibsToObject(context);
+addBuiltinLibsToObject(context,'<REPL>');
 
 returncontext;
 };

test/parallel/test-repl-built-in-modules.js

@@ -0,0 +1,32 @@
+'use strict';
+
+require('../common');
+constassert=require('assert');
+constcp=require('child_process');
+
+functionrunREPLWithAdditionalFlags(flags){
+// Use -i to force node into interactive mode, despite stdout not being a TTY
+constargs=['-i'].concat(flags);
+constret=cp.execFileSync(process.execPath,args,{
+input: 'require(\'events\');\nrequire(\'wasi\');',
+encoding: 'utf8',
+});
+returnret;
+}
+
+// Run REPL in normal mode.
+letstdout=runREPLWithAdditionalFlags([]);
+assert.match(stdout,/\[Function:EventEmitter\]{/);
+assert.match(
+stdout,
+/UncaughtError:Cannotfindmodule'wasi'[\w\W]+-<repl>\n/);
+
+// Run REPL with '--experimental-wasi-unstable-preview1'
+stdout=runREPLWithAdditionalFlags([
+'--experimental-wasi-unstable-preview1',
+]);
+assert.match(stdout,/\[Function:EventEmitter\]{/);
+assert.doesNotMatch(
+stdout,
+/UncaughtError:Cannotfindmodule'wasi'[\w\W]+-<repl>\n/);
+assert.match(stdout,/{WASI:\[classWASI\]}/);