perf: zoomzoom. Even more concurrency!

index.js

@@ -111,17 +111,12 @@ class Installer {
       if (dep.dev && this.config.config.production) { return }
       const depPath = dep.path(this.prefix)
       // Process children first, then extract this child
-      return BB.resolve()
-      .then(() => {
-        if (dep !== this.tree) {
-          // Don't try to extract the root
-          return extract.child(dep.name, dep, depPath, this.config)
-        }
-      })
-      .then(next)
-      .then(() => {
-        dep !== this.tree && this.pkgCount++
-      })
+      return BB.join(
+        !dep.isRoot && extract.child(
+          dep.name, dep, depPath, this.config
+        ).then(() => { this.pkgCount++ }),
+        next()
+      )
     }, {concurrency: 50, Promise: BB})
   }
 

lib/worker.js

@@ -4,7 +4,6 @@ const BB = require('bluebird')
 
 const log = require('npmlog')
 const pacote = require('pacote')
-const rimraf = BB.promisify(require('rimraf'))
 
 module.exports = (args, cb) => {
   const parsed = typeof args === 'string' ? JSON.parse(args) : args
@@ -13,7 +12,5 @@ module.exports = (args, cb) => {
   const opts = parsed[2]
   opts.log = log
   log.level = opts.loglevel
-  return rimraf(extractTo, {ignore: 'node_modules'}).then(() => {
-    return pacote.extract(spec, extractTo, opts)
-  }).nodeify(cb)
+  return BB.resolve(pacote.extract(spec, extractTo, opts)).nodeify(cb)
 }