Spaces:
Paused
Paused
OpenClawBot / src /auto-reply /reply.triggers.trigger-handling.ignores-inline-elevated-directive-unapproved-sender.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"; | |
| 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: "openclaw-triggers-" }, | |
| ); | |
| } | |
| function makeCfg(home: string) { | |
| return { | |
| agents: { | |
| defaults: { | |
| model: "anthropic/claude-opus-4-5", | |
| workspace: join(home, "openclaw"), | |
| }, | |
| }, | |
| channels: { | |
| whatsapp: { | |
| allowFrom: ["*"], | |
| }, | |
| }, | |
| session: { store: join(home, "sessions.json") }, | |
| }; | |
| } | |
| afterEach(() => { | |
| vi.restoreAllMocks(); | |
| }); | |
| describe("trigger handling", () => { | |
| it("ignores inline elevated directive for unapproved sender", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockResolvedValue({ | |
| payloads: [{ text: "ok" }], | |
| meta: { | |
| durationMs: 1, | |
| agentMeta: { sessionId: "s", provider: "p", model: "m" }, | |
| }, | |
| }); | |
| const cfg = { | |
| agents: { | |
| defaults: { | |
| model: "anthropic/claude-opus-4-5", | |
| workspace: join(home, "openclaw"), | |
| }, | |
| }, | |
| tools: { | |
| elevated: { | |
| allowFrom: { whatsapp: ["+1000"] }, | |
| }, | |
| }, | |
| channels: { | |
| whatsapp: { | |
| allowFrom: ["+1000"], | |
| }, | |
| }, | |
| session: { store: join(home, "sessions.json") }, | |
| }; | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "please /elevated on now", | |
| From: "+2000", | |
| To: "+2000", | |
| Provider: "whatsapp", | |
| SenderE164: "+2000", | |
| }, | |
| {}, | |
| cfg, | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).not.toContain("elevated is not available right now"); | |
| expect(runEmbeddedPiAgent).toHaveBeenCalled(); | |
| }); | |
| }); | |
| it("uses tools.elevated.allowFrom.discord for elevated approval", async () => { | |
| await withTempHome(async (home) => { | |
| const cfg = { | |
| agents: { | |
| defaults: { | |
| model: "anthropic/claude-opus-4-5", | |
| workspace: join(home, "openclaw"), | |
| }, | |
| }, | |
| tools: { elevated: { allowFrom: { discord: ["steipete"] } } }, | |
| session: { store: join(home, "sessions.json") }, | |
| }; | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "/elevated on", | |
| From: "discord:123", | |
| To: "user:123", | |
| Provider: "discord", | |
| SenderName: "Peter Steinberger", | |
| SenderUsername: "steipete", | |
| SenderTag: "steipete", | |
| CommandAuthorized: true, | |
| }, | |
| {}, | |
| cfg, | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).toContain("Elevated mode set to ask"); | |
| const storeRaw = await fs.readFile(cfg.session.store, "utf-8"); | |
| const store = JSON.parse(storeRaw) as Record<string, { elevatedLevel?: string }>; | |
| expect(store[MAIN_SESSION_KEY]?.elevatedLevel).toBe("on"); | |
| }); | |
| }); | |
| it("treats explicit discord elevated allowlist as override", async () => { | |
| await withTempHome(async (home) => { | |
| const cfg = { | |
| agents: { | |
| defaults: { | |
| model: "anthropic/claude-opus-4-5", | |
| workspace: join(home, "openclaw"), | |
| }, | |
| }, | |
| tools: { | |
| elevated: { | |
| allowFrom: { discord: [] }, | |
| }, | |
| }, | |
| session: { store: join(home, "sessions.json") }, | |
| }; | |
| const res = await getReplyFromConfig( | |
| { | |
| Body: "/elevated on", | |
| From: "discord:123", | |
| To: "user:123", | |
| Provider: "discord", | |
| SenderName: "steipete", | |
| }, | |
| {}, | |
| cfg, | |
| ); | |
| const text = Array.isArray(res) ? res[0]?.text : res?.text; | |
| expect(text).toContain("tools.elevated.allowFrom.discord"); | |
| expect(runEmbeddedPiAgent).not.toHaveBeenCalled(); | |
| }); | |
| }); | |
| it("returns a context overflow fallback when the embedded agent throws", async () => { | |
| await withTempHome(async (home) => { | |
| vi.mocked(runEmbeddedPiAgent).mockRejectedValue(new Error("Context window exceeded")); | |
| 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( | |
| "⚠️ Context overflow — prompt too large for this model. Try a shorter message or a larger-context model.", | |
| ); | |
| expect(runEmbeddedPiAgent).toHaveBeenCalledOnce(); | |
| }); | |
| }); | |
| }); | |