fix(types): fix URL building to handle query parameters in sign-in links (#3281)

The buildUrl function was incorrectly concatenating query parameters as part of the pathname, causing malformed URLs for sign-in links with parameters like "?source=vscode". This change separates pathname and search parameters properly, ensuring correct URL construction. Added corresponding test case to verify the fix.

Author: hassoncs

.changeset/hip-lands-search.md

@@ -0,0 +1,5 @@
+---
+"kilo-code": patch
+---
+
+Fix broken sign-in links

packages/types/src/__tests__/kilocode.test.ts

@@ -282,6 +282,9 @@ describe("URL functions", () => {
 			expect(getAppUrl("/profile")).toBe("https://kilocode.ai/profile")
 			expect(getAppUrl("/support")).toBe("https://kilocode.ai/support")
 			expect(getAppUrl("/sign-in-to-editor")).toBe("https://kilocode.ai/sign-in-to-editor")
+			expect(getAppUrl("/sign-in-to-editor?source=vscode")).toBe(
+				"https://kilocode.ai/sign-in-to-editor?source=vscode",
+			)
 		})
 
 		it("should handle development environment", () => {

packages/types/src/kilocode/kilocode.ts

@@ -106,7 +106,13 @@ function buildUrl(path: string = ""): string {
 	try {
 		const backend = new URL(getGlobalKilocodeBackendUrl())
 		const result = new URL(backend)
-		result.pathname = path ? ensureLeadingSlash(path) : ""
+
+		// Separate pathname and search parameters
+		const [pathname, search] = path.split("?")
+		result.pathname = pathname ? ensureLeadingSlash(pathname) : ""
+		if (search) {
+			result.search = `?${search}`
+		}
 
 		return removeTrailingSlash(result.toString(), result.pathname)
 	} catch (error) {