diff --git a/site/src/pages/AgentsPage/components/MCPServerAdminPanel.stories.tsx b/site/src/pages/AgentsPage/components/MCPServerAdminPanel.stories.tsx index d7cf50f4ff..7611cb4221 100644 --- a/site/src/pages/AgentsPage/components/MCPServerAdminPanel.stories.tsx +++ b/site/src/pages/AgentsPage/components/MCPServerAdminPanel.stories.tsx @@ -162,21 +162,18 @@ export const CreateServer: Story = { await body.findByRole("button", { name: /Add your first server/i }), ); - // Fill in the display name via the inline header input. + // Fill in the Display name field. const nameInput = await body.findByLabelText(/Display Name/i); await userEvent.type(nameInput, "Sentry"); - // Required fields (slug, server URL) are always visible. Optional - // sections start collapsed and the Enabled switch is edit-only. - // PencilIcon sits next to the editable name. + // Required fields (display name, slug, server URL) are always visible. + // Optional sections start collapsed and the Enabled switch is edit-only. expect(body.getByLabelText(/^Slug/i)).toBeInTheDocument(); expect(body.getByLabelText(/Server URL/i)).toBeInTheDocument(); expect(body.queryByLabelText(/Description/i)).not.toBeInTheDocument(); expect( body.queryByRole("switch", { name: /Enabled/i }), ).not.toBeInTheDocument(); - const nameRow = nameInput.closest("div")?.parentElement; - expect(nameRow?.querySelector("svg.lucide-pencil")).toBeTruthy(); // Slug should auto-populate from the display name. await expect(body.getByLabelText(/^Slug/i)).toHaveValue("sentry"); diff --git a/site/src/pages/AgentsPage/components/MCPServerAdminPanel.tsx b/site/src/pages/AgentsPage/components/MCPServerAdminPanel.tsx index 21c49fcf66..2fb7e64f9f 100644 --- a/site/src/pages/AgentsPage/components/MCPServerAdminPanel.tsx +++ b/site/src/pages/AgentsPage/components/MCPServerAdminPanel.tsx @@ -4,7 +4,6 @@ import { ChevronDownIcon, ChevronRightIcon, CircleIcon, - PencilIcon, PlusIcon, ServerIcon, XIcon, @@ -471,41 +470,15 @@ const ServerForm: FC = ({ return (
- {/* Back */} - {/* Header with icon + editable name + enabled toggle */}
- -
-
- - { - form.setFieldValue("displayName", e.target.value); - if (!form.values.slugTouched) { - form.setFieldValue("slug", slugify(e.target.value)); - } - }} - disabled={isDisabled} - spellCheck={false} - className="col-start-1 row-start-1 m-0 min-w-0 border-0 bg-transparent p-0 text-lg font-medium text-content-primary outline-none placeholder:text-content-secondary focus:ring-0" - placeholder="Server display name" - aria-label="Display Name" - /> -
- -
+ + {form.values.displayName || + (isEditing ? "Server display name" : "New MCP server")} + {isEditing && ( @@ -538,6 +511,26 @@ const ServerForm: FC = ({ >
+ + { + form.setFieldValue("displayName", e.target.value); + if (!form.values.slugTouched) { + form.setFieldValue("slug", slugify(e.target.value)); + } + }} + placeholder="e.g. Sentry" + disabled={isDisabled} + aria-label="Display Name" + /> +