macro-agent 0.1.2 → 0.1.3

package/src/map/adapter/__tests__/acp-over-map-history.test.ts

@@ -1,1386 +0,0 @@
-/**
- * ACP-over-MAP History Persistence Tests
- *
- * Verifies that prompts sent through the ACP-over-MAP handler
- * correctly record conversation turns in the EventStore, and that
- * _macro/getHistory returns them.
- */
-
-import { describe, it, expect, afterEach, vi } from "vitest";
-import { ACPOverMAPHandler } from "../acp-over-map.js";
-import type { ACPEnvelope } from "../acp-over-map.js";
-import { createEventStore, type EventStore } from "../../../store/event-store.js";
-import type { AgentManager } from "../../../agent/agent-manager.js";
-import type { TaskManager } from "../../../task/task-manager.js";
-import type { Agent, Task } from "../../../store/types/index.js";
-import type { AgentId } from "../../../store/types/index.js";
-
-// ─────────────────────────────────────────────────────────────────
-// Helpers
-// ─────────────────────────────────────────────────────────────────
-
-function createMockAgent(overrides: Partial<Agent> = {}): Agent {
-  return {
-    id: "agent-1" as AgentId,
-    session_id: "session-1",
-    state: "running",
-    task: "Test task",
-    task_id: "task-1",
-    parent: null,
-    lineage: [],
-    config: {},
-    cwd: "/test/cwd",
-    plan: [],
-    created_at: Date.now(),
-    started_at: Date.now(),
-    ...overrides,
-  };
-}
-
-function createMockTask(overrides: Partial<Task> = {}): Task {
-  return {
-    id: "task-1",
-    description: "Test task",
-    status: "in_progress",
-    created_by: "agent-1",
-    created_at: Date.now(),
-    ...overrides,
-  };
-}
-
-function createMockAgentManager(
-  promptUpdates: unknown[] = [],
-): AgentManager {
-  const mockAgent = createMockAgent();
-
-  return {
-    spawn: vi.fn().mockResolvedValue({
-      id: "agent-new",
-      session_id: "session-new",
-      agent: createMockAgent({ id: "agent-new" as AgentId, session_id: "session-new" }),
-      session: {},
-    }),
-    get: vi.fn().mockReturnValue(mockAgent),
-    list: vi.fn().mockReturnValue([mockAgent]),
-    listHeadManagers: vi.fn().mockReturnValue([mockAgent]),
-    getChildren: vi.fn().mockReturnValue([]),
-    getHierarchy: vi.fn().mockReturnValue({
-      root: { agent: mockAgent, children: [] },
-      depth: 1,
-      totalAgents: 1,
-    }),
-    getOrCreateHeadManager: vi.fn().mockResolvedValue({
-      id: "agent-1",
-      session_id: "session-1",
-      agent: mockAgent,
-      session: {},
-    }),
-    hasActiveSession: vi.fn().mockReturnValue(true),
-    resume: vi.fn().mockResolvedValue({
-      id: "agent-1",
-      session_id: "session-1",
-      agent: mockAgent,
-      session: {},
-    }),
-    terminate: vi.fn().mockResolvedValue(undefined),
-    prompt: vi.fn().mockReturnValue({
-      [Symbol.asyncIterator]: async function* () {
-        for (const update of promptUpdates) {
-          yield update;
-        }
-      },
-    }),
-    getSession: vi.fn().mockReturnValue(null),
-    onLifecycleEvent: vi.fn().mockReturnValue(() => {}),
-    close: vi.fn().mockResolvedValue(undefined),
-    respondToPermission: vi.fn().mockReturnValue(true),
-    cancelPermission: vi.fn().mockReturnValue(true),
-  } as unknown as AgentManager;
-}
-
-function createMockTaskManager(): TaskManager {
-  return {
-    get: vi.fn().mockReturnValue(createMockTask()),
-    list: vi.fn().mockReturnValue([createMockTask()]),
-    create: vi.fn().mockReturnValue(createMockTask()),
-  } as unknown as TaskManager;
-}
-
-/** Build an ACP envelope for processRequest */
-function envelope(
-  streamId: string,
-  method: string,
-  params?: unknown,
-  sessionId?: string,
-): ACPEnvelope {
-  return {
-    acp: {
-      jsonrpc: "2.0",
-      id: `${streamId}-${method}-${Date.now()}`,
-      method,
-      params,
-    },
-    acpContext: {
-      streamId,
-      sessionId,
-      direction: "client-to-agent",
-    },
-  };
-}
-
-// ─────────────────────────────────────────────────────────────────
-// Tests
-// ─────────────────────────────────────────────────────────────────
-
-describe("ACP-over-MAP history persistence", () => {
-  let eventStore: EventStore;
-  let handler: ACPOverMAPHandler;
-
-  afterEach(async () => {
-    await eventStore.close();
-  });
-
-  async function setup(promptUpdates: unknown[] = []) {
-    eventStore = await createEventStore({ inMemory: true });
-    const agentManager = createMockAgentManager(promptUpdates);
-    const taskManager = createMockTaskManager();
-
-    handler = new ACPOverMAPHandler({
-      agentManager,
-      eventStore,
-      taskManager,
-      defaultCwd: "/test/cwd",
-    });
-
-    return { agentManager, taskManager };
-  }
-
-  /** Register an agent in the EventStore so loadSession can resolve it */
-  function registerAgent(agentId: string, sessionId: string): void {
-    eventStore.emit({
-      type: "spawn",
-      source: { agent_id: agentId },
-      payload: {
-        agent_id: agentId,
-        session_id: sessionId,
-        task: "Test task",
-        task_id: "task-1",
-        cwd: "/test/cwd",
-      },
-    });
-    // Mark as running
-    eventStore.emit({
-      type: "lifecycle",
-      source: { agent_id: agentId },
-      payload: {
-        agent_id: agentId,
-        action: "started",
-      },
-    });
-  }
-
-  /** Initialize a stream and create a session, returning the sessionId */
-  async function initAndCreateSession(
-    streamId: string,
-    targetAgentId: AgentId = "agent-1" as AgentId,
-  ): Promise<string> {
-    // Initialize
-    await handler.processRequest(
-      targetAgentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    // Create session
-    const sessionResult = await handler.processRequest(
-      targetAgentId,
-      envelope(streamId, "session/new", {
-        cwd: "/test",
-        mcpServers: [],
-      }),
-    );
-
-    const sessionId = (sessionResult.acp.result as { sessionId?: string })?.sessionId;
-    if (!sessionId) throw new Error("session/new did not return sessionId");
-
-    // Register the agent in EventStore so loadSession can resolve it later
-    registerAgent(targetAgentId, sessionId);
-
-    return sessionId;
-  }
-
-  it("should record user and assistant text turns after prompt via ACP-over-MAP", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Hello " },
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "world!" },
-      },
-    ]);
-
-    const streamId = "test-stream-1";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    // Send prompt
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Say hello" }],
-      }, sessionId),
-    );
-
-    // Get history via extension
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    expect(turns).toHaveLength(2);
-
-    // User turn
-    expect(turns[0].role).toBe("user");
-    expect(turns[0].content).toBe("Say hello");
-
-    // Assistant turn — accumulated text chunks
-    expect(turns[1].role).toBe("assistant");
-    const content = turns[1].content as { parts: { type: string; text?: string }[] };
-    expect(content.parts[0].type).toBe("text");
-    expect(content.parts[0].text).toBe("Hello world!");
-  });
-
-  it("should record tool calls in assistant turns", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Let me check." },
-      },
-      {
-        sessionUpdate: "tool_call",
-        toolCallId: "tc-1",
-        title: "Read file",
-        status: "completed",
-        rawInput: { path: "/test.txt" },
-        rawOutput: "file contents",
-      },
-    ]);
-
-    const streamId = "test-stream-2";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Read test.txt" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    expect(turns).toHaveLength(2);
-
-    const assistantContent = turns[1].content as {
-      parts: { type: string; text?: string; toolCallId?: string; title?: string; output?: unknown }[];
-    };
-    expect(assistantContent.parts).toHaveLength(2);
-    expect(assistantContent.parts[0]).toEqual({ type: "text", text: "Let me check." });
-    expect(assistantContent.parts[1]).toMatchObject({
-      type: "tool",
-      toolCallId: "tc-1",
-      title: "Read file",
-      status: "completed",
-      output: "file contents",
-    });
-  });
-
-  it("should extract tool output from rawOutput ContentBlock array", async () => {
-    await setup([
-      {
-        sessionUpdate: "tool_call",
-        toolCallId: "tc-array",
-        title: "Read file",
-        status: "completed",
-        rawInput: { path: "/test.txt" },
-        rawOutput: [
-          { type: "text", text: "line 1" },
-          { type: "text", text: "line 2" },
-        ],
-      },
-    ]);
-
-    const streamId = "test-stream-array-output";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Read it" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    const assistantContent = turns[1].content as {
-      parts: { type: string; output?: string }[];
-    };
-    const toolPart = assistantContent.parts.find((p) => p.type === "tool");
-    expect(toolPart?.output).toBe("line 1\nline 2");
-  });
-
-  it("should merge title from initial tool_call into tool_call_update", async () => {
-    // Simulates WebSearch/WebFetch: initial tool_call has title, but
-    // tool_call_update (completed) does not include title.
-    await setup([
-      {
-        sessionUpdate: "tool_call",
-        toolCallId: "tc-ws",
-        title: "Search query here",
-        status: "pending",
-        rawInput: { query: "test" },
-        _meta: { claudeCode: { toolName: "WebSearch" } },
-      },
-      {
-        sessionUpdate: "tool_call_update",
-        toolCallId: "tc-ws",
-        status: "completed",
-        rawOutput: "search results",
-        // No title, no rawInput, no _meta — should use cached values
-      },
-    ]);
-
-    const streamId = "test-stream-merge-title";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Search for test" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    const assistantContent = turns[1].content as {
-      parts: { type: string; title?: string; name?: string; input?: unknown; output?: string }[];
-    };
-    const toolPart = assistantContent.parts.find((p) => p.type === "tool");
-    expect(toolPart?.title).toBe("Search query here");
-    expect(toolPart?.name).toBe("WebSearch");
-    expect(toolPart?.input).toEqual({ query: "test" });
-    expect(toolPart?.output).toBe("search results");
-  });
-
-  it("should accumulate history across multiple prompts", async () => {
-    const { agentManager } = await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Hello" },
-      },
-    ]);
-
-    const streamId = "test-stream-3";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    // First prompt
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Hi" }],
-      }, sessionId),
-    );
-
-    // Update mock for second prompt
-    (agentManager.prompt as ReturnType<typeof vi.fn>).mockReturnValue({
-      [Symbol.asyncIterator]: async function* () {
-        yield {
-          sessionUpdate: "agent_message_chunk",
-          content: { type: "text", text: "Goodbye" },
-        };
-      },
-    } as any);
-
-    // Second prompt
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Bye" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    // 2 prompts × 2 turns = 4 turns total
-    expect(turns).toHaveLength(4);
-    expect(turns[0].role).toBe("user");
-    expect(turns[0].content).toBe("Hi");
-    expect(turns[1].role).toBe("assistant");
-    expect(turns[2].role).toBe("user");
-    expect(turns[2].content).toBe("Bye");
-    expect(turns[3].role).toBe("assistant");
-  });
-
-  it("should return empty turns for session with no history", async () => {
-    await setup();
-
-    const streamId = "test-stream-4";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: unknown[] }).turns;
-    expect(turns).toEqual([]);
-  });
-
-  it("should only record completed tool calls, not running ones", async () => {
-    await setup([
-      {
-        sessionUpdate: "tool_call",
-        toolCallId: "tc-running",
-        title: "Running tool",
-        status: "running",
-        rawInput: {},
-      },
-      {
-        sessionUpdate: "tool_call",
-        toolCallId: "tc-done",
-        title: "Done tool",
-        status: "completed",
-        rawInput: { x: 1 },
-        rawOutput: "result",
-      },
-    ]);
-
-    const streamId = "test-stream-5";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Run tools" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    const assistantContent = turns[1].content as {
-      parts: { type: string; toolCallId?: string }[];
-    };
-
-    const toolParts = assistantContent.parts.filter((p) => p.type === "tool");
-    expect(toolParts).toHaveLength(1);
-    expect(toolParts[0].toolCallId).toBe("tc-done");
-  });
-
-  it("should restore history after reconnection via _resolve_ pattern (full TUI flow)", async () => {
-    // This test simulates the full TUI reconnection scenario:
-    // 1. Stream 1: Create session, send prompt (records turns)
-    // 2. Stream 2: Reconnect with _resolve_, loadSession returns sessionId, getHistory returns turns
-
-    const { agentManager } = await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Hello from agent!" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-
-    // ── Stream 1: Initial session ──
-    const stream1 = "stream-reconnect-1";
-    const sessionId = await initAndCreateSession(stream1, agentId);
-
-    // Send a prompt (records turns in EventStore)
-    await handler.processRequest(
-      agentId,
-      envelope(stream1, "session/prompt", {
-        prompt: [{ type: "text", text: "Hello agent" }],
-      }, sessionId),
-    );
-
-    // Verify turns exist
-    const check = await handler.processRequest(
-      agentId,
-      envelope(stream1, "_macro/getHistory", { sessionId }),
-    );
-    expect((check.acp.result as { turns: unknown[] }).turns).toHaveLength(2);
-
-    // ── Stream 2: Simulates TUI restart / reconnection ──
-    const stream2 = "stream-reconnect-2";
-
-    // Initialize the new stream
-    await handler.processRequest(
-      agentId,
-      envelope(stream2, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test-reconnect", version: "1.0" },
-      }),
-    );
-
-    // loadSession with _resolve_ pattern (TUI sends agentId in _meta)
-    // The mock agent has session_id: "session-1", so loadSession should resolve it
-    const loadResult = await handler.processRequest(
-      agentId,
-      envelope(stream2, "session/load", {
-        sessionId: "_resolve_",
-        cwd: "/test",
-        mcpServers: [],
-        _meta: { agentId },
-      }),
-    );
-
-    // The loadResult MUST include the resolved sessionId
-    const resolvedSessionId = (loadResult.acp.result as { sessionId?: string })?.sessionId;
-    expect(resolvedSessionId).toBeDefined();
-    expect(resolvedSessionId).toBe(sessionId);
-    expect(resolvedSessionId).not.toBe("_resolve_");
-
-    // Now the TUI uses the resolved sessionId to call getHistory
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(stream2, "_macro/getHistory", { sessionId: resolvedSessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    expect(turns).toHaveLength(2);
-    expect(turns[0].role).toBe("user");
-    expect(turns[0].content).toBe("Hello agent");
-    expect(turns[1].role).toBe("assistant");
-    const content = turns[1].content as { parts: { type: string; text: string }[] };
-    expect(content.parts[0].text).toBe("Hello from agent!");
-  });
-
-  it("should return sessionId from loadSession on all code paths", async () => {
-    // Verify that loadSession always returns { sessionId } so the TUI
-    // never falls back to using agentId for history queries.
-    await setup();
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "stream-sessionid-test";
-
-    // Initialize
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    // Create a session first so the agent exists
-    const newResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/new", {
-        cwd: "/test",
-        mcpServers: [],
-      }),
-    );
-    const sessionId = (newResult.acp.result as { sessionId?: string })?.sessionId;
-    expect(sessionId).toBeDefined();
-
-    // Register the agent in EventStore so loadSession can resolve it
-    registerAgent(agentId, sessionId!);
-
-    // Now loadSession on a new stream — should return the sessionId
-    const stream2 = "stream-sessionid-test-2";
-    await handler.processRequest(
-      agentId,
-      envelope(stream2, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    const loadResult = await handler.processRequest(
-      agentId,
-      envelope(stream2, "session/load", {
-        sessionId: "_resolve_",
-        cwd: "/test",
-        mcpServers: [],
-        _meta: { agentId },
-      }),
-    );
-
-    const resolved = (loadResult.acp.result as { sessionId?: string })?.sessionId;
-    expect(resolved).toBe(sessionId);
-  });
-
-  it("should stream notifications back via emitNotification callback", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Hello" },
-      },
-    ]);
-
-    const streamId = "test-stream-6";
-    const agentId = "agent-1" as AgentId;
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    const notifications: ACPEnvelope[] = [];
-    const emitNotification = (n: ACPEnvelope) => notifications.push(n);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Say hello" }],
-      }, sessionId),
-      emitNotification,
-    );
-
-    // Should have at least the session update notification
-    const sessionUpdates = notifications.filter(
-      (n) => n.acp.method === "session/update",
-    );
-    expect(sessionUpdates.length).toBeGreaterThan(0);
-
-    // AND history should still be recorded
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: unknown[] }).turns;
-    expect(turns).toHaveLength(2);
-  });
-
-  it("should resolve history via agentId when sessionId is different (restart scenario)", async () => {
-    // This test simulates the scenario where:
-    // 1. Session is created, prompt is sent (history recorded under sessionId)
-    // 2. Server restarts, resume() fails, TUI creates a NEW session
-    // 3. TUI calls _macro/getHistory with the NEW sessionId + agentId
-    // 4. Server uses agentId to resolve the OLD session and return its history
-
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "I remember you!" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "stream-agentid-lookup";
-    const originalSessionId = await initAndCreateSession(streamId, agentId);
-
-    // Send a prompt (records turns under originalSessionId)
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Remember me" }],
-      }, originalSessionId),
-    );
-
-    // Verify turns exist under original session
-    const check = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId: originalSessionId }),
-    );
-    expect((check.acp.result as { turns: unknown[] }).turns).toHaveLength(2);
-
-    // Simulate calling getHistory with a DIFFERENT sessionId + agentId
-    // (as happens when resume fails and TUI creates a new session).
-    // agentId takes priority — resolves to the original session with history.
-    const newSessionId = "completely-different-session-id";
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", {
-        sessionId: newSessionId,
-        agentId,
-      }),
-    );
-
-    // agentId resolves to the OLD session, so we should get history
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    expect(turns).toHaveLength(2);
-    expect(turns[0].role).toBe("user");
-    expect(turns[0].content).toBe("Remember me");
-    expect(turns[1].role).toBe("assistant");
-    const content = turns[1].content as { parts: { text: string }[] };
-    expect(content.parts[0].text).toBe("I remember you!");
-
-    // Without agentId, only sessionId is used — new session has no turns
-    const historyNoAgent = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", {
-        sessionId: newSessionId,
-      }),
-    );
-    const turnsNoAgent = (historyNoAgent.acp.result as { turns: unknown[] }).turns;
-    expect(turnsNoAgent).toHaveLength(0);
-  });
-
-  it("should include agent cwd in getHistory response", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Working in dir" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-cwd";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Where are you?" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-
-    const result = historyResult.acp.result as { turns: unknown[]; cwd: string | null };
-    expect(result.cwd).toBe("/test/cwd");
-  });
-
-  it("should return null cwd when agentId is not provided", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Hello" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-cwd-null";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Hi" }],
-      }, sessionId),
-    );
-
-    // Query without agentId — cwd should be null
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId }),
-    );
-
-    const result = historyResult.acp.result as { turns: unknown[]; cwd: string | null };
-    expect(result.cwd).toBeNull();
-  });
-
-  it("should capture plan entries from streaming and include in getHistory", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Planning..." },
-      },
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Analyze codebase", priority: "high", status: "in_progress" },
-          { content: "Write tests", priority: "medium", status: "pending" },
-          { content: "Deploy", priority: "low", status: "pending" },
-        ],
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: " Done." },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-plan";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Make a plan" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-
-    const result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-    };
-
-    expect(result.plan).toHaveLength(3);
-    expect(result.plan[0]).toEqual({
-      content: "Analyze codebase",
-      priority: "high",
-      status: "in_progress",
-    });
-    expect(result.plan[1]).toEqual({
-      content: "Write tests",
-      priority: "medium",
-      status: "pending",
-    });
-    expect(result.plan[2]).toEqual({
-      content: "Deploy",
-      priority: "low",
-      status: "pending",
-    });
-  });
-
-  it("should return empty plan when no plan updates were received", async () => {
-    await setup([
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "No plan here" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-no-plan";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Just chat" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-
-    const result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: unknown[];
-    };
-
-    expect(result.plan).toEqual([]);
-  });
-
-  it("should use latest plan when multiple plan updates are received", async () => {
-    await setup([
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Step 1", priority: "high", status: "pending" },
-        ],
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Working..." },
-      },
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Step 1", priority: "high", status: "completed" },
-          { content: "Step 2", priority: "medium", status: "in_progress" },
-        ],
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-plan-latest";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Work on it" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-
-    const result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-    };
-
-    // Should have the LATEST plan (second update)
-    expect(result.plan).toHaveLength(2);
-    expect(result.plan[0].status).toBe("completed");
-    expect(result.plan[1].status).toBe("in_progress");
-  });
-
-  it("should persist plan across multiple prompts (latest wins)", async () => {
-    const { agentManager } = await setup([
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Initial task", priority: "high", status: "in_progress" },
-        ],
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "First response" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-plan-persist";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    // First prompt — sets initial plan
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Start working" }],
-      }, sessionId),
-    );
-
-    // Verify plan from first prompt
-    let historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-    let result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-    };
-    expect(result.plan).toHaveLength(1);
-    expect(result.plan[0].content).toBe("Initial task");
-
-    // Second prompt with updated plan
-    (agentManager.prompt as ReturnType<typeof vi.fn>).mockReturnValue({
-      [Symbol.asyncIterator]: async function* () {
-        yield {
-          sessionUpdate: "plan",
-          entries: [
-            { content: "Initial task", priority: "high", status: "completed" },
-            { content: "New task", priority: "medium", status: "in_progress" },
-          ],
-        };
-        yield {
-          sessionUpdate: "agent_message_chunk",
-          content: { type: "text", text: "Second response" },
-        };
-      },
-    } as any);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Continue" }],
-      }, sessionId),
-    );
-
-    // Plan should now reflect the second prompt's update
-    historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-    result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-    };
-    expect(result.plan).toHaveLength(2);
-    expect(result.plan[0].status).toBe("completed");
-    expect(result.plan[1].content).toBe("New task");
-  });
-
-  it("should keep plan from earlier prompt if new prompt has no plan updates", async () => {
-    const { agentManager } = await setup([
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Persistent task", priority: "high", status: "in_progress" },
-        ],
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "First" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-plan-keep";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    // First prompt — sets plan
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Plan it" }],
-      }, sessionId),
-    );
-
-    // Second prompt with NO plan updates
-    (agentManager.prompt as ReturnType<typeof vi.fn>).mockReturnValue({
-      [Symbol.asyncIterator]: async function* () {
-        yield {
-          sessionUpdate: "agent_message_chunk",
-          content: { type: "text", text: "No plan this time" },
-        };
-      },
-    } as any);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Just chat" }],
-      }, sessionId),
-    );
-
-    // Plan should still be present from the first prompt
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-    const result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-    };
-    expect(result.plan).toHaveLength(1);
-    expect(result.plan[0].content).toBe("Persistent task");
-  });
-
-  // ─────────────────────────────────────────────────────────────────
-  // Fork history tests
-  // ─────────────────────────────────────────────────────────────────
-
-  /** Register a forked agent with fork_of metadata */
-  function registerForkedAgent(
-    agentId: string,
-    sessionId: string,
-    sourceAgentId: string,
-    createdAt?: number,
-  ): void {
-    eventStore.emit({
-      type: "spawn",
-      source: { agent_id: sourceAgentId },
-      payload: {
-        agent_id: agentId,
-        session_id: sessionId,
-        task: `[Fork of ${sourceAgentId}]`,
-        task_id: "task-fork",
-        cwd: "/test/cwd",
-      },
-    });
-    // Set fork_of metadata (mirrors what agent-manager.ts does)
-    eventStore.updateAgentMetadata(agentId as AgentId, {
-      metadata: { fork_of: sourceAgentId },
-    });
-    // Backdate created_at if specified (for timestamp filtering tests)
-    if (createdAt !== undefined) {
-      // Overwrite the agent row's created_at by re-emitting a lifecycle event
-      // at the desired time — but since we can't change created_at directly,
-      // we rely on the spawn event timestamp. For testing, we'll emit the
-      // spawn before recording turns to ensure correct ordering.
-    }
-    // Mark as running
-    eventStore.emit({
-      type: "lifecycle",
-      source: { agent_id: agentId },
-      payload: {
-        agent_id: agentId,
-        action: "started",
-      },
-    });
-  }
-
-  /** Record a turn directly in the event store (bypasses prompt flow) */
-  function recordTurn(
-    agentId: string,
-    conversationId: string,
-    role: "user" | "assistant",
-    content: unknown,
-  ): void {
-    const now = Date.now();
-    eventStore.emit({
-      type: "turn",
-      source: { agent_id: agentId },
-      payload: {
-        action: "recorded",
-        turn_id: `turn_${role}_${now}_${Math.random().toString(36).slice(2, 8)}`,
-        conversation_id: conversationId,
-        participant: role,
-        timestamp: now,
-        content_type: role === "user" ? "user_prompt" : "assistant_response",
-        content,
-        source_type: "acp",
-      },
-    });
-  }
-
-  it("should include source agent history for forked agents", async () => {
-    await setup();
-
-    const sourceAgentId = "agent-source" as AgentId;
-    const sourceSessionId = "session-source";
-    const forkedAgentId = "agent-forked" as AgentId;
-    const forkedSessionId = "session-forked";
-
-    // Register source agent and record turns
-    registerAgent(sourceAgentId, sourceSessionId);
-    recordTurn(sourceAgentId, sourceSessionId, "user", "Hello source");
-    recordTurn(sourceAgentId, sourceSessionId, "assistant", { parts: [{ type: "text", text: "Hi from source" }] });
-
-    // Register forked agent with fork_of metadata
-    registerForkedAgent(forkedAgentId, forkedSessionId, sourceAgentId);
-
-    // Initialize stream for forked agent
-    const streamId = "fork-history-stream";
-    await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    // Query history for forked agent
-    const historyResult = await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "_macro/getHistory", { agentId: forkedAgentId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    // Should include source agent's 2 turns
-    expect(turns).toHaveLength(2);
-    expect(turns[0].role).toBe("user");
-    expect(turns[0].content).toBe("Hello source");
-    expect(turns[1].role).toBe("assistant");
-  });
-
-  it("should combine source and forked agent turns in order", async () => {
-    await setup();
-
-    const sourceAgentId = "agent-src" as AgentId;
-    const sourceSessionId = "session-src";
-    const forkedAgentId = "agent-fork" as AgentId;
-    const forkedSessionId = "session-fork";
-
-    // Source agent conversation
-    registerAgent(sourceAgentId, sourceSessionId);
-    recordTurn(sourceAgentId, sourceSessionId, "user", "Question 1");
-    recordTurn(sourceAgentId, sourceSessionId, "assistant", { parts: [{ type: "text", text: "Answer 1" }] });
-
-    // Fork the agent
-    registerForkedAgent(forkedAgentId, forkedSessionId, sourceAgentId);
-
-    // Forked agent has its own turns
-    recordTurn(forkedAgentId, forkedSessionId, "user", "Question 2 (forked)");
-    recordTurn(forkedAgentId, forkedSessionId, "assistant", { parts: [{ type: "text", text: "Answer 2 (forked)" }] });
-
-    const streamId = "fork-combined-stream";
-    await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    const historyResult = await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "_macro/getHistory", { agentId: forkedAgentId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    // Source turns (2) + forked turns (2) = 4
-    expect(turns).toHaveLength(4);
-    expect(turns[0].content).toBe("Question 1");
-    expect(turns[2].content).toBe("Question 2 (forked)");
-  });
-
-  it("should not include source turns recorded after the fork", async () => {
-    await setup();
-
-    const sourceAgentId = "agent-pre" as AgentId;
-    const sourceSessionId = "session-pre";
-    const forkedAgentId = "agent-post" as AgentId;
-    const forkedSessionId = "session-post";
-
-    // Source agent: record a turn before the fork
-    registerAgent(sourceAgentId, sourceSessionId);
-    recordTurn(sourceAgentId, sourceSessionId, "user", "Before fork");
-    recordTurn(sourceAgentId, sourceSessionId, "assistant", { parts: [{ type: "text", text: "Pre-fork reply" }] });
-
-    // Small delay to ensure fork timestamp is after source turns
-    await new Promise((r) => setTimeout(r, 10));
-
-    // Fork the agent
-    registerForkedAgent(forkedAgentId, forkedSessionId, sourceAgentId);
-
-    // Small delay to ensure post-fork turn timestamp is after fork
-    await new Promise((r) => setTimeout(r, 10));
-
-    // Source agent continues after fork — these should NOT appear in forked history
-    recordTurn(sourceAgentId, sourceSessionId, "user", "After fork on source");
-    recordTurn(sourceAgentId, sourceSessionId, "assistant", { parts: [{ type: "text", text: "Post-fork source reply" }] });
-
-    const streamId = "fork-filter-stream";
-    await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    const historyResult = await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "_macro/getHistory", { agentId: forkedAgentId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-
-    // Should only include the 2 pre-fork turns, not the 2 post-fork ones
-    expect(turns).toHaveLength(2);
-    expect(turns[0].content).toBe("Before fork");
-    expect(turns.find((t: { content: unknown }) => t.content === "After fork on source")).toBeUndefined();
-  });
-
-  it("should return empty history for forked agent when source has no turns", async () => {
-    await setup();
-
-    const sourceAgentId = "agent-empty-src" as AgentId;
-    const sourceSessionId = "session-empty-src";
-    const forkedAgentId = "agent-empty-fork" as AgentId;
-    const forkedSessionId = "session-empty-fork";
-
-    // Source agent exists but has no conversation turns
-    registerAgent(sourceAgentId, sourceSessionId);
-    registerForkedAgent(forkedAgentId, forkedSessionId, sourceAgentId);
-
-    const streamId = "fork-empty-stream";
-    await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    const historyResult = await handler.processRequest(
-      forkedAgentId,
-      envelope(streamId, "_macro/getHistory", { agentId: forkedAgentId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: unknown[] }).turns;
-    expect(turns).toEqual([]);
-  });
-
-  it("should not affect non-forked agent history queries", async () => {
-    await setup();
-
-    const agentId = "agent-normal" as AgentId;
-    const sessionId = "session-normal";
-
-    registerAgent(agentId, sessionId);
-    recordTurn(agentId, sessionId, "user", "Normal question");
-    recordTurn(agentId, sessionId, "assistant", { parts: [{ type: "text", text: "Normal reply" }] });
-
-    const streamId = "normal-history-stream";
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "initialize", {
-        protocolVersion: 1,
-        capabilities: {},
-        clientInfo: { name: "test", version: "1.0" },
-      }),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { agentId }),
-    );
-
-    const turns = (historyResult.acp.result as { turns: { role: string; content: unknown }[] }).turns;
-    expect(turns).toHaveLength(2);
-    expect(turns[0].content).toBe("Normal question");
-  });
-
-  it("should include both plan and cwd together in getHistory response", async () => {
-    await setup([
-      {
-        sessionUpdate: "plan",
-        entries: [
-          { content: "Do something", priority: "high", status: "pending" },
-        ],
-      },
-      {
-        sessionUpdate: "agent_message_chunk",
-        content: { type: "text", text: "Got it" },
-      },
-    ]);
-
-    const agentId = "agent-1" as AgentId;
-    const streamId = "test-stream-plan-cwd";
-    const sessionId = await initAndCreateSession(streamId, agentId);
-
-    await handler.processRequest(
-      agentId,
-      envelope(streamId, "session/prompt", {
-        prompt: [{ type: "text", text: "Go" }],
-      }, sessionId),
-    );
-
-    const historyResult = await handler.processRequest(
-      agentId,
-      envelope(streamId, "_macro/getHistory", { sessionId, agentId }),
-    );
-
-    const result = historyResult.acp.result as {
-      turns: unknown[];
-      plan: Array<{ content: string; priority: string; status: string }>;
-      cwd: string | null;
-    };
-
-    // Both fields present
-    expect(result.turns).toHaveLength(2);
-    expect(result.plan).toHaveLength(1);
-    expect(result.plan[0].content).toBe("Do something");
-    expect(result.cwd).toBe("/test/cwd");
-  });
-});