bare-module 2.3.1 → 2.5.0

package/index.js

@@ -239,9 +239,10 @@ const Module = module.exports = exports = class Module {
     }
 
     let {
-      imports = null,
-      protocol = this._protocolFor(specifier, this._protocols['file:']),
       referrer = null,
+      protocol = this._protocolFor(specifier, this._protocols['file:']),
+      imports = null,
+      builtins = null,
       dynamic = false,
       main = referrer ? referrer._main : null,
       type = 0,
@@ -270,22 +271,26 @@ const Module = module.exports = exports = class Module {
 
     const module = this._cache[specifier] = new this(specifier)
 
-    module._defaultType = defaultType
-    module._main = main || module
-    module._info = this._loadPackageManifest(path.dirname(module._filename), protocol)
+    if (builtins && specifier in builtins) {
+      module._exports = builtins[specifier]
+    } else {
+      module._defaultType = defaultType
+      module._main = main || module
+      module._info = this._loadPackageManifest(path.dirname(module._filename), protocol)
 
-    let extension = this._extensionFor(type) || path.extname(specifier)
+      let extension = this._extensionFor(type) || path.extname(specifier)
 
-    if (extension in this._extensions === false) {
-      if (defaultType) extension = this._extensionFor(defaultType) || '.js'
-      else extension = '.js'
-    }
+      if (extension in this._extensions === false) {
+        if (defaultType) extension = this._extensionFor(defaultType) || '.js'
+        else extension = '.js'
+      }
 
-    if (extension === '.bundle' && path.extname(specifier) !== extension) {
-      throw errors.INVALID_BUNDLE_EXTENSION(`Invalid extension for bundle '${specifier}'`)
-    }
+      if (extension === '.bundle' && path.extname(specifier) !== extension) {
+        throw errors.INVALID_BUNDLE_EXTENSION(`Invalid extension for bundle '${specifier}'`)
+      }
 
-    this._extensions[extension].call(this, module, source, referrer, protocol, imports)
+      this._extensions[extension].call(this, module, source, referrer, protocol, imports)
+    }
 
     return this._transform(module, referrer, dynamic)
   }
@@ -320,9 +325,11 @@ const Module = module.exports = exports = class Module {
     }
 
     let {
-      imports = null,
+      referrer = null,
       protocol = this._protocols['file:'],
-      referrer = null
+      imports = null,
+      builtins = null,
+      conditions = ['import', 'require', 'bare', 'node']
     } = opts
 
     const bundle = this._bundleFor(path.dirname(specifier), protocol)
@@ -343,7 +350,7 @@ const Module = module.exports = exports = class Module {
       })
     }
 
-    const [resolved = null] = this._resolve(specifier, dirname, protocol, imports)
+    const [resolved = null] = this._resolve(specifier, dirname, protocol, imports, builtins, conditions)
 
     if (resolved === null) {
       let msg = `Cannot find module '${specifier}'`
@@ -356,10 +363,14 @@ const Module = module.exports = exports = class Module {
     return protocol.postresolve(resolved, dirname)
   }
 
-  static * _resolve (specifier, dirname, protocol, imports) {
+  static * _resolve (specifier, dirname, protocol, imports, builtins, conditions) {
     const info = this._loadPackageManifest(dirname, protocol)
 
-    specifier = this._mapConditionalSpecifier(specifier, specifier, imports, protocol.imports, info.imports)
+    specifier = this._mapConditionalSpecifier(
+      specifier,
+      conditions,
+      [imports, protocol.imports, info.imports]
+    )
 
     protocol = this._protocolFor(specifier, protocol)
 
@@ -367,12 +378,14 @@ const Module = module.exports = exports = class Module {
 
     yield * protocol.resolve(specifier, dirname, imports)
 
+    if (builtins && specifier in builtins) yield specifier
+
     if (path.isAbsolute(specifier)) {
       yield * this._resolveFile(specifier, protocol)
-      yield * this._resolveDirectory(specifier, protocol)
+      yield * this._resolveDirectory(specifier, protocol, conditions)
     }
 
-    yield * this._resolveNodeModules(specifier, dirname, protocol)
+    yield * this._resolveNodeModules(specifier, dirname, protocol, conditions)
   }
 
   static * _resolveFile (filename, protocol) {
@@ -394,13 +407,18 @@ const Module = module.exports = exports = class Module {
     yield * this._resolveFile(path.join(dirname, 'index'), protocol)
   }
 
-  static * _resolveDirectory (dirname, protocol) {
+  static * _resolveDirectory (dirname, protocol, conditions) {
     const info = this._loadPackageManifest(dirname, protocol, { traverse: false })
 
     let specifier = null
 
     if (info.exports) {
-      specifier = this._mapConditionalSpecifier('.', null, info.exports)
+      specifier = this._mapConditionalSpecifier(
+        '.',
+        conditions,
+        [info.exports],
+        null // Disable fallback
+      )
 
       if (specifier) specifier = path.join(dirname, specifier)
       else return // Unexported
@@ -416,7 +434,7 @@ const Module = module.exports = exports = class Module {
     yield * this._resolveIndex(dirname, protocol)
   }
 
-  static * _resolveNodeModules (specifier, dirname, protocol) {
+  static * _resolveNodeModules (specifier, dirname, protocol, conditions) {
     const [, name, expansion = '.'] = /^((?:@[^/\\%]+\/)?[^./\\%][^/\\%]*)(\/.*)?$/.exec(specifier) || []
 
     for (const nodeModules of this._resolveNodeModulesPaths(dirname)) {
@@ -426,7 +444,12 @@ const Module = module.exports = exports = class Module {
         const info = this._loadPackageManifest(path.join(nodeModules, name), protocol, { traverse: false })
 
         if (info.exports) {
-          resolved = this._mapConditionalSpecifier(expansion, null, info.exports)
+          resolved = this._mapConditionalSpecifier(
+            expansion,
+            conditions,
+            [info.exports],
+            null // Disable fallback
+          )
 
           if (resolved) resolved = path.join(name, resolved)
           else return // Unexported
@@ -436,7 +459,7 @@ const Module = module.exports = exports = class Module {
       const filename = path.join(nodeModules, resolved)
 
       yield * this._resolveFile(filename, protocol)
-      yield * this._resolveDirectory(filename, protocol)
+      yield * this._resolveDirectory(filename, protocol, conditions)
     }
   }
 
@@ -452,7 +475,7 @@ const Module = module.exports = exports = class Module {
     }
   }
 
-  static _mapConditionalSpecifier (specifier, fallback, ...specifierMaps) {
+  static _mapConditionalSpecifier (specifier, conditions, specifierMaps, fallback = specifier) {
     const specifiers = this._flattenSpecifierMaps(specifierMaps)
 
     if (specifier in specifiers) {
@@ -467,19 +490,15 @@ const Module = module.exports = exports = class Module {
       while (true) {
         if (typeof specifiers === 'string') return specifiers
         if (specifiers === null || typeof specifiers !== 'object') return specifiers
+
         specifiers = first(specifiers)
       }
     }
 
     function first (specifiers) {
       for (const key in specifiers) {
-        switch (key) {
-          case 'require':
-          case 'import':
-          case 'bare':
-          case 'node':
-          case 'default':
-            return specifiers[key]
+        if (key === 'default' || conditions.includes(key)) {
+          return specifiers[key]
         }
       }
 
@@ -490,14 +509,17 @@ 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)
+    for (let map of specifierMaps) {
+      if (typeof map === 'string') map = { '.': map }
+      if (map === null || typeof map !== 'object') continue
+
+      this._mergeSpecifierMaps(specifiers, map)
     }
 
     return specifiers
   }
 
-  static _mergeSpecifierMaps (source, destination) {
+  static _mergeSpecifierMaps (destination, source) {
     // TODO Do a deep merge
     Object.assign(destination, source)
   }
@@ -769,7 +791,8 @@ Module._protocols['file:'] = new Protocol({
   },
 
   postresolve (specifier) {
-    return binding.realpath(specifier)
+    if (path.isAbsolute(specifier)) return binding.realpath(specifier)
+    return specifier
   },
 
   exists (filename) {

package/package.json

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