fix: set delivered:false and error on announce delivery failure paths

This commit is contained in:
YuzuruS
2026-03-02 09:16:31 +09:00
committed by Tak Hoffman
parent f599afbe07
commit 86df703d91
2 changed files with 8 additions and 2 deletions

View File

@@ -288,8 +288,9 @@ describe("runCronIsolatedAgentTurn", () => {
// Announce delivery failure should not mark a successful agent execution
// as error. The execution succeeded; only delivery failed.
expect(res.status).toBe("ok");
expect(res.delivered).not.toBe(true);
expect(res.delivered).toBe(false);
expect(res.deliveryAttempted).toBe(true);
expect(res.error).toBe("cron announce delivery failed");
expect(deps.sendMessageTelegram).not.toHaveBeenCalled();
});
});
@@ -345,8 +346,9 @@ describe("runCronIsolatedAgentTurn", () => {
// Even when announce throws (e.g. "pairing required"), the agent
// execution succeeded so the job status should be ok.
expect(res.status).toBe("ok");
expect(res.delivered).not.toBe(true);
expect(res.delivered).toBe(false);
expect(res.deliveryAttempted).toBe(true);
expect(res.error).toContain("pairing required");
expect(deps.sendMessageTelegram).not.toHaveBeenCalled();
});
});

View File

@@ -337,6 +337,8 @@ export async function dispatchCronDelivery(
status: "ok",
summary,
outputText,
error: message,
delivered: false,
deliveryAttempted,
...params.telemetry,
});
@@ -351,6 +353,8 @@ export async function dispatchCronDelivery(
status: "ok",
summary,
outputText,
error: String(err),
delivered: false,
deliveryAttempted,
...params.telemetry,
});