fix(exports): scope inlined dependencies to configs with exports (#1040)

Author: lazerg

src/features/pkg/index.ts

@@ -80,10 +80,9 @@ export async function bundleDone(
     }
 
     const chunks: ChunksByFormat = {}
-    const inlinedDeps = mergeInlinedDeps(ctx.bundles)
-    for (const bundle of ctx.bundles) {
-      if (!bundle.config.exports) continue
-
+    const exportsBundles = ctx.bundles.filter((bundle) => bundle.config.exports)
+    const inlinedDeps = mergeInlinedDeps(exportsBundles)
+    for (const bundle of exportsBundles) {
       chunks[bundle.config.format] ||= []
       chunks[bundle.config.format]!.push(...bundle.chunks)
     }

tests/e2e.test.ts

@@ -538,6 +538,43 @@ describe('deps', () => {
       )
       expect(pkg.inlinedDependencies).toBeUndefined()
     })
+
+    test('should not include inlined deps from configs without exports', async (context) => {
+      const { testDir } = await testBuild({
+        context,
+        files: {
+          ...node_modules,
+          'index.ts': `export { lib } from 'my-lib'`,
+          'standalone.ts': `import { lib } from 'my-lib'\nconsole.log(lib)`,
+          'package.json': JSON.stringify({
+            name: 'test-pkg',
+            version: '1.0.0',
+            dependencies: { 'my-lib': '1.2.3' },
+          }),
+          'tsdown.config.ts': `export default [
+          { entry: ['index.ts'], format: ['esm'], exports: true, dts: false },
+          {
+            entry: ['standalone.ts'],
+            format: ['cjs'],
+            outDir: 'dist/standalone',
+            dts: false,
+            deps: { alwaysBundle: [/.*/] },
+          },
+        ]`,
+        },
+        options: {
+          entry: undefined,
+          config: 'tsdown.config.ts',
+          dts: false,
+        },
+        snapshot: false,
+      })
+
+      const pkg = JSON.parse(
+        await readFile(path.join(testDir, 'package.json'), 'utf8'),
+      )
+      expect(pkg.inlinedDependencies).toBeUndefined()
+    })
   })
 })