fix(outdated): add dependent location in long output

Author: milaninfy

lib/commands/outdated.js

@@ -195,6 +195,9 @@ class Outdated extends ArboristWorkspaceCmd {
           wanted: wanted.version,
           latest: latest.version,
           workspaceDependent: edge.from?.isWorkspace ? edge.from.pkgid : null,
+          dependentLocation: edge.from?.name
+            ? edge.from?.location
+            : 'global',
           dependent: edge.from?.name ?? 'global',
           homepage: packument.homepage,
         })
@@ -226,7 +229,7 @@ class Outdated extends ArboristWorkspaceCmd {
         'Latest',
         'Location',
         'Depended by',
-        ...long ? ['Package Type', 'Homepage'] : [],
+        ...long ? ['Package Type', 'Homepage', 'Dependent Location'] : [],
       ].map(h => bold.underline(h)),
       ...list.map((d) => [
         d.current === d.wanted ? yellow(d.name) : red(d.name),
@@ -235,7 +238,7 @@ class Outdated extends ArboristWorkspaceCmd {
         blue(d.latest),
         d.location ?? '-',
         d.workspaceDependent ? blue(d.workspaceDependent) : d.dependent,
-        ...long ? [d.type, blue(d.homepage ?? '')] : [],
+        ...long ? [d.type, blue(d.homepage ?? ''), d.dependentLocation] : [],
       ]),
     ], {
       align: ['l', 'r', 'r', 'r', 'l'],
@@ -252,7 +255,7 @@ class Outdated extends ArboristWorkspaceCmd {
       d.current ? `${d.name}@${d.current}` : 'MISSING',
       `${d.name}@${d.latest}`,
       d.dependent,
-      ...this.npm.config.get('long') ? [d.type, d.homepage] : [],
+      ...this.npm.config.get('long') ? [d.type, d.homepage, d.dependentLocation] : [],
     ].join(':')).join('\n')
   }
 
@@ -268,7 +271,10 @@ class Outdated extends ArboristWorkspaceCmd {
         latest: d.latest,
         dependent: d.dependent,
         location: d.path,
-        ...this.npm.config.get('long') ? { type: d.type, homepage: d.homepage } : {},
+        ...this.npm.config.get('long') ? {
+          type: d.type,
+          homepage: d.homepage,
+          dependentLocation: d.dependentLocation } : {},
       }
       acc[d.name] = acc[d.name]
         // If this item alread has an outdated dep then we turn it into an array

tap-snapshots/test/lib/commands/outdated.js.test.cjs

@@ -31,29 +31,33 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
     "latest": "1.0.1",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/cat",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   },
   "chai": {
     "current": "1.0.0",
     "wanted": "1.0.1",
     "latest": "1.0.1",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/chai",
-    "type": "peerDependencies"
+    "type": "peerDependencies",
+    "dependentLocation": ""
   },
   "dog": {
     "current": "1.0.1",
     "wanted": "1.0.1",
     "latest": "2.0.0",
     "dependent": "prefix",
     "location": "{CWD}/prefix/node_modules/dog",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   },
   "theta": {
     "wanted": "1.0.1",
     "latest": "1.0.1",
     "dependent": "prefix",
-    "type": "dependencies"
+    "type": "dependencies",
+    "dependentLocation": ""
   }
 }
 `
@@ -90,7 +94,7 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --long > must match snapshot 1`] = `
-Package  Current  Wanted  Latest  Location           Depended by  Package Type      Homepage
+Package  Current  Wanted  Latest  Location           Depended by  Package Type      Homepage  Dependent Location
 cat        1.0.0   1.0.1   1.0.1  node_modules/cat   prefix       dependencies
 chai       1.0.0   1.0.1   1.0.1  node_modules/chai  prefix       peerDependencies
 dog        1.0.1   1.0.1   2.0.0  node_modules/dog   prefix       dependencies
@@ -120,10 +124,10 @@ exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --parseable --long > must match snapshot 1`] = `
-{CWD}/prefix/node_modules/cat:[email protected]:[email protected]:[email protected]:prefix:dependencies:
-{CWD}/prefix/node_modules/chai:[email protected]:[email protected]:[email protected]:prefix:peerDependencies:
-{CWD}/prefix/node_modules/dog:[email protected]:[email protected]:[email protected]:prefix:dependencies:
-:[email protected]:MISSING:[email protected]:prefix:dependencies:
+{CWD}/prefix/node_modules/cat:[email protected]:[email protected]:[email protected]:prefix:dependencies::
+{CWD}/prefix/node_modules/chai:[email protected]:[email protected]:[email protected]:prefix:peerDependencies::
+{CWD}/prefix/node_modules/dog:[email protected]:[email protected]:[email protected]:prefix:dependencies::
+:[email protected]:MISSING:[email protected]:prefix:dependencies::
 `
 
 exports[`test/lib/commands/outdated.js TAP should display outdated deps outdated --parseable > must match snapshot 1`] = `