Spaces:
Configuration error
Configuration error
melbot / src /auto-reply /reply.triggers.trigger-handling.includes-error-cause-embedded-agent-throws.e2e.test.ts
| import fs from "node:fs/promises"; | |
| import { join } from "node:path"; | |
| import { afterEach, describe, expect, it, vi } from "vitest"; | |
| import { withTempHome as withTempHomeBase } from "../../test/helpers/temp-home.js"; | |
| vi.mock("../agents/pi-embedded.js", () => ({ | |
| abortEmbeddedPiRun: vi.fn().mockReturnValue(false), | |
| compactEmbeddedPiSession: vi.fn(), | |
| runEmbeddedPiAgent: vi.fn(), | |
| queueEmbeddedPiMessage: vi.fn().mockReturnValue(false), | |
| resolveEmbeddedSessionLane: (key: string) => `session:${key.trim() || "main"}`, | |
| isEmbeddedPiRunActive: vi.fn().mockReturnValue(false), | |
| isEmbeddedPiRunStreaming: vi.fn().mockReturnValue(false), | |
| })); | |
| const usageMocks = vi.hoisted(() => ({ | |
| loadProviderUsageSummary: vi.fn().mockResolvedValue({ | |
| updatedAt: 0, | |
| providers: [], | |
| }), | |
| formatUsageSummaryLine: vi.fn().mockReturnValue("📊 Usage: Claude 80% left"), | |
| resolveUsageProviderId: vi.fn((provider: string) => provider.split("/")[0]), | |
| })); | |
| vi.mock("../infra/provider-usage.js", () => usageMocks); | |
| const modelCatalogMocks = vi.hoisted(() => ({ | |
| loadModelCatalog: vi.fn().mockResolvedValue([ | |
| { | |
| provider: "anthropic", | |
| id: "claude-opus-4-5", | |
| name: "Claude Opus 4.5", | |
| contextWindow: 200000, | |
| }, | |
| { | |
| provider: "openrouter", | |
| id: "anthropic/claude-opus-4-5", | |
| name: "Claude Opus 4.5 (OpenRouter)", | |
| contextWindow: 200000, | |
| }, | |
| { provider: "openai", id: "gpt-4.1-mini", name: "GPT-4.1 mini" }, | |
| { provider: "openai", id: "gpt-5.2", name: "GPT-5.2" }, | |
| { provider: "openai-codex", id: "gpt-5.2", name: "GPT-5.2 (Codex)" }, | |
| { provider: "minimax", id: "MiniMax-M2.1", name: "MiniMax M2.1" }, | |
| ]), | |
| resetModelCatalogCacheForTest: vi.fn(), | |
| })); | |
| vi.mock("../agents/model-catalog.js", () => modelCatalogMocks); | |
| import { abortEmbeddedPiRun, runEmbeddedPiAgent } from "../agents/pi-embedded.js"; | |
| import { getReplyFromConfig } from "./reply.js"; | |
| import { HEARTBEAT_TOKEN } from "./tokens.js"; | |
| const _MAIN_SESSION_KEY = "agent:main:main"; | |
| const webMocks = vi.hoisted(() => ({ | |
| webAuthExists: vi.fn().mockResolvedValue(true), | |
| getWebAuthAgeMs: vi.fn().mockReturnValue(120_000), | |
| readWebSelfId: vi.fn().mockReturnValue({ e164: "+1999" }), | |
| })); | |
| vi.mock("../web/session.js", () => webMocks); | |
| async function withTempHome<T>(fn: (home: string) => Promise<T>): Promise<T> { | |
| return withTempHomeBase( | |
| async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockClear(); | |
| vi.mocked(abortEmbeddedPiRun).mockClear(); | |
| return await fn(home); | |
| }, | |
| { prefix: "moltbot-triggers-" }, | |
| ); | |
| } | |
| function makeCfg(home: string) { | |
| return { | |
| agents: { | |
| defaults: { | |
| model: "anthropic/claude-opus-4-5", | |
| workspace: join(home, "clawd"), | |
| }, | |
| }, | |
| channels: { | |
| whatsapp: { | |
| allowFrom: ["*"], | |
| }, | |
| }, | |
| session: { store: join(home, "sessions.json") }, | |
| }; | |
| } | |
| afterEach(() => { | |
| vi.restoreAllMocks(); | |
| }); | |
| describe("trigger handling", () => { | |
| it("includes the error cause when the embedded agent throws", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockRejectedValue(new Error("sandbox is not defined.")); | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "hello", | |
| From: "+1002", | |
| To: "+2000", | |
| }, | |
| {}, | |
| makeCfg(home), | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).toBe( | |
| "⚠️ Agent failed before reply: sandbox is not defined.\nLogs: moltbot logs --follow", | |
| ); | |
| expect(runEmbeddedPiAgent).toHaveBeenCalledOnce(); | |
| }); | |
| }); | |
| it("uses heartbeat model override for heartbeat runs", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockResolvedValue({ | |
| payloads: [{ text: "ok" }], | |
| meta: { | |
| durationMs: 1, | |
| agentMeta: { sessionId: "s", provider: "p", model: "m" }, | |
| }, | |
| }); | |
| const cfg = makeCfg(home); | |
| cfg.agents = { | |
| ...cfg.agents, | |
| defaults: { | |
| ...cfg.agents?.defaults, | |
| heartbeat: { model: "anthropic/claude-haiku-4-5-20251001" }, | |
| }, | |
| }; | |
| await getReplyFromConfig( | |
| { | |
| Body: "hello", | |
| From: "+1002", | |
| To: "+2000", | |
| }, | |
| { isHeartbeat: true }, | |
| cfg, | |
| ); | |
| const call = vi.mocked(runEmbeddedPiAgent).mock.calls[0]?.[0]; | |
| expect(call?.provider).toBe("anthropic"); | |
| expect(call?.model).toBe("claude-haiku-4-5-20251001"); | |
| }); | |
| }); | |
| it("suppresses HEARTBEAT_OK replies outside heartbeat runs", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockResolvedValue({ | |
| payloads: [{ text: HEARTBEAT_TOKEN }], | |
| meta: { | |
| durationMs: 1, | |
| agentMeta: { sessionId: "s", provider: "p", model: "m" }, | |
| }, | |
| }); | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "hello", | |
| From: "+1002", | |
| To: "+2000", | |
| }, | |
| {}, | |
| makeCfg(home), | |
| ); | |
| expect(res).toBeUndefined(); | |
| expect(runEmbeddedPiAgent).toHaveBeenCalledOnce(); | |
| }); | |
| }); | |
| it("strips HEARTBEAT_OK at edges outside heartbeat runs", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockResolvedValue({ | |
| payloads: [{ text: `${HEARTBEAT_TOKEN} hello` }], | |
| meta: { | |
| durationMs: 1, | |
| agentMeta: { sessionId: "s", provider: "p", model: "m" }, | |
| }, | |
| }); | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "hello", | |
| From: "+1002", | |
| To: "+2000", | |
| }, | |
| {}, | |
| makeCfg(home), | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).toBe("hello"); | |
| }); | |
| }); | |
| it("updates group activation when the owner sends /activation", async () => { | |
| await withTempHome(async (home) => { | |
| const cfg = makeCfg(home); | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "/activation always", | |
| From: "123@g.us", | |
| To: "+2000", | |
| ChatType: "group", | |
| Provider: "whatsapp", | |
| SenderE164: "+2000", | |
| CommandAuthorized: true, | |
| }, | |
| {}, | |
| cfg, | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).toContain("Group activation set to always"); | |
| const store = JSON.parse(await fs.readFile(cfg.session.store, "utf-8")) as Record< | |
| string, | |
| { groupActivation?: string } | |
| >; | |
| expect(store["agent:main:whatsapp:group:123@g.us"]?.groupActivation).toBe("always"); | |
| expect(runEmbeddedPiAgent).not.toHaveBeenCalled(); | |
| }); | |
| }); | |
| }); | |