fix: properly handle ts worker source

fix #1385
vitejs · Jan 6, 2021 · eea1224 · eea1224
1 parent 756a0f2
commit eea1224
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/packages/playground/worker/my-worker.js → packages/playground/worker/my-worker.ts b/packages/playground/worker/my-worker.js → packages/playground/worker/my-worker.ts
diff --git a/packages/vite/src/node/plugins/esbuild.ts b/packages/vite/src/node/plugins/esbuild.ts
@@ -8,7 +8,7 @@ import {
   TransformOptions,
   TransformResult
 } from 'esbuild'
-import { createDebugger, generateCodeFrame } from '../utils'
+import { cleanUrl, createDebugger, generateCodeFrame } from '../utils'
 import merge from 'merge-source-map'
 import { SourceMap } from 'rollup'
 import { ResolvedConfig } from '..'
@@ -51,9 +51,13 @@ export async function transformWithEsbuild(
   inMap?: object
 ): Promise<EsbuildTransformResult> {
   const service = await ensureService()
-
+  // if the id ends with a valid ext, use it (e.g. vue blocks)
+  // otherwise, cleanup the query before checking the ext
+  const ext = path.extname(
+    /\.\w+$/.test(filename) ? filename : cleanUrl(filename)
+  )
   const resolvedOptions = {
-    loader: path.extname(filename).slice(1) as Loader,
+    loader: ext.slice(1) as Loader,
     sourcemap: true,
     // ensure source file name contains full query
     sourcefile: filename,