Skip to content

Commit 4dc8386

Browse files
Dumbrisclaude
andauthored
fix: use consistent server count across UI components (smart-mcp-proxy#266)
The top header and NavBar were using systemStore.upstreamStats which counts raw connection status, while the Servers page uses serversStore.serverCount which counts health.level === 'healthy'. This inconsistency caused the top header to show different connected server counts than the server page (e.g., 8/23 vs 14/23). Changed TopHeader.vue and NavBar.vue to use serversStore.serverCount as the single source of truth, ensuring consistent counts across all UI components. Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 7a64add commit 4dc8386

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

frontend/src/components/NavBar.vue

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@
7171

7272
<!-- Server stats -->
7373
<div class="flex items-center space-x-1 text-sm">
74-
<span class="hidden md:inline">{{ systemStore.upstreamStats.connected_servers }}/{{ systemStore.upstreamStats.total_servers }}</span>
74+
<span class="hidden md:inline">{{ serversStore.serverCount.connected }}/{{ serversStore.serverCount.total }}</span>
7575
<span class="text-xs opacity-60 hidden md:inline">servers</span>
7676
</div>
7777
</div>
@@ -106,6 +106,7 @@
106106
import { computed, ref } from 'vue'
107107
import { useRoute } from 'vue-router'
108108
import { useSystemStore } from '@/stores/system'
109+
import { useServersStore } from '@/stores/servers'
109110
110111
// Icons (we'll use SVG icons directly for simplicity)
111112
const DashboardIcon = 'svg'
@@ -117,6 +118,7 @@ const SettingsIcon = 'svg'
117118
118119
const route = useRoute()
119120
const systemStore = useSystemStore()
121+
const serversStore = useServersStore()
120122
121123
const menuItems = [
122124
{ name: 'Dashboard', path: '/', icon: DashboardIcon },

frontend/src/components/TopHeader.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,9 @@
5252
systemStore.isRunning ? 'bg-success animate-pulse' : 'bg-error'
5353
]"
5454
/>
55-
<span class="font-bold">{{ systemStore.upstreamStats.connected_servers }}</span>
55+
<span class="font-bold">{{ serversStore.serverCount.connected }}</span>
5656
<span class="opacity-60">/</span>
57-
<span>{{ systemStore.upstreamStats.total_servers }}</span>
57+
<span>{{ serversStore.serverCount.total }}</span>
5858
<span class="text-xs opacity-60">Servers</span>
5959
</div>
6060

0 commit comments

Comments
 (0)