diff --git a/webview-ui/src/components/settings/ApiConfigManager.tsx b/webview-ui/src/components/settings/ApiConfigManager.tsx index 2374b90c0a..62aae27642 100644 --- a/webview-ui/src/components/settings/ApiConfigManager.tsx +++ b/webview-ui/src/components/settings/ApiConfigManager.tsx @@ -49,6 +49,18 @@ const ApiConfigManager = ({ return null } + const resetCreateState = () => { + setIsCreating(false) + setNewProfileName("") + setError(null) + } + + const resetRenameState = () => { + setIsRenaming(false) + setInputValue("") + setError(null) + } + // Focus input when entering rename mode useEffect(() => { if (isRenaming) { @@ -67,17 +79,13 @@ const ApiConfigManager = ({ // Reset state when current profile changes useEffect(() => { - setIsRenaming(false) - setIsCreating(false) - setInputValue("") - setNewProfileName("") - setError(null) + resetCreateState() + resetRenameState() }, [currentApiConfigName]) const handleAdd = () => { + resetCreateState() setIsCreating(true) - setNewProfileName("") - setError(null) } const handleStartRename = () => { @@ -87,9 +95,7 @@ const ApiConfigManager = ({ } const handleCancel = () => { - setIsRenaming(false) - setInputValue("") - setError(null) + resetRenameState() } const handleSave = () => { @@ -103,17 +109,13 @@ const ApiConfigManager = ({ if (isRenaming && currentApiConfigName) { if (currentApiConfigName === trimmedValue) { - setIsRenaming(false) - setInputValue("") - setError(null) + resetRenameState() return } onRenameConfig(currentApiConfigName, trimmedValue) } - setIsRenaming(false) - setInputValue("") - setError(null) + resetRenameState() } const handleNewProfileSave = () => { @@ -126,9 +128,7 @@ const ApiConfigManager = ({ } onUpsertConfig(trimmedValue) - setIsCreating(false) - setNewProfileName("") - setError(null) + resetCreateState() } const handleDelete = () => { @@ -293,9 +293,7 @@ const ApiConfigManager = ({ setNewProfileName("") setError(null) } else { - setIsCreating(false) - setNewProfileName("") - setError(null) + resetCreateState() } }} aria-labelledby="new-profile-title"> @@ -303,14 +301,7 @@ const ApiConfigManager = ({