esm: improve commonjs hint on module not found

Adds hint when module specifier is a file URL. PR-URL: #33220 Fixes: #33219 Reviewed-By: Guy Bedford <guybedford@gmail.com>
nodejs · May 16, 2020 · e53de96 · e53de96
1 parent 35aae31
commit e53de96
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/lib/internal/modules/esm/resolve.js b/lib/internal/modules/esm/resolve.js
@@ -758,6 +758,9 @@ function defaultResolve(specifier, context = {}, defaultResolveUnused) {
     // resolved CommonJS module
     if (error.code === 'ERR_MODULE_NOT_FOUND' ||
         error.code === 'ERR_UNSUPPORTED_DIR_IMPORT') {
+      if (StringPrototypeStartsWith(specifier, 'file://')) {
+        specifier = fileURLToPath(specifier);
+      }
       const found = resolveAsCommonJS(specifier, parentURL);
       if (found) {
         // Modify the stack and message string to include the hint