Skip to content

Commit

Permalink
repl: fix referrer for dynamic import
Browse files Browse the repository at this point in the history
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 <me@gus.host>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Coe <bencoe@gmail.com>
  • Loading branch information
coreyfarrell authored and targos committed Dec 1, 2019
1 parent 6257408 commit b91d22c
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 3 deletions.
8 changes: 5 additions & 3 deletions lib/repl.js
Original file line number Diff line number Diff line change
Expand Up @@ -332,10 +332,12 @@ function REPLServer(prompt,
if (code === '\n')
return cb(null);

let pwd;
let parentURL;
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) {
Expand All @@ -350,7 +352,7 @@ function REPLServer(prompt,
filename: file,
displayErrors: true,
importModuleDynamically: async (specifier) => {
return asyncESM.ESMLoader.import(specifier, pwd);
return asyncESM.ESMLoader.import(specifier, parentURL);
}
});
} catch (e) {
Expand Down
24 changes: 24 additions & 0 deletions test/parallel/test-repl-import-referrer.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
'use strict';
const common = require('../common');
const assert = require('assert');
const cp = require('child_process');
const fixtures = require('../common/fixtures');

const args = ['--interactive', '--experimental-repl-await'];
const opts = { cwd: fixtures.path('es-modules') };
const child = cp.spawn(process.execPath, args, opts);

let output = '';
child.stdout.setEncoding('utf8');
child.stdout.on('data', (data) => {
output += data;
});

child.on('exit', common.mustCall(() => {
const results = 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();

0 comments on commit b91d22c

Please sign in to comment.