fix: missing bins info in package-lock

When migrating from package-lock v1 -> v2 in a project that currently
has an actual tree + a package-lock file the reify process fails to
properly update the package-lock with the bins info.

This fixes it by setting build-ideal-tree to always inflate old
lockfiles even if `complete: true` isn't set.

Fixes: npm/cli#1957

lib/arborist/build-ideal-tree.js

@@ -578,7 +578,7 @@ module.exports = cls => class IdealTreeBuilder extends cls {
     const { meta, inventory } = this.idealTree
     const ancient = meta.ancientLockfile
     const old = meta.loadedFromDisk && !(meta.originalLockfileVersion >= 2)
-    if (inventory.size === 0 || !(ancient || old && this[_complete]))
+    if (inventory.size === 0 || !(ancient || old))
       return
 
     // if the lockfile is from node v5 or earlier, then we'll have to reload