Fix bug: Remove self from list of dependencies.

See npm/read-installed#37
timoxley · Jan 26, 2015 · f1e3973 · f1e3973
1 parent 5bb7c00
commit f1e3973
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -61,7 +61,10 @@ function getDependencies(mod, result, visited, depth) {
   var dependencies = mod.dependencies || []
   Object.keys(dependencies).forEach(name => {
     var dep = mod.dependencies[name]
-    if (dep === mod) return
+    if (dep === mod) {
+      delete mod.dependencies[name]
+      return
+    }
     if (typeof dep === 'string') return
     if (visited[dep.realPath]) return
     visited[dep.realPath] = true

diff --git a/installed.js b/installed.js
@@ -95,7 +95,10 @@ function getDependencies(mod, result, visited, depth) {
   var dependencies = mod.dependencies || [];
   Object.keys(dependencies).forEach(function (name) {
     var dep = mod.dependencies[name];
-    if (dep === mod) return;
+    if (dep === mod) {
+      delete mod.dependencies[name];
+      return;
+    }
     if (typeof dep === "string") return;
     if (visited[dep.realPath]) return;
     visited[dep.realPath] = true;