vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 11:00:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(cron): cover mode none implicit recipient leak

Browse Source
This commit is contained in:
Ayaan Zaidi
2026-04-20 08:12:07 +05:30
parent 7d9a9d83ff
commit 03de50e70b
1 changed files with 66 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
src/cron/isolated-agent/run.message-tool-policy.test.ts
View File

@@ -166,6 +166,72 @@ describe("runCronIsolatedAgentTurn message tool policy", () => {
});
});
it("does not resolve implicit last-target context for delivery.mode none with only accountId", async () => {
mockRunCronFallbackPassthrough();
resolveCronDeliveryPlanMock.mockReturnValue({
requested: false,
mode: "none",
channel: "last",
accountId: "ops",
});
await runCronIsolatedAgentTurn({
...makeParams(),
job: {
id: "message-tool-policy",
name: "Message Tool Policy",
schedule: { kind: "every", everyMs: 60_000 },
sessionTarget: "isolated",
payload: { kind: "agentTurn", message: "send a message" },
delivery: { mode: "none", accountId: "ops" },
} as never,
});
expect(resolveDeliveryTargetMock).not.toHaveBeenCalled();
expect(runEmbeddedPiAgentMock).toHaveBeenCalledTimes(1);
expect(runEmbeddedPiAgentMock.mock.calls[0]?.[0]).toMatchObject({
disableMessageTool: false,
messageChannel: undefined,
messageTo: undefined,
messageThreadId: undefined,
currentChannelId: undefined,
agentAccountId: undefined,
});
});
it("does not resolve implicit last-target context for delivery.mode none with only threadId", async () => {
mockRunCronFallbackPassthrough();
resolveCronDeliveryPlanMock.mockReturnValue({
requested: false,
mode: "none",
channel: "last",
threadId: 42,
});
await runCronIsolatedAgentTurn({
...makeParams(),
job: {
id: "message-tool-policy",
name: "Message Tool Policy",
schedule: { kind: "every", everyMs: 60_000 },
sessionTarget: "isolated",
payload: { kind: "agentTurn", message: "send a message" },
delivery: { mode: "none", threadId: 42 },
} as never,
});
expect(resolveDeliveryTargetMock).not.toHaveBeenCalled();
expect(runEmbeddedPiAgentMock).toHaveBeenCalledTimes(1);
expect(runEmbeddedPiAgentMock.mock.calls[0]?.[0]).toMatchObject({
disableMessageTool: false,
messageChannel: undefined,
messageTo: undefined,
messageThreadId: undefined,
currentChannelId: undefined,
agentAccountId: undefined,
});
});
it("forwards explicit message targets into the embedded run", async () => {
mockRunCronFallbackPassthrough();
const executor = createCronPromptExecutor({