From 595df172a8526afa81de791e7cff4b00bec7089d Mon Sep 17 00:00:00 2001
From: zxhlyh <jasonapring2015@outlook.com>
Date: Mon, 29 Sep 2025 13:51:48 +0800
Subject: [PATCH] fix: model list refresh when change credential (#26421)

---
 .../header/account-setting/model-provider-page/hooks.ts  | 9 ++++++---
 .../model-provider-page/model-auth/hooks/use-auth.ts     | 6 +++---
 .../provider-added-card/model-load-balancing-modal.tsx   | 2 +-
 3 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/web/app/components/header/account-setting/model-provider-page/hooks.ts b/web/app/components/header/account-setting/model-provider-page/hooks.ts
index b10aeeb47e..48dc609795 100644
--- a/web/app/components/header/account-setting/model-provider-page/hooks.ts
+++ b/web/app/components/header/account-setting/model-provider-page/hooks.ts
@@ -323,15 +323,18 @@ export const useRefreshModel = () => {
   const { eventEmitter } = useEventEmitterContextContext()
   const updateModelProviders = useUpdateModelProviders()
   const updateModelList = useUpdateModelList()
-  const handleRefreshModel = useCallback((provider: ModelProvider, configurationMethod: ConfigurationMethodEnum, CustomConfigurationModelFixedFields?: CustomConfigurationModelFixedFields) => {
+  const handleRefreshModel = useCallback((
+    provider: ModelProvider,
+    CustomConfigurationModelFixedFields?: CustomConfigurationModelFixedFields,
+    refreshModelList?: boolean,
+  ) => {
     updateModelProviders()
 
     provider.supported_model_types.forEach((type) => {
       updateModelList(type)
     })
 
-    if (configurationMethod === ConfigurationMethodEnum.customizableModel
-        && provider.custom_configuration.status === CustomConfigurationStatusEnum.active) {
+    if (refreshModelList && provider.custom_configuration.status === CustomConfigurationStatusEnum.active) {
       eventEmitter?.emit({
         type: UPDATE_MODEL_PROVIDER_CUSTOM_MODEL_LIST,
         payload: provider.provider,
diff --git a/web/app/components/header/account-setting/model-provider-page/model-auth/hooks/use-auth.ts b/web/app/components/header/account-setting/model-provider-page/model-auth/hooks/use-auth.ts
index 14b21be7f7..3136a70563 100644
--- a/web/app/components/header/account-setting/model-provider-page/model-auth/hooks/use-auth.ts
+++ b/web/app/components/header/account-setting/model-provider-page/model-auth/hooks/use-auth.ts
@@ -90,7 +90,7 @@ export const useAuth = (
         type: 'success',
         message: t('common.api.actionSuccess'),
       })
-      handleRefreshModel(provider, configurationMethod, undefined)
+      handleRefreshModel(provider, undefined, true)
     }
     finally {
       handleSetDoingAction(false)
@@ -125,7 +125,7 @@ export const useAuth = (
         type: 'success',
         message: t('common.api.actionSuccess'),
       })
-      handleRefreshModel(provider, configurationMethod, undefined)
+      handleRefreshModel(provider, undefined, true)
       onRemove?.(pendingOperationCredentialId.current ?? '')
       closeConfirmDelete()
     }
@@ -147,7 +147,7 @@ export const useAuth = (
 
       if (res.result === 'success') {
         notify({ type: 'success', message: t('common.actionMsg.modifiedSuccessfully') })
-        handleRefreshModel(provider, configurationMethod, undefined)
+        handleRefreshModel(provider, undefined, !payload.credential_id)
       }
     }
     finally {
diff --git a/web/app/components/header/account-setting/model-provider-page/provider-added-card/model-load-balancing-modal.tsx b/web/app/components/header/account-setting/model-provider-page/provider-added-card/model-load-balancing-modal.tsx
index 070c2ee90f..090147897b 100644
--- a/web/app/components/header/account-setting/model-provider-page/provider-added-card/model-load-balancing-modal.tsx
+++ b/web/app/components/header/account-setting/model-provider-page/provider-added-card/model-load-balancing-modal.tsx
@@ -159,7 +159,7 @@ const ModelLoadBalancingModal = ({
       )
       if (res.result === 'success') {
         notify({ type: 'success', message: t('common.actionMsg.modifiedSuccessfully') })
-        handleRefreshModel(provider, configurateMethod, currentCustomConfigurationModelFixedFields)
+        handleRefreshModel(provider, currentCustomConfigurationModelFixedFields, false)
         onSave?.(provider.provider)
         onClose?.()
       }