From 47aa4c6beede0ce8e151952ed6c81e6bcb53ee93 Mon Sep 17 00:00:00 2001
From: Jason <farion1231@gmail.com>
Date: Fri, 2 Jan 2026 23:51:03 +0800
Subject: [PATCH] fix(skills): show loading indicator when refreshing discovery
 list

Use isFetching instead of just isLoading to show the loading spinner.
isLoading is only true on initial load, while isFetching is true
during any fetch operation including refetch.
---
 src/components/skills/SkillsPage.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/components/skills/SkillsPage.tsx b/src/components/skills/SkillsPage.tsx
index 16bbce978..721ac9665 100644
--- a/src/components/skills/SkillsPage.tsx
+++ b/src/components/skills/SkillsPage.tsx
@@ -54,6 +54,7 @@ export const SkillsPage = forwardRef<SkillsPageHandle, SkillsPageProps>(
     const {
       data: discoverableSkills,
       isLoading: loadingDiscoverable,
+      isFetching: fetchingDiscoverable,
       refetch: refetchDiscoverable,
     } = useDiscoverableSkills();
     const { data: installedSkills } = useInstalledSkills();
@@ -86,7 +87,7 @@ export const SkillsPage = forwardRef<SkillsPageHandle, SkillsPageProps>(
       });
     }, [discoverableSkills, installedDirs]);
 
-    const loading = loadingDiscoverable;
+    const loading = loadingDiscoverable || fetchingDiscoverable;
 
     useImperativeHandle(ref, () => ({
       refresh: () => {