fix: revert userSavedProxy from auto-selection useEffect dependencies

blink-so[bot] · kylecarbs · blink-so[bot] · commit 7ef638f7a3f3 · 2025-07-17T21:28:51.000Z
The auto-selection useEffect should not include userSavedProxy in its
dependencies as it should only run when latencies are loaded, not when
the user proxy changes. The check inside the useEffect is sufficient.

Co-authored-by: kylecarbs &lt;7122116+kylecarbs@users.noreply.github.com&gt;
diff --git a/site/src/contexts/ProxyContext.tsx b/site/src/contexts/ProxyContext.tsx
@@ -211,7 +211,7 @@ export const ProxyProvider: FC<PropsWithChildren> = ({ children }) => {
 			saveUserSelectedProxy(best.proxy);
 			updateProxy();
 		}
-	}, [latenciesLoaded, proxiesResp, proxyLatencies, userSavedProxy]);
+	}, [latenciesLoaded, proxiesResp, proxyLatencies]);
 
 	return (
 		<ProxyContext.Provider

Original file line number	Diff line number	Diff line change
`@@ -211,7 +211,7 @@ export const ProxyProvider: FC<PropsWithChildren> = ({ children }) => {`
`211`	`211`	`saveUserSelectedProxy(best.proxy);`
`212`	`212`	`updateProxy();`
`213`	`213`	`}`
`214`		`- }, [latenciesLoaded, proxiesResp, proxyLatencies, userSavedProxy]);`
	`214`	`+ }, [latenciesLoaded, proxiesResp, proxyLatencies]);`
`215`	`215`
`216`	`216`	`return (`
`217`	`217`	`<ProxyContext.Provider`