fix: listing deps of a linked dep

npm ls was failing to properly follow symlink targets in order to find its dependencies to get printed output. PR-URL: #1871 Credit: @ruyadorno Close: #1871 Reviewed-by: @nlf
npm · Sep 29, 2020 · 798a6e3 · 798a6e3
1 parent ee79c55
commit 798a6e3
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/lib/ls.js b/lib/ls.js
@@ -428,7 +428,7 @@ const ls = async (args) => {
         !(node instanceof Arborist.Node) || (node[_depth] > depthToPrint)
       return (shouldSkipChildren)
         ? []
-        : [...node.edgesOut.values()]
+        : [...(node.target || node).edgesOut.values()]
           .filter(filterByEdgesTypes({
             dev,
             development,

diff --git a/tap-snapshots/test-lib-ls.js-TAP.test.js b/tap-snapshots/test-lib-ls.js-TAP.test.js
@@ -453,6 +453,7 @@ [email protected] {CWD}/ls-ls-loading-a-tree-containing-works
 exports[`test/lib/ls.js TAP ls loading a tree containing workspaces > should list workspaces properly 1`] = `
 [email protected] {CWD}/ls-ls-loading-a-tree-containing-workspaces
 +-- [email protected] -> {CWD}/ls-ls-loading-a-tree-containing-workspaces/a
+| \`-- [email protected]
 \`-- [email protected] -> {CWD}/ls-ls-loading-a-tree-containing-workspaces/b
 
 `

diff --git a/test/lib/ls.js b/test/lib/ls.js
@@ -1396,12 +1396,21 @@ t.test('ls', (t) => {
       }),
       node_modules: {
         a: t.fixture('symlink', '../a'),
-        b: t.fixture('symlink', '../b')
+        b: t.fixture('symlink', '../b'),
+        c: {
+          'package.json': JSON.stringify({
+            name: 'c',
+            version: '1.0.0'
+          })
+        }
       },
       a: {
         'package.json': JSON.stringify({
           name: 'a',
-          version: '1.0.0'
+          version: '1.0.0',
+          dependencies: {
+            c: '^1.0.0'
+          }
         })
       },
       b: {