fix: reactPlugin skip inject react hmr code to workerFile (fix vitejs…

…#6148)
aleclarson · Dec 30, 2021 · 136883d · 136883d
1 parent a8a0d84
commit 136883d
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/packages/plugin-react/src/index.ts b/packages/plugin-react/src/index.ts
@@ -109,6 +109,11 @@ export default function viteReact(opts: Options = {}): PluginOption[] {
 
       if (/\.(mjs|[tj]sx?)$/.test(extension)) {
         const isJSX = extension.endsWith('x')
+        // as worker, querystring could be
+        // import MyWorker from './worker?worker' => worker
+        // import MyWorker from './worker?worker&inline' => worker&inline
+        // transformed by webWorkerPlugin :'/worker?worker_file' => worker_file
+        const isWorker = querystring.includes('worker')
         const isNodeModules = id.includes('/node_modules/')
         const isProjectFile =
           !isNodeModules && (id[0] === '\0' || id.startsWith(projectRoot + '/'))
@@ -118,7 +123,7 @@ export default function viteReact(opts: Options = {}): PluginOption[] {
         let useFastRefresh = false
         if (!skipFastRefresh && !ssr && !isNodeModules) {
           // Modules with .js or .ts extension must import React.
-          const isReactModule = isJSX || code.includes('react')
+          const isReactModule = isJSX || (code.includes('react') && !isWorker)
           if (isReactModule && filter(id)) {
             useFastRefresh = true
             plugins.push([