chore(logging): remove Copy All button from LogStreamPanel

Export now produces the same JSON the Copy All button used to dump to
the clipboard, so the button is redundant. History and Network panels
have no equivalent — dropping Copy All also brings Logs into parity.

Removes the prop from LogStreamPanel + LoggingScreen + InspectorView +
App.tsx, and updates the matching tests/stories. Tests now assert the
button is *not* in the document so a future re-introduction would have
to update the assertion deliberately.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: cliffhall

clients/web/src/App.tsx

@@ -936,7 +936,6 @@ function App() {
         onSetLogLevel={onSetLogLevel}
         onClearLogs={onClearLogs}
         onExportLogs={onExportLogs}
-        onCopyAllLogs={todoNoop}
         onClearHistory={onClearHistory}
         onExportHistory={onExportHistory}
         onReplayHistory={todoNoop}

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

@@ -21,7 +21,6 @@ const meta: Meta<typeof LogStreamPanel> = {
     },
     autoScroll: true,
     onToggleAutoScroll: fn(),
-    onCopyAll: fn(),
     onClear: fn(),
     onExport: fn(),
   },

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

@@ -45,7 +45,6 @@ const baseProps = {
   visibleLevels: allVisible,
   autoScroll: true,
   onToggleAutoScroll: vi.fn(),
-  onCopyAll: vi.fn(),
   onClear: vi.fn(),
   onExport: vi.fn(),
 };
@@ -57,8 +56,8 @@ describe("LogStreamPanel", () => {
     expect(screen.getByRole("button", { name: "Clear" })).toBeInTheDocument();
     expect(screen.getByRole("button", { name: "Export" })).toBeInTheDocument();
     expect(
-      screen.getByRole("button", { name: "Copy All" }),
-    ).toBeInTheDocument();
+      screen.queryByRole("button", { name: "Copy All" }),
+    ).not.toBeInTheDocument();
   });
 
   it("renders log entries when provided", () => {
@@ -150,14 +149,6 @@ describe("LogStreamPanel", () => {
     expect(onExport).toHaveBeenCalledTimes(1);
   });
 
-  it("invokes onCopyAll when Copy All is clicked", async () => {
-    const user = userEvent.setup();
-    const onCopyAll = vi.fn();
-    renderWithMantine(<LogStreamPanel {...baseProps} onCopyAll={onCopyAll} />);
-    await user.click(screen.getByRole("button", { name: "Copy All" }));
-    expect(onCopyAll).toHaveBeenCalledTimes(1);
-  });
-
   it("invokes onToggleAutoScroll when the auto-scroll checkbox is clicked", async () => {
     const user = userEvent.setup();
     const onToggleAutoScroll = vi.fn();

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

@@ -19,7 +19,6 @@ export interface LogStreamPanelProps {
   visibleLevels: Record<LoggingLevel, boolean>;
   autoScroll: boolean;
   onToggleAutoScroll: () => void;
-  onCopyAll: () => void;
   onClear: () => void;
   onExport: () => void;
 }
@@ -64,7 +63,6 @@ export function LogStreamPanel({
   visibleLevels,
   autoScroll,
   onToggleAutoScroll,
-  onCopyAll,
   onClear,
   onExport,
 }: LogStreamPanelProps) {
@@ -97,13 +95,6 @@ export function LogStreamPanel({
           >
             Export
           </Button>
-          <Button
-            variant="default"
-            onClick={onCopyAll}
-            disabled={entries.length === 0}
-          >
-            Copy All
-          </Button>
         </Group>
       </Group>
       {filteredEntries.length > 0 ? (

clients/web/src/components/screens/LoggingScreen/LoggingScreen.stories.tsx

@@ -15,7 +15,6 @@ const meta: Meta<typeof LoggingScreen> = {
     onExport: fn(),
     autoScroll: true,
     onToggleAutoScroll: fn(),
-    onCopyAll: fn(),
   },
 };
 

clients/web/src/components/screens/LoggingScreen/LoggingScreen.test.tsx

@@ -11,7 +11,6 @@ const baseProps = {
   onExport: vi.fn(),
   autoScroll: true,
   onToggleAutoScroll: vi.fn(),
-  onCopyAll: vi.fn(),
 };
 
 describe("LoggingScreen", () => {
@@ -45,24 +44,13 @@ describe("LoggingScreen", () => {
     expect(onClear).toHaveBeenCalledTimes(1);
   });
 
-  it("invokes onCopyAll when Copy All is clicked", async () => {
-    const user = userEvent.setup();
-    const onCopyAll = vi.fn();
-    const entries = [
-      { receivedAt: new Date(), params: { level: "info" as const, data: "x" } },
-    ];
-    renderWithMantine(
-      <LoggingScreen {...baseProps} entries={entries} onCopyAll={onCopyAll} />,
-    );
-    await user.click(screen.getByRole("button", { name: "Copy All" }));
-    expect(onCopyAll).toHaveBeenCalledTimes(1);
-  });
-
-  it("disables Clear / Export / Copy All when there are no entries", () => {
+  it("disables Clear / Export when there are no entries", () => {
     renderWithMantine(<LoggingScreen {...baseProps} />);
     expect(screen.getByRole("button", { name: "Clear" })).toBeDisabled();
     expect(screen.getByRole("button", { name: "Export" })).toBeDisabled();
-    expect(screen.getByRole("button", { name: "Copy All" })).toBeDisabled();
+    expect(
+      screen.queryByRole("button", { name: "Copy All" }),
+    ).not.toBeInTheDocument();
   });
 
   it("toggles a single level on level button click", async () => {

clients/web/src/components/screens/LoggingScreen/LoggingScreen.tsx

@@ -13,7 +13,6 @@ export interface LoggingScreenProps {
   onExport: () => void;
   autoScroll: boolean;
   onToggleAutoScroll: () => void;
-  onCopyAll: () => void;
 }
 
 const ALL_LEVELS_VISIBLE: Record<LoggingLevel, boolean> = {
@@ -63,7 +62,6 @@ export function LoggingScreen({
   onExport,
   autoScroll,
   onToggleAutoScroll,
-  onCopyAll,
 }: LoggingScreenProps) {
   const [filterText, setFilterText] = useState("");
   const [visibleLevels, setVisibleLevels] =
@@ -99,7 +97,6 @@ export function LoggingScreen({
         visibleLevels={visibleLevels}
         autoScroll={autoScroll}
         onToggleAutoScroll={onToggleAutoScroll}
-        onCopyAll={onCopyAll}
         onClear={onClear}
         onExport={onExport}
       />

clients/web/src/components/views/InspectorView/InspectorView.stories.tsx

@@ -350,7 +350,6 @@ const meta: Meta<typeof InspectorView> = {
     onSetLogLevel: fn(),
     onClearLogs: fn(),
     onExportLogs: fn(),
-    onCopyAllLogs: fn(),
     onClearHistory: fn(),
     onExportHistory: fn(),
     onReplayHistory: fn(),

clients/web/src/components/views/InspectorView/InspectorView.test.tsx

@@ -79,7 +79,6 @@ function makeProps(
     onSetLogLevel: vi.fn(),
     onClearLogs: vi.fn(),
     onExportLogs: vi.fn(),
-    onCopyAllLogs: vi.fn(),
     onClearHistory: vi.fn(),
     onExportHistory: vi.fn(),
     onReplayHistory: vi.fn(),

clients/web/src/components/views/InspectorView/InspectorView.tsx

@@ -199,7 +199,6 @@ export interface InspectorViewProps {
   onSetLogLevel: (level: LoggingLevel) => void;
   onClearLogs: () => void;
   onExportLogs: () => void;
-  onCopyAllLogs: () => void;
 
   onClearHistory: () => void;
   onExportHistory: () => void;
@@ -270,7 +269,6 @@ export function InspectorView({
   onSetLogLevel,
   onClearLogs,
   onExportLogs,
-  onCopyAllLogs,
   onClearHistory,
   onExportHistory,
   onReplayHistory,
@@ -453,7 +451,6 @@ export function InspectorView({
               onExport={onExportLogs}
               autoScroll={autoScroll}
               onToggleAutoScroll={() => setAutoScroll((prev) => !prev)}
-              onCopyAll={onCopyAllLogs}
             />
           </ScreenStage>
           <ScreenStage active={activeTab === "History"}>