fix(ls): respect --include-workspace-root (#4481)

Author: fritzy

lib/commands/ls.js

@@ -95,10 +95,15 @@ class LS extends ArboristWorkspaceCmd {
         return true
       }
 
+      if (this.npm.flatOptions.includeWorkspaceRoot
+          && !edge.to.isWorkspace) {
+        return true
+      }
+
       if (edge.from.isProjectRoot) {
-        return edge.to &&
-          edge.to.isWorkspace &&
-          wsNodes.includes(edge.to.target)
+        return (edge.to
+          && edge.to.isWorkspace
+          && wsNodes.includes(edge.to.target))
       }
 
       return true

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

@@ -504,6 +504,15 @@ [email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspac
 
 `
 
+exports[`test/lib/commands/ls.js TAP ls loading a tree containing workspaces > should inlude root and specified workspace 1`] = `
+[email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspaces
++-- [email protected] -> ./d
+| \`-- [email protected]
+|   \`-- [email protected]
+\`-- [email protected]
+
+`
+
 exports[`test/lib/commands/ls.js TAP ls loading a tree containing workspaces > should list --all workspaces properly 1`] = `
 [email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspaces
 +-- [email protected] -> ./a
@@ -515,7 +524,8 @@ [email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspac
 | \`-- [email protected]
 |   \`-- [email protected]
 +-- [email protected] -> ./group/e
-\`-- [email protected] -> ./group/f
++-- [email protected] -> ./group/f
+\`-- [email protected]
 
 `
 
@@ -529,7 +539,8 @@ [email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspac
 | \`-- [email protected]
 |   \`-- [email protected]
 +-- [email protected] -> ./group/e
-\`-- [email protected] -> ./group/f
++-- [email protected] -> ./group/f
+\`-- [email protected]
 
 `
 
@@ -543,13 +554,14 @@ exports[`test/lib/commands/ls.js TAP ls loading a tree containing workspaces > s
 +-- [[email protected] -> ./d
 | \`-- [email protected]
 +-- [[email protected] -> ./group/e
-\`-- [[email protected] -> ./group/f
++-- [[email protected] -> ./group/f
+\`-- [email protected]
 
 `
 
 exports[`test/lib/commands/ls.js TAP ls loading a tree containing workspaces > should not list workspaces with --no-workspaces 1`] = `
 [[email protected] {CWD}/tap-testdir-ls-ls-loading-a-tree-containing-workspaces
-[0m\`-- (empty)[0m
+[0m\`-- [email protected][0m
 
 `
 

test/lib/commands/ls.js

@@ -1384,6 +1384,7 @@ t.test('ls', t => {
         name: 'workspaces-tree',
         version: '1.0.0',
         workspaces: ['./a', './b', './d', './group/*'],
+        dependencies: { pacote: '1.0.0' },
       }),
       node_modules: {
         a: t.fixture('symlink', '../a'),
@@ -1412,6 +1413,9 @@ t.test('ls', t => {
         baz: {
           'package.json': JSON.stringify({ name: 'baz', version: '1.0.0' }),
         },
+        pacote: {
+          'package.json': JSON.stringify({ name: 'pacote', version: '1.0.0' }),
+        },
       },
       a: {
         'package.json': JSON.stringify({
@@ -1469,6 +1473,7 @@ t.test('ls', t => {
     npm.flatOptions.workspacesEnabled = false
     await ls.exec([])
     t.matchSnapshot(redactCwd(result), 'should not list workspaces with --no-workspaces')
+
     config.all = true
     config.depth = Infinity
     npm.color = false
@@ -1495,6 +1500,12 @@ t.test('ls', t => {
 
     t.matchSnapshot(redactCwd(result), 'should filter using workspace config')
 
+    // filter out a single workspace and include root
+    npm.flatOptions.includeWorkspaceRoot = true
+    await ls.execWorkspaces([], ['d'])
+    t.matchSnapshot(redactCwd(result), 'should inlude root and specified workspace')
+    npm.flatOptions.includeWorkspaceRoot = false
+
     // filter out a workspace by parent path
     await ls.execWorkspaces([], ['./group'])