Support excluding sources from sourcemaps (#2531)

src/Chunk.ts

@@ -1147,9 +1147,16 @@ export default class Chunk {
 
 				if (this.graph.pluginDriver.hasLoadersOrTransforms) {
 					const decodedMap = magicString.generateDecodedMap({});
-					map = collapseSourcemaps(this, file, decodedMap, this.usedModules, chunkSourcemapChain);
+					map = collapseSourcemaps(
+						this,
+						file,
+						decodedMap,
+						this.usedModules,
+						chunkSourcemapChain,
+						options.sourcemapExcludeSources
+					);
 				} else {
-					map = magicString.generateMap({ file, includeContent: true });
+					map = magicString.generateMap({ file, includeContent: !options.sourcemapExcludeSources });
 				}
 
 				map.sources = map.sources.map(sourcePath =>

src/rollup/types.d.ts

@@ -283,6 +283,7 @@ export interface OutputOptions {
 	intro?: string | (() => string | Promise<string>);
 	outro?: string | (() => string | Promise<string>);
 	sourcemap?: boolean | 'inline';
+	sourcemapExcludeSources?: boolean;
 	sourcemapFile?: string;
 	sourcemapPathTransform?: (sourcePath: string) => string;
 	interop?: boolean;

src/utils/collapseSourcemaps.ts

@@ -140,7 +140,8 @@ export default function collapseSourcemaps(
 	file: string,
 	map: DecodedSourceMap,
 	modules: Module[],
-	bundleSourcemapChain: RawSourceMap[]
+	bundleSourcemapChain: RawSourceMap[],
+	excludeContent: boolean
 ) {
 	const moduleSources = modules.filter(module => !module.excludeFromSourcemap).map(module => {
 		let sourcemapChain = module.sourcemapChain;
@@ -206,5 +207,7 @@ export default function collapseSourcemaps(
 		file = basename(file);
 	}
 
+	sourcesContent = excludeContent ? null : sourcesContent;
+
 	return new SourceMap({ file, sources, sourcesContent, names, mappings });
 }

src/utils/mergeOptions.ts

@@ -271,6 +271,7 @@ function getOutputOptions(
 		outro: getOption('outro'),
 		paths: getOption('paths'),
 		sourcemap: getOption('sourcemap'),
+		sourcemapExcludeSources: getOption('sourcemapExcludeSources'),
 		sourcemapFile: getOption('sourcemapFile'),
 		sourcemapPathTransform: getOption('sourcemapPathTransform'),
 		strict: getOption('strict', true)

test/form/index.js

@@ -78,14 +78,18 @@ function generateAndTestBundle(bundle, outputOptions, expectedFile, { show }) {
 
 		try {
 			actualMap = JSON.parse(sander.readFileSync(outputOptions.file + '.map').toString());
-			actualMap.sourcesContent = actualMap.sourcesContent.map(normaliseOutput);
+			actualMap.sourcesContent = actualMap.sourcesContent
+				? actualMap.sourcesContent.map(normaliseOutput)
+				: null;
 		} catch (err) {
 			assert.equal(err.code, 'ENOENT');
 		}
 
 		try {
 			expectedMap = JSON.parse(sander.readFileSync(expectedFile + '.map').toString());
-			expectedMap.sourcesContent = expectedMap.sourcesContent.map(normaliseOutput);
+			expectedMap.sourcesContent = actualMap.sourcesContent
+				? expectedMap.sourcesContent.map(normaliseOutput)
+				: null;
 		} catch (err) {
 			assert.equal(err.code, 'ENOENT');
 		}

test/form/samples/sourcemaps-excludesources/_config.js

@@ -0,0 +1,7 @@
+module.exports = {
+	description: 'correct sourcemaps are written (excluding sourceContent)',
+	skipIfWindows: true,
+	options: {
+		output: { sourcemap: true, sourcemapExcludeSources: true }
+	}
+};

test/form/samples/sourcemaps-excludesources/bar.js

@@ -0,0 +1,3 @@
+export default function bar () {
+	console.log( 'hello from bar.js' );
+}

test/form/samples/sourcemaps-excludesources/foo.js

@@ -0,0 +1,3 @@
+export default function foo () {
+	console.log( 'hello from foo.js' );
+}

test/form/samples/sourcemaps-excludesources/main.js

@@ -0,0 +1,7 @@
+import foo from './foo';
+import bar from './bar';
+
+console.log( 'hello from main.js' );
+
+foo();
+bar();