fix(ui): live-refresh dashboard stat tiles on feature-set / client / server changes

The Home dashboard loaded its FeatureSets / Clients / Servers counts once (on
mount + Space switch + gateway/server-status events) but ignored
feature-set-changed and client-changed. So a FeatureSet composed by an MCP
client via mcpmux_manage_feature_set — or a newly authenticated app — left the
tile counts stale until a Space switch or reload.

Subscribe loadStats to feature-set-changed, client-changed, and server-changed
(memoized via useCallback so the subscription is stable per Space). The
FeatureSets *page* already live-refreshed via its own feature-set-changed
listener; this closes the dashboard gap.

Adds HomePageStats.test.tsx: the FeatureSets tile count updates from 1 → 2 when
a feature-set-changed event arrives.

Signed-off-by: Mohammod Al Amin Ashik <maa.ashik00@gmail.com>

Author: its-mash

apps/desktop/src/features/home/HomePage.tsx

@@ -9,12 +9,12 @@
  * row of stat tiles that double as navigation — every tile is a button into
  * the page that manages what it counts.
  */
-import { useEffect, useState } from 'react';
+import { useEffect, useState, useCallback } from 'react';
 import { Server, Wrench, Monitor, Globe, ArrowUpRight, Compass, ArrowRight } from 'lucide-react';
 import type { LucideIcon } from 'lucide-react';
 import { PageHeader } from '@mcpmux/ui';
 import { ConnectionCard } from '@/components/ConnectionCard';
-import { useGatewayEvents, useServerStatusEvents } from '@/hooks/useDomainEvents';
+import { useGatewayEvents, useServerStatusEvents, useDomainEvents } from '@/hooks/useDomainEvents';
 import { useViewSpace, useNavigateTo } from '@/stores';
 import type { NavItem } from '@/stores/types';
 import { spaceAccentColor } from '@/lib/spaceAccent';
@@ -162,7 +162,7 @@ export function HomePage() {
   const [statsLoaded, setStatsLoaded] = useState(false);
   const viewSpace = useViewSpace();
 
-  const loadStats = async () => {
+  const loadStats = useCallback(async () => {
     try {
       const [clients, featureSets, gateway, installedServers] = await Promise.all([
         import('@/lib/api/clients').then((m) => m.listClients()),
@@ -182,13 +182,12 @@ export function HomePage() {
     } catch (e) {
       console.error('Failed to load home stats:', e);
     }
-  };
+  }, [viewSpace?.id]);
 
   // Load on mount and when the viewed Space changes.
   useEffect(() => {
     loadStats();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [viewSpace?.id]);
+  }, [loadStats]);
 
   // Keep `Tools: X/Y` honest across gateway start/stop and backend churn.
   // ConnectionCard owns the actual running/URL UI.
@@ -206,6 +205,20 @@ export function HomePage() {
     }
   });
 
+  // Keep the FeatureSets + Clients tiles live when those change anywhere —
+  // e.g. an MCP client composing a FeatureSet via `mcpmux_manage_feature_set`,
+  // or a new app authenticating. Without this the counts go stale until a
+  // Space switch or reload.
+  const { subscribe } = useDomainEvents();
+  useEffect(() => {
+    const unsubs = [
+      subscribe('feature-set-changed', () => void loadStats()),
+      subscribe('client-changed', () => void loadStats()),
+      subscribe('server-changed', () => void loadStats()),
+    ];
+    return () => unsubs.forEach((u) => u());
+  }, [subscribe, loadStats]);
+
   return (
     <div className="space-y-6">
       <PageHeader

tests/ts/components/HomePageStats.test.tsx

@@ -0,0 +1,74 @@
+/**
+ * Dashboard stat tiles must reflect changes made anywhere — including a
+ * FeatureSet composed by an MCP client via `mcpmux_manage_feature_set`, which
+ * arrives as a `feature-set-changed` domain event. Guards the fix for
+ * "created a FeatureSet via the tool but the count didn't update live".
+ */
+
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, act, waitFor } from '@testing-library/react';
+
+const { handlers, mockListClients, mockListFS, mockGatewayStatus, mockListInstalled } = vi.hoisted(
+  () => ({
+    handlers: new Map<string, ((e: { payload: unknown }) => void)[]>(),
+    mockListClients: vi.fn(),
+    mockListFS: vi.fn(),
+    mockGatewayStatus: vi.fn(),
+    mockListInstalled: vi.fn(),
+  })
+);
+
+vi.mock('@tauri-apps/api/event', () => ({
+  listen: vi.fn((name: string, cb: (e: { payload: unknown }) => void) => {
+    const arr = handlers.get(name) ?? [];
+    arr.push(cb);
+    handlers.set(name, arr);
+    return Promise.resolve(() => {});
+  }),
+}));
+vi.mock('@/lib/api/clients', () => ({ listClients: mockListClients }));
+vi.mock('@/lib/api/featureSets', () => ({
+  listFeatureSetsBySpace: mockListFS,
+  listFeatureSets: mockListFS,
+}));
+vi.mock('@/lib/api/gateway', () => ({ getGatewayStatus: mockGatewayStatus }));
+vi.mock('@/lib/api/registry', () => ({ listInstalledServers: mockListInstalled }));
+vi.mock('@/stores', () => ({
+  useViewSpace: () => ({ id: 'space-1', name: 'My Space' }),
+  useNavigateTo: () => () => {},
+}));
+vi.mock('@/components/ConnectionCard', () => ({ ConnectionCard: () => null }));
+
+import { HomePage } from '@/features/home/HomePage';
+
+function emit(channel: string, payload: unknown) {
+  act(() => {
+    handlers.get(channel)?.forEach((cb) => cb({ payload }));
+  });
+}
+
+describe('HomePage dashboard stats', () => {
+  beforeEach(() => {
+    handlers.clear();
+    mockListClients.mockReset().mockResolvedValue([]);
+    mockGatewayStatus.mockReset().mockResolvedValue({ connected_backends: 0 });
+    mockListInstalled.mockReset().mockResolvedValue([{}]); // 1 server → skip onboarding strip
+    mockListFS.mockReset();
+  });
+
+  it('refreshes the FeatureSets count when a feature-set-changed event arrives', async () => {
+    // First load → 1 FeatureSet; after the event → 2.
+    mockListFS.mockResolvedValueOnce([{}]).mockResolvedValue([{}, {}]);
+
+    render(<HomePage />);
+    await waitFor(() =>
+      expect(screen.getByTestId('stat-featuresets-value')).toHaveTextContent('1')
+    );
+
+    emit('feature-set-changed', { action: 'created' });
+
+    await waitFor(() =>
+      expect(screen.getByTestId('stat-featuresets-value')).toHaveTextContent('2')
+    );
+  });
+});