From f69b6988349011a2ac48c8d7eedd2c6c827cb785 Mon Sep 17 00:00:00 2001
From: Felarof <nithin.sonti@gmail.com>
Date: Sat, 14 Mar 2026 09:03:25 -0700
Subject: [PATCH] fix: repair stale defaultProviderId in storage on load

When the stored default provider ID doesn't match any loaded provider,
write back the corrected ID (providers[0].id) to storage so it doesn't
silently persist across sessions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../apps/agent/lib/llm-providers/useLlmProviders.ts      | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/packages/browseros-agent/apps/agent/lib/llm-providers/useLlmProviders.ts b/packages/browseros-agent/apps/agent/lib/llm-providers/useLlmProviders.ts
index 06c24831..905094f4 100644
--- a/packages/browseros-agent/apps/agent/lib/llm-providers/useLlmProviders.ts
+++ b/packages/browseros-agent/apps/agent/lib/llm-providers/useLlmProviders.ts
@@ -60,6 +60,15 @@ export function useLlmProviders(): UseLlmProvidersReturn {
           await defaultProviderIdStorage.setValue(loadedDefaultId)
         }
 
+        // Repair stale default ID that doesn't match any provider
+        const defaultExists = loadedProviders.some(
+          (p) => p.id === loadedDefaultId,
+        )
+        if (!defaultExists && loadedProviders.length > 0) {
+          loadedDefaultId = loadedProviders[0].id
+          await defaultProviderIdStorage.setValue(loadedDefaultId)
+        }
+
         setProviders(loadedProviders)
         setDefaultProviderId(loadedDefaultId)
       } catch {