Add support for bundleDependencies: true to bundle all

Author: isaacs

index.js

@@ -92,17 +92,23 @@ class BundleWalker extends EE {
     this.onPackage(this.package)
   }
 
+  allDepsBundled (pkg) {
+    return Object.keys(pkg.dependencies || {}).concat(
+      Object.keys(pkg.optionalDependencies || {}))
+  }
+
   onPackage (pkg) {
     // all deps are bundled if we got here as a child.
     // otherwise, only bundle bundledDeps
     // Get a unique-ified array with a short-lived Set
     const bdRaw = this.parent
-      ? Object.keys(pkg.dependencies || {}).concat(
-        Object.keys(pkg.optionalDependencies || {}))
+      ? this.allDepsBundled(pkg)
       : pkg.bundleDependencies || pkg.bundledDependencies || []
 
     const bd = Array.from(new Set(
-      Array.isArray(bdRaw) ? bdRaw : Object.keys(bdRaw)))
+      Array.isArray(bdRaw) ? bdRaw
+      : bdRaw === true ? this.allDepsBundled(pkg)
+      : Object.keys(bdRaw)))
 
     if (!bd.length)
       return this.done()

test/all-bundle.js

@@ -0,0 +1,58 @@
+'use strict'
+const t = require('tap')
+const path = require('path')
+const pkg = path.resolve(__dirname, path.basename(__filename, '.js'))
+
+require('./pkgtree.js')(pkg, {
+  $package: {
+    name: 'a',
+    version: '1.2.3',
+    bundledDependencies: true,
+    dependencies: {
+      b: '1.2.3',
+      d: '1.2.3'
+    }
+  },
+  b: {
+    d: { $package: {
+      name: 'd',
+      version: '1.2.3',
+      dependencies: {
+        e: '1.2.3'
+      }
+    }},
+    $package: {
+      name: 'b',
+      version: '1.2.3',
+      dependencies: {
+        c: '1.2.3',
+        d: '1.2.3'
+      }
+    }
+  },
+  c: { $package: {
+    name: 'c',
+    version: '1.2.3'
+  }},
+  d: { $package: {
+    name: 'd',
+    version: '1.2.3',
+    dependencies: {
+      c: '1.2.3'
+    }
+  }},
+  e: { $package: {
+    name: 'e',
+    version: '1.2.3'
+  }}
+}, t)
+
+const walk = require('../')
+
+const check = (result, t) => {
+  t.same(result.sort(), ['b', 'c', 'd', 'e'])
+  t.end()
+}
+
+t.test('sync', t => check(walk.sync({path: pkg}), t))
+t.test('async', t => walk({path: pkg}).then(res => check(res, t)))

test/no-bundle.js

@@ -7,6 +7,7 @@ require('./pkgtree.js')(pkg, {
   $package: {
     name: 'a',
     version: '1.2.3',
+    bundledDependencies: false,
     dependencies: {
       b: '1.2.3',
       d: '1.2.3'
@@ -16,6 +17,7 @@ require('./pkgtree.js')(pkg, {
     d: { $package: {
       name: 'd',
       version: '1.2.3',
+      bundleDependencies: false,
       dependencies: {
         e: '1.2.3'
       }