Merge pull request #56 from DMontgomery40/codex/stability-20260309-2343

fix: avoid hidden system-status startup load on dashboard monitoring deep-link

Author: DMontgomery40

web/src/pages/Dashboard.tsx

@@ -34,7 +34,7 @@ export function Dashboard() {
 
       {/* System Status Subtab */}
       <div style={{ display: activeSubtab === 'system' ? 'block' : 'none' }}>
-        <SystemStatusSubtab />
+        {activeSubtab === 'system' ? <SystemStatusSubtab /> : null}
       </div>
 
       {/* Monitoring Subtab */}

web/tests/e2e/exhaustive/dashboard_startup_load.spec.ts

@@ -20,3 +20,26 @@ test('Dashboard default subtab does not trigger storage startup requests', async
   const storageRequests = [...observedApiPaths].filter((path) => path.includes('/api/index/stats'));
   expect(storageRequests).toEqual([]);
 });
+
+test('Dashboard monitoring deep-link does not trigger system status startup requests', async ({ page, baseURL }) => {
+  const observedApiPaths = new Set<string>();
+
+  page.on('request', (request) => {
+    const url = request.url();
+    if (!url.includes('/api/')) return;
+    try {
+      observedApiPaths.add(new URL(url).pathname);
+    } catch {
+      // Ignore malformed URLs in diagnostics.
+    }
+  });
+
+  await page.goto(new URL('dashboard?subtab=monitoring', baseURL).toString());
+  await page.waitForURL(/\/dashboard\?subtab=monitoring(?:&|$)/);
+  await page.waitForTimeout(1500);
+
+  const systemStatusRequests = [...observedApiPaths].filter((path) =>
+    ['/api/mcp/status', '/api/docker/status'].includes(path),
+  );
+  expect(systemStatusRequests).toEqual([]);
+});