repl: fix referrer for dynamic import

The ESM loader does not accept a directory as the referrer, it requires a path within the directory. Add `/repl` to ensure relative dynamic imports can succeed. Fixes: #19570 PR-URL: #30609 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Ben Coe <[email protected]>

Author: coreyfarrell

lib/repl.js

@@ -324,10 +324,12 @@ function REPLServer(prompt,
 if(code==='\n')
 returncb(null);
 
-letpwd;
+letparentURL;
 try{
 const{ pathToFileURL }=require('url');
-pwd=pathToFileURL(process.cwd()).href;
+// Adding `/repl` prevents dynamic imports from loading relative
+// to the parent of `process.cwd()`.
+parentURL=pathToFileURL(path.join(process.cwd(),'repl')).href;
 }catch{
 }
 while(true){
@@ -342,7 +344,7 @@ function REPLServer(prompt,
 filename: file,
 displayErrors: true,
 importModuleDynamically: experimentalModules ? async(specifier)=>{
-returnasyncESM.ESMLoader.import(specifier,pwd);
+returnasyncESM.ESMLoader.import(specifier,parentURL);
 } : undefined
 });
 }catch(e){

test/parallel/test-repl-import-referrer.js

@@ -0,0 +1,29 @@
+'use strict';
+constcommon=require('../common');
+constassert=require('assert');
+constcp=require('child_process');
+constfixtures=require('../common/fixtures');
+
+constargs=[
+'--interactive',
+'--experimental-repl-await',
+'--experimental-modules'
+];
+
+constopts={cwd: fixtures.path('es-modules')};
+constchild=cp.spawn(process.execPath,args,opts);
+
+letoutput='';
+child.stdout.setEncoding('utf8');
+child.stdout.on('data',(data)=>{
+output+=data;
+});
+
+child.on('exit',common.mustCall(()=>{
+constresults=output.replace(/^>/mg,'').split('\n').slice(2);
+assert.deepStrictEqual(results,['[Module]{message: \'A message\'}','']);
+}));
+
+child.stdin.write('await import(\'./message.mjs\');\n');
+child.stdin.write('.exit');
+child.stdin.end();