repoburg 1.3.202 → 1.3.204

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

@@ -1,297 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { render, screen, fireEvent, cleanup } from '@testing-library/react';
-import WorkspaceSidebar from './WorkspaceSidebar.tsx';
-import { resetStore } from '../test/component-helpers.ts';
-import { makeSession, makeWorkspace } from '../test/fixtures.ts';
-
-describe('WorkspaceSidebar', () => {
-  beforeEach(() => {
-    resetStore();
-  });
-  afterEach(() => {
-    cleanup();
-  });
-
-  it('shows empty state when no workspaces', () => {
-    render(<WorkspaceSidebar />);
-    expect(screen.getByText(/No workspaces yet/)).toBeInTheDocument();
-  });
-
-  it('renders workspace groups with session counts', () => {
-    const ws = makeWorkspace('w1', '/projects/app');
-    const s1 = makeSession('s1', { cwd: '/projects/app', title: 'Session A' });
-    resetStore({ workspaces: [ws], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    expect(screen.getByText('app')).toBeInTheDocument(); // label from path basename
-    expect(screen.getByText('Session A')).toBeInTheDocument();
-    expect(screen.getByText('1')).toBeInTheDocument(); // session count
-  });
-
-  it('active session has active class', () => {
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Active One' });
-    resetStore({ workspaces: [ws], sessions: [s1], activeSessionId: 's1' });
-    render(<WorkspaceSidebar />);
-    const treeItem = screen.getByText('Active One').closest('.tree-item');
-    expect(treeItem).toHaveClass('active');
-  });
-
-  it('click on session calls selectSession', () => {
-    const selectSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Click Me' });
-    resetStore({ workspaces: [ws], sessions: [s1], selectSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.click(screen.getByText('Click Me'));
-    expect(selectSession).toHaveBeenCalledWith('s1');
-  });
-
-  it('collapse section header toggles section', () => {
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Session 1' });
-    resetStore({ workspaces: [ws], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    const header = document.querySelector('.tree-section-header')!;
-    fireEvent.click(header);
-    expect(document.querySelector('.tree-section')!.classList.contains('collapsed')).toBe(true);
-  });
-
-  it('plus button on workspace calls createSessionInWorkspace', () => {
-    const createSessionInWorkspace = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Session 1' });
-    resetStore({ workspaces: [ws], sessions: [s1], createSessionInWorkspace });
-    render(<WorkspaceSidebar />);
-    const addBtn = document.querySelector('.tree-section-add')!;
-    fireEvent.click(addBtn);
-    expect(createSessionInWorkspace).toHaveBeenCalledWith('/app');
-  });
-
-  it('context menu on workspace shows New Session and Hide Workspace', () => {
-    const ws = makeWorkspace('w1', '/app');
-    resetStore({ workspaces: [ws], sessions: [] });
-    render(<WorkspaceSidebar />);
-    const header = document.querySelector('.tree-section-header')!;
-    fireEvent.contextMenu(header);
-    expect(screen.getByText('New Session')).toBeInTheDocument();
-    expect(screen.getByText('Hide Workspace')).toBeInTheDocument();
-  });
-
-  it('context menu "Hide Workspace" calls removeWorkspace', () => {
-    const removeWorkspace = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    resetStore({ workspaces: [ws], sessions: [], removeWorkspace });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(document.querySelector('.tree-section-header')!);
-    fireEvent.click(screen.getByText('Hide Workspace'));
-    expect(removeWorkspace).toHaveBeenCalledWith('/app');
-  });
-
-  it('context menu on working session shows Rename, Mark Finished, Delete', () => {
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Working Session', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('Working Session'));
-    expect(screen.getByText('Rename')).toBeInTheDocument();
-    expect(screen.getByText('Mark as Finished')).toBeInTheDocument();
-    expect(screen.getByText('Delete Session')).toBeInTheDocument();
-  });
-
-  it('context menu on finished session shows Reopen and Delete', () => {
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Finished Session', lifecycle: 'finished' });
-    resetStore({ workspaces: [ws], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('Finished Session'));
-    expect(screen.getByText('Reopen')).toBeInTheDocument();
-    expect(screen.getByText('Delete Session')).toBeInTheDocument();
-  });
-
-  it('rename: inline input appears, Enter calls renameSession', () => {
-    const renameSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Old Name', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1], renameSession });
-    render(<WorkspaceSidebar />);
-    // Open context menu and click Rename
-    fireEvent.contextMenu(screen.getByText('Old Name'));
-    fireEvent.click(screen.getByText('Rename'));
-    // Rename input appears
-    const input = document.querySelector('.tree-rename-input') as HTMLInputElement;
-    expect(input).toBeInTheDocument();
-    expect(input.value).toBe('Old Name');
-    // Type new name and press Enter
-    fireEvent.change(input, { target: { value: 'New Name' } });
-    fireEvent.keyDown(input, { key: 'Enter' });
-    expect(renameSession).toHaveBeenCalledWith('s1', 'New Name');
-  });
-
-  it('footer "Add workspace" calls pickAndAddWorkspace', () => {
-    const pickAndAddWorkspace = vi.fn().mockResolvedValue(undefined);
-    resetStore({ pickAndAddWorkspace });
-    render(<WorkspaceSidebar />);
-    fireEvent.click(screen.getByText('Add workspace'));
-    expect(pickAndAddWorkspace).toHaveBeenCalled();
-  });
-
-  // --- context menu action clicks ---
-
-  it('context menu "New Session" calls createSessionInWorkspace', () => {
-    const createSessionInWorkspace = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    resetStore({ workspaces: [ws], sessions: [], createSessionInWorkspace });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(document.querySelector('.tree-section-header')!);
-    fireEvent.click(screen.getByText('New Session'));
-    expect(createSessionInWorkspace).toHaveBeenCalledWith('/app');
-  });
-
-  it('context menu "Mark as Finished" calls markFinished', () => {
-    const markFinished = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Working', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1], markFinished });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('Working'));
-    fireEvent.click(screen.getByText('Mark as Finished'));
-    expect(markFinished).toHaveBeenCalledWith('s1');
-  });
-
-  it('context menu "Delete Session" on working session calls deleteSession', () => {
-    const deleteSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'ToDelete', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1], deleteSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('ToDelete'));
-    fireEvent.click(screen.getByText('Delete Session'));
-    expect(deleteSession).toHaveBeenCalledWith('s1');
-  });
-
-  it('context menu "Reopen" on finished session calls reopenSession', () => {
-    const reopenSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'ReopenMe', lifecycle: 'finished' });
-    resetStore({ workspaces: [ws], sessions: [s1], reopenSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('ReopenMe'));
-    fireEvent.click(screen.getByText('Reopen'));
-    expect(reopenSession).toHaveBeenCalledWith('s1');
-  });
-
-  it('context menu "Delete Session" on finished session calls deleteSession', () => {
-    const deleteSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'FinDel', lifecycle: 'finished' });
-    resetStore({ workspaces: [ws], sessions: [s1], deleteSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('FinDel'));
-    fireEvent.click(screen.getByText('Delete Session'));
-    expect(deleteSession).toHaveBeenCalledWith('s1');
-  });
-
-  // --- finished sub-group ---
-
-  it('finished sub-group toggle collapses and expands', () => {
-    const ws = makeWorkspace('w1', '/app');
-    const working = makeSession('s1', { cwd: '/app', title: 'Active', lifecycle: 'working' });
-    const finished = makeSession('s2', { cwd: '/app', title: 'Done', lifecycle: 'finished' });
-    resetStore({ workspaces: [ws], sessions: [working, finished] });
-    render(<WorkspaceSidebar />);
-    // Finished sub-group visible
-    expect(screen.getByText('Finished')).toBeInTheDocument();
-    expect(screen.getByText('Done')).toBeInTheDocument();
-    // Click finished header to collapse
-    fireEvent.click(screen.getByText('Finished'));
-    expect(screen.queryByText('Done')).not.toBeInTheDocument();
-  });
-
-  it('workspace with no sessions shows "No sessions"', () => {
-    const ws = makeWorkspace('w1', '/app');
-    resetStore({ workspaces: [ws], sessions: [] });
-    render(<WorkspaceSidebar />);
-    expect(screen.getByText('No sessions')).toBeInTheDocument();
-  });
-
-  // --- implicit groups ---
-
-  it('session with cwd not in any workspace creates implicit group', () => {
-    const s1 = makeSession('s1', { cwd: '/unknown/path', title: 'Lonely' });
-    resetStore({ workspaces: [], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    // Label from path basename
-    expect(screen.getByText('path')).toBeInTheDocument();
-    expect(screen.getByText('Lonely')).toBeInTheDocument();
-  });
-
-  it('deleted workspace excluded from implicit groups', () => {
-    const ws = makeWorkspace('w1', '/app', { deleted: 1 });
-    const s1 = makeSession('s1', { cwd: '/app', title: 'Orphan' });
-    resetStore({ workspaces: [ws], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    // Deleted workspace → no group for /app (session not shown in sidebar)
-    expect(screen.queryByText('Orphan')).not.toBeInTheDocument();
-  });
-
-  it('session with null cwd shows "Other" group', () => {
-    const s1 = makeSession('s1', { cwd: null, title: 'NoCwd' });
-    resetStore({ workspaces: [], sessions: [s1] });
-    render(<WorkspaceSidebar />);
-    expect(screen.getByText('Other')).toBeInTheDocument();
-    expect(screen.getByText('NoCwd')).toBeInTheDocument();
-  });
-
-  // --- rename edge cases ---
-
-  it('rename Escape cancels without calling renameSession', () => {
-    const renameSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'OldName', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1], renameSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('OldName'));
-    fireEvent.click(screen.getByText('Rename'));
-    const input = document.querySelector('.tree-rename-input') as HTMLInputElement;
-    fireEvent.keyDown(input, { key: 'Escape' });
-    expect(renameSession).not.toHaveBeenCalled();
-    expect(document.querySelector('.tree-rename-input')).not.toBeInTheDocument();
-  });
-
-  it('rename blur confirms and calls renameSession', () => {
-    const renameSession = vi.fn().mockResolvedValue(undefined);
-    const ws = makeWorkspace('w1', '/app');
-    const s1 = makeSession('s1', { cwd: '/app', title: 'BlurMe', lifecycle: 'working' });
-    resetStore({ workspaces: [ws], sessions: [s1], renameSession });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(screen.getByText('BlurMe'));
-    fireEvent.click(screen.getByText('Rename'));
-    const input = document.querySelector('.tree-rename-input') as HTMLInputElement;
-    fireEvent.change(input, { target: { value: 'NewBlur' } });
-    fireEvent.blur(input);
-    expect(renameSession).toHaveBeenCalledWith('s1', 'NewBlur');
-  });
-
-  // --- sidebar header + button ---
-
-  it('sidebar header "+" button calls pickAndAddWorkspace', () => {
-    const pickAndAddWorkspace = vi.fn().mockResolvedValue(undefined);
-    resetStore({ pickAndAddWorkspace });
-    render(<WorkspaceSidebar />);
-    const addBtn = document.querySelector('.sidebar-add')!;
-    fireEvent.click(addBtn);
-    expect(pickAndAddWorkspace).toHaveBeenCalled();
-  });
-
-  // --- context menu close ---
-
-  it('Escape key closes context menu', () => {
-    const ws = makeWorkspace('w1', '/app');
-    resetStore({ workspaces: [ws], sessions: [] });
-    render(<WorkspaceSidebar />);
-    fireEvent.contextMenu(document.querySelector('.tree-section-header')!);
-    expect(screen.getByText('New Session')).toBeInTheDocument();
-    fireEvent.keyDown(document, { key: 'Escape' });
-    expect(screen.queryByText('New Session')).not.toBeInTheDocument();
-  });
-});

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

@@ -1,351 +0,0 @@
-import { useState, useRef, useEffect } from 'react';
-import { useStore } from '../store/store.ts';
-import { deriveSessionStatus } from '../hooks/useSessionStatus.ts';
-import { useOutsideClick } from '../hooks/useOutsideClick.ts';
-import { useEscapeKey } from '../hooks/useEscapeKey.ts';
-import type { Session, Workspace } from '../types/index.ts';
-import StatusDot from './StatusDot.tsx';
-import RuntimeIcon from './RuntimeIcon.tsx';
-import { Plus, ChevronDown, Folder, FolderPlus, Trash2, Edit3, CheckCircle2, RotateCcw } from 'lucide-react';
-
-interface ContextMenuState {
-  x: number;
-  y: number;
-  type: 'workspace' | 'session';
-  path?: string;
-  sessionId?: string;
-  sessionTitle?: string | null;
-  sessionLifecycle?: string;
-}
-
-interface WorkspaceGroup {
-  path: string;
-  label: string;
-  sessions: Session[];
-  workspace: Workspace | null;
-}
-
-/** Build workspace groups from active workspaces + session cwd values.
- *  Deleted workspace paths are excluded from implicit groups. */
-function buildGroups(workspaces: Workspace[], sessions: Session[]): WorkspaceGroup[] {
-  const sessionByCwd = new Map<string, Session[]>();
-
-  for (const s of sessions) {
-    const key = s.cwd || '(no directory)';
-    const list = sessionByCwd.get(key) ?? [];
-    list.push(s);
-    sessionByCwd.set(key, list);
-  }
-
-  const seenPaths = new Set<string>();
-  const deletedPaths = new Set(workspaces.filter((w) => w.deleted === 1).map((w) => w.path));
-  const groups: WorkspaceGroup[] = [];
-
-  // Active explicit workspaces first (deleted = 0)
-  for (const ws of workspaces) {
-    if (ws.deleted === 1) {
-      seenPaths.add(ws.path);
-      continue;
-    }
-    seenPaths.add(ws.path);
-    groups.push({
-      path: ws.path,
-      label: ws.label || ws.path.split('/').pop() || ws.path,
-      sessions: sessionByCwd.get(ws.path) ?? [],
-      workspace: ws,
-    });
-  }
-
-  // Implicit groups — session cwd values not in any workspace (active or deleted)
-  for (const [path, sess] of sessionByCwd) {
-    if (seenPaths.has(path) || deletedPaths.has(path)) continue;
-    groups.push({
-      path,
-      label: path === '(no directory)' ? 'Other' : path.split('/').pop() || path,
-      sessions: sess,
-      workspace: null,
-    });
-  }
-
-  return groups;
-}
-
-export default function WorkspaceSidebar() {
-  const {
-    sessions,
-    workspaces,
-    activeSessionId,
-    selectSession,
-    deleteSession,
-    removeWorkspace,
-    createSessionInWorkspace,
-    pickAndAddWorkspace,
-    renameSession,
-    markFinished,
-    reopenSession,
-  } = useStore();
-  const [collapsed, setCollapsed] = useState<Set<string>>(new Set());
-  const [subCollapsed, setSubCollapsed] = useState<Set<string>>(new Set());
-  const [ctxMenu, setCtxMenu] = useState<ContextMenuState | null>(null);
-  const [renamingId, setRenamingId] = useState<string | null>(null);
-  const [renameValue, setRenameValue] = useState('');
-  const ctxRef = useRef<HTMLDivElement>(null);
-  const renameRef = useRef<HTMLInputElement>(null);
-
-  useOutsideClick(ctxRef, () => setCtxMenu(null), !!ctxMenu);
-  useEscapeKey(() => setCtxMenu(null), !!ctxMenu);
-
-  function toggleSection(path: string) {
-    setCollapsed((prev) => {
-      const next = new Set(prev);
-      if (next.has(path)) next.delete(path);
-      else next.add(path);
-      return next;
-    });
-  }
-
-  function toggleSubGroup(key: string) {
-    setSubCollapsed((prev) => {
-      const next = new Set(prev);
-      if (next.has(key)) next.delete(key);
-      else next.add(key);
-      return next;
-    });
-  }
-
-  // Focus rename input when it appears
-  useEffect(() => {
-    if (renamingId && renameRef.current) {
-      renameRef.current.focus();
-      renameRef.current.select();
-    }
-  }, [renamingId]);
-
-  function startRename(id: string, currentTitle: string) {
-    setRenamingId(id);
-    setRenameValue(currentTitle || id.slice(0, 8));
-  }
-
-  function confirmRename() {
-    if (renamingId) {
-      renameSession(renamingId, renameValue);
-    }
-    setRenamingId(null);
-  }
-
-  function cancelRename() {
-    setRenamingId(null);
-    setRenameValue('');
-  }
-
-  const groups = buildGroups(workspaces, sessions);
-
-  function renderSession(s: Session) {
-    const isActive = activeSessionId === s.id;
-    const status = deriveSessionStatus(s);
-    return (
-      <div
-        key={s.id}
-        className={`tree-item ${isActive ? 'active' : ''} ${status.isFinished ? 'tree-item-finished' : ''}`}
-        onClick={() => selectSession(s.id)}
-        onContextMenu={(e) => {
-          e.preventDefault();
-          setCtxMenu({
-            x: e.clientX,
-            y: e.clientY,
-            type: 'session',
-            sessionId: s.id,
-            sessionTitle: s.title,
-            sessionLifecycle: s.lifecycle,
-          });
-        }}
-      >
-        <RuntimeIcon iconClass={status.iconClass} iconText={status.iconText} size={16} />
-        {renamingId === s.id ? (
-          <input
-            ref={renameRef}
-            className="tree-rename-input"
-            value={renameValue}
-            onChange={(e) => setRenameValue(e.target.value)}
-            onClick={(e) => e.stopPropagation()}
-            onKeyDown={(e) => {
-              if (e.key === 'Enter') confirmRename();
-              if (e.key === 'Escape') cancelRename();
-            }}
-            onBlur={confirmRename}
-          />
-        ) : (
-          <span className="tree-label">{s.title || s.id.slice(0, 8)}</span>
-        )}
-        {status.dot && <StatusDot status={status.dot} />}
-      </div>
-    );
-  }
-
-  return (
-    <div className="ide-sidebar">
-      <div className="sidebar-header">
-        <span className="sidebar-header-title">Workspaces</span>
-        <button className="sidebar-add" title="Add workspace" onClick={() => pickAndAddWorkspace()}>
-          <Plus size={14} />
-        </button>
-      </div>
-
-      {groups.length === 0 ? (
-        <div className="empty-state" style={{ padding: '16px' }}>
-          No workspaces yet. Click + to add one.
-        </div>
-      ) : (
-        groups.map((group) => {
-          const isCollapsed = collapsed.has(group.path);
-          const workingSessions = group.sessions.filter((s) => s.lifecycle !== 'finished');
-          const finishedSessions = group.sessions.filter((s) => s.lifecycle === 'finished');
-          const finishedKey = `${group.path}:finished`;
-          const finishedCollapsed = subCollapsed.has(finishedKey);
-
-          return (
-            <div key={group.path} className={`tree-section ${isCollapsed ? 'collapsed' : ''}`}>
-              <div
-                className="tree-section-header"
-                onClick={() => toggleSection(group.path)}
-                onContextMenu={(e) => {
-                  e.preventDefault();
-                  setCtxMenu({ x: e.clientX, y: e.clientY, type: 'workspace', path: group.path });
-                }}
-              >
-                <ChevronDown size={9} className="tree-chevron" />
-                <Folder size={14} />
-                <span className="tree-section-label">{group.label}</span>
-                <button
-                  className="tree-section-add"
-                  title="New session in this workspace"
-                  onClick={(e) => {
-                    e.stopPropagation();
-                    createSessionInWorkspace(group.path);
-                  }}
-                >
-                  <Plus size={11} />
-                </button>
-                <span className="tree-count">{group.sessions.length}</span>
-              </div>
-              <div className="tree-children">
-                {/* Working sessions — rendered directly, no sub-group header */}
-                {workingSessions.length === 0 && finishedSessions.length === 0 ? (
-                  <div className="tree-empty">No sessions</div>
-                ) : (
-                  workingSessions.map(renderSession)
-                )}
-
-                {/* Finished sub-group — only show if there are finished sessions */}
-                {finishedSessions.length > 0 && (
-                  <div className={`tree-subgroup ${finishedCollapsed ? 'collapsed' : ''}`}>
-                    <div className="tree-subgroup-header" onClick={() => toggleSubGroup(finishedKey)}>
-                      <ChevronDown size={8} className="tree-chevron" />
-                      <span className="tree-subgroup-label">Finished</span>
-                      <span className="tree-subgroup-count">{finishedSessions.length}</span>
-                    </div>
-                    {!finishedCollapsed && (
-                      <div className="tree-subgroup-items">{finishedSessions.map(renderSession)}</div>
-                    )}
-                  </div>
-                )}
-              </div>
-            </div>
-          );
-        })
-      )}
-
-      <div className="sidebar-footer" onClick={() => pickAndAddWorkspace()}>
-        <FolderPlus size={14} />
-        <span>Add workspace</span>
-      </div>
-
-      {ctxMenu && (
-        <div
-          ref={ctxRef}
-          className="ctx-menu"
-          style={{ position: 'fixed', left: ctxMenu.x, top: ctxMenu.y, zIndex: 500 }}
-        >
-          {ctxMenu.type === 'workspace' && (
-            <>
-              <div
-                className="ctx-menu-item"
-                onClick={() => {
-                  createSessionInWorkspace(ctxMenu.path!);
-                  setCtxMenu(null);
-                }}
-              >
-                <Plus size={12} /> New Session
-              </div>
-              <div
-                className="ctx-menu-item ctx-danger"
-                onClick={() => {
-                  removeWorkspace(ctxMenu.path!);
-                  setCtxMenu(null);
-                }}
-              >
-                <Trash2 size={12} /> Hide Workspace
-              </div>
-            </>
-          )}
-          {ctxMenu.type === 'session' && (
-            <>
-              {ctxMenu.sessionLifecycle !== 'finished' ? (
-                <>
-                  <div
-                    className="ctx-menu-item"
-                    onClick={() => {
-                      startRename(ctxMenu.sessionId!, ctxMenu.sessionTitle ?? '');
-                      setCtxMenu(null);
-                    }}
-                  >
-                    <Edit3 size={12} /> Rename
-                  </div>
-                  <div
-                    className="ctx-menu-item"
-                    onClick={() => {
-                      markFinished(ctxMenu.sessionId!);
-                      setCtxMenu(null);
-                    }}
-                  >
-                    <CheckCircle2 size={12} /> Mark as Finished
-                  </div>
-                  <div
-                    className="ctx-menu-item ctx-danger"
-                    onClick={() => {
-                      deleteSession(ctxMenu.sessionId!);
-                      setCtxMenu(null);
-                    }}
-                  >
-                    <Trash2 size={12} /> Delete Session
-                  </div>
-                </>
-              ) : (
-                <>
-                  <div
-                    className="ctx-menu-item"
-                    onClick={() => {
-                      reopenSession(ctxMenu.sessionId!);
-                      setCtxMenu(null);
-                    }}
-                  >
-                    <RotateCcw size={12} /> Reopen
-                  </div>
-                  <div
-                    className="ctx-menu-item ctx-danger"
-                    onClick={() => {
-                      deleteSession(ctxMenu.sessionId!);
-                      setCtxMenu(null);
-                    }}
-                  >
-                    <Trash2 size={12} /> Delete Session
-                  </div>
-                </>
-              )}
-            </>
-          )}
-        </div>
-      )}
-    </div>
-  );
-}

package/agent-pane/frontend/src/components/input/OptionSubPopup.tsx

@@ -1,32 +0,0 @@
-interface OptionSubPopupProps {
-  title: string;
-  options: { value: string; label: string; description?: string }[];
-  currentValue: string | null | undefined;
-  onSelect: (value: string) => void;
-  onClose: () => void;
-}
-
-/** Radio-style sub-popup for /sandbox, /websearch, /network commands. */
-export default function OptionSubPopup({ title, options, currentValue, onSelect, onClose }: OptionSubPopupProps) {
-  return (
-    <div className="slash-popup">
-      <div className="slash-popup-header">
-        <button className="slash-popup-back" onClick={onClose}>← Back</button>
-        <span>{title}</span>
-      </div>
-      {options.map((opt) => (
-        <div
-          key={opt.value}
-          className={`slash-popup-item ${opt.value === currentValue ? 'selected' : ''}`}
-          onClick={() => onSelect(opt.value)}
-        >
-          <div className="slash-popup-item-main">
-            <span className="slash-popup-label">{opt.label}</span>
-            {opt.description && <span className="slash-popup-hint">{opt.description}</span>}
-          </div>
-          {opt.value === currentValue && <span className="slash-popup-check">✓</span>}
-        </div>
-      ))}
-    </div>
-  );
-}