macro-agent 0.2.5 → 0.2.6

package/src/cli/__tests__/stable-instance-id.test.ts

@@ -1,57 +0,0 @@
-/**
- * Tests for getStableInstanceId
- *
- * Verifies that the stable instance ID is deterministic, consistent,
- * and produces valid instance IDs for the EventStore.
- */
-
-import { describe, it, expect } from "vitest";
-import { resolve } from "node:path";
-import { getStableInstanceId } from "../stable-instance-id.js";
-
-describe("getStableInstanceId", () => {
-  it("should return the same ID for the same path", () => {
-    const id1 = getStableInstanceId("/Users/test/project");
-    const id2 = getStableInstanceId("/Users/test/project");
-    expect(id1).toBe(id2);
-  });
-
-  it("should return different IDs for different paths", () => {
-    const id1 = getStableInstanceId("/Users/test/project-a");
-    const id2 = getStableInstanceId("/Users/test/project-b");
-    expect(id1).not.toBe(id2);
-  });
-
-  it("should start with inst_ prefix", () => {
-    const id = getStableInstanceId("/tmp/test");
-    expect(id).toMatch(/^inst_/);
-  });
-
-  it("should produce a valid instance ID format (alphanumeric + underscore)", () => {
-    const id = getStableInstanceId("/tmp/test");
-    expect(id).toMatch(/^[a-zA-Z0-9_]+$/);
-  });
-
-  it("should normalize relative paths to absolute", () => {
-    // Both should resolve to the same absolute path
-    const absPath = resolve("./src");
-    const id1 = getStableInstanceId("./src");
-    const id2 = getStableInstanceId(absPath);
-    expect(id1).toBe(id2);
-  });
-
-  it("should handle paths with trailing slashes consistently", () => {
-    // resolve() strips trailing slashes, so these should be equal
-    const id1 = getStableInstanceId("/tmp/project");
-    const id2 = getStableInstanceId("/tmp/project/");
-    expect(id1).toBe(id2);
-  });
-
-  it("should produce a fixed-length ID", () => {
-    const short = getStableInstanceId("/a");
-    const long = getStableInstanceId("/very/long/path/to/some/deeply/nested/project/directory");
-    // Both should be inst_ + 12 hex chars = 17 chars total
-    expect(short.length).toBe(17);
-    expect(long.length).toBe(17);
-  });
-});

package/src/cli/acp.ts

@@ -1,101 +0,0 @@
-#!/usr/bin/env node
-/**
- * Multi-Agent Server Entry Point (V2)
- *
- * Runs macro-agent using the V2 architecture:
- * - AgentStore for lifecycle state
- * - InboxAdapter (embedded agent-inbox) for messaging
- * - TasksAdapter (opentasks client) for task management
- * - AgentManagerV2 for agent lifecycle
- *
- * Usage:
- *   multiagent [options]
- *
- * Options:
- *   --cwd <path>       Working directory for agents
- *   --acp              Stdio ACP-only mode (for embedded use with acp-factory)
- */
-
-import { bootV2 } from "../boot-v2.js";
-import { parseArgs } from "./parse-args.js";
-
-// Re-export utilities for backwards compatibility
-export { parseArgs, type ACPServerOptions } from "./parse-args.js";
-export { getStableInstanceId } from "./stable-instance-id.js";
-
-// ─────────────────────────────────────────────────────────────────
-// Main Entry Point
-// ─────────────────────────────────────────────────────────────────
-
-async function main() {
-  const options = parseArgs();
-  const defaultCwd = options.cwd ?? process.cwd();
-
-  try {
-    const system = await bootV2({
-      cwd: defaultCwd,
-      // `--instance-id` picks the on-disk compartment under ~/.macro-agent/
-      // so multiple runs with the same id share state. Omit to auto-derive
-      // from the cwd hash. See `BootV2Config.instanceId`.
-      ...(options.instanceId ? { instanceId: options.instanceId } : {}),
-      // Enable ACP WebSocket server if port is specified (server mode)
-      ...(options.port
-        ? {
-            acp: {
-              enabled: true,
-              port: options.port,
-              host: options.host ?? "localhost",
-            },
-          }
-        : {}),
-    });
-
-    const acpUrl = system.acpServer
-      ? `ws://${options.host ?? "localhost"}:${options.port}/acp`
-      : null;
-    console.error("[acp] V2 system booted successfully.");
-    if (acpUrl) {
-      console.error(`[acp] MAP WebSocket: ${acpUrl}`);
-    }
-
-    // Cleanup function
-    const cleanup = async () => {
-      try {
-        await system.shutdown();
-      } catch (err) {
-        console.error(`[cleanup] Shutdown failed: ${err}`);
-      }
-    };
-
-    // Handle graceful shutdown
-    process.on("SIGINT", async () => {
-      await cleanup();
-      process.exit(0);
-    });
-
-    process.on("SIGTERM", async () => {
-      await cleanup();
-      process.exit(0);
-    });
-
-    if (options.acp) {
-      // Stdio ACP-only mode - keep process alive until stdin closes
-      process.stdin.resume();
-      process.stdin.on("end", async () => {
-        await cleanup();
-        process.exit(0);
-      });
-    } else {
-      // Full server mode - keep process alive
-      console.error(`[acp] System ready. Press Ctrl+C to stop.`);
-      await new Promise<void>(() => {
-        // Never resolves - process stays alive until signal
-      });
-    }
-  } catch (error) {
-    console.error(`ACP server error: ${error}`);
-    process.exit(1);
-  }
-}
-
-main();

package/src/cli/inbox-mcp-proxy.ts

@@ -1,56 +0,0 @@
-/**
- * inbox-mcp-proxy.ts — stdio entry that exposes agent-inbox tools as an
- * MCP server to a spawned worker.
- *
- * Why this exists:
- *
- *   The agent-inbox package ships an `InboxMcpProxy` (in dist/mcp/mcp-proxy.js)
- *   designed to bridge agent-inbox IPC ↔ MCP-stdio. macro-agent's
- *   `agentManager.spawn` configures this as a per-spawn MCP server so the
- *   spawned worker has access to `send_message`, `check_inbox`,
- *   `read_thread`, `list_agents` — the tools the macro-agent architecture
- *   docs claim are available, but which were not actually being mounted on
- *   workers prior to this fix.
- *
- *   Particularly important for mail-inbound dispatch workers, which run with
- *   `isolatedSettings: true` and therefore can't pick up host-level plugin
- *   MCPs that would otherwise have provided agent-inbox.
- *
- * Env vars consumed:
- *
- *   INBOX_SOCKET_PATH  (required)  — path to agent-inbox's IPC socket. Set
- *                                    by `buildMcpServerConfig` in
- *                                    agent-manager-v2.ts.
- *   MACRO_AGENT_ID     (optional)  — the spawned worker's agent id; used as
- *                                    the proxy's `defaultAgentId` so tools
- *                                    like check_inbox auto-target the
- *                                    caller's mailbox.
- *
- * Failure mode:
- *
- *   If INBOX_SOCKET_PATH is unset, the script exits with a non-zero status
- *   so Claude Code's MCP-init reports the misconfiguration loudly rather
- *   than silently leaving the worker without inbox tools.
- */
-
-import { InboxMcpProxy } from "agent-inbox";
-
-async function main(): Promise<void> {
-  const socketPath = process.env.INBOX_SOCKET_PATH;
-  if (!socketPath) {
-    console.error(
-      "[inbox-mcp-proxy] INBOX_SOCKET_PATH is unset — cannot start. " +
-        "macro-agent's agentManager.spawn should always inject this env var.",
-    );
-    process.exit(1);
-  }
-
-  const agentId = process.env.MACRO_AGENT_ID || "anonymous";
-  const proxy = new InboxMcpProxy(socketPath, agentId);
-  await proxy.start();
-}
-
-main().catch((err) => {
-  console.error(`[inbox-mcp-proxy] Fatal: ${(err as Error).message}`);
-  process.exit(1);
-});