From 51c68ef1921b38ccfe5f65749136b4dcfe242b30 Mon Sep 17 00:00:00 2001
From: Jason <farion1231@gmail.com>
Date: Thu, 16 Oct 2025 22:54:51 +0800
Subject: [PATCH] chore: clean up TODO comments in ProviderForm

- Remove obsolete TODO comments for Codex Base URL handling
- Codex Base URL is already fully managed by useCodexConfigState hook
- useBaseUrlState is now only used for Claude mode
- Add clarifying comments about the architecture
---
 src/components/providers/forms/ProviderForm.tsx | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/components/providers/forms/ProviderForm.tsx b/src/components/providers/forms/ProviderForm.tsx
index 86dcba4..89aa87d 100644
--- a/src/components/providers/forms/ProviderForm.tsx
+++ b/src/components/providers/forms/ProviderForm.tsx
@@ -120,20 +120,15 @@ export function ProviderForm({
     selectedPresetId,
   });
 
-  // 使用 Base URL hook
-  const {
-    baseUrl,
-    // codexBaseUrl,  // TODO: 等 Codex 支持时使用
-    handleClaudeBaseUrlChange,
-    // handleCodexBaseUrlChange, // TODO: 等 Codex 支持时使用
-  } = useBaseUrlState({
+  // 使用 Base URL hook (仅 Claude 模式)
+  const { baseUrl, handleClaudeBaseUrlChange } = useBaseUrlState({
     appType,
     category,
     settingsConfig: form.watch("settingsConfig"),
-    codexConfig: "", // TODO: 从 settingsConfig 中提取 codex config
+    codexConfig: "",
     onSettingsConfigChange: (config) => form.setValue("settingsConfig", config),
     onCodexConfigChange: () => {
-      // TODO: 更新 codex config
+      // Codex 使用 useCodexConfigState 管理 Base URL
     },
   });