fix: make sure we pack workspace targets, not the link node

lib/index.js

@@ -321,8 +321,12 @@ class PackWalker extends IgnoreWalker {
       const node = this.tree.edgesOut.get(dep).to
       // and start building options to be passed to the walker for this package
       const walkerOpts = {
+        // we use node.path for the path because we want the location the node was linked to,
+        // not where it actually lives on disk
         path: node.path,
-        tree: node,
+        // but link nodes don't have edgesOut, so we need to pass in the target of the node
+        // in order to make sure we correctly traverse its dependencies
+        tree: node.target,
         isPackage: true,
         ignoreFiles: [],
         seen: this.seen, // pass through seen so we can prevent infinite circular loops

test/bundled-workspace.js

@@ -0,0 +1,54 @@
+'use strict'
+
+const t = require('tap')
+const packlist = require('../')
+
+t.test('packs workspace dependencies correctly', async (t) => {
+  const pkg = t.testdir({
+    'package.json': JSON.stringify({
+      name: 'root',
+      version: '1.2.3',
+      main: 'index.js',
+      files: ['index.js'],
+      dependencies: {
+        foo: '1.0.0',
+        bar: '1.0.0',
+      },
+      bundleDependencies: ['foo', 'bar'],
+      workspaces: ['./workspaces/*'],
+    }),
+    'index.js': '',
+    workspaces: {
+      foo: {
+        'package.json': JSON.stringify({
+          name: 'foo',
+          version: '1.0.0',
+          main: 'index.js',
+        }),
+        'index.js': '',
+      },
+      bar: {
+        'package.json': JSON.stringify({
+          name: 'bar',
+          version: '1.0.0',
+          main: 'index.js',
+        }),
+        'index.js': '',
+      },
+    },
+    node_modules: {
+      foo: t.fixture('symlink', '../workspaces/foo'),
+      bar: t.fixture('symlink', '../workspaces/bar'),
+    },
+  })
+
+  const files = await packlist({ path: pkg })
+  t.same(files, [
+    'index.js',
+    'node_modules/bar/index.js',
+    'node_modules/foo/index.js',
+    'node_modules/bar/package.json',
+    'node_modules/foo/package.json',
+    'package.json',
+  ])
+})