Skip to content

Commit

Permalink
fix: duplicated sourceMappingURL for worker bundles (fix vitejs#11601)
Browse files Browse the repository at this point in the history
  • Loading branch information
Karshilov committed Jan 5, 2023
1 parent 896475d commit 6801a09
Showing 1 changed file with 1 addition and 21 deletions.
22 changes: 1 addition & 21 deletions packages/vite/src/node/plugins/worker.ts
Original file line number Diff line number Diff line change
Expand Up @@ -142,13 +142,7 @@ function emitSourcemapForWorkerEntry(
const { map: sourcemap } = chunk

if (sourcemap) {
if (config.build.sourcemap === 'inline') {
// Manually add the sourcemap to the code if configured for inline sourcemaps.
// TODO: Remove when https://github.com/rollup/rollup/issues/3913 is resolved
// Currently seems that it won't be resolved until Rollup 3
const dataUrl = sourcemap.toUrl()
chunk.code += `//# sourceMappingURL=${dataUrl}`
} else if (
if (
config.build.sourcemap === 'hidden' ||
config.build.sourcemap === true
) {
Expand All @@ -159,20 +153,6 @@ function emitSourcemapForWorkerEntry(
type: 'asset',
source: data,
})

// Emit the comment that tells the JS debugger where it can find the
// sourcemap file.
// 'hidden' causes the sourcemap file to be created but
// the comment in the file to be omitted.
if (config.build.sourcemap === true) {
// inline web workers need to use the full sourcemap path
// non-inline web workers can use a relative path
const sourceMapUrl =
query?.inline != null
? mapFileName
: path.relative(config.build.assetsDir, mapFileName)
chunk.code += `//# sourceMappingURL=${sourceMapUrl}`
}
}
}

Expand Down

0 comments on commit 6801a09

Please sign in to comment.