From 4dc59dff212a1ff62ad2d781f5e84d1c1928539d Mon Sep 17 00:00:00 2001
From: Jason <farion1231@gmail.com>
Date: Fri, 2 Jan 2026 23:41:28 +0800
Subject: [PATCH] fix(skills): remove refresh button from installed skills
 panel

The refresh button is only needed in the discovery panel (to fetch
latest skills from GitHub). The installed skills panel uses local
database which auto-updates on install/uninstall operations.
---
 src/App.tsx | 38 +++++++++-----------------------------
 1 file changed, 9 insertions(+), 29 deletions(-)

diff --git a/src/App.tsx b/src/App.tsx
index 21d8b0ad2..db2abe7d2 100644
--- a/src/App.tsx
+++ b/src/App.tsx
@@ -111,15 +111,6 @@ function App() {
   const currentProviderId = data?.currentProviderId ?? "";
   const hasSkillsSupport = true;
 
-  const refreshSkillsData = async () => {
-    try {
-      await queryClient.invalidateQueries({ queryKey: ["skills"] });
-      await queryClient.refetchQueries({ queryKey: ["skills"], type: "active" });
-    } catch (error) {
-      console.error("[App] Failed to refresh skills data", error);
-    }
-  };
-
   // 🎯 使用 useProviderActions Hook 统一管理所有 Provider 操作
   const {
     addProvider,
@@ -656,26 +647,15 @@ function App() {
               </Button>
             )}
             {currentView === "skills" && (
-              <>
-                <Button
-                  variant="ghost"
-                  size="sm"
-                  onClick={refreshSkillsData}
-                  className="hover:bg-black/5 dark:hover:bg-white/5"
-                >
-                  <RefreshCw className="w-4 h-4 mr-2" />
-                  {t("skills.refresh")}
-                </Button>
-                <Button
-                  variant="ghost"
-                  size="sm"
-                  onClick={() => setCurrentView("skillsDiscovery")}
-                  className="hover:bg-black/5 dark:hover:bg-white/5"
-                >
-                  <Search className="w-4 h-4 mr-2" />
-                  {t("skills.discover")}
-                </Button>
-              </>
+              <Button
+                variant="ghost"
+                size="sm"
+                onClick={() => setCurrentView("skillsDiscovery")}
+                className="hover:bg-black/5 dark:hover:bg-white/5"
+              >
+                <Search className="w-4 h-4 mr-2" />
+                {t("skills.discover")}
+              </Button>
             )}
             {currentView === "skillsDiscovery" && (
               <>