bare-module 2.0.2 → 2.2.0

package/index.js

@@ -51,6 +51,10 @@ const Module = module.exports = exports = class Module {
     this._exports = value
   }
 
+  get protocol () {
+    return this._protocol
+  }
+
   // For Node.js compatibility
   get id () {
     return this.filename
@@ -87,7 +91,6 @@ const Module = module.exports = exports = class Module {
 
   static _extensions = Object.create(null)
   static _protocols = Object.create(null)
-  static _imports = Object.create(null)
   static _cache = Object.create(null)
   static _bundles = Object.create(null)
   static _modules = new Set()
@@ -171,7 +174,7 @@ const Module = module.exports = exports = class Module {
 
   static createRequire (filename, opts = {}) {
     const {
-      imports = this._imports,
+      imports = null,
       protocol = this._protocolFor(filename, this._protocols['file:']),
       type = constants.types.SCRIPT,
       defaultType = constants.types.SCRIPT
@@ -226,7 +229,7 @@ const Module = module.exports = exports = class Module {
     }
 
     let {
-      imports = this._imports,
+      imports = null,
       protocol = this._protocolFor(specifier, this._protocols['file:']),
       referrer = null,
       dynamic = false,
@@ -258,22 +261,8 @@ const Module = module.exports = exports = class Module {
     const module = this._cache[specifier] = new this(specifier)
 
     module._defaultType = defaultType
-
-    let dirname = path.dirname(module._filename)
-    do {
-      const pkg = path.join(dirname, 'package.json')
-
-      if (protocol.exists(pkg)) {
-        try {
-          module._info = this.load(pkg, { protocol })._exports
-        } catch {}
-        break
-      }
-
-      dirname = path.dirname(dirname)
-    } while (dirname !== path.sep && dirname !== '.')
-
     module._main = main || module
+    module._info = this._loadPackageManifest(path.dirname(module._filename), protocol)
 
     let extension = this._extensionFor(type) || path.extname(specifier)
 
@@ -291,6 +280,22 @@ const Module = module.exports = exports = class Module {
     return this._transform(module, referrer, dynamic)
   }
 
+  static _loadPackageManifest (dirname, protocol) {
+    do {
+      const pkg = path.join(dirname, 'package.json')
+
+      if (protocol.exists(pkg)) {
+        try {
+          return this.load(pkg, { protocol })._exports
+        } catch {}
+      }
+
+      dirname = path.dirname(dirname)
+    } while (dirname !== path.sep && dirname !== '.')
+
+    return {}
+  }
+
   static resolve (specifier, dirname = os.cwd(), opts = {}) {
     if (typeof dirname !== 'string') {
       opts = dirname
@@ -298,7 +303,7 @@ const Module = module.exports = exports = class Module {
     }
 
     let {
-      imports = this._imports,
+      imports = null,
       protocol = this._protocols['file:'],
       referrer = null
     } = opts
@@ -335,8 +340,9 @@ const Module = module.exports = exports = class Module {
   }
 
   static * _resolve (specifier, dirname, protocol, imports) {
-    if (specifier in imports) specifier = imports[specifier]
-    else if (specifier in protocol.imports) specifier = protocol.imports[specifier]
+    const info = this._loadPackageManifest(dirname, protocol)
+
+    specifier = this._mapConditionalSpecifier(specifier, specifier, imports, protocol.imports, info.imports)
 
     protocol = this._protocolFor(specifier, protocol)
 
@@ -384,7 +390,7 @@ const Module = module.exports = exports = class Module {
         let specifier
 
         if (info.exports) {
-          specifier = this._mapConditionalExport('.', dirname, info.exports)
+          specifier = this._mapConditionalSpecifier('.', null, info.exports)
 
           if (specifier) specifier = path.join(dirname, specifier)
           else return
@@ -417,7 +423,7 @@ const Module = module.exports = exports = class Module {
 
           if (info) {
             if (info.exports) {
-              resolved = this._mapConditionalExport(expansion, dirname, info.exports)
+              resolved = this._mapConditionalSpecifier(expansion, null, info.exports)
 
               if (resolved) resolved = path.join(name, resolved)
               else return
@@ -445,33 +451,31 @@ const Module = module.exports = exports = class Module {
     }
   }
 
-  static _mapConditionalExport (specifier, dirname, exports) {
-    if (typeof exports !== 'object') exports = { '.': exports }
+  static _mapConditionalSpecifier (specifier, fallback, ...specifierMaps) {
+    const specifiers = this._flattenSpecifierMaps(specifierMaps)
 
-    if (specifier in exports) {
-      specifier = search(exports[specifier])
+    if (specifier in specifiers) {
+      specifier = search(specifiers[specifier])
     } else {
-      specifier = search(exports)
+      specifier = search(specifiers)
     }
 
-    if (specifier) specifier = path.join(dirname, specifier)
+    return specifier || fallback
 
-    return specifier
-
-    function search (specifier) {
+    function search (specifiers) {
       while (true) {
-        if (typeof specifier === 'string') return specifier
-        if (specifier === null || typeof specifier !== 'object') return specifier
-        specifier = first(specifier)
+        if (typeof specifiers === 'string') return specifiers
+        if (specifiers === null || typeof specifiers !== 'object') return specifiers
+        specifiers = first(specifiers)
       }
     }
 
-    function first (exports) {
-      for (const key in exports) {
+    function first (specifiers) {
+      for (const key in specifiers) {
         switch (key) {
           case 'require':
           case 'import':
-            return exports[key]
+            return specifiers[key]
         }
       }
 
@@ -479,6 +483,21 @@ const Module = module.exports = exports = class Module {
     }
   }
 
+  static _flattenSpecifierMaps (specifierMaps) {
+    const specifiers = Object.create(null)
+
+    for (const map of specifierMaps) {
+      this._mergeSpecifierMaps(typeof map === 'object' ? map : { '.': map }, specifiers)
+    }
+
+    return specifiers
+  }
+
+  static _mergeSpecifierMaps (source, destination) {
+    // TODO Do a deep merge
+    Object.assign(destination, source)
+  }
+
   static _extensionFor (type) {
     switch (type) {
       case constants.types.SCRIPT:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-module",
-  "version": "2.0.2",
+  "version": "2.2.0",
   "description": "Module support for JavaScript",
   "main": "index.js",
   "files": [