repoburg 1.3.202 → 1.3.204

package/agent-pane/frontend/package.json

@@ -1,35 +0,0 @@
-{
-  "name": "agent-pane-frontend",
-  "version": "0.1.0",
-  "private": true,
-  "type": "module",
-  "scripts": {
-    "dev": "vite",
-    "build": "tsc --noEmit && vite build",
-    "preview": "vite preview",
-    "typecheck": "tsc --noEmit",
-    "test": "vitest run",
-    "test:watch": "vitest"
-  },
-  "dependencies": {
-    "dompurify": "^3.2.0",
-    "lucide-react": "^0.469.0",
-    "marked": "^15.0.0",
-    "react": "^18.3.1",
-    "react-dom": "^18.3.1",
-    "zustand": "^5.0.0"
-  },
-  "devDependencies": {
-    "@testing-library/jest-dom": "^6.4.0",
-    "@testing-library/react": "^16.1.0",
-    "@types/dompurify": "^3.0.5",
-    "@types/react": "^18.3.0",
-    "@types/react-dom": "^18.3.0",
-    "@vitejs/plugin-react": "^4.3.0",
-    "@vitest/coverage-v8": "^3.2.7",
-    "jsdom": "^25.0.0",
-    "typescript": "^5.7.0",
-    "vite": "^6.0.0",
-    "vitest": "^3.2.4"
-  }
-}

package/agent-pane/frontend/src/App.test.tsx

@@ -1,74 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { render, screen, fireEvent, cleanup } from '@testing-library/react';
-import App from './App.tsx';
-import { resetStore } from './test/component-helpers.ts';
-
-// Mock child components that have side effects (WebSocket, init)
-vi.mock('./api/websocket.ts', () => ({
-  connectWebSocket: vi.fn(),
-  disconnectWebSocket: vi.fn(),
-}));
-
-// Mock API modules to prevent real network calls
-vi.mock('./api/sessions.ts', () => ({
-  fetchSessions: vi.fn(),
-  fetchSessionDetail: vi.fn(),
-  createSession: vi.fn(),
-  deleteSession: vi.fn(),
-  submitPrompt: vi.fn(),
-  cancelTurn: vi.fn(),
-  renameSession: vi.fn(),
-  markFinished: vi.fn(),
-  reopenSession: vi.fn(),
-  clearUnreviewed: vi.fn(),
-  rewindSession: vi.fn(),
-  branchSession: vi.fn(),
-  updateModelConfig: vi.fn(),
-}));
-
-vi.mock('./api/workspaces.ts', () => ({
-  fetchWorkspaces: vi.fn(),
-  createWorkspace: vi.fn(),
-  hideWorkspace: vi.fn(),
-  deleteWorkspace: vi.fn(),
-  pickDirectory: vi.fn(),
-}));
-
-describe('App', () => {
-  beforeEach(() => {
-    // Mock init as no-op to prevent real store init from running
-    resetStore({ init: vi.fn().mockResolvedValue(undefined) });
-  });
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('shows "Loading sessions…" when loading and no sessions', () => {
-    resetStore({ loading: true, sessions: [] });
-    render(<App />);
-    expect(screen.getByText('Loading sessions…')).toBeInTheDocument();
-  });
-
-  it('shows error message when error and no sessions', () => {
-    const init = vi.fn().mockResolvedValue(undefined); // no-op init
-    resetStore({ loading: false, error: 'Connection refused', sessions: [], init });
-    render(<App />);
-    expect(screen.getByText('Error: Connection refused')).toBeInTheDocument();
-  });
-
-  it('renders IdeWorkspace when sessions loaded', () => {
-    const session = { id: 's1', title: 'Test', runtime_type: 'codex', model: null, model_reasoning_effort: null, cwd: null, external_thread_id: null, status: 'active' as const, is_running: 0, lifecycle: 'working', has_unreviewed: 0, created_at: '2025-01-01T00:00:00Z', updated_at: '2025-01-01T00:00:00Z' };
-    resetStore({ loading: false, sessions: [session], openTabs: ['s1'], activeSessionId: 's1' });
-    render(<App />);
-    // IdeWorkspace renders the workspace sidebar
-    expect(document.querySelector('.ide-root')).toBeInTheDocument();
-  });
-
-  it('Cmd+K opens command palette', () => {
-    const setShowCommandPalette = vi.fn();
-    resetStore({ loading: false, sessions: [], setShowCommandPalette });
-    render(<App />);
-    fireEvent.keyDown(window, { key: 'k', metaKey: true });
-    expect(setShowCommandPalette).toHaveBeenCalledWith(true);
-  });
-});

package/agent-pane/frontend/src/App.tsx

@@ -1,44 +0,0 @@
-import { useEffect } from 'react';
-import { useStore } from './store/store.ts';
-import IdeWorkspace from './components/IdeWorkspace.tsx';
-import NewSessionDialog from './components/NewSessionDialog.tsx';
-import CommandPalette from './components/CommandPalette.tsx';
-import Toast from './components/Toast.tsx';
-import AppInitializer from './AppInitializer.tsx';
-
-export default function App() {
-  const loading = useStore((s) => s.loading);
-  const error = useStore((s) => s.error);
-  const sessions = useStore((s) => s.sessions);
-  const setShowCommandPalette = useStore((s) => s.setShowCommandPalette);
-
-  // Cmd+K / Ctrl+K listener
-  useEffect(() => {
-    function handleKeyDown(e: KeyboardEvent) {
-      if ((e.metaKey || e.ctrlKey) && e.key === 'k') {
-        e.preventDefault();
-        setShowCommandPalette(true);
-      }
-    }
-    window.addEventListener('keydown', handleKeyDown);
-    return () => window.removeEventListener('keydown', handleKeyDown);
-  }, [setShowCommandPalette]);
-
-  return (
-    <>
-      <AppInitializer />
-      {loading && sessions.length === 0 ? (
-        <div className="empty-state">Loading sessions…</div>
-      ) : error && sessions.length === 0 ? (
-        <div className="empty-state">Error: {error}</div>
-      ) : (
-        <>
-          <IdeWorkspace />
-          <NewSessionDialog />
-          <CommandPalette />
-          <Toast />
-        </>
-      )}
-    </>
-  );
-}

package/agent-pane/frontend/src/AppInitializer.test.tsx

@@ -1,142 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { render, cleanup } from '@testing-library/react';
-import AppInitializer from './AppInitializer.tsx';
-import { useStore } from './store/store.ts';
-import { resetStore } from './test/component-helpers.ts';
-
-// Mock WebSocket module
-vi.mock('./api/websocket.ts', () => ({
-  connectWebSocket: vi.fn(),
-  disconnectWebSocket: vi.fn(),
-}));
-
-// Mock API modules
-vi.mock('./api/sessions.ts', () => ({
-  fetchSessions: vi.fn(),
-  fetchSessionDetail: vi.fn(),
-  createSession: vi.fn(),
-  deleteSession: vi.fn(),
-  submitPrompt: vi.fn(),
-  cancelTurn: vi.fn(),
-  renameSession: vi.fn(),
-  markFinished: vi.fn(),
-  reopenSession: vi.fn(),
-  clearUnreviewed: vi.fn(),
-  rewindSession: vi.fn(),
-  branchSession: vi.fn(),
-  updateModelConfig: vi.fn(),
-}));
-
-vi.mock('./api/workspaces.ts', () => ({
-  fetchWorkspaces: vi.fn(),
-  createWorkspace: vi.fn(),
-  hideWorkspace: vi.fn(),
-  deleteWorkspace: vi.fn(),
-  pickDirectory: vi.fn(),
-}));
-
-/** Helper: set document.visibilityState then dispatch visibilitychange */
-function dispatchVisibility(state: string) {
-  Object.defineProperty(document, 'visibilityState', {
-    value: state,
-    configurable: true,
-  });
-  document.dispatchEvent(new Event('visibilitychange'));
-}
-
-describe('AppInitializer', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    resetStore();
-    // Restore default visibility
-    Object.defineProperty(document, 'visibilityState', {
-      value: 'visible',
-      configurable: true,
-    });
-  });
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('calls init on mount', () => {
-    const init = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ init });
-    render(<AppInitializer />);
-    expect(init).toHaveBeenCalled();
-  });
-
-  it('calls connectWebSocket on mount', async () => {
-    const { connectWebSocket } = await import('./api/websocket.ts');
-    render(<AppInitializer />);
-    expect(connectWebSocket).toHaveBeenCalled();
-  });
-
-  it('cleanup calls disconnectWebSocket', async () => {
-    const { disconnectWebSocket } = await import('./api/websocket.ts');
-    const init = vi.fn().mockResolvedValue(undefined);
-    useStore.setState({ init });
-    const { unmount } = render(<AppInitializer />);
-    unmount();
-    expect(disconnectWebSocket).toHaveBeenCalled();
-  });
-
-  it('visibilitychange hidden: does not reconnect or refresh', async () => {
-    const { connectWebSocket } = await import('./api/websocket.ts');
-    const refreshActiveSession = vi.fn();
-    const addLogEntry = vi.fn();
-    useStore.setState({ refreshActiveSession, addLogEntry });
-    render(<AppInitializer />);
-    vi.clearAllMocks();
-    dispatchVisibility('hidden');
-    expect(connectWebSocket).not.toHaveBeenCalled();
-    expect(refreshActiveSession).not.toHaveBeenCalled();
-  });
-
-  it('visibilitychange visible with no active session: no reconnect, no refresh', async () => {
-    const { connectWebSocket } = await import('./api/websocket.ts');
-    const refreshActiveSession = vi.fn();
-    useStore.setState({ activeSessionId: null, refreshActiveSession });
-    render(<AppInitializer />);
-    vi.clearAllMocks();
-    dispatchVisibility('visible');
-    expect(connectWebSocket).not.toHaveBeenCalled();
-    expect(refreshActiveSession).not.toHaveBeenCalled();
-  });
-
-  it('visibilitychange visible with WS down: reconnects, no refresh', async () => {
-    const { connectWebSocket } = await import('./api/websocket.ts');
-    const refreshActiveSession = vi.fn();
-    useStore.setState({
-      activeSessionId: 's1',
-      wsConnected: false,
-      sessionDetails: {},
-      refreshActiveSession,
-    });
-    render(<AppInitializer />);
-    vi.clearAllMocks();
-    dispatchVisibility('visible');
-    expect(connectWebSocket).toHaveBeenCalledTimes(1);
-    expect(refreshActiveSession).not.toHaveBeenCalled();
-  });
-
-  it('visibilitychange visible with running session: refreshes and logs', async () => {
-    const { connectWebSocket } = await import('./api/websocket.ts');
-    const refreshActiveSession = vi.fn();
-    const addLogEntry = vi.fn();
-    useStore.setState({
-      activeSessionId: 's1',
-      wsConnected: true,
-      sessionDetails: {
-        s1: { id: 's1', is_running: 1 } as never,
-      },
-      refreshActiveSession,
-      addLogEntry,
-    });
-    render(<AppInitializer />);
-    vi.clearAllMocks();
-    dispatchVisibility('visible');
-    expect(refreshActiveSession).toHaveBeenCalledTimes(1);
-    expect(addLogEntry).toHaveBeenCalledWith('INFO', 'Tab returned — reconciling session state');
-    expect(connectWebSocket).not.toHaveBeenCalled();
-  });
-});

package/agent-pane/frontend/src/AppInitializer.tsx

@@ -1,46 +0,0 @@
-import { useEffect } from 'react';
-import { useStore } from './store/store.ts';
-import { connectWebSocket, disconnectWebSocket } from './api/websocket.ts';
-
-export default function AppInitializer() {
-  const init = useStore((s) => s.init);
-
-  useEffect(() => {
-    // Fetch initial sessions from REST API
-    init();
-
-    // Connect WebSocket for live streaming
-    connectWebSocket();
-
-    // visibilitychange — reconcile when tab returns to foreground
-    function handleVisibilityChange() {
-      if (document.visibilityState !== 'visible') return;
-
-      const state = useStore.getState();
-      const activeId = state.activeSessionId;
-      if (!activeId) return;
-
-      const detail = state.sessionDetails[activeId];
-
-      // If WS is not connected, reconnect
-      if (!state.wsConnected) {
-        connectWebSocket();
-      }
-
-      // If session was running, re-fetch to catch missed events
-      if (detail && detail.is_running === 1) {
-        state.refreshActiveSession();
-        state.addLogEntry('INFO', 'Tab returned — reconciling session state');
-      }
-    }
-
-    document.addEventListener('visibilitychange', handleVisibilityChange);
-
-    return () => {
-      disconnectWebSocket();
-      document.removeEventListener('visibilitychange', handleVisibilityChange);
-    };
-  }, [init]);
-
-  return null;
-}

package/agent-pane/frontend/src/__tests__/ThreadsPanel.test.tsx

@@ -1,192 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { render, screen, fireEvent, cleanup } from '@testing-library/react';
-import ThreadsPanel from '../components/panels/ThreadsPanel.tsx';
-import { useStore } from '../store/store.ts';
-import { makeSession, makeDetail, makeTurn } from '../test/fixtures.ts';
-import type { DiscussionThread } from '../types/index.ts';
-
-const NOW = '2025-01-01T00:00:00Z';
-
-function makeThread(id: string, sessionId: string, overrides: Partial<DiscussionThread> = {}): DiscussionThread {
-  return {
-    id,
-    session_id: sessionId,
-    item_id: 'i1',
-    turn_id: 't1',
-    start_offset: 0,
-    end_offset: 10,
-    selected_text: 'Some selected text',
-    status: 'open',
-    resolved_at: null,
-    created_at: NOW,
-    updated_at: NOW,
-    external_thread_id: null,
-    ...overrides,
-  };
-}
-
-describe('ThreadsPanel', () => {
-  beforeEach(() => {
-    // Reset store to a clean state with a session
-    useStore.setState({
-      sessions: [],
-      sessionDetails: {},
-      activeSessionId: null,
-      openTabs: [],
-      activeTurnIndex: 0,
-      submittingPrompt: null,
-      restoredPrompt: null,
-      activeRightTab: 'threads',
-    });
-  });
-
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('shows empty state when no session selected', () => {
-    render(<ThreadsPanel />);
-    expect(screen.getByText('No session selected')).toBeInTheDocument();
-  });
-
-  it('shows empty state with instructions when no threads exist', () => {
-    const session = makeSession('s1');
-    const detail = makeDetail('s1', []);
-    useStore.setState({
-      sessions: [session],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    expect(screen.getByText('No threads yet.')).toBeInTheDocument();
-    expect(screen.getByText(/Select text in an agent response/)).toBeInTheDocument();
-  });
-
-  it('renders an open thread card with quoted text', () => {
-    const thread = makeThread('th1', 's1');
-    const detail = makeDetail('s1', [], { threads: [thread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    expect(screen.getByText(/Some selected text/)).toBeInTheDocument();
-    expect(screen.getByText('Open')).toBeInTheDocument();
-  });
-
-  it('shows filter tabs with counts', () => {
-    const openThread = makeThread('th1', 's1', { status: 'open' });
-    const resolvedThread = makeThread('th2', 's1', { status: 'resolved' });
-    const detail = makeDetail('s1', [], { threads: [openThread, resolvedThread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    expect(screen.getByText(/Open \(1\)/)).toBeInTheDocument();
-    expect(screen.getByText(/Resolved \(1\)/)).toBeInTheDocument();
-    expect(screen.getByText(/All \(2\)/)).toBeInTheDocument();
-  });
-
-  it('filters threads by resolved status', () => {
-    const openThread = makeThread('th1', 's1', { status: 'open', selected_text: 'Open thread text' });
-    const resolvedThread = makeThread('th2', 's1', { status: 'resolved', selected_text: 'Resolved thread text' });
-    const detail = makeDetail('s1', [], { threads: [openThread, resolvedThread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    // Click "Resolved" filter
-    fireEvent.click(screen.getByText(/Resolved/));
-    expect(screen.getByText(/Resolved thread text/)).toBeInTheDocument();
-    expect(screen.queryByText(/Open thread text/)).not.toBeInTheDocument();
-  });
-
-  it('shows thread conversation history from turns', () => {
-    const thread = makeThread('th1', 's1');
-    const threadTurn = makeTurn('tt1', 's1', {
-      thread_id: 'th1',
-      user_prompt: '[Quoting from your previous response]:\n> Some text\n\nWhat is this?',
-      status: 'completed',
-      items: [{
-        id: 'ti1',
-        turn_id: 'tt1',
-        external_item_id: 'ext-ti1',
-        type: 'agent_message' as const,
-        data: { text: 'This is an explanation.' },
-        status: 'completed' as const,
-        created_at: NOW,
-        updated_at: NOW,
-      }],
-    });
-    const detail = makeDetail('s1', [threadTurn], { threads: [thread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    // User message (with quote prefix stripped)
-    expect(screen.getByText('What is this?')).toBeInTheDocument();
-    // AI response
-    expect(screen.getByText('This is an explanation.')).toBeInTheDocument();
-  });
-
-  it('shows follow-up input for open threads', () => {
-    const thread = makeThread('th1', 's1', { status: 'open' });
-    const detail = makeDetail('s1', [], { threads: [thread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    expect(screen.getByPlaceholderText('Type a follow-up…')).toBeInTheDocument();
-  });
-
-  it('shows resolve and delete actions for open threads', () => {
-    const thread = makeThread('th1', 's1', { status: 'open' });
-    const detail = makeDetail('s1', [], { threads: [thread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    expect(screen.getByText('Resolve')).toBeInTheDocument();
-    expect(screen.getByText('Delete')).toBeInTheDocument();
-  });
-
-  it('shows reopen action for resolved threads', () => {
-    const thread = makeThread('th1', 's1', { status: 'resolved' });
-    const detail = makeDetail('s1', [], { threads: [thread] });
-
-    useStore.setState({
-      sessions: [makeSession('s1')],
-      sessionDetails: { s1: detail },
-      activeSessionId: 's1',
-    });
-
-    render(<ThreadsPanel />);
-    // Switch to "All" filter to see resolved threads
-    fireEvent.click(screen.getByText(/All/));
-    expect(screen.getByText('Reopen')).toBeInTheDocument();
-  });
-});

package/agent-pane/frontend/src/api/client.test.ts

@@ -1,126 +0,0 @@
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { apiGet, apiPost, apiDelete, apiPatch } from './client.ts';
-
-describe('apiGet', () => {
-  beforeEach(() => {
-    global.fetch = vi.fn();
-  });
-
-  it('sends GET request and returns parsed JSON', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ id: '123' }),
-    });
-
-    const result = await apiGet('/sessions');
-    expect(result).toEqual({ id: '123' });
-    expect(global.fetch).toHaveBeenCalledWith('/api/sessions', {
-      method: 'GET',
-      headers: undefined,
-      body: undefined,
-    });
-  });
-});
-
-describe('apiPost', () => {
-  beforeEach(() => {
-    global.fetch = vi.fn();
-  });
-
-  it('sends POST with JSON body and Content-Type header', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ success: true }),
-    });
-
-    const result = await apiPost('/sessions', { title: 'Test' });
-    expect(result).toEqual({ success: true });
-    expect(global.fetch).toHaveBeenCalledWith('/api/sessions', {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ title: 'Test' }),
-    });
-  });
-
-  it('sends POST without body when body is undefined', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ ok: true }),
-    });
-
-    await apiPost('/sessions/1/cancel');
-    expect(global.fetch).toHaveBeenCalledWith('/api/sessions/1/cancel', {
-      method: 'POST',
-      headers: undefined,
-      body: undefined,
-    });
-  });
-});
-
-describe('apiPatch', () => {
-  beforeEach(() => {
-    global.fetch = vi.fn();
-  });
-
-  it('sends PATCH with JSON body', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ id: '1', title: 'Updated' }),
-    });
-
-    const result = await apiPatch('/sessions/1', { title: 'Updated' });
-    expect(result).toEqual({ id: '1', title: 'Updated' });
-    expect(global.fetch).toHaveBeenCalledWith('/api/sessions/1', {
-      method: 'PATCH',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ title: 'Updated' }),
-    });
-  });
-});
-
-describe('apiDelete', () => {
-  beforeEach(() => {
-    global.fetch = vi.fn();
-  });
-
-  it('sends DELETE request', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ success: true }),
-    });
-
-    const result = await apiDelete('/sessions/1');
-    expect(result).toEqual({ success: true });
-    expect(global.fetch).toHaveBeenCalledWith('/api/sessions/1', {
-      method: 'DELETE',
-      headers: undefined,
-      body: undefined,
-    });
-  });
-});
-
-describe('error handling', () => {
-  beforeEach(() => {
-    global.fetch = vi.fn();
-  });
-
-  it('throws with method, path, and status on non-ok response', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: false,
-      status: 404,
-      statusText: 'Not Found',
-    });
-
-    await expect(apiGet('/sessions/missing')).rejects.toThrow('GET /sessions/missing failed: 404 Not Found');
-  });
-
-  it('throws for POST error', async () => {
-    (global.fetch as ReturnType<typeof vi.fn>).mockResolvedValue({
-      ok: false,
-      status: 500,
-      statusText: 'Internal Server Error',
-    });
-
-    await expect(apiPost('/sessions', {})).rejects.toThrow('POST /sessions failed: 500 Internal Server Error');
-  });
-});