From 552f7abee40d3e50a28f7693bbaca171c2ae4f30 Mon Sep 17 00:00:00 2001
From: Jason <farion1231@gmail.com>
Date: Fri, 20 Mar 2026 15:23:18 +0800
Subject: [PATCH] refactor(ui): remove duplicate OAuth tab from
 AddProviderDialog

- Remove AuthCenterPanel import and OAuth TabsContent
- Narrow activeTab type from three values to "app-specific" | "universal"
- Simplify footer by removing oauth branch, reducing to two-way conditional
- Change TabsList from grid-cols-3 to grid-cols-2
- OAuth authentication remains available in settings page and CopilotAuthSection
---
 .../providers/AddProviderDialog.tsx           | 22 +++----------------
 1 file changed, 3 insertions(+), 19 deletions(-)

diff --git a/src/components/providers/AddProviderDialog.tsx b/src/components/providers/AddProviderDialog.tsx
index 34a722982..a4f223d7c 100644
--- a/src/components/providers/AddProviderDialog.tsx
+++ b/src/components/providers/AddProviderDialog.tsx
@@ -14,7 +14,6 @@ import {
 } from "@/components/providers/forms/ProviderForm";
 import { UniversalProviderFormModal } from "@/components/universal/UniversalProviderFormModal";
 import { UniversalProviderPanel } from "@/components/universal";
-import { AuthCenterPanel } from "@/components/settings/AuthCenterPanel";
 import { providerPresets } from "@/config/claudeProviderPresets";
 import { codexProviderPresets } from "@/config/codexProviderPresets";
 import { geminiProviderPresets } from "@/config/geminiProviderPresets";
@@ -44,7 +43,7 @@ export function AddProviderDialog({
   // OpenCode and OpenClaw don't support universal providers
   const showUniversalTab = appId !== "opencode" && appId !== "openclaw";
   const [activeTab, setActiveTab] = useState<
-    "app-specific" | "universal" | "oauth"
+    "app-specific" | "universal"
   >("app-specific");
   const [universalFormOpen, setUniversalFormOpen] = useState(false);
   const [selectedUniversalPreset, setSelectedUniversalPreset] =
@@ -256,14 +255,6 @@ export function AddProviderDialog({
           {t("common.add")}
         </Button>
       </>
-    ) : activeTab === "oauth" ? (
-      <Button
-        variant="outline"
-        onClick={() => onOpenChange(false)}
-        className="border-border/20 hover:bg-accent hover:text-accent-foreground"
-      >
-        {t("common.close", { defaultValue: "关闭" })}
-      </Button>
     ) : (
       <>
         <Button
@@ -294,19 +285,16 @@ export function AddProviderDialog({
         <Tabs
           value={activeTab}
           onValueChange={(v) =>
-            setActiveTab(v as "app-specific" | "universal" | "oauth")
+            setActiveTab(v as "app-specific" | "universal")
           }
         >
-          <TabsList className="grid w-full grid-cols-3 mb-6">
+          <TabsList className="grid w-full grid-cols-2 mb-6">
             <TabsTrigger value="app-specific">
               {t(`apps.${appId}`)} {t("provider.tabProvider")}
             </TabsTrigger>
             <TabsTrigger value="universal">
               {t("provider.tabUniversal")}
             </TabsTrigger>
-            <TabsTrigger value="oauth">
-              {t("provider.tabOAuth", { defaultValue: "OAuth 认证源" })}
-            </TabsTrigger>
           </TabsList>
 
           <TabsContent value="app-specific" className="mt-0">
@@ -323,10 +311,6 @@ export function AddProviderDialog({
           <TabsContent value="universal" className="mt-0">
             <UniversalProviderPanel />
           </TabsContent>
-
-          <TabsContent value="oauth" className="mt-0">
-            <AuthCenterPanel />
-          </TabsContent>
         </Tabs>
       ) : (
         // OpenCode/OpenClaw: directly show form without tabs