repoburg 1.3.201 → 1.3.204

package/agent-pane/backend/src/codex/app-server-types.ts

@@ -1,259 +0,0 @@
-/** TypeScript interfaces for the Codex app-server JSON-RPC protocol. */
-
-// --- Skill metadata (from `skills/list`) ---
-
-export interface AppServerSkill {
-  name: string;
-  description: string;
-  path: string;
-  scope: 'user' | 'repo' | 'system' | 'admin';
-  enabled: boolean;
-  displayName?: string;
-  shortDescription?: string;
-  interface?: {
-    displayName?: string;
-    shortDescription?: string;
-    brandColor?: string;
-    defaultPrompt?: string;
-    iconSmall?: string | null;
-    iconLarge?: string | null;
-  };
-}
-
-export interface SkillsListResult {
-  data: Array<{
-    cwd: string;
-    skills: AppServerSkill[];
-    errors: string[];
-  }>;
-}
-
-// --- Model metadata (from `model/list`) ---
-
-export interface AppServerModel {
-  id: string;
-  model: string;
-  displayName: string;
-  description: string;
-  defaultReasoningEffort: string;
-  supportedReasoningEfforts: Array<{
-    reasoningEffort: string;
-    description: string;
-  }>;
-  inputModalities?: string[];
-  isDefault?: boolean;
-  hidden?: boolean;
-}
-
-export interface ModelListResult {
-  data: AppServerModel[];
-  nextCursor: string | null;
-}
-
-// --- Plugin metadata (from `plugin/list`) ---
-
-export interface AppServerPlugin {
-  id: string;
-  name: string;
-  installed: boolean;
-  enabled: boolean;
-  availability: string;
-  localVersion?: string | null;
-  version?: string | null;
-  interface?: {
-    displayName?: string;
-    shortDescription?: string;
-    longDescription?: string;
-    brandColor?: string;
-    developerName?: string;
-    category?: string;
-  };
-}
-
-export interface PluginListResult {
-  marketplaces: Array<{
-    name: string;
-    plugins: AppServerPlugin[];
-  }>;
-}
-
-// --- MCP server metadata (from `mcpServerStatus/list`) ---
-
-export interface AppServerMcpServer {
-  name: string;
-  serverInfo?: {
-    name?: string;
-    title?: string | null;
-    version?: string;
-    description?: string | null;
-  };
-  authStatus?: string;
-  tools?: Record<string, unknown>;
-}
-
-export interface McpListResult {
-  data: AppServerMcpServer[];
-}
-
-// --- File search (from `fuzzyFileSearch`) ---
-
-export interface AppServerFileSearchResult {
-  file_name: string;
-  path: string;
-  match_type: 'file' | 'directory';
-  score: number;
-  root: string;
-}
-
-export interface FileSearchResult {
-  results: AppServerFileSearchResult[];
-}
-
-// --- Thread / Turn ---
-
-export interface ThreadStartParams {
-  cwd: string;
-  approvalPolicy: string;
-  sandbox: string;
-  model?: string;
-}
-
-export interface ThreadStartResult {
-  thread: { id: string };
-}
-
-export interface ThreadResumeParams {
-  threadId: string;
-  cwd: string;
-  approvalPolicy: string;
-  sandbox: string;
-  model?: string;
-}
-
-export interface TurnStartParams {
-  threadId: string;
-  input: Array<{ type: 'text'; text: string } | { type: 'image'; url: string }>;
-  approvalPolicy: string;
-  sandbox: { type: 'read-only' | 'workspace-write' | 'danger-full-access' };
-  model?: string;
-  effort?: string;
-}
-
-export interface TurnStartResult {
-  turn: { id: string };
-}
-
-export interface TurnInterruptParams {
-  threadId: string;
-  turnId: string;
-}
-
-// --- Notification params ---
-
-export interface ItemNotification {
-  threadId: string;
-  turnId: string;
-  item: AppServerItem;
-}
-
-export interface TurnCompletedNotification {
-  threadId: string;
-  turnId: string;
-  usage?: unknown;
-}
-
-export interface TurnFailedNotification {
-  threadId: string;
-  turnId: string;
-  error: { message: string };
-}
-
-/** Params for `item/agentMessage/delta` — incremental text fragment. */
-export interface AgentMessageDeltaNotification {
-  threadId: string;
-  turnId: string;
-  itemId: string;
-  delta: string;
-}
-
-/** CamelCase token usage from app-server (needs mapping to snake_case TokenUsage). */
-export interface AppServerTokenUsageData {
-  totalTokens: number;
-  inputTokens: number;
-  cachedInputTokens: number;
-  cacheWriteInputTokens: number;
-  outputTokens: number;
-  reasoningOutputTokens: number;
-}
-
-/** Params for `thread/tokenUsage/updated` — token usage for a turn. */
-export interface TokenUsageNotification {
-  threadId: string;
-  turnId: string;
-  tokenUsage: {
-    total: AppServerTokenUsageData;
-    last: AppServerTokenUsageData;
-    modelContextWindow: number;
-  };
-}
-
-/** Map app-server camelCase token usage to shared snake_case TokenUsage. */
-export function mapTokenUsage(data: AppServerTokenUsageData): {
-  input_tokens: number;
-  cached_input_tokens: number;
-  cache_write_input_tokens: number;
-  output_tokens: number;
-  reasoning_output_tokens: number;
-} {
-  return {
-    input_tokens: data.inputTokens,
-    cached_input_tokens: data.cachedInputTokens,
-    cache_write_input_tokens: data.cacheWriteInputTokens,
-    output_tokens: data.outputTokens,
-    reasoning_output_tokens: data.reasoningOutputTokens,
-  };
-}
-
-// --- Generic item type (replaces SDK ThreadItem) ---
-
-export interface AppServerItem {
-  id: string;
-  type: string;
-  text?: string;
-  summary?: unknown[];
-  content?: unknown[];
-  command?: string;
-  aggregated_output?: string;
-  exit_code?: number | null;
-  status?: 'in_progress' | 'completed' | 'failed';
-  changes?: unknown;
-  server?: string;
-  tool?: string;
-  arguments?: unknown;
-  result?: unknown;
-  error?: string | null;
-  query?: string;
-  items?: unknown[];
-  message?: string;
-  [key: string]: unknown;
-}
-
-// --- Initialize handshake ---
-
-export interface InitializeParams {
-  clientInfo: {
-    name: string;
-    title: string;
-    version: string;
-  };
-  capabilities: {
-    experimentalApi: boolean;
-  };
-}
-
-export interface InitializeResult {
-  userAgent: string;
-  codexHome: string;
-  platformFamily: string;
-  platformOs: string;
-}

package/agent-pane/backend/src/codex/codex-provider-runtime-config.test.ts

@@ -1,56 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-
-// Mock the codex-provider module to get access to getThreadOptions
-// We mock everything except getThreadOptions which we want to test
-
-vi.mock('../db/index.ts', () => ({
-  getDb: vi.fn(),
-  initDb: vi.fn(),
-}));
-
-vi.mock('../ws/index.ts', () => ({
-  broadcast: vi.fn(),
-  initWs: vi.fn(),
-  handleUpgrade: vi.fn(),
-  getWss: vi.fn(),
-}));
-
-vi.mock('./item-mapper.ts', () => ({
-  mapItemData: vi.fn(),
-  getItemStatus: vi.fn(),
-}));
-
-// Import after mocks are set up
-import type { Session } from '@agentpane/shared';
-
-// We need to test getThreadOptions indirectly since it's not exported.
-// Instead, we test the behavior by checking that runtime_config is parsed correctly.
-// Since getThreadOptions is private, we test the parsing logic here.
-
-describe('SessionRuntimeConfig parsing', () => {
-  function parseRuntimeConfig(session: Pick<Session, 'runtime_config'>): Record<string, unknown> | null {
-    if (!session.runtime_config) return null;
-    try {
-      return JSON.parse(session.runtime_config) as Record<string, unknown>;
-    } catch {
-      return null;
-    }
-  }
-
-  it('returns null when runtime_config is null', () => {
-    expect(parseRuntimeConfig({ runtime_config: null })).toBeNull();
-  });
-
-  it('returns parsed config when valid JSON', () => {
-    const config = { sandboxMode: 'read-only', webSearchMode: 'live', networkAccessEnabled: true };
-    expect(parseRuntimeConfig({ runtime_config: JSON.stringify(config) })).toEqual(config);
-  });
-
-  it('returns null when invalid JSON', () => {
-    expect(parseRuntimeConfig({ runtime_config: 'invalid json' })).toBeNull();
-  });
-
-  it('returns empty object when runtime_config is empty string JSON', () => {
-    expect(parseRuntimeConfig({ runtime_config: '{}' })).toEqual({});
-  });
-});

package/agent-pane/backend/src/codex/codex-provider.test.ts

@@ -1,60 +0,0 @@
-import { describe, it, expect, vi } from 'vitest';
-
-// Mock the app-server client to avoid spawning real processes
-vi.mock('../codex/app-server-client.ts', () => ({
-  AppServerClient: vi.fn().mockImplementation(() => ({
-    initialize: vi.fn().mockResolvedValue(undefined),
-    request: vi.fn().mockResolvedValue({ data: [] }),
-    onNotification: vi.fn(),
-    close: vi.fn().mockResolvedValue(undefined),
-    isClosed: vi.fn().mockReturnValue(false),
-    events: { once: vi.fn(), on: vi.fn() },
-  })),
-}));
-
-// Mock the shared app-server to avoid spawning a real process
-vi.mock('../codex/shared-app-server.ts', () => ({
-  getSharedAppServer: vi.fn().mockReturnValue({
-    request: vi.fn().mockResolvedValue({ data: [] }),
-  }),
-  closeSharedAppServer: vi.fn().mockResolvedValue(undefined),
-}));
-
-// Mock broadcast to avoid WebSocket setup
-vi.mock('../ws/index.ts', () => ({
-  broadcast: vi.fn(),
-}));
-
-import { codexProvider } from './codex-provider.ts';
-import type { RuntimeProvider } from '../providers/runtime-provider.ts';
-
-describe('codexProvider', () => {
-  it('implements all RuntimeProvider methods', () => {
-    const provider: RuntimeProvider = codexProvider;
-    expect(typeof provider.startTurn).toBe('function');
-    expect(typeof provider.interruptTurn).toBe('function');
-    expect(typeof provider.clearThread).toBe('function');
-    expect(typeof provider.clearAllThreads).toBe('function');
-    expect(typeof provider.getModels).toBe('function');
-    expect(typeof provider.shutdown).toBe('function');
-  });
-
-  it('getModels returns an array', async () => {
-    const models = await codexProvider.getModels();
-    expect(Array.isArray(models)).toBe(true);
-  });
-
-  it('shutdown calls closeSharedAppServer and clearAllThreads', async () => {
-    await codexProvider.shutdown();
-    // No error thrown — shutdown completed
-    // closeSharedAppServer and clearAllThreads are mocked, so this just verifies no crash
-  });
-
-  it('clearAllThreads does not throw', () => {
-    expect(() => codexProvider.clearAllThreads()).not.toThrow();
-  });
-
-  it('clearThread does not throw for unknown session', () => {
-    expect(() => codexProvider.clearThread('nonexistent')).not.toThrow();
-  });
-});