Generate sourcemaps but omit the comment

src/rollup/index.ts

@@ -402,7 +402,9 @@ function writeOutputFile(
 				url = `${basename(outputFile.fileName)}.map`;
 				writeSourceMapPromise = writeFile(`${fileName}.map`, outputFile.map.toString());
 			}
-			source += `//# ${SOURCEMAPPING_URL}=${url}\n`;
+			if (outputOptions.sourcemap !== 'hidden') {
+				source += `//# ${SOURCEMAPPING_URL}=${url}\n`;
+			}
 		}
 	}
 

src/rollup/types.d.ts

@@ -475,7 +475,7 @@ export interface OutputOptions {
 	outro?: string | (() => string | Promise<string>);
 	paths?: OptionsPaths;
 	preferConst?: boolean;
-	sourcemap?: boolean | 'inline';
+	sourcemap?: boolean | 'inline' | 'hidden';
 	sourcemapExcludeSources?: boolean;
 	sourcemapFile?: string;
 	sourcemapPathTransform?: (sourcePath: string) => string;

test/cli/samples/sourcemap-hidden/_config.js

@@ -0,0 +1,13 @@
+const fs = require('fs');
+const assert = require('assert');
+
+module.exports = {
+	description: 'adds a newline after the sourceMappingURL comment (#756)',
+	command: 'rollup -i main.js -f es -m hidden -o output.js',
+	test() {
+		const output = fs.readFileSync('output.js', 'utf-8');
+		assert.equal(output.trim(), 'console.log( 42 );');
+		fs.unlinkSync('output.js');
+		fs.unlinkSync('output.js.map');
+	}
+};

test/cli/samples/sourcemap-hidden/main.js

@@ -0,0 +1 @@
+console.log( 42 );

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

@@ -0,0 +1,7 @@
+module.exports = {
+	description: 'correct sourcemaps are written (separate file) without comment',
+	skipIfWindows: true,
+	options: {
+		output: { sourcemap: 'hidden' }
+	}
+};

test/form/samples/sourcemaps-hidden/_expected/amd.js

@@ -0,0 +1,16 @@
+define(function () { 'use strict';
+
+	function foo () {
+		console.log( 'hello from foo.js' );
+	}
+
+	function bar () {
+		console.log( 'hello from bar.js' );
+	}
+
+	console.log( 'hello from main.js' );
+
+	foo();
+	bar();
+
+});

test/form/samples/sourcemaps-hidden/_expected/cjs.js

@@ -0,0 +1,14 @@
+'use strict';
+
+function foo () {
+	console.log( 'hello from foo.js' );
+}
+
+function bar () {
+	console.log( 'hello from bar.js' );
+}
+
+console.log( 'hello from main.js' );
+
+foo();
+bar();

test/form/samples/sourcemaps-hidden/_expected/es.js

@@ -0,0 +1,12 @@
+function foo () {
+	console.log( 'hello from foo.js' );
+}
+
+function bar () {
+	console.log( 'hello from bar.js' );
+}
+
+console.log( 'hello from main.js' );
+
+foo();
+bar();

test/form/samples/sourcemaps-hidden/_expected/iife.js

@@ -0,0 +1,17 @@
+(function () {
+	'use strict';
+
+	function foo () {
+		console.log( 'hello from foo.js' );
+	}
+
+	function bar () {
+		console.log( 'hello from bar.js' );
+	}
+
+	console.log( 'hello from main.js' );
+
+	foo();
+	bar();
+
+}());

test/form/samples/sourcemaps-hidden/_expected/system.js

@@ -0,0 +1,21 @@
+System.register([], function () {
+	'use strict';
+	return {
+		execute: function () {
+
+			function foo () {
+				console.log( 'hello from foo.js' );
+			}
+
+			function bar () {
+				console.log( 'hello from bar.js' );
+			}
+
+			console.log( 'hello from main.js' );
+
+			foo();
+			bar();
+
+		}
+	};
+});

test/form/samples/sourcemaps-hidden/_expected/umd.js

@@ -0,0 +1,19 @@
+(function (factory) {
+	typeof define === 'function' && define.amd ? define(factory) :
+	factory();
+}(function () { 'use strict';
+
+	function foo () {
+		console.log( 'hello from foo.js' );
+	}
+
+	function bar () {
+		console.log( 'hello from bar.js' );
+	}
+
+	console.log( 'hello from main.js' );
+
+	foo();
+	bar();
+
+}));

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

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

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

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

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

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