dedupe: Handle dev/extraneous deps better

devDeps are ignored.  Extraneous deps are treated as if they are
required with '' as the range.

1 files changed, 7 insertions, 1 deletions

diff --git a/lib/dedupe.js b/lib/dedupe.js
index 50908e319..9a357908a 100644
--- a/lib/dedupe.js
+++ b/lib/dedupe.js
@@ -83,7 +83,7 @@ function dedupe_ (dir, filter, unavoidable, dryrun, silent, cb) {
         return d && d.parent && !d.parent.duplicate && !d.unavoidable
       }).map(function M (d) {
         return [d.path, d.version, d.dependents.map(function (k) {
-          return [k.path, k.version, k.dependencies[d.name]]
+          return [k.path, k.version, k.dependencies[d.name] || ""]
         })]
       })]
     }).map(function (item) {
@@ -298,6 +298,12 @@ function readInstalled (dir, counter, parent, cb) {
 
   function next () {
     if (!children || !pkg || !realpath) return
+
+    // ignore devDependencies.  Just leave them where they are.
+    children = children.filter(function (c) {
+      return !pkg.devDependencies.hasOwnProperty(c)
+    })
+
     pkg.realPath = realpath
     if (pkg.realPath !== pkg.path) children = []
     var d = path.resolve(dir, "node_modules")