fix(exports): include non-entry chunks in exports when all is true

sxzz · sxzz · commit 7949216eb518 · 2026-02-15T16:17:23.000+08:00
Closes #752
diff --git a/src/features/pkg/exports.test.ts b/src/features/pkg/exports.test.ts
@@ -405,6 +405,82 @@ describe('generateExports', () => {
     `)
   })
 
+  test('export all includes non-entry chunks', async ({ expect }) => {
+    const results = generateExports(
+      {
+        es: [genChunk('index.js'), genChunk('utils.js', false)],
+      },
+      {
+        exports: { all: true },
+      },
+    )
+    await expect(results).resolves.toMatchInlineSnapshot(`
+      {
+        "exports": {
+          ".": "./index.js",
+          "./*": "./*",
+          "./utils": "./utils.js",
+        },
+        "main": undefined,
+        "module": undefined,
+        "publishExports": undefined,
+        "types": undefined,
+      }
+    `)
+  })
+
+  test('export all excludes virtual modules', async ({ expect }) => {
+    const results = generateExports(
+      {
+        es: [
+          genChunk('index.js'),
+          genChunk('virtual.js', false, '\0virtual-module'),
+        ],
+      },
+      {
+        exports: { all: true },
+      },
+    )
+    await expect(results).resolves.toMatchInlineSnapshot(`
+      {
+        "exports": {
+          ".": "./index.js",
+          "./*": "./*",
+        },
+        "main": undefined,
+        "module": undefined,
+        "publishExports": undefined,
+        "types": undefined,
+      }
+    `)
+  })
+
+  test('export all excludes node_modules chunks', async ({ expect }) => {
+    const results = generateExports(
+      {
+        es: [
+          genChunk('index.js'),
+          genChunk('lodash.js', false, '/project/node_modules/lodash/index.js'),
+        ],
+      },
+      {
+        exports: { all: true },
+      },
+    )
+    await expect(results).resolves.toMatchInlineSnapshot(`
+      {
+        "exports": {
+          ".": "./index.js",
+          "./*": "./*",
+        },
+        "main": undefined,
+        "module": undefined,
+        "publishExports": undefined,
+        "types": undefined,
+      }
+    `)
+  })
+
   test('windows-like paths for subpackages', async ({ expect }) => {
     const results = generateExports({
       es: [
@@ -604,13 +680,13 @@ describe('generateExports', () => {
   })
 })
 
-function genChunk(fileName: string, isEntry = true) {
+function genChunk(fileName: string, isEntry = true, facadeModuleId?: string) {
   // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
   return {
     type: 'chunk',
     fileName,
     isEntry,
-    facadeModuleId: path.resolve(`./SRC/${fileName}`),
+    facadeModuleId: facadeModuleId ?? path.resolve(`./SRC/${fileName}`),
     outDir: cwd,
   } as RolldownChunk
 }
diff --git a/src/features/pkg/exports.ts b/src/features/pkg/exports.ts
@@ -1,6 +1,6 @@
 import { readFileSync, writeFileSync } from 'node:fs'
 import path from 'node:path'
-import { RE_CSS, RE_DTS } from 'rolldown-plugin-dts/filename'
+import { RE_CSS, RE_DTS, RE_NODE_MODULES } from 'rolldown-plugin-dts/filename'
 import { detectIndentation } from '../../utils/format.ts'
 import { stripExtname } from '../../utils/fs.ts'
 import { matchPattern, slash, typeAssert } from '../../utils/general.ts'
@@ -183,10 +183,21 @@ export async function generateExports(
     // Filter out non-entry chunks and excluded files
     const filteredChunks = chunksByFormat.filter(
       (chunk): chunk is RolldownCodeChunk => {
-        if (chunk.type !== 'chunk' || !chunk.isEntry) {
+        if (chunk.type !== 'chunk') {
           return false
         }
 
+        if (!chunk.isEntry) {
+          if (!all) return false
+
+          if (
+            chunk.facadeModuleId?.[0] === '\0' ||
+            (chunk.facadeModuleId && RE_NODE_MODULES.test(chunk.facadeModuleId))
+          ) {
+            return false
+          }
+        }
+
         const [name] = getExportName(chunk)
         return !shouldExclude(name, exclude)
       },
