fix(server): only warn sourcemap when explicitly turned on in loadAndTransform

antfu · antfu · commit 28e1c5337866 · 2024-01-24T18:02:39.000+01:00
diff --git a/packages/vite/src/node/server/transformRequest.ts b/packages/vite/src/node/server/transformRequest.ts
@@ -240,9 +240,11 @@ async function loadAndTransform(
 
         code = code.replace(convertSourceMap.mapFileCommentRegex, blankReplacer)
       } catch (e) {
-        logger.warn(`Failed to load source map for ${url}.`, {
-          timestamp: true,
-        })
+        if (server.config.build.sourcemap) {
+          logger.warn(`Failed to load source map for ${url}.`, {
+            timestamp: true,
+          })
+        }
       }
     }
   } else {

Original file line number	Diff line number	Diff line change
`@@ -240,9 +240,11 @@ async function loadAndTransform(`
`240`	`240`
`241`	`241`	`code = code.replace(convertSourceMap.mapFileCommentRegex, blankReplacer)`
`242`	`242`	`} catch (e) {`
`243`		- logger.warn(`Failed to load source map for ${url}.`, {
`244`		`- timestamp: true,`
`245`		`- })`
	`243`	`+ if (server.config.build.sourcemap) {`
	`244`	+ logger.warn(`Failed to load source map for ${url}.`, {
	`245`	`+ timestamp: true,`
	`246`	`+ })`
	`247`	`+ }`
`246`	`248`	`}`
`247`	`249`	`}`
`248`	`250`	`} else {`