fix: strip query strings before matching exclude patterns

Vite appends cache-busting query strings (e.g. ?t=1707350000) to module
URLs during HMR. The exclude patterns use $ anchors that expect the URL
to end with the file extension (e.g. /.*\.tsx?$/), which fails to match
URLs with query parameters.

This causes HMR module requests to leak through to the app server, which
returns 404 with text/plain MIME type. The browser rejects the module and
the page goes blank.

Fix by also testing exclude patterns against the URL path (without query
string), so /frontend/App.tsx?t=123 correctly matches /.*\.tsx?$/ via
the path portion /frontend/App.tsx.

Author: NathanFlurry

src/dev-server.ts

@@ -63,15 +63,16 @@ function createMiddleware(server: ViteDevServer, options: DevServerOptions) {
 		}
 
 		const exclude = options.exclude ?? defaultOptions.exclude ?? [];
+		const urlPath = req.url?.split("?")[0];
 
 		for (const pattern of exclude) {
 			if (req.url) {
 				if (pattern instanceof RegExp) {
-					if (pattern.test(req.url)) {
+					if (pattern.test(req.url) || (urlPath && pattern.test(urlPath))) {
 						return next();
 					}
 				} else if (typeof pattern === "string") {
-					if (req.url.startsWith(pattern)) {
+					if (req.url.startsWith(pattern) || (urlPath && urlPath.startsWith(pattern))) {
 						return next();
 					}
 				}