bare-module-traverse 1.5.1 → 1.6.0

package/index.js

@@ -123,7 +123,10 @@ exports.module = function* (
     source = yield { module: url }
 
     if (source === null) {
-      throw errors.MODULE_NOT_FOUND(`Cannot find module '${url.href}'`)
+      throw errors.MODULE_NOT_FOUND(
+        `Cannot find module '${url.href}'`,
+        url.href
+      )
     }
   }
 
@@ -307,6 +310,7 @@ exports.imports = function* (
     matchedConditions.push(condition)
 
     const resolver = resolve(entry, parentURL, { ...opts, matchedConditions })
+    const candidates = []
 
     let next = resolver.next()
     let resolutions = 0
@@ -319,6 +323,8 @@ exports.imports = function* (
       } else {
         const url = value.resolution
 
+        candidates.push(url)
+
         let resolved = false
 
         if (
@@ -391,19 +397,35 @@ exports.imports = function* (
 
     matchedConditions.pop()
 
+    let message = `Cannot find ${condition === 'default' ? 'module' : condition} '${specifier}' imported from '${parentURL.href}'`
+
+    if (candidates.length > 0) {
+      message += '\nCandidates:'
+      message += '\n' + candidates.map((url) => '- ' + url.href).join('\n')
+    }
+
     if (resolutions === 0) {
       switch (condition) {
         case 'addon':
           throw errors.ADDON_NOT_FOUND(
-            `Cannot find addon '${specifier}' imported from '${parentURL.href}'`
+            message,
+            specifier,
+            parentURL,
+            candidates
           )
         case 'asset':
           throw errors.ASSET_NOT_FOUND(
-            `Cannot find asset '${specifier}' imported from '${parentURL.href}'`
+            message,
+            specifier,
+            parentURL,
+            candidates
           )
         default:
           throw errors.MODULE_NOT_FOUND(
-            `Cannot find module '${specifier}' imported from '${parentURL.href}'`
+            message,
+            specifier,
+            parentURL,
+            candidates
           )
       }
     }

package/lib/errors.js

@@ -12,28 +12,46 @@ module.exports = class ModuleTraverseError extends Error {
     return 'ModuleTraverseError'
   }
 
-  static MODULE_NOT_FOUND(msg) {
-    return new ModuleTraverseError(
+  static MODULE_NOT_FOUND(msg, specifier, referrer = null, candidates = []) {
+    const err = new ModuleTraverseError(
       msg,
       'MODULE_NOT_FOUND',
       ModuleTraverseError.MODULE_NOT_FOUND
     )
+
+    err.specifier = specifier
+    err.referrer = referrer
+    err.candidates = candidates
+
+    return err
   }
 
-  static ADDON_NOT_FOUND(msg) {
-    return new ModuleTraverseError(
+  static ADDON_NOT_FOUND(msg, specifier, referrer = null, candidates = []) {
+    const err = new ModuleTraverseError(
       msg,
       'ADDON_NOT_FOUND',
       ModuleTraverseError.ADDON_NOT_FOUND
     )
+
+    err.specifier = specifier
+    err.referrer = referrer
+    err.candidates = candidates
+
+    return err
   }
 
-  static ASSET_NOT_FOUND(msg) {
-    return new ModuleTraverseError(
+  static ASSET_NOT_FOUND(msg, specifier, referrer = null, candidates = []) {
+    const err = new ModuleTraverseError(
       msg,
       'ASSET_NOT_FOUND',
       ModuleTraverseError.ASSET_NOT_FOUND
     )
+
+    err.specifier = specifier
+    err.referrer = referrer
+    err.candidates = candidates
+
+    return err
   }
 
   static INVALID_IMPORTS_MAP(msg) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-module-traverse",
-  "version": "1.5.1",
+  "version": "1.6.0",
   "description": "Low-level module graph traversal for Bare",
   "exports": {
     ".": {