module: also enable subpath imports in REPL

PR-URL: #43450
Fixes: #43410
Reviewed-By: Guy Bedford <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: rayw000

lib/internal/modules/cjs/loader.js

@@ -909,20 +909,19 @@ Module._resolveFilename = function(request, parent, isMain, options) {
     paths = Module._resolveLookupPaths(request, parent);
   }
 
-  if (parent?.filename) {
-    if (request[0] === '#') {
-      const pkg = readPackageScope(parent.filename) || {};
-      if (pkg.data?.imports != null) {
-        try {
-          return finalizeEsmResolution(
-            packageImportsResolve(request, pathToFileURL(parent.filename),
-                                  cjsConditions), request, parent.filename,
-            pkg.path);
-        } catch (e) {
-          if (e.code === 'ERR_MODULE_NOT_FOUND')
-            throw createEsmNotFoundErr(request);
-          throw e;
-        }
+  if (request[0] === '#' && (parent?.filename || parent?.id === '<repl>')) {
+    const parentPath = parent?.filename ?? process.cwd() + path.sep;
+    const pkg = readPackageScope(parentPath) || {};
+    if (pkg.data?.imports != null) {
+      try {
+        return finalizeEsmResolution(
+          packageImportsResolve(request, pathToFileURL(parentPath),
+                                cjsConditions), parentPath, request,
+          pkg.path);
+      } catch (e) {
+        if (e.code === 'ERR_MODULE_NOT_FOUND')
+          throw createEsmNotFoundErr(request);
+        throw e;
       }
     }
   }

test/es-module/test-esm-repl-imports.js

@@ -0,0 +1,19 @@
+'use strict';
+const { mustCall } = require('../common');
+const assert = require('assert');
+const fixtures = require('../common/fixtures');
+const { spawn } = require('child_process');
+
+const child = spawn(process.execPath, [
+  '--interactive',
+], {
+  cwd: fixtures.path('es-modules', 'pkgimports'),
+});
+
+child.stdin.end(
+  'try{require("#test");await import("#test")}catch{process.exit(-1)}'
+);
+
+child.on('exit', mustCall((code) => {
+  assert.strictEqual(code, 0);
+}));