Fix cleanUrl to properly handle subpath imports with query/hash

- Remove special case that prevented query/hash stripping from subpath imports
- Add comprehensive test coverage for subpath imports with query parameters and hash fragments
- Fixes edge case where #path/to/file.html?param=value would remain unchanged instead of becoming #path/to/file.html

Co-authored-by: Somhairle MacLeòid <penalosa@users.noreply.github.com>

Author: claude[bot]

packages/vite-plugin-cloudflare/src/__tests__/utils.spec.ts

@@ -18,6 +18,24 @@ describe("cleanUrl", () => {
 		);
 	});
 
+	test("removes query parameters from subpath imports", () => {
+		expect(cleanUrl("#path/to/file.html?param=value")).toBe("#path/to/file.html");
+		expect(cleanUrl("#components/template.txt?version=1&debug=true")).toBe(
+			"#components/template.txt"
+		);
+	});
+
+	test("removes hash fragments from subpath imports", () => {
+		expect(cleanUrl("#path/to/file.html#section")).toBe("#path/to/file.html");
+		expect(cleanUrl("#components/template.txt#anchor")).toBe(
+			"#components/template.txt"
+		);
+	});
+
+	test("removes both query and hash from subpath imports", () => {
+		expect(cleanUrl("#path/to/file.html?param=value#section")).toBe("#path/to/file.html");
+	});
+
 	test("returns unchanged URL when no query or hash present", () => {
 		expect(cleanUrl("./file.html")).toBe("./file.html");
 		expect(cleanUrl("/absolute/path.txt")).toBe("/absolute/path.txt");

packages/vite-plugin-cloudflare/src/utils.ts

@@ -45,10 +45,6 @@ export function getOutputDirectory(
 
 const postfixRE = /[?#].*$/;
 export function cleanUrl(url: string): string {
-	// Don't strip subpath imports (URLs starting with #)
-	if (url.startsWith("#")) {
-		return url;
-	}
 	return url.replace(postfixRE, "");
 }