[feat] add enableSourcemap option

site/content/docs/04-compile-time.md

@@ -82,6 +82,8 @@ The following options can be passed to the compiler. None are required:
 | `loopGuardTimeout` | 0 | A `number` that tells Svelte to break the loop if it blocks the thread for more than `loopGuardTimeout` ms. This is useful to prevent infinite loops. **Only available when `dev: true`**
 | `preserveComments` | `false` | If `true`, your HTML comments will be preserved during server-side rendering. By default, they are stripped out.
 | `preserveWhitespace` | `false` | If `true`, whitespace inside and between elements is kept as you typed it, rather than removed or collapsed to a single space where possible.
+| `sourcemap` | `object | string` | An initial sourcemap that will be merged into the final output sourcemap. This is usually the preprocessor sourcemap.
+| `enableSourcemap` | `boolean | 'js' | 'css'` | If `true`, Svelte generate sourcemaps for components. Use `js` or `css` for more granular control of sourcemap generation. By default, this is `true`.
 | `outputFilename` | `null` | A `string` used for your JavaScript sourcemap.
 | `cssOutputFilename` | `null` | A `string` used for your CSS sourcemap.
 | `sveltePath` | `"svelte"` | The location of the `svelte` package. Any imports from `svelte` or `svelte/[module]` will be modified accordingly.

src/compiler/compile/Component.ts

@@ -342,22 +342,29 @@ export default class Component {
 			css = compile_options.customElement
 				? { code: null, map: null }
 				: result.css;
+
+			const jsSourcemapEnabled = compile_options.enableSourcemap === true || compile_options.enableSourcemap === 'js';
 
-			const sourcemap_source_filename = get_sourcemap_source_filename(compile_options);
+			if (!jsSourcemapEnabled) {
+				js = print(program);
+				js.map = null;
+			} else {
+				const sourcemap_source_filename = get_sourcemap_source_filename(compile_options);
 
-			js = print(program, {
-				sourceMapSource: sourcemap_source_filename
-			});
+				js = print(program, {
+					sourceMapSource: sourcemap_source_filename
+				});
 
-			js.map.sources = [
-				sourcemap_source_filename
-			];
+				js.map.sources = [
+					sourcemap_source_filename
+				];
 
-			js.map.sourcesContent = [
-				this.source
-			];
+				js.map.sourcesContent = [
+					this.source
+				];
 
-			js.map = apply_preprocessor_sourcemap(sourcemap_source_filename, js.map, compile_options.sourcemap as (string | RawSourceMap | DecodedSourceMap));
+				js.map = apply_preprocessor_sourcemap(sourcemap_source_filename, js.map, compile_options.sourcemap as (string | RawSourceMap | DecodedSourceMap));
+			}
 		}
 
 		return {

src/compiler/compile/index.ts

@@ -13,6 +13,7 @@ const valid_options = [
 	'name',
 	'filename',
 	'sourcemap',
+	'enableSourcemap',
 	'generate',
 	'errorMode',
 	'varsReport',
@@ -82,7 +83,7 @@ function validate_options(options: CompileOptions, warnings: Warning[]) {
 }
 
 export default function compile(source: string, options: CompileOptions = {}) {
-	options = Object.assign({ generate: 'dom', dev: false }, options);
+	options = Object.assign({ generate: 'dom', dev: false, enableSourcemap: true }, options);
 
 	const stats = new Stats();
 	const warnings = [];

src/compiler/compile/render_dom/index.ts

@@ -33,10 +33,16 @@ export default function dom(
 	}
 
 	const css = component.stylesheet.render(options.filename, !options.customElement);
+
+	const cssSourcemapEnabled = options.enableSourcemap === true || options.enableSourcemap === 'css';
 
-	css.map = apply_preprocessor_sourcemap(options.filename, css.map, options.sourcemap as string | RawSourceMap | DecodedSourceMap);
+	if (cssSourcemapEnabled) {
+		css.map = apply_preprocessor_sourcemap(options.filename, css.map, options.sourcemap as string | RawSourceMap | DecodedSourceMap);
+	} else {
+		css.map = null;
+	}
 
-	const styles = component.stylesheet.has_styles && options.dev
+	const styles = cssSourcemapEnabled && component.stylesheet.has_styles && options.dev
 		? `${css.code}\n/*# sourceMappingURL=${css.map.toUrl()} */`
 		: css.code;
 
@@ -521,7 +527,7 @@ export default function dom(
 				constructor(options) {
 					super();
 
-					${css.code && b`this.shadowRoot.innerHTML = \`<style>${css.code.replace(/\\/g, '\\\\')}${options.dev ? `\n/*# sourceMappingURL=${css.map.toUrl()} */` : ''}</style>\`;`}
+					${css.code && b`this.shadowRoot.innerHTML = \`<style>${css.code.replace(/\\/g, '\\\\')}${cssSourcemapEnabled && options.dev ? `\n/*# sourceMappingURL=${css.map.toUrl()} */` : ''}</style>\`;`}
 
 					@init(this, { target: this.shadowRoot, props: ${init_props}, customElement: true }, ${definition}, ${has_create_fragment ? 'create_fragment' : 'null'}, ${not_equal}, ${prop_indexes}, null, ${dirty});
 

src/compiler/compile/render_ssr/index.ts

@@ -200,11 +200,13 @@ export default function ssr(
 		main
 	].filter(Boolean);
 
+	const cssSourcemapEnabled = options.enableSourcemap === true || options.enableSourcemap === 'css';
+
 	const js = b`
 		${css.code ? b`
 		const #css = {
 			code: "${css.code}",
-			map: ${css.map ? string_literal(css.map.toString()) : 'null'}
+			map: ${cssSourcemapEnabled && css.map ? string_literal(css.map.toString()) : 'null'}
 		};` : null}
 
 		${component.extract_javascript(component.ast.module)}

src/compiler/interfaces.ts

@@ -147,6 +147,7 @@ export interface CompileOptions {
 	varsReport?: 'full' | 'strict' | false;
 
 	sourcemap?: object | string;
+	enableSourcemap?: boolean | 'js' | 'css';
 	outputFilename?: string;
 	cssOutputFilename?: string;
 	sveltePath?: string;

test/sourcemaps/index.ts

@@ -85,11 +85,13 @@ describe('sourcemaps', () => {
 				);
 			}
 
-			assert.deepEqual(
-				js.map.sources.slice().sort(),
-				(config.js_map_sources || ['input.svelte']).sort(),
-				'js.map.sources is wrong'
-			);
+			if (js.map) {
+				assert.deepEqual(
+					js.map.sources.slice().sort(),
+					(config.js_map_sources || ['input.svelte']).sort(),
+					'js.map.sources is wrong'
+				);
+			}
 			if (css.map) {
 				assert.deepEqual(
 					css.map.sources.slice().sort(),

test/sourcemaps/samples/no-sourcemap/_config.js

@@ -0,0 +1,5 @@
+export default {
+  compile_options: {
+    enableSourcemap: false
+  }
+};

test/sourcemaps/samples/no-sourcemap/input.svelte

@@ -0,0 +1,11 @@
+<script>
+	export let foo;
+</script>
+
+<p>{foo}</p>
+
+<style>
+	p {
+		color: red;
+	}
+</style>

test/sourcemaps/samples/no-sourcemap/test.js

@@ -0,0 +1,4 @@
+export function test({ assert, js, css }) {
+	assert.equal(js.map, null);
+	assert.equal(css.map, null);
+}

test/sourcemaps/samples/only-css-sourcemap/_config.js

@@ -0,0 +1,5 @@
+export default {
+  compile_options: {
+    enableSourcemap: 'css'
+  }
+};

test/sourcemaps/samples/only-css-sourcemap/input.svelte

@@ -0,0 +1,11 @@
+<script>
+	export let foo;
+</script>
+
+<p>{foo}</p>
+
+<style>
+	p {
+		color: red;
+	}
+</style>

test/sourcemaps/samples/only-css-sourcemap/test.js

@@ -0,0 +1,4 @@
+export function test({ assert, js, css }) {
+	assert.equal(js.map, null);
+	assert.notEqual(css.map, null);
+}

test/sourcemaps/samples/only-js-sourcemap/_config.js

@@ -0,0 +1,5 @@
+export default {
+  compile_options: {
+    enableSourcemap: 'js'
+  }
+};

test/sourcemaps/samples/only-js-sourcemap/input.svelte

@@ -0,0 +1,11 @@
+<script>
+	export let foo;
+</script>
+
+<p>{foo}</p>
+
+<style>
+	p {
+		color: red;
+	}
+</style>

test/sourcemaps/samples/only-js-sourcemap/test.js

@@ -0,0 +1,4 @@
+export function test({ assert, js, css }) {
+	assert.notEqual(js.map, null);
+	assert.equal(css.map, null);
+}