wip: use sourcemap

vitejs · Jun 16, 2023 · 3e6073c · 3e6073c
1 parent c2e0149
commit 3e6073c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/packages/vite/rollup.config.ts b/packages/vite/rollup.config.ts
@@ -194,7 +194,7 @@ function createCjsConfig(isProduction: boolean) {
       ...Object.keys(pkg.dependencies),
       ...(isProduction ? [] : Object.keys(pkg.devDependencies)),
     ],
-    plugins: [...createNodePlugins(false, false, false), bundleSizeLimit(120)],
+    plugins: [...createNodePlugins(false, false, false), bundleSizeLimit(150)],
   })
 }
 

diff --git a/packages/vite/src/node/server/send.ts b/packages/vite/src/node/server/send.ts
@@ -5,6 +5,7 @@ import type {
 } from 'node:http'
 import getEtag from 'etag'
 import type { SourceMap } from 'rollup'
+import MagicString from 'magic-string'
 import { removeTimestampQuery } from '../utils'
 import { getCodeWithSourcemap } from './sourcemap'
 
@@ -63,7 +64,15 @@ export function send(
     }
   } else {
     if (type === 'js') {
-      content += `\n//# sourceURL=${removeTimestampQuery(req.url!)}`
+      const urlWithoutTimestamp = removeTimestampQuery(req.url!)
+      if (req.url! !== urlWithoutTimestamp) {
+        const ms = new MagicString(content.toString())
+        content = getCodeWithSourcemap(
+          type,
+          content.toString(),
+          ms.generateMap({ source: urlWithoutTimestamp, hires: true }),
+        )
+      }
     }
   }