Remove unnecessary enableSourcemap usage (#862)

sveltejs · Feb 24, 2024 · 324f569 · 324f569
1 parent 0c2695d
commit 324f569
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 14 deletions.
diff --git a/.changeset/late-pans-guess.md b/.changeset/late-pans-guess.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/vite-plugin-svelte': patch
+---
+
+Remove unnecessary `enableSourcemap` option usage and prevent passing it in Svelte 5
diff --git a/packages/vite-plugin-svelte/src/utils/compile.js b/packages/vite-plugin-svelte/src/utils/compile.js
@@ -71,13 +71,6 @@ export const _createCompileSvelte = (makeHot) => {
 			const hash = `s-${safeBase64Hash(normalizedFilename)}`;
 			compileOptions.cssHash = () => hash;
 		}
-		if (ssr && compileOptions.enableSourcemap !== false) {
-			if (typeof compileOptions.enableSourcemap === 'object') {
-				compileOptions.enableSourcemap.css = false;
-			} else {
-				compileOptions.enableSourcemap = { js: true, css: false };
-			}
-		}
 
 		let preprocessed;
 		let preprocessors = options.preprocess;

diff --git a/packages/vite-plugin-svelte/src/utils/load-raw.js b/packages/vite-plugin-svelte/src/utils/load-raw.js
@@ -1,7 +1,7 @@
 import fs from 'node:fs';
 import { toRollupError } from './error.js';
 import { log } from './log.js';
-import { isSvelte4 } from './svelte-version.js';
+import { isSvelte4, isSvelte5 } from './svelte-version.js';
 /**
  * utility function to compile ?raw and ?direct requests in load hook
  *
@@ -27,12 +27,14 @@ export async function loadRaw(svelteRequest, compileSvelte, options) {
 			compilerOptions: {
 				dev: false,
 				css: 'external',
-				enableSourcemap: query.sourcemap
-					? {
-							js: type === 'script' || type === 'all',
-							css: type === 'style' || type === 'all'
-						}
-					: false,
+				enableSourcemap: isSvelte5
+					? undefined
+					: query.sourcemap
+						? {
+								js: type === 'script' || type === 'all',
+								css: type === 'style' || type === 'all'
+							}
+						: false,
 				...svelteRequest.query.compilerOptions
 			},
 			hot: false,