macro-agent 0.2.4 → 0.2.6

package/src/mcp/__tests__/mcp-server-v2.test.ts

@@ -1,236 +0,0 @@
-/**
- * Tests for MCP Server V2 — simplified tool surface
- */
-
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-import { createMCPServerV2 } from "../mcp-server-v2.js";
-import { AgentStore } from "../../agent/agent-store.js";
-import type { AgentManager } from "../../agent/agent-manager.js";
-import type { InboxAdapter, TasksAdapter } from "../../adapters/types.js";
-import type { ToolContext } from "../types.js";
-import { DefaultRoleRegistry } from "../../roles/registry.js";
-
-// =============================================================================
-// Mocks
-// =============================================================================
-
-function createMockAgentManager(): AgentManager {
-  return {
-    spawn: vi.fn().mockResolvedValue({
-      id: "agent_child",
-      session_id: "session_child",
-      agent: { id: "agent_child", name: "child-agent", task_id: "task_child", state: "running" },
-      session: {},
-    }),
-    terminate: vi.fn().mockResolvedValue(undefined),
-    get: vi.fn().mockReturnValue(null),
-    list: vi.fn().mockReturnValue([]),
-    getChildren: vi.fn().mockReturnValue([]),
-    getHierarchy: vi.fn().mockReturnValue({
-      root: {
-        agent: { id: "agent_1", name: "root", task: "test", state: "running" },
-        children: [],
-      },
-      depth: 0,
-      totalAgents: 1,
-    }),
-    getSession: vi.fn().mockReturnValue(null),
-    hasActiveSession: vi.fn().mockReturnValue(false),
-    isPrompting: vi.fn().mockReturnValue(false),
-    prompt: vi.fn(),
-    setSpawnInterceptor: vi.fn(),
-    getRoleRegistry: vi.fn().mockReturnValue(new DefaultRoleRegistry()),
-    onLifecycleEvent: vi.fn().mockReturnValue(() => {}),
-    close: vi.fn().mockResolvedValue(undefined),
-  } as unknown as AgentManager;
-}
-
-function createMockInboxAdapter(): InboxAdapter {
-  return {
-    registerAgent: vi.fn().mockResolvedValue(undefined),
-    deregisterAgent: vi.fn().mockResolvedValue(undefined),
-    send: vi.fn().mockResolvedValue("msg-1"),
-    onDelivery: vi.fn(),
-    offDelivery: vi.fn(),
-    checkInbox: vi.fn().mockResolvedValue([]),
-    readThread: vi.fn().mockResolvedValue([]),
-    setSignalFilter: vi.fn(),
-    setEmissionValidator: vi.fn(),
-    socketPath: "/tmp/test.sock",
-    stop: vi.fn().mockResolvedValue(undefined),
-  } as unknown as InboxAdapter;
-}
-
-function createMockTasksAdapter(): TasksAdapter {
-  return {
-    createTask: vi.fn().mockResolvedValue("t-1"),
-    assignTask: vi.fn().mockResolvedValue(undefined),
-    transitionTask: vi.fn().mockResolvedValue(undefined),
-    getTask: vi.fn().mockResolvedValue({ id: "t-1", status: "open" }),
-    queryReady: vi.fn().mockResolvedValue([]),
-    listTasks: vi.fn().mockResolvedValue([]),
-    addBlocker: vi.fn().mockResolvedValue(undefined),
-    removeBlocker: vi.fn().mockResolvedValue(undefined),
-    claimTask: vi.fn().mockResolvedValue(null),
-    unclaimTask: vi.fn().mockResolvedValue(undefined),
-    listClaimable: vi.fn().mockResolvedValue([]),
-    connect: vi.fn().mockResolvedValue(undefined),
-    disconnect: vi.fn(),
-    connected: true,
-  } as unknown as TasksAdapter;
-}
-
-// =============================================================================
-// Tests
-// =============================================================================
-
-describe("MCP Server V2", () => {
-  let agentStore: AgentStore;
-  let agentManager: AgentManager;
-  let inboxAdapter: InboxAdapter;
-  let tasksAdapter: TasksAdapter;
-  let context: ToolContext;
-
-  beforeEach(() => {
-    agentStore = new AgentStore(":memory:");
-    agentManager = createMockAgentManager();
-    inboxAdapter = createMockInboxAdapter();
-    tasksAdapter = createMockTasksAdapter();
-
-    // Register the calling agent in store
-    agentStore.putAgent({
-      id: "agent_1",
-      role: "coordinator",
-      state: "running",
-      parent_id: null,
-      lineage: [],
-      scope: "default",
-      task: "test",
-      cwd: "/tmp",
-      capabilities: [],
-      created_at: Date.now(),
-    });
-
-    context = {
-      agent_id: "agent_1",
-      session_id: "session_1",
-      task_id: "task_1",
-      lineage: [],
-      cwd: "/tmp",
-    };
-  });
-
-  afterEach(() => {
-    agentStore.close();
-  });
-
-  describe("creation", () => {
-    it("should create an MCP server instance", () => {
-      const instance = createMCPServerV2(context, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-
-      expect(instance.server).toBeDefined();
-      expect(instance.start).toBeDefined();
-      expect(instance.close).toBeDefined();
-    });
-  });
-
-  describe("tool registration", () => {
-    it("should register core tools for coordinator role", () => {
-      const instance = createMCPServerV2(context, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-
-      // McpServer doesn't expose tool list directly, but we can verify
-      // it was created without errors (tools registered successfully)
-      expect(instance.server).toBeDefined();
-    });
-
-    it("should register tools for worker role", () => {
-      agentStore.putAgent({
-        id: "worker_1",
-        role: "worker",
-        state: "running",
-        parent_id: "agent_1",
-        lineage: ["agent_1"],
-        scope: "default",
-        task: "work",
-        cwd: "/tmp",
-        capabilities: [],
-        created_at: Date.now(),
-      });
-
-      const workerContext: ToolContext = {
-        agent_id: "worker_1",
-        session_id: "session_w1",
-        task_id: "task_w1",
-        lineage: ["agent_1"],
-        cwd: "/tmp",
-      };
-
-      const instance = createMCPServerV2(workerContext, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-
-      expect(instance.server).toBeDefined();
-    });
-  });
-
-  describe("tool surface comparison with V1", () => {
-    it("V2 should NOT have emit_status (replaced by inbox events)", () => {
-      // V2 MCP server only registers: done, spawn_agent, stop_agent, get_hierarchy, inject_context
-      // All messaging tools (send_message, check_messages) are in agent-inbox
-      // All task tools (create_task, get_task, claim_task, etc.) are in opentasks
-      const instance = createMCPServerV2(context, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-
-      // Verify server created successfully with reduced tool set
-      expect(instance.server).toBeDefined();
-    });
-
-    it("V2 should have exactly 5 tools for coordinator", () => {
-      // coordinator gets: done, spawn_agent, stop_agent, get_hierarchy, inject_context
-      // This is verified by the fact that createMCPServerV2 succeeds without errors
-      // and the tool count is limited to what's defined in the V2 factory
-      const instance = createMCPServerV2(context, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-      expect(instance.server).toBeDefined();
-    });
-  });
-
-  describe("inject_context uses inbox", () => {
-    it("should use inboxAdapter.send for context injection", async () => {
-      // We can't easily call tool handlers directly on McpServer,
-      // but we can verify the inject_context tool uses inboxAdapter
-      // by checking that it's registered with the right dependencies
-      const instance = createMCPServerV2(context, {
-        agentStore,
-        agentManager,
-        inboxAdapter,
-        tasksAdapter,
-      });
-
-      // The tool is registered on the server — actual handler verification
-      // would require running the MCP server with a transport
-      expect(instance.server).toBeDefined();
-    });
-  });
-});

package/src/mcp/index.ts

@@ -1,6 +0,0 @@
-/**
- * MCP module exports (V2)
- */
-
-export * from "./types.js";
-export * from "./mcp-server-v2.js";