paperclip-plugin-telegram 0.2.0 → 0.2.2

package/tests/acp-bridge.test.ts

@@ -1,314 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { getSessions, routeMessageToAgent, handleHandoffToolCall } from "../src/acp-bridge.js";
-import type { PluginContext } from "@paperclipai/plugin-sdk";
-
-let sentMessages: Array<{ chatId: string; text: string; options?: Record<string, unknown> }> = [];
-let stateStore: Record<string, unknown> = {};
-let emittedEvents: Array<{ event: string; companyId: string; payload: unknown }> = [];
-
-vi.mock("../src/telegram-api.js", async () => {
-  const actual = await vi.importActual("../src/telegram-api.js") as Record<string, unknown>;
-  return {
-    ...actual,
-    sendMessage: vi.fn(async (_ctx: unknown, _token: string, chatId: string, text: string, options?: Record<string, unknown>) => {
-      sentMessages.push({ chatId, text, options });
-      return 100;
-    }),
-    sendChatAction: vi.fn(),
-    editMessage: vi.fn().mockResolvedValue(true),
-  };
-});
-
-function mockCtx(): PluginContext {
-  return {
-    http: { fetch: vi.fn() },
-    metrics: { write: vi.fn() },
-    state: {
-      get: vi.fn(async (key: { stateKey: string }) => stateStore[key.stateKey] ?? null),
-      set: vi.fn(async (key: { stateKey: string }, value: unknown) => {
-        stateStore[key.stateKey] = value;
-      }),
-    },
-    logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-    events: {
-      emit: vi.fn((event: string, companyId: string, payload: unknown) => {
-        emittedEvents.push({ event, companyId, payload });
-      }),
-      on: vi.fn(),
-    },
-    agents: {
-      get: vi.fn().mockResolvedValue(null),
-      sessions: {
-        create: vi.fn().mockResolvedValue({ sessionId: "native-session-1" }),
-        sendMessage: vi.fn(),
-        close: vi.fn(),
-      },
-    },
-  } as unknown as PluginContext;
-}
-
-beforeEach(() => {
-  sentMessages = [];
-  stateStore = {};
-  emittedEvents = [];
-});
-
-describe("getSessions", () => {
-  it("returns empty array when no sessions exist", async () => {
-    const ctx = mockCtx();
-    const sessions = await getSessions(ctx, "chat-1", 42);
-    expect(sessions).toEqual([]);
-  });
-
-  it("returns stored sessions", async () => {
-    const session = {
-      sessionId: "s1",
-      agentId: "a1",
-      agentName: "builder",
-      agentDisplayName: "Builder",
-      transport: "acp",
-      spawnedAt: "2026-01-01T00:00:00Z",
-      status: "active",
-      lastActivityAt: "2026-01-01T00:00:00Z",
-    };
-    stateStore["sessions_chat-1_42"] = [session];
-    const ctx = mockCtx();
-    const sessions = await getSessions(ctx, "chat-1", 42);
-    expect(sessions).toHaveLength(1);
-    expect(sessions[0].sessionId).toBe("s1");
-  });
-});
-
-describe("routeMessageToAgent - @mention extraction", () => {
-  it("routes to agent matching @mention (exact match)", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [
-      {
-        sessionId: "s1",
-        agentId: "a1",
-        agentName: "builder",
-        agentDisplayName: "Builder",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-      {
-        sessionId: "s2",
-        agentId: "a2",
-        agentName: "tester",
-        agentDisplayName: "Tester",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-    ];
-
-    const result = await routeMessageToAgent(ctx, "token", "chat-1", 42, "@builder hello", undefined, "company-1");
-
-    expect(result).toBe(true);
-    // Should have emitted to builder session (s1)
-    const emitted = emittedEvents.find(e => e.event === "acp-spawn");
-    expect(emitted).toBeDefined();
-    expect((emitted!.payload as Record<string, unknown>).sessionId).toBe("s1");
-  });
-
-  it("routes to agent matching @mention case-insensitively", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [
-      {
-        sessionId: "s1",
-        agentId: "a1",
-        agentName: "builder",
-        agentDisplayName: "Builder",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-    ];
-
-    const result = await routeMessageToAgent(ctx, "token", "chat-1", 42, "@BUILDER help me", undefined, "company-1");
-    expect(result).toBe(true);
-  });
-
-  it("routes via partial @mention match", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [
-      {
-        sessionId: "s1",
-        agentId: "a1",
-        agentName: "codebuilder",
-        agentDisplayName: "Codebuilder",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-    ];
-
-    const result = await routeMessageToAgent(ctx, "token", "chat-1", 42, "@code do this", undefined, "company-1");
-    expect(result).toBe(true);
-    const emitted = emittedEvents.find(e => e.event === "acp-spawn");
-    expect((emitted!.payload as Record<string, unknown>).sessionId).toBe("s1");
-  });
-});
-
-describe("routeMessageToAgent - reply-to fallback", () => {
-  it("routes to agent via reply-to message mapping", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [
-      {
-        sessionId: "s1",
-        agentId: "a1",
-        agentName: "builder",
-        agentDisplayName: "Builder",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-      {
-        sessionId: "s2",
-        agentId: "a2",
-        agentName: "tester",
-        agentDisplayName: "Tester",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-    ];
-    stateStore["agent_msg_chat-1_99"] = { sessionId: "s2" };
-
-    const result = await routeMessageToAgent(ctx, "token", "chat-1", 42, "continue this", 99, "company-1");
-    expect(result).toBe(true);
-
-    const emitted = emittedEvents.find(e => e.event === "acp-spawn");
-    expect((emitted!.payload as Record<string, unknown>).sessionId).toBe("s2");
-  });
-
-  it("falls back to most recently active when no mention or reply match", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [
-      {
-        sessionId: "s1",
-        agentId: "a1",
-        agentName: "builder",
-        agentDisplayName: "Builder",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-01T00:00:00Z",
-      },
-      {
-        sessionId: "s2",
-        agentId: "a2",
-        agentName: "tester",
-        agentDisplayName: "Tester",
-        transport: "acp",
-        spawnedAt: "2026-01-01T00:00:00Z",
-        status: "active",
-        lastActivityAt: "2026-01-02T00:00:00Z", // more recent
-      },
-    ];
-
-    await routeMessageToAgent(ctx, "token", "chat-1", 42, "do something", undefined, "company-1");
-
-    const emitted = emittedEvents.find(e => e.event === "acp-spawn");
-    expect((emitted!.payload as Record<string, unknown>).sessionId).toBe("s2");
-  });
-
-  it("returns false when no active sessions exist", async () => {
-    const ctx = mockCtx();
-    const result = await routeMessageToAgent(ctx, "token", "chat-1", 42, "hello", undefined, "company-1");
-    expect(result).toBe(false);
-  });
-});
-
-describe("Session registry - max enforcement", () => {
-  it("getSessions returns sessions from state key based on chatId and threadId", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_10"] = [{ sessionId: "a" }];
-    stateStore["sessions_chat-1_20"] = [{ sessionId: "b" }];
-
-    const s1 = await getSessions(ctx, "chat-1", 10);
-    const s2 = await getSessions(ctx, "chat-1", 20);
-    expect(s1[0].sessionId).toBe("a");
-    expect(s2[0].sessionId).toBe("b");
-  });
-});
-
-describe("handleHandoffToolCall - approval callback data", () => {
-  it("returns error when required fields are missing", async () => {
-    const ctx = mockCtx();
-    const result = await handleHandoffToolCall(ctx, "token", {}, "company-1", "agent-1");
-    expect(result.error).toContain("Missing required fields");
-  });
-
-  it("creates pending handoff with approval buttons when requiresApproval is true", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [{
-      sessionId: "s1",
-      agentId: "agent-1",
-      agentName: "builder",
-      agentDisplayName: "Builder",
-      transport: "acp",
-      spawnedAt: "2026-01-01T00:00:00Z",
-      status: "active",
-      lastActivityAt: "2026-01-01T00:00:00Z",
-    }];
-
-    const result = await handleHandoffToolCall(ctx, "token", {
-      targetAgent: "tester",
-      reason: "needs testing",
-      contextSummary: "code is ready",
-      requiresApproval: true,
-      chatId: "chat-1",
-      threadId: 42,
-    }, "company-1", "agent-1");
-
-    expect(result.content).toBeDefined();
-    const parsed = JSON.parse(result.content!);
-    expect(parsed.status).toBe("pending_approval");
-
-    // Should have sent message with approve/reject buttons
-    const msg = sentMessages.find(m => m.text.includes("Handing off"));
-    expect(msg).toBeDefined();
-    const keyboard = msg!.options?.inlineKeyboard as Array<Array<{ text: string; callback_data: string }>>;
-    expect(keyboard).toBeDefined();
-    const approveBtn = keyboard.flat().find((b: { text: string }) => b.text === "Approve");
-    const rejectBtn = keyboard.flat().find((b: { text: string }) => b.text === "Reject");
-    expect(approveBtn).toBeDefined();
-    expect(rejectBtn).toBeDefined();
-    expect(approveBtn!.callback_data).toMatch(/^handoff_approve_/);
-    expect(rejectBtn!.callback_data).toMatch(/^handoff_reject_/);
-  });
-
-  it("executes handoff immediately when requiresApproval is false", async () => {
-    const ctx = mockCtx();
-    stateStore["sessions_chat-1_42"] = [{
-      sessionId: "s1",
-      agentId: "agent-1",
-      agentName: "builder",
-      agentDisplayName: "Builder",
-      transport: "acp",
-      spawnedAt: "2026-01-01T00:00:00Z",
-      status: "active",
-      lastActivityAt: "2026-01-01T00:00:00Z",
-    }];
-
-    const result = await handleHandoffToolCall(ctx, "token", {
-      targetAgent: "tester",
-      reason: "needs testing",
-      contextSummary: "code is ready",
-      requiresApproval: false,
-      chatId: "chat-1",
-      threadId: 42,
-    }, "company-1", "agent-1");
-
-    const parsed = JSON.parse(result.content!);
-    expect(parsed.status).toBe("handed_off");
-  });
-});

package/tests/command-registry.test.ts

@@ -1,283 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { handleCommandsCommand, tryCustomCommand } from "../src/command-registry.js";
-import type { PluginContext } from "@paperclipai/plugin-sdk";
-
-let sentMessages: Array<{ chatId: string; text: string; options?: Record<string, unknown> }> = [];
-let stateStore: Record<string, unknown> = {};
-
-vi.mock("../src/telegram-api.js", async () => {
-  const actual = await vi.importActual("../src/telegram-api.js") as Record<string, unknown>;
-  return {
-    ...actual,
-    sendMessage: vi.fn(async (_ctx: unknown, _token: string, chatId: string, text: string, options?: Record<string, unknown>) => {
-      sentMessages.push({ chatId, text, options });
-      return 1;
-    }),
-    sendChatAction: vi.fn(),
-  };
-});
-
-function mockCtx(): PluginContext {
-  return {
-    http: {
-      fetch: vi.fn().mockResolvedValue({
-        json: () => Promise.resolve({ id: "created-issue-1" }),
-        text: () => Promise.resolve("ok"),
-      }),
-    },
-    metrics: { write: vi.fn() },
-    state: {
-      get: vi.fn(async (key: { stateKey: string }) => stateStore[key.stateKey] ?? null),
-      set: vi.fn(async (key: { stateKey: string }, value: unknown) => {
-        stateStore[key.stateKey] = value;
-      }),
-    },
-    logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-    config: { get: vi.fn().mockResolvedValue({ paperclipBaseUrl: "http://localhost:3100" }) },
-    issues: { get: vi.fn().mockResolvedValue({ id: "i1", title: "Test", status: "open" }) },
-    agents: { invoke: vi.fn().mockResolvedValue({ runId: "run-1" }) },
-  } as unknown as PluginContext;
-}
-
-beforeEach(() => {
-  sentMessages = [];
-  stateStore = {};
-});
-
-describe("handleCommandsCommand - subcommands", () => {
-  it("shows help when no subcommand given", async () => {
-    const ctx = mockCtx();
-    await handleCommandsCommand(ctx, "token", "123", "", undefined, "co-1");
-    expect(sentMessages.length).toBe(1);
-    expect(sentMessages[0].text).toContain("Custom Commands");
-  });
-
-  it("lists empty registry", async () => {
-    const ctx = mockCtx();
-    await handleCommandsCommand(ctx, "token", "123", "list", undefined, "co-1");
-    expect(sentMessages[0].text).toContain("No custom commands");
-  });
-
-  it("imports a valid command", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "deploy",
-      description: "Deploy to prod",
-      steps: [{ id: "s1", type: "send_message", text: "Deploying..." }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("deploy");
-    expect(sentMessages[0].text).toContain("imported");
-
-    // Verify it's stored
-    const stored = stateStore["commands_co-1"] as Array<{ name: string }>;
-    expect(stored).toBeDefined();
-    expect(stored.length).toBe(1);
-    expect(stored[0].name).toBe("deploy");
-  });
-
-  it("rejects import of invalid JSON", async () => {
-    const ctx = mockCtx();
-    await handleCommandsCommand(ctx, "token", "123", "import {invalid", undefined, "co-1");
-    expect(sentMessages[0].text).toContain("Invalid JSON");
-  });
-
-  it("rejects import without name field", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({ steps: [{ id: "s1", type: "send_message", text: "hi" }] });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("must have");
-  });
-
-  it("deletes a command", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "deploy",
-      description: "Deploy",
-      steps: [],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    await handleCommandsCommand(ctx, "token", "123", "delete deploy", undefined, "co-1");
-    expect(sentMessages[0].text).toContain("deleted");
-
-    const stored = stateStore["commands_co-1"] as Array<{ name: string }>;
-    expect(stored.length).toBe(0);
-  });
-
-  it("reports not found when deleting nonexistent command", async () => {
-    const ctx = mockCtx();
-    await handleCommandsCommand(ctx, "token", "123", "delete nonexistent", undefined, "co-1");
-    expect(sentMessages[0].text).toContain("not found");
-  });
-});
-
-describe("Namespace protection - built-in commands cannot be overridden", () => {
-  it("rejects import of /status as custom command", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "status",
-      description: "Override status",
-      steps: [{ id: "s1", type: "send_message", text: "hi" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("Cannot override built-in");
-  });
-
-  it("rejects import of /help as custom command", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "help",
-      description: "Override help",
-      steps: [{ id: "s1", type: "send_message", text: "hi" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("Cannot override built-in");
-  });
-
-  it("rejects import of /acp as custom command", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "acp",
-      description: "Override acp",
-      steps: [{ id: "s1", type: "send_message", text: "hi" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("Cannot override built-in");
-  });
-});
-
-describe("tryCustomCommand - command lookup", () => {
-  it("returns false for built-in commands", async () => {
-    const ctx = mockCtx();
-    const result = await tryCustomCommand(ctx, "token", "123", "status", "", undefined, "co-1");
-    expect(result).toBe(false);
-  });
-
-  it("returns false when command not in registry", async () => {
-    const ctx = mockCtx();
-    const result = await tryCustomCommand(ctx, "token", "123", "unknown", "", undefined, "co-1");
-    expect(result).toBe(false);
-  });
-
-  it("returns true and executes when command found in registry", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "greet",
-      description: "Greet user",
-      steps: [{ id: "s1", type: "send_message", text: "Hello {{args}}" }],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    const result = await tryCustomCommand(ctx, "token", "123", "greet", "world", undefined, "co-1");
-    expect(result).toBe(true);
-    expect(sentMessages.some(m => m.text === "Hello world")).toBe(true);
-  });
-});
-
-describe("Workflow step template interpolation", () => {
-  it("interpolates {{arg0}} and {{arg1}} parameters", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "deploy",
-      description: "Deploy",
-      steps: [{ id: "s1", type: "send_message", text: "Deploy {{arg0}} to {{arg1}}" }],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    await tryCustomCommand(ctx, "token", "123", "deploy", "v1.2 prod", undefined, "co-1");
-    expect(sentMessages.some(m => m.text === "Deploy v1.2 to prod")).toBe(true);
-  });
-
-  it("interpolates {{args}} as full args string", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "echo",
-      description: "Echo",
-      steps: [{ id: "s1", type: "send_message", text: "You said: {{args}}" }],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    await tryCustomCommand(ctx, "token", "123", "echo", "hello world", undefined, "co-1");
-    expect(sentMessages.some(m => m.text === "You said: hello world")).toBe(true);
-  });
-
-  it("interpolates {{prev.result}} from previous step", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "chain",
-      description: "Chain",
-      steps: [
-        { id: "s1", type: "send_message", text: "step one" },
-        { id: "s2", type: "send_message", text: "prev was: {{prev.result}}" },
-      ],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    await tryCustomCommand(ctx, "token", "123", "chain", "", undefined, "co-1");
-    expect(sentMessages.some(m => m.text === "prev was: sent")).toBe(true);
-  });
-
-  it("interpolates {{step_id.result}} from specific step", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "multi",
-      description: "Multi step",
-      steps: [
-        { id: "first", type: "send_message", text: "one" },
-        { id: "second", type: "send_message", text: "two" },
-        { id: "third", type: "send_message", text: "first said: {{first.result}}" },
-      ],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    await tryCustomCommand(ctx, "token", "123", "multi", "", undefined, "co-1");
-    expect(sentMessages.some(m => m.text === "first said: sent")).toBe(true);
-  });
-});
-
-describe("Command import validation", () => {
-  it("rejects steps missing type field", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "bad",
-      description: "bad",
-      steps: [{ id: "s1" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("must have");
-  });
-
-  it("rejects steps with invalid type", async () => {
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "bad",
-      description: "bad",
-      steps: [{ id: "s1", type: "invalid_type" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("Invalid step type");
-  });
-
-  it("updates existing command on re-import", async () => {
-    stateStore["commands_co-1"] = [{
-      name: "deploy",
-      description: "Old deploy",
-      steps: [{ id: "s1", type: "send_message", text: "old" }],
-      createdBy: "test",
-      createdAt: "2026-01-01",
-    }];
-    const ctx = mockCtx();
-    const cmd = JSON.stringify({
-      name: "deploy",
-      description: "New deploy",
-      steps: [{ id: "s1", type: "send_message", text: "new" }],
-    });
-    await handleCommandsCommand(ctx, "token", "123", `import ${cmd}`, undefined, "co-1");
-    expect(sentMessages[0].text).toContain("updated");
-
-    const stored = stateStore["commands_co-1"] as Array<{ description: string }>;
-    expect(stored.length).toBe(1);
-    expect(stored[0].description).toBe("New deploy");
-  });
-});