repoburg 1.3.201 → 1.3.204

package/agent-pane/frontend/src/components/CollapsibleToolCalls.test.tsx

@@ -1,49 +0,0 @@
-import { describe, it, expect, afterEach } from 'vitest';
-import { render, screen, fireEvent, cleanup } from '@testing-library/react';
-import CollapsibleToolCalls, { isToolItem } from './CollapsibleToolCalls.tsx';
-import { makeItem, makeTurn } from '../test/fixtures.ts';
-
-describe('isToolItem', () => {
-  it('returns true for tool types', () => {
-    expect(isToolItem(makeItem('i1', 't1', 'command_execution'))).toBe(true);
-    expect(isToolItem(makeItem('i2', 't1', 'file_change'))).toBe(true);
-    expect(isToolItem(makeItem('i3', 't1', 'mcp_tool_call'))).toBe(true);
-    expect(isToolItem(makeItem('i4', 't1', 'web_search'))).toBe(true);
-  });
-
-  it('returns false for non-tool types', () => {
-    expect(isToolItem(makeItem('i1', 't1', 'agent_message'))).toBe(false);
-    expect(isToolItem(makeItem('i2', 't1', 'reasoning'))).toBe(false);
-    expect(isToolItem(makeItem('i3', 't1', 'todo_list'))).toBe(false);
-    expect(isToolItem(makeItem('i4', 't1', 'error'))).toBe(false);
-  });
-});
-
-describe('CollapsibleToolCalls', () => {
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('returns null when items is empty', () => {
-    const { container } = render(<CollapsibleToolCalls items={[]} turn={makeTurn('t1', 's1')} />);
-    expect(container.firstChild).toBeNull();
-  });
-
-  it('shows collapsed state by default (no item content visible)', () => {
-    const item = makeItem('i1', 't1', 'command_execution');
-    item.data = { command: 'ls', aggregated_output: '' };
-    render(<CollapsibleToolCalls items={[item]} turn={makeTurn('t1', 's1')} />);
-    expect(screen.getByText(/1 tool call/)).toBeInTheDocument();
-    // Item card content should NOT be rendered when collapsed
-    expect(screen.queryByText('Command')).not.toBeInTheDocument();
-  });
-
-  it('expands on header click and shows item cards', () => {
-    const item = makeItem('i1', 't1', 'command_execution');
-    item.data = { command: 'echo hello', aggregated_output: 'hello' };
-    render(<CollapsibleToolCalls items={[item]} turn={makeTurn('t1', 's1')} />);
-    const header = document.querySelector('.tool-calls-header')!;
-    fireEvent.click(header);
-    expect(screen.getByText('Command')).toBeInTheDocument();
-  });
-});

package/agent-pane/frontend/src/components/CollapsibleToolCalls.tsx

@@ -1,56 +0,0 @@
-import { useState } from 'react';
-import { Wrench, ChevronRight } from 'lucide-react';
-import { MemoItemCard } from './TurnContent.tsx';
-import type { Item, Turn, DiscussionThread } from '../types/index.ts';
-
-const TOOL_TYPES = new Set(['command_execution', 'file_change', 'mcp_tool_call', 'web_search']);
-
-export function isToolItem(item: Item): boolean {
-  return TOOL_TYPES.has(item.type);
-}
-
-interface Props {
-  items: Item[];
-  runtimeType?: string;
-  turn: Turn;
-  threads?: DiscussionThread[];
-}
-
-export default function CollapsibleToolCalls({ items, runtimeType, turn, threads = [] }: Props) {
-  const [expanded, setExpanded] = useState(false);
-
-  if (items.length === 0) return null;
-
-  const count = items.length;
-
-  return (
-    <div className="tool-calls">
-      <div className="tool-calls-header" onClick={() => setExpanded((e) => !e)}>
-        <span className="tool-calls-icon">
-          <Wrench size={12} />
-        </span>
-        <span className="tool-calls-summary">
-          {count} tool call{count > 1 ? 's' : ''}
-        </span>
-        <ChevronRight
-          size={14}
-          className="tool-calls-chevron"
-          style={{ transform: expanded ? 'rotate(90deg)' : 'none' }}
-        />
-      </div>
-      {expanded && (
-        <div className="tool-calls-body">
-          {items.map((item) => (
-            <MemoItemCard
-              key={item.id}
-              item={item}
-              runtimeType={runtimeType}
-              turn={turn}
-              threads={threads}
-            />
-          ))}
-        </div>
-      )}
-    </div>
-  );
-}

package/agent-pane/frontend/src/components/CommandInputBar.test.tsx

@@ -1,133 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { render, screen, fireEvent, cleanup } from '@testing-library/react';
-
-// Mock useProvider to avoid async fetch in tests
-vi.mock('../hooks/useProvider.ts', () => ({
-  useProvider: () => ({
-    provider: { id: 'codex', label: 'Codex', available: true, capabilities: {
-      dynamicModels: true, skills: true, plugins: true, mcpServers: true,
-      imageAttachments: true, sandbox: true, webSearch: true, networkAccess: true, fileReferences: true,
-    } },
-    capabilities: {
-      dynamicModels: true, skills: true, plugins: true, mcpServers: true,
-      imageAttachments: true, sandbox: true, webSearch: true, networkAccess: true, fileReferences: true,
-    },
-    loading: false,
-  }),
-}));
-
-import CommandInputBar from './CommandInputBar.tsx';
-import { useStore } from '../store/store.ts';
-import { resetStore } from '../test/component-helpers.ts';
-import { makeSession, makeDetail } from '../test/fixtures.ts';
-
-describe('CommandInputBar', () => {
-  beforeEach(() => {
-    vi.stubGlobal('requestAnimationFrame', (cb: () => void) => cb());
-    resetStore();
-  });
-  afterEach(() => {
-    cleanup();
-    vi.unstubAllGlobals();
-  });
-
-  it('shows "Select or create a session…" placeholder when no active session', () => {
-    render(<CommandInputBar />);
-    expect(screen.getByPlaceholderText('Select or create a session…')).toBeInTheDocument();
-  });
-
-  it('shows "Agent is running…" placeholder when is_running=1', () => {
-    const detail = makeDetail('s1', [], { is_running: 1 });
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    render(<CommandInputBar />);
-    expect(screen.getByPlaceholderText('Agent is running…')).toBeInTheDocument();
-  });
-
-  it('shows "Type a message…" placeholder when idle', () => {
-    const detail = makeDetail('s1', []);
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    render(<CommandInputBar />);
-    const textarea = screen.getByRole('textbox') as HTMLTextAreaElement;
-    expect(textarea.placeholder).toMatch(/Type a message/);
-  });
-
-  it('disables textarea when no active session', () => {
-    render(<CommandInputBar />);
-    expect(screen.getByRole('textbox')).toBeDisabled();
-  });
-
-  it('sends prompt on Cmd+Enter and clears input', async () => {
-    const detail = makeDetail('s1', []);
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    const submitPrompt = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ submitPrompt });
-
-    render(<CommandInputBar />);
-    const textarea = screen.getByRole('textbox') as HTMLTextAreaElement;
-    fireEvent.change(textarea, { target: { value: 'hello world' } });
-    fireEvent.keyDown(textarea, { key: 'Enter', metaKey: true });
-
-    expect(submitPrompt).toHaveBeenCalledWith('hello world', undefined);
-  });
-
-  it('send button calls submitPrompt with trimmed value', async () => {
-    const detail = makeDetail('s1', []);
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    const submitPrompt = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ submitPrompt });
-
-    render(<CommandInputBar />);
-    const textarea = screen.getByRole('textbox') as HTMLTextAreaElement;
-    fireEvent.change(textarea, { target: { value: '  test  ' } });
-    const sendBtn = document.querySelector('.cmd-send-btn') as HTMLElement;
-    fireEvent.click(sendBtn);
-
-    expect(submitPrompt).toHaveBeenCalledWith('test', undefined);
-  });
-
-  it('send button is disabled when input is empty', () => {
-    const detail = makeDetail('s1', []);
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    render(<CommandInputBar />);
-    const sendBtn = document.querySelector('.cmd-send-btn') as HTMLButtonElement;
-    expect(sendBtn.disabled).toBe(true);
-  });
-
-  it('stop button is visible when running and calls cancelTurn', async () => {
-    const detail = makeDetail('s1', [], { is_running: 1 });
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    const cancelTurn = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ cancelTurn });
-
-    render(<CommandInputBar />);
-    const stopBtn = document.querySelector('.cmd-stop-btn') as HTMLElement;
-    expect(stopBtn).toBeInTheDocument();
-    fireEvent.click(stopBtn);
-    expect(cancelTurn).toHaveBeenCalled();
-  });
-
-  it('ESC key calls cancelTurn when running', () => {
-    const detail = makeDetail('s1', [], { is_running: 1 });
-    resetStore({ activeSessionId: 's1', sessionDetails: { s1: detail }, sessions: [makeSession('s1')] });
-    const cancelTurn = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ cancelTurn });
-
-    render(<CommandInputBar />);
-    fireEvent.keyDown(window, { key: 'Escape' });
-    expect(cancelTurn).toHaveBeenCalled();
-  });
-
-  it('restoredPrompt sets textarea value and clears store', () => {
-    const detail = makeDetail('s1', []);
-    resetStore({
-      activeSessionId: 's1',
-      sessionDetails: { s1: detail },
-      sessions: [makeSession('s1')],
-      restoredPrompt: 'restored text',
-    });
-    render(<CommandInputBar />);
-    const textarea = screen.getByRole('textbox') as HTMLTextAreaElement;
-    expect(textarea.value).toBe('restored text');
-    expect(useStore.getState().restoredPrompt).toBeNull();
-  });
-});