esm: improve commonjs hint on module not found

aduh95 · guybedford · commit 1cb80d1e0590 · 2020-05-15T19:09:01.000-07:00
Adds hint when module specifier is a file URL. PR-URL: nodejs/node#33220 Fixes: nodejs/node#33219 Reviewed-By: Guy Bedford <guybedford@gmail.com>
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
