From ec6e3d9d82b0f91a5f8007745006dd74b4cfd01d Mon Sep 17 00:00:00 2001 From: npub1mn7jgtj4w2pd0g0zeuhxsa6jy6p0rewxz4kujt98my82ahfmp72sxjexk7 Date: Mon, 13 Jul 2026 14:38:55 -0400 Subject: [PATCH] fix(desktop): restore the MCP Servers group label The compact rows expose raw transport kinds, so restore a text-only group label that identifies them as MCP servers. Keep screenshot coverage aligned for populated and empty buzz-agent sections. Co-authored-by: Will Pfleger Signed-off-by: Will Pfleger --- desktop/src/features/agents/ui/McpServersSection.tsx | 9 +++++++++ desktop/tests/e2e/config-bridge-screenshots.spec.ts | 6 +++--- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/desktop/src/features/agents/ui/McpServersSection.tsx b/desktop/src/features/agents/ui/McpServersSection.tsx index b4d92af1ce..3e6c93ca5d 100644 --- a/desktop/src/features/agents/ui/McpServersSection.tsx +++ b/desktop/src/features/agents/ui/McpServersSection.tsx @@ -29,6 +29,15 @@ export function McpServersSection({ variant === "compact" ? "mt-3 pt-2" : "divide-y divide-border/50", )} > +

+ MCP Servers +

+ {isBuzzAgent && buzzAgentSlot ? buzzAgentSlot : null} {extensions.length > 0 ? ( diff --git a/desktop/tests/e2e/config-bridge-screenshots.spec.ts b/desktop/tests/e2e/config-bridge-screenshots.spec.ts index 754c79f9c5..ef20e7b4a6 100644 --- a/desktop/tests/e2e/config-bridge-screenshots.spec.ts +++ b/desktop/tests/e2e/config-bridge-screenshots.spec.ts @@ -247,10 +247,10 @@ test.describe("config bridge screenshots", () => { // Advanced runtime fields render directly in the profile panel's flat list. await expect(panel.getByText("active_provider")).toBeVisible(); - // MCP servers render once each, without the removed summary row. + // MCP servers render once each under their group label. await expect(panel.getByText("developer", { exact: true })).toHaveCount(1); await expect(panel.getByText("MCP Servers", { exact: true })).toHaveCount( - 0, + 1, ); await settleAnimations(panel); @@ -275,7 +275,7 @@ test.describe("config bridge screenshots", () => { panel.getByText("No custom servers configured", { exact: true }), ).toBeVisible(); await expect(panel.getByText("MCP Servers", { exact: true })).toHaveCount( - 0, + 1, ); });