Revert "refactor(webview): keep selected model cache out of provider settings"

This reverts commit d22864b.

Author: WebMad

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

@@ -58,7 +58,10 @@ export const LiteLLM = ({
 				if (refreshStatus === "loading") {
 					if (!litellmErrorJustReceived.current) {
 						setRefreshStatus("success")
-						// Refresh the shared cache used by ApiOptions without invalidating unrelated queries.
+						// Refresh the provider-scoped cache used by useSelectedModel and the shared cache used by
+						// ApiOptions. Target both exact keys rather than the bare ["routerModels"] prefix, which
+						// would needlessly invalidate every other provider's query too.
+						void queryClient.invalidateQueries({ queryKey: ["routerModels", providerIdentifiers.litellm] })
 						void queryClient.invalidateQueries({ queryKey: ["routerModels", "all"] })
 					}
 					// If litellmErrorJustReceived.current is true, status is already (or will be) "error".

webview-ui/src/components/settings/providers/__tests__/LiteLLM.spec.tsx

@@ -47,7 +47,7 @@ describe("LiteLLM", () => {
 		mockUseExtensionState.mockReturnValue({ routerModels: { [providerIdentifiers.litellm]: {} } })
 	})
 
-	it("invalidates the shared LiteLLM cache after a successful model refresh", async () => {
+	it("invalidates both LiteLLM caches after a successful model refresh", async () => {
 		const queryClient = new QueryClient({ defaultOptions: { queries: { retry: false } } })
 		const invalidateQueries = vi.spyOn(queryClient, "invalidateQueries")
 		const apiConfiguration: ProviderSettings = {
@@ -72,6 +72,9 @@ describe("LiteLLM", () => {
 		})
 
 		await waitFor(() => {
+			expect(invalidateQueries).toHaveBeenCalledWith({
+				queryKey: ["routerModels", providerIdentifiers.litellm],
+			})
 			expect(invalidateQueries).toHaveBeenCalledWith({ queryKey: ["routerModels", "all"] })
 		})
 	})