From 7d9d4fa2bc85df0a2fd06fd04faf88c6b6b90b20 Mon Sep 17 00:00:00 2001 From: ZGaetano Date: Sat, 30 May 2026 13:00:25 -0400 Subject: [PATCH] refactor: single source of truth for model persistence (remove redundant global writes) --- src/components/chat/view/subcomponents/ModelSelectorBar.tsx | 1 - .../chat/view/subcomponents/ProviderSelectionEmptyState.tsx | 4 ---- 2 files changed, 5 deletions(-) diff --git a/src/components/chat/view/subcomponents/ModelSelectorBar.tsx b/src/components/chat/view/subcomponents/ModelSelectorBar.tsx index 9a8ae3b..602fc36 100644 --- a/src/components/chat/view/subcomponents/ModelSelectorBar.tsx +++ b/src/components/chat/view/subcomponents/ModelSelectorBar.tsx @@ -97,7 +97,6 @@ export default function ModelSelectorBar({ gemini: setGeminiModel, }; setters[provider]?.(modelValue); - localStorage.setItem(`${provider}-model`, modelValue); setOpen(false); setSearch(''); }; diff --git a/src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx b/src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx index df7941f..7473074 100644 --- a/src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx +++ b/src/components/chat/view/subcomponents/ProviderSelectionEmptyState.tsx @@ -174,16 +174,12 @@ export default function ProviderSelectionEmptyState({ (providerId: LLMProvider, modelValue: string) => { if (providerId === "claude") { setClaudeModel(modelValue); - localStorage.setItem("claude-model", modelValue); } else if (providerId === "codex") { setCodexModel(modelValue); - localStorage.setItem("codex-model", modelValue); } else if (providerId === "gemini") { setGeminiModel(modelValue); - localStorage.setItem("gemini-model", modelValue); } else { setCursorModel(modelValue); - localStorage.setItem("cursor-model", modelValue); } }, [setClaudeModel, setCursorModel, setCodexModel, setGeminiModel],