From 536d292838379af8e415044c82276e9a1e9ffbeb Mon Sep 17 00:00:00 2001
From: Mark McDowall <mark@mcdowall.ca>
Date: Sat, 28 Mar 2026 14:10:11 -0700
Subject: [PATCH] Prevent duplicating providers when adding a new provider

---
 frontend/src/Settings/useProviderSettings.ts | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/frontend/src/Settings/useProviderSettings.ts b/frontend/src/Settings/useProviderSettings.ts
index 809547b5c..3bee87878 100644
--- a/frontend/src/Settings/useProviderSettings.ts
+++ b/frontend/src/Settings/useProviderSettings.ts
@@ -122,13 +122,17 @@ export const useSaveProviderSettings = <T extends ModelBase>(
     },
     onSuccess: (updatedSettings: T) => {
       queryClient.setQueryData<T[]>([path], (oldData = []) => {
-        if (id) {
-          return oldData.map((item) =>
-            item.id === updatedSettings.id ? updatedSettings : item
-          );
+        const existingIndex = oldData.findIndex(
+          (item) => item.id === updatedSettings.id
+        );
+
+        if (existingIndex === -1) {
+          return [...oldData, updatedSettings];
         }
 
-        return [...oldData, updatedSettings];
+        return oldData.map((item) =>
+          item.id === updatedSettings.id ? updatedSettings : item
+        );
       });
       onSuccess?.(updatedSettings);
     },