diff-trees: Only avoid adding bundled deps when they're the result of…

… a link Credit: @iarna
npm · Apr 10, 2018 · bd16485 · bd16485
1 parent 4785f13
commit bd16485
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/install/diff-trees.js b/lib/install/diff-trees.js
@@ -221,9 +221,8 @@ var diffTrees = module.exports._diffTrees = function (oldTree, newTree) {
         pkg.fromPath = toMv.pkg.path
         setAction(differences, 'move', pkg)
         delete toRemove[toMv.flatname]
-      // we don't generate add actions for things found in links (which already exist on disk) or
-      // for bundled modules (which will be installed when we install their parent)
-      } else if (!(pkg.isInLink && pkg.fromBundle)) {
+      // we don't generate add actions for things found in links (which already exist on disk)
+      } else if (!pkg.isInLink || !(pkg.fromBundle && pkg.fromBundle.isLink)) {
         setAction(differences, 'add', pkg)
       }
     }