From 5aa408805116e9a4bf68403cb321e08b124a38af Mon Sep 17 00:00:00 2001 From: yyh Date: Mon, 19 Jan 2026 19:51:47 +0800 Subject: [PATCH] fix(sandbox): use deleteConfig when switching to managed mode Delete user config instead of saving empty config when switching to managed mode, allowing the system to fall back to system defaults. --- .../sandbox-provider-page/config-modal.tsx | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/web/app/components/header/account-setting/sandbox-provider-page/config-modal.tsx b/web/app/components/header/account-setting/sandbox-provider-page/config-modal.tsx index 99dcbeec51..d293e71141 100644 --- a/web/app/components/header/account-setting/sandbox-provider-page/config-modal.tsx +++ b/web/app/components/header/account-setting/sandbox-provider-page/config-modal.tsx @@ -102,13 +102,10 @@ function ConfigModal({ provider, onClose }: ConfigModalProps) { }, [provider.config_schema, provider.config, t]) const handleSave = useCallback(async () => { - // For managed mode, save empty config to use system defaults + // For managed mode, delete user config to use system defaults if (shouldShowModeSelection && configMode === 'managed') { try { - await saveConfig({ - providerType: provider.provider_type, - config: {}, - }) + await deleteConfig(provider.provider_type) notify({ type: 'success', message: t('api.saved', { ns: 'common' }) }) onClose() } @@ -137,7 +134,7 @@ function ConfigModal({ provider, onClose }: ConfigModalProps) { catch { // Error toast is handled by fetch layer } - }, [shouldShowModeSelection, configMode, saveConfig, provider.provider_type, notify, t, onClose]) + }, [shouldShowModeSelection, configMode, saveConfig, deleteConfig, provider.provider_type, notify, t, onClose]) const handleRevoke = useCallback(async () => { try {