fix: do not emit sourcemap for types (#649)

src/rollup/input.ts

@@ -147,6 +147,7 @@ export async function buildInputConfig(
   } as const
 
   const hasBrowserslistConfig = !!(browserslistConfig && !hasSpecifiedTsTarget)
+  const sourceMap = bundleConfig.sourcemap
   const swcOptions = {
     jsc: {
       ...(!hasSpecifiedTsTarget &&
@@ -162,11 +163,11 @@ export async function buildInputConfig(
       ...(shouldMinify && {
         minify: {
           ...swcMinifyOptions,
-          sourceMap: bundleConfig.sourcemap,
+          sourceMap: sourceMap,
         },
       }),
     },
-    sourceMaps: bundleConfig.sourcemap,
+    sourceMaps: sourceMap,
     inlineSourcesContent: false,
     isModule: true,
     ...(hasBrowserslistConfig && {

src/rollup/output.ts

@@ -51,7 +51,7 @@ export async function buildOutputConfigs(
     interop: 'auto',
     freeze: false,
     strict: false,
-    sourcemap: bundleConfig.sourcemap,
+    sourcemap: bundleConfig.sourcemap && !dts,
     manualChunks: createSplitChunks(
       pluginContext.moduleDirectiveLayerMap,
       entryFiles,

test/fixtures/integration-test-template/index.test.ts

@@ -1,11 +1,12 @@
 import { describe, it } from 'vitest'
-import { createJob, assertContainFiles } from '../../testing-utils'
+import { createJob, getFileNamesFromDirectory } from '../../testing-utils'
 
 describe('integration - <name>', () => {
   const { distDir } = createJob({
     directory: __dirname,
   })
   it('should work', async () => {
-    assertContainFiles(distDir, ['index.js'])
+    const files = getFileNamesFromDirectory(distDir)
+    // expect(files).toEqual([])
   })
 })

test/integration/sourcemap-dts/index.test.ts

@@ -0,0 +1,13 @@
+import { expect, describe, it } from 'vitest'
+import { createJob, getFileNamesFromDirectory } from '../../testing-utils'
+
+describe('integration - sourcemap-dts', () => {
+  const { distDir } = createJob({
+    directory: __dirname,
+    args: ['--sourcemap'],
+  })
+  it('should not generate sourcemap for types', async () => {
+    const files = await getFileNamesFromDirectory(distDir)
+    expect(files).toEqual(['index.d.ts', 'index.js', 'index.js.map'])
+  })
+})

test/integration/sourcemap-dts/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "sourcemap-dts",
+  "main": "./dist/index.js",
+  "type": "module",
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js"
+    }
+  }
+}

test/integration/sourcemap-dts/src/index.ts

@@ -0,0 +1,3 @@
+export function foo() {
+  return 'foo'
+}

test/testing-utils/index.ts

@@ -9,9 +9,12 @@ export { runCli } from './cli'
 export { createJob } from './integration'
 
 export async function getFileNamesFromDirectory(directory: string) {
-  const files = await glob(['**/*.{,c,m}js', '**/*.{,c,m}d.ts'], {
-    cwd: directory,
-  })
+  const files = await glob(
+    ['**/*.{,c,m}js', '**/*.{,c,m}d.ts', '**/*.{,c,m}js.map'],
+    {
+      cwd: directory,
+    },
+  )
 
   return files.sort().map((file) => normalizePath(file))
 }