feat(web): add Mantine clear buttons to text inputs app-wide (#1333)

clients/web/src/App.test.tsx

@@ -659,7 +659,7 @@ describe("App pending server-initiated request modal", () => {
 
     // Resolving via the modal calls the queued request's respond() — this is
     // what unblocks the originating call (the "spinner clears" criterion).
-    await user.click(screen.getByRole("button", { name: "Auto-respond" }));
+    await user.click(screen.getByRole("button", { name: "Send Response" }));
     expect(respond).toHaveBeenCalledTimes(1);
 
     // The client clearing its queue (empty event) closes the modal.

clients/web/src/components/groups/AppControls/AppControls.tsx

@@ -1,4 +1,5 @@
 import {
+  CloseButton,
   Group,
   ScrollArea,
   Stack,
@@ -61,6 +62,15 @@ export function AppControls({
         placeholder="Search apps..."
         value={searchText}
         onChange={(e) => onSearchChange(e.currentTarget.value)}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          searchText ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onSearchChange("")}
+            />
+          ) : null
+        }
       />
       <ScrollArea.Autosize viewportRef={viewportRef} mah={LIST_MAX_HEIGHT}>
         <Stack gap="xs">

clients/web/src/components/groups/ExperimentalFeaturesPanel/ExperimentalFeaturesPanel.tsx

@@ -5,6 +5,7 @@ import {
   Button,
   Card,
   Checkbox,
+  CloseButton,
   Divider,
   Group,
   Stack,
@@ -244,13 +245,31 @@ export function ExperimentalFeaturesPanel({
             onChange={(e) =>
               onHeaderChange(index, e.currentTarget.value, header.value)
             }
+            rightSectionPointerEvents="auto"
+            rightSection={
+              header.key ? (
+                <CloseButton
+                  aria-label="Clear"
+                  onClick={() => onHeaderChange(index, "", header.value)}
+                />
+              ) : null
+            }
           />
           <TextInput
             placeholder="Header value"
             value={header.value}
             onChange={(e) =>
               onHeaderChange(index, header.key, e.currentTarget.value)
             }
+            rightSectionPointerEvents="auto"
+            rightSection={
+              header.value ? (
+                <CloseButton
+                  aria-label="Clear"
+                  onClick={() => onHeaderChange(index, header.key, "")}
+                />
+              ) : null
+            }
           />
           <RemoveIcon onClick={() => onRemoveHeader(index)}>
             <Text size="xs">✕</Text>
@@ -269,6 +288,15 @@ export function ExperimentalFeaturesPanel({
         onChange={(e) => onRequestChange(e.currentTarget.value)}
         autosize
         minRows={6}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          requestDraft ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onRequestChange("")}
+            />
+          ) : null
+        }
       />
 
       <Button onClick={onSendRequest}>Send Request</Button>

clients/web/src/components/groups/HistoryControls/HistoryControls.test.tsx

@@ -31,6 +31,20 @@ describe("HistoryControls", () => {
     expect(onSearchChange).toHaveBeenCalledWith("a");
   });
 
+  it("clears the search input when the Clear button is clicked", async () => {
+    const user = userEvent.setup();
+    const onSearchChange = vi.fn();
+    renderWithMantine(
+      <HistoryControls
+        {...baseProps}
+        searchText="abc"
+        onSearchChange={onSearchChange}
+      />,
+    );
+    await user.click(screen.getByRole("button", { name: "Clear" }));
+    expect(onSearchChange).toHaveBeenCalledWith("");
+  });
+
   it("renders the method filter placeholder", () => {
     renderWithMantine(<HistoryControls {...baseProps} />);
     expect(screen.getByPlaceholderText("All methods")).toBeInTheDocument();

clients/web/src/components/groups/HistoryControls/HistoryControls.tsx

@@ -1,4 +1,4 @@
-import { Select, Stack, TextInput, Title } from "@mantine/core";
+import { CloseButton, Select, Stack, TextInput, Title } from "@mantine/core";
 import type {
   MessageMethod,
   MessageOrigin,
@@ -33,6 +33,15 @@ export function HistoryControls({
         placeholder="Search..."
         value={searchText}
         onChange={(event) => onSearchChange(event.currentTarget.value)}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          searchText ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onSearchChange("")}
+            />
+          ) : null
+        }
       />
 
       <Title order={6}>Filter by Method</Title>

clients/web/src/components/groups/ImportServerJsonPanel/ImportServerJsonPanel.test.tsx

@@ -196,6 +196,40 @@ describe("ImportServerJsonPanel", () => {
     expect(onServerNameChange).toHaveBeenCalledWith("X");
   });
 
+  it("clears the paste textarea, env-var, and name-override fields via Clear buttons", async () => {
+    const user = userEvent.setup();
+    const onJsonChange = vi.fn();
+    const onEnvVarChange = vi.fn();
+    const onServerNameChange = vi.fn();
+    const envVars: EnvVarInfo[] = [
+      { name: "DEBUG", required: false, value: "true" },
+    ];
+    renderWithMantine(
+      <ImportServerJsonPanel
+        {...baseHandlers}
+        onJsonChange={onJsonChange}
+        onEnvVarChange={onEnvVarChange}
+        onServerNameChange={onServerNameChange}
+        draft={{
+          rawText: "{}",
+          envOverrides: {},
+          nameOverride: "Custom Name",
+        }}
+        validation={[]}
+        envVars={envVars}
+      />,
+    );
+    // DOM order: paste textarea, env-var input, name-override input.
+    const clearButtons = screen.getAllByRole("button", { name: "Clear" });
+    expect(clearButtons).toHaveLength(3);
+    await user.click(clearButtons[0]);
+    expect(onJsonChange).toHaveBeenCalledWith("");
+    await user.click(clearButtons[1]);
+    expect(onEnvVarChange).toHaveBeenCalledWith("DEBUG", "");
+    await user.click(clearButtons[2]);
+    expect(onServerNameChange).toHaveBeenCalledWith("");
+  });
+
   it("renders the existing nameOverride value", () => {
     renderWithMantine(
       <ImportServerJsonPanel

clients/web/src/components/groups/ImportServerJsonPanel/ImportServerJsonPanel.tsx

@@ -1,5 +1,6 @@
 import {
   Button,
+  CloseButton,
   Divider,
   Group,
   Radio,
@@ -88,6 +89,12 @@ export function ImportServerJsonPanel({
         autosize
         minRows={8}
         maxRows={15}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          draft.rawText ? (
+            <CloseButton aria-label="Clear" onClick={() => onJsonChange("")} />
+          ) : null
+        }
       />
 
       <Divider />
@@ -139,6 +146,15 @@ export function ImportServerJsonPanel({
               onChange={(e) =>
                 onEnvVarChange(envVar.name, e.currentTarget.value)
               }
+              rightSectionPointerEvents="auto"
+              rightSection={
+                envVar.value ? (
+                  <CloseButton
+                    aria-label="Clear"
+                    onClick={() => onEnvVarChange(envVar.name, "")}
+                  />
+                ) : null
+              }
             />
           ))}
         </>
@@ -150,6 +166,15 @@ export function ImportServerJsonPanel({
         label="Server Name (optional override)"
         value={draft.nameOverride ?? ""}
         onChange={(e) => onServerNameChange(e.currentTarget.value)}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          draft.nameOverride ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onServerNameChange("")}
+            />
+          ) : null
+        }
       />
 
       <Group justify="flex-end">

clients/web/src/components/groups/LogControls/LogControls.tsx

@@ -1,5 +1,6 @@
 import {
   Button,
+  CloseButton,
   Group,
   Select,
   Stack,
@@ -64,6 +65,15 @@ export function LogControls({
         placeholder="Search..."
         value={filterText}
         onChange={(e) => onFilterChange(e.currentTarget.value)}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          filterText ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onFilterChange("")}
+            />
+          ) : null
+        }
       />
 
       <Title order={5}>Set Active Level</Title>

clients/web/src/components/groups/NetworkControls/NetworkControls.tsx

@@ -1,5 +1,6 @@
 import {
   Button,
+  CloseButton,
   Group,
   Stack,
   Text,
@@ -45,6 +46,15 @@ export function NetworkControls({
         placeholder="Search..."
         value={filterText}
         onChange={(e) => onFilterChange(e.currentTarget.value)}
+        rightSectionPointerEvents="auto"
+        rightSection={
+          filterText ? (
+            <CloseButton
+              aria-label="Clear"
+              onClick={() => onFilterChange("")}
+            />
+          ) : null
+        }
       />
 
       <Group justify="space-between">

clients/web/src/components/groups/PendingClientRequestModal/PendingClientRequestModal.stories.tsx

@@ -71,7 +71,7 @@ export const Sampling: Story = {
     expect(
       body.getByText("The server is requesting an LLM completion."),
     ).toBeInTheDocument();
-    await userEvent.click(body.getByRole("button", { name: "Auto-respond" }));
+    await userEvent.click(body.getByRole("button", { name: "Send Response" }));
     expect(args.onSamplingRespond).toHaveBeenCalled();
   },
 };

clients/web/src/components/groups/PendingClientRequestModal/PendingClientRequestModal.test.tsx

@@ -79,12 +79,12 @@ describe("PendingClientRequestModal", () => {
     ).toBeInTheDocument();
   });
 
-  it("auto-responds with a stub sampling result", async () => {
+  it("sends the default stub sampling result when the draft is untouched", async () => {
     const user = userEvent.setup();
     renderWithMantine(
       <PendingClientRequestModal {...baseProps} request={samplingContent} />,
     );
-    await user.click(screen.getByRole("button", { name: "Auto-respond" }));
+    await user.click(screen.getByRole("button", { name: "Send Response" }));
     expect(baseProps.onSamplingRespond).toHaveBeenCalledWith({
       model: "stub-model",
       stopReason: "endTurn",
@@ -124,10 +124,10 @@ describe("PendingClientRequestModal", () => {
     renderWithMantine(
       <PendingClientRequestModal {...baseProps} request={samplingContent} />,
     );
-    const autoRespond = screen.getByRole("button", { name: "Auto-respond" });
-    await user.click(autoRespond);
+    const sendResponse = screen.getByRole("button", { name: "Send Response" });
+    await user.click(sendResponse);
     // After the first dispatch the actions lock (busy); a second click no-ops.
-    await user.click(autoRespond);
+    await user.click(sendResponse);
     expect(baseProps.onSamplingRespond).toHaveBeenCalledTimes(1);
     expect(
       screen.getByRole("button", { name: "Send Response" }),

clients/web/src/components/groups/PendingClientRequestModal/PendingClientRequestModal.tsx

@@ -55,8 +55,8 @@ const TitleText = Text.withProps({ fw: 600 });
 const QueueLabel = Text.withProps({ size: "xs", c: "dimmed" });
 
 /**
- * The stub result pre-filled into a sampling draft. "Auto-respond" sends this
- * as-is; "Send Response" sends whatever the user edited it into.
+ * The stub result pre-filled into a sampling draft. "Send Response" sends this
+ * as-is when untouched, or whatever the user edited it into.
  */
 function createDefaultSamplingResult(): CreateMessageResult {
   return {
@@ -119,7 +119,6 @@ function SamplingModalBody({
       request={request}
       draftResult={draftResult}
       onResultChange={setDraftResult}
-      onAutoRespond={once(() => onRespond(createDefaultSamplingResult()))}
       onSend={once(() => onRespond(draftResult))}
       onReject={once(onReject)}
       busy={responded}

clients/web/src/components/groups/PromptArgumentsForm/PromptArgumentsForm.test.tsx

@@ -144,6 +144,23 @@ describe("PromptArgumentsForm", () => {
     expect(onArgumentChange).toHaveBeenCalledWith("text", "h");
   });
 
+  it("clears an argument via its Clear button (onArgumentChange with empty value)", async () => {
+    const user = userEvent.setup();
+    const onArgumentChange = vi.fn();
+    renderWithMantine(
+      <PromptArgumentsForm
+        prompt={promptWithArgs}
+        argumentValues={{ text: "Hello" }}
+        onArgumentChange={onArgumentChange}
+        onGetPrompt={vi.fn()}
+      />,
+    );
+    // Non-autocomplete branch (completions unsupported) renders a TextInput
+    // with a Clear button whenever the value is non-empty.
+    await user.click(screen.getByRole("button", { name: "Clear" }));
+    expect(onArgumentChange).toHaveBeenCalledWith("text", "");
+  });
+
   it("invokes onGetPrompt when Get Prompt is clicked", async () => {
     const user = userEvent.setup();
     const onGetPrompt = vi.fn();

clients/web/src/components/groups/PromptArgumentsForm/PromptArgumentsForm.tsx

@@ -2,6 +2,7 @@ import { useCallback, useEffect, useRef, useState } from "react";
 import {
   Autocomplete,
   Button,
+  CloseButton,
   Group,
   Stack,
   Text,
@@ -221,6 +222,15 @@ export function PromptArgumentsForm({
                   onChange={(event) =>
                     handleChange(arg.name, event.currentTarget.value)
                   }
+                  rightSectionPointerEvents="auto"
+                  rightSection={
+                    argumentValues[arg.name] ? (
+                      <CloseButton
+                        aria-label="Clear"
+                        onClick={() => handleChange(arg.name, "")}
+                      />
+                    ) : null
+                  }
                 />
               ),
             )}