plugin-react-refresh should skip worker files

[buffaybu]

Describe the bug

If a worker file includes react string and a function, react-refresh code including /@vite/client which references HTMLElement will be injected to the worker code, causing Uncaught ReferenceError: HTMLElement is not defined .

vite/packages/plugin-react-refresh/index.js

Lines 70 to 74 in be9bc3f

	// plain js/ts files can't use React without importing it, so skip
	// them whenever possible
	if (!id.endsWith('x') && !code.includes('react')) {
	return
	}

Reproduction

https://github.com/buffaybu/vite-env-example/tree/react-refresh (react-refresh branch)

Replace "react" with "1" and the error is gone.

System Info

• @vitejs/plugin-react-refresh version: 1.1.3
• vite version: 2.0.0-beta.50
• Operating System: macOS 11.1
• Node version: 14.15.0
• Package manager (npm/yarn/pnpm) and version: yarn

[yyx990803]

Incidentally this is now fixed after 5735692 because worker files now have a query.