Optimizes code to prevent memory leak, add error messages

Author: pugazhendhi-m

src/api/providers/fetchers/unbound.ts

@@ -46,7 +46,7 @@ export async function getUnboundModels(apiKey?: string | null): Promise<Record<s
 		}
 	} catch (error) {
 		console.error(`Error fetching Unbound models: ${JSON.stringify(error, Object.getOwnPropertyNames(error), 2)}`)
-		return {}
+		throw new Error(`Failed to fetch Unbound models: ${error instanceof Error ? error.message : "Unknown error"}`)
 	}
 
 	return models

webview-ui/src/components/settings/providers/Unbound.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useState } from "react"
+import { useCallback, useState, useRef } from "react"
 import { VSCodeTextField } from "@vscode/webview-ui-toolkit/react"
 import { useQueryClient } from "@tanstack/react-query"
 
@@ -24,6 +24,10 @@ export const Unbound = ({ apiConfiguration, setApiConfigurationField, routerMode
 	const [isInvalidKey, setIsInvalidKey] = useState<boolean>(false)
 	const queryClient = useQueryClient()
 
+	// Add refs to store timer IDs
+	const didRefetchTimerRef = useRef<NodeJS.Timeout>()
+	const invalidKeyTimerRef = useRef<NodeJS.Timeout>()
+
 	const handleInputChange = useCallback(
 		<K extends keyof ProviderSettings, E>(
 			field: K,
@@ -42,18 +46,28 @@ export const Unbound = ({ apiConfiguration, setApiConfigurationField, routerMode
 			apiConfiguration: apiConfiguration,
 		})
 
-		const waitForStateUpdate = new Promise<void>((resolve) => {
+		const waitForStateUpdate = new Promise<void>((resolve, reject) => {
+			const timeoutId = setTimeout(() => {
+				window.removeEventListener("message", messageHandler)
+				reject(new Error("Timeout waiting for state update"))
+			}, 10000) // 10 second timeout
+
 			const messageHandler = (event: MessageEvent) => {
 				const message = event.data
 				if (message.type === "state") {
+					clearTimeout(timeoutId)
 					window.removeEventListener("message", messageHandler)
 					resolve()
 				}
 			}
 			window.addEventListener("message", messageHandler)
 		})
 
-		await waitForStateUpdate
+		try {
+			await waitForStateUpdate
+		} catch (error) {
+			console.error("Failed to save configuration:", error)
+		}
 	}, [apiConfiguration])
 
 	const requestModels = useCallback(async () => {
@@ -102,10 +116,10 @@ export const Unbound = ({ apiConfiguration, setApiConfigurationField, routerMode
 
 		if (requestModelsResult) {
 			setDidRefetch(true)
-			setTimeout(() => setDidRefetch(false), 2000)
+			didRefetchTimerRef.current = setTimeout(() => setDidRefetch(false), 3000)
 		} else {
 			setIsInvalidKey(true)
-			setTimeout(() => setIsInvalidKey(false), 2000)
+			invalidKeyTimerRef.current = setTimeout(() => setIsInvalidKey(false), 3000)
 		}
 	}, [saveConfiguration, requestModels])