@robota-sdk/agent-transport 3.0.0-beta.75 → 3.0.0-beta.77

package/src/tui/__tests__/TextPrompt.test.tsx

@@ -1,98 +0,0 @@
-import React from 'react';
-import { render } from 'ink-testing-library';
-import { describe, it, expect } from 'vitest';
-import TextPrompt from '../TextPrompt.js';
-
-describe('TextPrompt', () => {
-  it('renders title', () => {
-    const { lastFrame } = render(
-      <TextPrompt title="Enter URL" onSubmit={() => {}} onCancel={() => {}} />,
-    );
-    expect(lastFrame()!).toContain('Enter URL');
-  });
-
-  it('renders placeholder when provided', () => {
-    const { lastFrame } = render(
-      <TextPrompt title="Enter" placeholder="owner/repo" onSubmit={() => {}} onCancel={() => {}} />,
-    );
-    expect(lastFrame()!).toContain('owner/repo');
-  });
-
-  it('calls onCancel on Escape', async () => {
-    let cancelled = false;
-    const { stdin } = render(
-      <TextPrompt
-        title="Enter"
-        onSubmit={() => {}}
-        onCancel={() => {
-          cancelled = true;
-        }}
-      />,
-    );
-    stdin.write('\x1B');
-    await new Promise((r) => setTimeout(r, 50));
-    expect(cancelled).toBe(true);
-  });
-
-  it('calls onSubmit with value on Enter', () => {
-    let submitted = '';
-    const { stdin } = render(
-      <TextPrompt
-        title="Enter"
-        onSubmit={(v) => {
-          submitted = v;
-        }}
-        onCancel={() => {}}
-      />,
-    );
-    stdin.write('hello');
-    stdin.write('\r');
-    expect(submitted).toBe('hello');
-  });
-
-  it('can submit an empty value when allowed', () => {
-    let submitted = 'not-called';
-    const { stdin } = render(
-      <TextPrompt
-        title="Enter"
-        allowEmpty
-        onSubmit={(v) => {
-          submitted = v;
-        }}
-        onCancel={() => {}}
-      />,
-    );
-    stdin.write('\r');
-    expect(submitted).toBe('');
-  });
-
-  it('masks typed values when requested', async () => {
-    const { stdin, lastFrame } = render(
-      <TextPrompt title="Secret" masked onSubmit={() => {}} onCancel={() => {}} />,
-    );
-    stdin.write('abc');
-    await new Promise((r) => setTimeout(r, 50));
-    expect(lastFrame()!).toContain('***');
-    expect(lastFrame()!).not.toContain('abc');
-  });
-
-  it('shows validation error and blocks submit', async () => {
-    let submitted = false;
-    const validate = (v: string) => (v.length < 3 ? 'Too short' : undefined);
-    const { stdin, lastFrame } = render(
-      <TextPrompt
-        title="Enter"
-        onSubmit={() => {
-          submitted = true;
-        }}
-        onCancel={() => {}}
-        validate={validate}
-      />,
-    );
-    stdin.write('ab');
-    stdin.write('\r');
-    expect(submitted).toBe(false);
-    await new Promise((r) => setTimeout(r, 50));
-    expect(lastFrame()!).toContain('Too short');
-  });
-});

package/src/tui/__tests__/TuiInteractionChannel.display-contract.test.ts

@@ -1,239 +0,0 @@
-/**
- * Display-contract tests for TuiInteractionChannel.
- *
- * Verifies user-visible state: which entries appear in stateManager.history,
- * with which roles, and in which order.
- *
- * These are deliberately separate from lifecycle.test.ts:
- *   lifecycle  → event routing / onChange propagation (mechanism layer)
- *   this file  → what the user sees on screen (display contract layer)
- *
- * Design principles:
- *   - user_message assertions do NOT require getFullHistory injection
- *   - every history assertion checks entry.type (= role) explicitly
- *   - timing: user message visible BEFORE complete fires
- */
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-vi.mock('@robota-sdk/agent-framework', async () => {
-  const actual = await vi.importActual<typeof import('@robota-sdk/agent-framework')>(
-    '@robota-sdk/agent-framework',
-  );
-  return {
-    ...actual,
-    InteractiveSession: vi.fn().mockImplementation(() => {
-      const handlers = new Map<string, Array<(...args: unknown[]) => void>>();
-      return {
-        getFullHistory: vi.fn().mockReturnValue([]),
-        setName: vi.fn(),
-        getName: vi.fn().mockReturnValue(undefined),
-        getPermissionMode: vi.fn().mockReturnValue('default'),
-        isInitialized: false,
-        on: vi.fn().mockImplementation((event: string, handler: (...args: unknown[]) => void) => {
-          if (!handlers.has(event)) handlers.set(event, []);
-          handlers.get(event)!.push(handler);
-        }),
-        off: vi.fn(),
-        emit: (event: string, ...args: unknown[]) => {
-          (handlers.get(event) ?? []).forEach((h) => h(...args));
-        },
-        submit: vi.fn().mockResolvedValue(undefined),
-        executeCommand: vi.fn().mockResolvedValue(null),
-        getPendingPrompt: vi.fn().mockReturnValue(null),
-        abort: vi.fn(),
-        cancelQueue: vi.fn(),
-        getContextState: vi.fn().mockReturnValue({
-          usedPercentage: 0,
-          usedTokens: 0,
-          maxTokens: 100_000,
-        }),
-        getExecutionWorkspaceSnapshot: vi.fn().mockReturnValue({ entries: [] }),
-        shutdown: vi.fn().mockResolvedValue(undefined),
-        sendAgentJob: vi.fn().mockResolvedValue(undefined),
-        readExecutionWorkspaceDetail: vi.fn().mockResolvedValue({}),
-      };
-    }),
-    CommandRegistry: vi.fn().mockImplementation(() => ({
-      addModule: vi.fn(),
-    })),
-  };
-});
-
-import {
-  createAssistantMessage,
-  createSystemMessage,
-  createUserMessage,
-  messageToHistoryEntry,
-} from '@robota-sdk/agent-core';
-import { TuiInteractionChannel } from '../TuiInteractionChannel.js';
-
-import type { IAIProvider, IHistoryEntry } from '@robota-sdk/agent-core';
-import type { IExecutionResult } from '@robota-sdk/agent-interface-transport';
-
-// ── Helpers ───────────────────────────────────────────────────────────────────
-
-type MockSession = {
-  getFullHistory: ReturnType<typeof vi.fn>;
-  on: ReturnType<typeof vi.fn>;
-  emit: (event: string, ...args: unknown[]) => void;
-};
-
-function getMockSession(channel: TuiInteractionChannel): MockSession {
-  return (channel as unknown as { interactiveSession: MockSession }).interactiveSession;
-}
-
-function emitSessionEvent(channel: TuiInteractionChannel, event: string, ...args: unknown[]): void {
-  getMockSession(channel).emit(event, ...args);
-}
-
-function makeChannel(): TuiInteractionChannel {
-  return new TuiInteractionChannel({
-    cwd: '/tmp/test',
-    provider: {} as IAIProvider,
-  });
-}
-
-function makeResult(overrides?: Partial<IExecutionResult>): IExecutionResult {
-  return {
-    contextState: { usedPercentage: 5, usedTokens: 500, maxTokens: 100_000 },
-    response: 'done',
-    ...overrides,
-  } as unknown as IExecutionResult;
-}
-
-const MOCK_TOOL_RUNNING = {
-  toolName: 'bash',
-  isRunning: true,
-  input: '{}',
-  startTime: Date.now(),
-};
-
-const MOCK_TOOL_DONE = {
-  toolName: 'bash',
-  isRunning: false,
-  input: '{}',
-  startTime: Date.now(),
-};
-
-function entryRole(entry: IHistoryEntry): string {
-  return entry.type;
-}
-
-beforeEach(() => {
-  vi.useFakeTimers();
-});
-
-afterEach(() => {
-  vi.useRealTimers();
-});
-
-// ── Group D: display contract (what the user sees) ────────────────────────────
-
-describe('Group D — display contract: history entries and active tools', () => {
-  it('D1 (CLI-B05): user_message event immediately adds role=user entry before complete', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    // user message fires BEFORE complete — must be in history immediately
-    emitSessionEvent(channel, 'user_message', 'hello');
-
-    expect(channel.stateManager.history).toHaveLength(1);
-    expect(entryRole(channel.stateManager.history[0])).toBe('user');
-    expect((channel.stateManager.history[0].data as { content: string }).content).toBe('hello');
-
-    await channel.stop();
-  });
-
-  it('D2: complete syncs assistant entry into history', async () => {
-    const channel = makeChannel();
-    const mockSession = getMockSession(channel);
-    const assistantEntry = messageToHistoryEntry(createAssistantMessage('Hi!'));
-    mockSession.getFullHistory.mockReturnValue([assistantEntry]);
-    await channel.start();
-
-    emitSessionEvent(channel, 'complete', makeResult());
-
-    const roles = channel.stateManager.history.map(entryRole);
-    expect(roles).toContain('assistant');
-    await channel.stop();
-  });
-
-  it('D3 (CLI-B06): error event syncs error entry into history — no silent failure', async () => {
-    const channel = makeChannel();
-    const mockSession = getMockSession(channel);
-    const userEntry = messageToHistoryEntry(createUserMessage('hello'));
-    const errorEntry = messageToHistoryEntry(createSystemMessage('Error: network failure'));
-    mockSession.getFullHistory.mockReturnValue([userEntry, errorEntry]);
-    await channel.start();
-
-    emitSessionEvent(channel, 'user_message', 'hello');
-    emitSessionEvent(channel, 'error');
-
-    // error entry from getFullHistory must be visible
-    const roles = channel.stateManager.history.map(entryRole);
-    expect(roles).toContain('system');
-    await channel.stop();
-  });
-
-  it('D4 (CLI-B07): tool_end marks tool isRunning=false; complete clears all (no stale spinner after complete)', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'tool_start', MOCK_TOOL_RUNNING);
-    expect(channel.stateManager.activeTools).toHaveLength(1);
-
-    // After tool_end: tool stays in activeTools with isRunning:false (shows "ran" status during streaming)
-    emitSessionEvent(channel, 'tool_end', MOCK_TOOL_DONE);
-    expect(channel.stateManager.activeTools).toHaveLength(1);
-    expect(channel.stateManager.activeTools[0]!.isRunning).toBe(false);
-
-    // After complete: all tools cleared — StreamingIndicator must be gone
-    emitSessionEvent(channel, 'complete', makeResult());
-    expect(channel.stateManager.activeTools).toHaveLength(0);
-
-    await channel.stop();
-  });
-
-  it('D5 (CLI-B08): thinking(false) clears activeTools even without complete', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'thinking', true);
-    emitSessionEvent(channel, 'tool_start', MOCK_TOOL_RUNNING);
-    expect(channel.stateManager.activeTools).toHaveLength(1);
-
-    // abort path: thinking(false) without complete
-    emitSessionEvent(channel, 'thinking', false);
-    expect(channel.stateManager.activeTools).toHaveLength(0);
-
-    await channel.stop();
-  });
-
-  it('D6: full turn — user entry appears first, assistant entry follows after complete', async () => {
-    const channel = makeChannel();
-    const mockSession = getMockSession(channel);
-    const userEntry = messageToHistoryEntry(createUserMessage('hello'));
-    const assistantEntry = messageToHistoryEntry(createAssistantMessage('world'));
-    // session history after complete includes both
-    mockSession.getFullHistory.mockReturnValue([userEntry, assistantEntry]);
-    await channel.start();
-
-    // 1. user message fires — visible immediately
-    emitSessionEvent(channel, 'user_message', 'hello');
-    expect(entryRole(channel.stateManager.history[0])).toBe('user');
-
-    // 2. streaming
-    emitSessionEvent(channel, 'text_delta', 'world');
-    expect(channel.stateManager.streamingText).toBe('world');
-
-    // 3. complete — syncHistory replaces with authoritative session history
-    emitSessionEvent(channel, 'complete', makeResult());
-    expect(channel.stateManager.streamingText).toBe('');
-
-    const roles = channel.stateManager.history.map(entryRole);
-    expect(roles[0]).toBe('user');
-    expect(roles[1]).toBe('assistant');
-
-    await channel.stop();
-  });
-});

package/src/tui/__tests__/TuiInteractionChannel.lifecycle.test.ts

@@ -1,297 +0,0 @@
-/**
- * Integration tests for TuiInteractionChannel lifecycle:
- * session event wiring, handleInput roundtrip, onChange propagation.
- *
- * No Ink rendering, no PTY — pure TypeScript.
- */
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-
-vi.mock('@robota-sdk/agent-framework', async () => {
-  const actual = await vi.importActual<typeof import('@robota-sdk/agent-framework')>(
-    '@robota-sdk/agent-framework',
-  );
-  return {
-    ...actual,
-    InteractiveSession: vi.fn().mockImplementation(() => {
-      const handlers = new Map<string, Array<(...args: unknown[]) => void>>();
-      return {
-        getFullHistory: vi.fn().mockReturnValue([]),
-        setName: vi.fn(),
-        getName: vi.fn().mockReturnValue(undefined),
-        getPermissionMode: vi.fn().mockReturnValue('default'),
-        isInitialized: false,
-        on: vi.fn().mockImplementation((event: string, handler: (...args: unknown[]) => void) => {
-          if (!handlers.has(event)) handlers.set(event, []);
-          handlers.get(event)!.push(handler);
-        }),
-        off: vi.fn(),
-        emit: (event: string, ...args: unknown[]) => {
-          (handlers.get(event) ?? []).forEach((h) => h(...args));
-        },
-        submit: vi.fn().mockResolvedValue(undefined),
-        executeCommand: vi.fn().mockResolvedValue(null),
-        getPendingPrompt: vi.fn().mockReturnValue(null),
-        abort: vi.fn(),
-        cancelQueue: vi.fn(),
-        getContextState: vi.fn().mockReturnValue({
-          usedPercentage: 0,
-          usedTokens: 0,
-          maxTokens: 100_000,
-        }),
-        getExecutionWorkspaceSnapshot: vi.fn().mockReturnValue({ entries: [] }),
-        shutdown: vi.fn().mockResolvedValue(undefined),
-        sendAgentJob: vi.fn().mockResolvedValue(undefined),
-        readExecutionWorkspaceDetail: vi.fn().mockResolvedValue({}),
-      };
-    }),
-    CommandRegistry: vi.fn().mockImplementation(() => ({
-      addModule: vi.fn(),
-    })),
-  };
-});
-
-import { TuiInteractionChannel } from '../TuiInteractionChannel.js';
-
-import type { IAIProvider } from '@robota-sdk/agent-core';
-import type {
-  IExecutionResult,
-  IInteractiveSession,
-  ITransportRegistryView,
-} from '@robota-sdk/agent-interface-transport';
-
-// ── Helpers ───────────────────────────────────────────────────────────────────
-
-type MockSession = {
-  getFullHistory: ReturnType<typeof vi.fn>;
-  submit: ReturnType<typeof vi.fn>;
-  executeCommand: ReturnType<typeof vi.fn>;
-  on: ReturnType<typeof vi.fn>;
-  emit: (event: string, ...args: unknown[]) => void;
-};
-
-function getMockSession(channel: TuiInteractionChannel): MockSession {
-  return (channel as unknown as { interactiveSession: MockSession }).interactiveSession;
-}
-
-function emitSessionEvent(channel: TuiInteractionChannel, event: string, ...args: unknown[]): void {
-  getMockSession(channel).emit(event, ...args);
-}
-
-function makeMockTransportRegistry(): {
-  registry: ITransportRegistryView<IInteractiveSession>;
-  startAll: ReturnType<typeof vi.fn>;
-  stopAll: ReturnType<typeof vi.fn>;
-} {
-  const startAll = vi.fn().mockResolvedValue(undefined);
-  const stopAll = vi.fn().mockResolvedValue(undefined);
-  return {
-    registry: { startAll, stopAll } as unknown as ITransportRegistryView<IInteractiveSession>,
-    startAll,
-    stopAll,
-  };
-}
-
-function makeChannel(opts?: {
-  transportRegistry?: ITransportRegistryView<IInteractiveSession>;
-}): TuiInteractionChannel {
-  return new TuiInteractionChannel({
-    cwd: '/tmp/test',
-    provider: {} as IAIProvider,
-    ...opts,
-  });
-}
-
-const MOCK_RESULT = {
-  contextState: { usedPercentage: 10, usedTokens: 1_000, maxTokens: 100_000 },
-  response: 'Hello!',
-} as unknown as IExecutionResult;
-
-const MOCK_TOOL = {
-  toolName: 'bash',
-  isRunning: true,
-  input: '{}',
-  startTime: Date.now(),
-} as unknown as Parameters<
-  InstanceType<typeof TuiInteractionChannel>['stateManager']['onToolStart']
->[0];
-
-beforeEach(() => {
-  vi.useFakeTimers();
-});
-
-afterEach(() => {
-  vi.useRealTimers();
-});
-
-// ── Group A: channel.start() / channel.stop() lifecycle ───────────────────────
-
-describe('Group A — channel.start() / channel.stop() lifecycle', () => {
-  it('A1: text_delta after start() updates stateManager.streamingText', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'text_delta', 'Hello!');
-
-    expect(channel.stateManager.streamingText).toBe('Hello!');
-    await channel.stop();
-  });
-
-  it('A2: complete after start() clears streaming state and updates contextState', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'text_delta', 'streaming...');
-    emitSessionEvent(channel, 'complete', MOCK_RESULT);
-
-    expect(channel.stateManager.streamingText).toBe('');
-    expect(channel.stateManager.contextState.percentage).toBe(10);
-    expect(channel.stateManager.contextState.usedTokens).toBe(1_000);
-    await channel.stop();
-  });
-
-  it('A3: tool_start after start() adds entry to stateManager.activeTools', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'tool_start', MOCK_TOOL);
-
-    expect(channel.stateManager.activeTools).toHaveLength(1);
-    expect(channel.stateManager.activeTools[0]).toMatchObject({ toolName: 'bash' });
-    await channel.stop();
-  });
-
-  it('A4: error after start() clears stateManager.streamingText', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    emitSessionEvent(channel, 'text_delta', 'partial...');
-    emitSessionEvent(channel, 'error');
-
-    expect(channel.stateManager.streamingText).toBe('');
-    await channel.stop();
-  });
-
-  it('A5: calling start() twice does not duplicate subscriptions', async () => {
-    const channel = makeChannel();
-    await channel.start();
-    await channel.start(); // second call is a no-op (sessionStarted guard)
-
-    emitSessionEvent(channel, 'text_delta', 'hi');
-
-    expect(channel.stateManager.streamingText).toBe('hi'); // not 'hihi'
-    await channel.stop();
-  });
-
-  it('A6: stop() calls transportRegistry.stopAll exactly once', async () => {
-    const { registry, stopAll } = makeMockTransportRegistry();
-    const channel = makeChannel({ transportRegistry: registry });
-    await channel.start();
-    await channel.stop();
-
-    expect(stopAll).toHaveBeenCalledOnce();
-  });
-});
-
-// ── Group B: handleInput() AI-response roundtrip ──────────────────────────────
-
-describe('Group B — handleInput() roundtrip', () => {
-  it('B1: handleInput("hello") calls session.submit with "hello"', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    await channel.handleInput('hello');
-
-    const mockSession = getMockSession(channel);
-    expect(mockSession.submit).toHaveBeenCalledWith('hello');
-    await channel.stop();
-  });
-
-  it('B2: text_delta + complete syncs history to stateManager', async () => {
-    const channel = makeChannel();
-    const mockSession = getMockSession(channel);
-    const historyEntry = {
-      role: 'assistant',
-      content: [{ type: 'text', text: 'Hi!' }],
-      timestamp: Date.now(),
-    };
-    mockSession.getFullHistory.mockReturnValue([historyEntry]);
-    await channel.start();
-
-    await channel.handleInput('hello');
-    emitSessionEvent(channel, 'text_delta', 'Hi!');
-    expect(channel.stateManager.streamingText).toBe('Hi!');
-
-    emitSessionEvent(channel, 'complete', MOCK_RESULT);
-    expect(channel.stateManager.streamingText).toBe('');
-    expect(channel.stateManager.history).toHaveLength(1);
-
-    await channel.stop();
-  });
-
-  it('B3: handleInput("/help") calls executeCommand, not session.submit', async () => {
-    const channel = makeChannel();
-    await channel.start();
-
-    await channel.handleInput('/help');
-
-    const mockSession = getMockSession(channel);
-    expect(mockSession.submit).not.toHaveBeenCalled();
-    expect(mockSession.executeCommand).toHaveBeenCalledWith('help', '');
-    await channel.stop();
-  });
-
-  it('B4: handleInput("hello") triggers channel.onChange at least once', async () => {
-    const channel = makeChannel();
-    const onChange = vi.fn();
-    channel.onChange = onChange;
-    await channel.start();
-    onChange.mockClear();
-
-    await channel.handleInput('hello');
-    emitSessionEvent(channel, 'text_delta', 'hey');
-
-    expect(onChange).toHaveBeenCalled();
-    await channel.stop();
-  });
-});
-
-// ── Group C: onChange propagation invariant ───────────────────────────────────
-
-describe('Group C — onChange propagation invariant', () => {
-  it('C1: session event after start() causes channel.onChange to fire', async () => {
-    const channel = makeChannel();
-    const onChange = vi.fn();
-    channel.onChange = onChange;
-    await channel.start();
-    onChange.mockClear();
-
-    // tool_start calls notify() directly (not debounced), so onChange fires immediately
-    emitSessionEvent(channel, 'tool_start', MOCK_TOOL);
-
-    expect(onChange).toHaveBeenCalled();
-    await channel.stop();
-  });
-
-  it('C2: channel.onChange does not fire for events before start()', () => {
-    const channel = makeChannel();
-    const onChange = vi.fn();
-    channel.onChange = onChange;
-    // Do NOT call channel.start() — handlers not registered yet
-    emitSessionEvent(channel, 'text_delta', 'hello'); // no-op: no handlers
-
-    expect(onChange).not.toHaveBeenCalled();
-  });
-
-  it('C3: channel.onChange does not fire for events after stop()', async () => {
-    const channel = makeChannel();
-    const onChange = vi.fn();
-    channel.onChange = onChange;
-    await channel.start();
-    await channel.stop(); // sets this.onChange = null
-    onChange.mockClear();
-
-    emitSessionEvent(channel, 'text_delta', 'hello');
-
-    expect(onChange).not.toHaveBeenCalled();
-  });
-});