From 9fe140419a5b062a6cedf1771b8ca9c8ed6879a3 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Fri, 31 Jul 2026 05:12:06 +0800 Subject: [PATCH] test(release): align retired commitment proof --- .../e2e/commitments-safety-docker-client.ts | 33 ++++++++----------- test/scripts/docker-e2e-clients.test.ts | 2 +- 2 files changed, 15 insertions(+), 20 deletions(-) diff --git a/scripts/e2e/commitments-safety-docker-client.ts b/scripts/e2e/commitments-safety-docker-client.ts index 9251e7f848d1..9a9b44d47d21 100644 --- a/scripts/e2e/commitments-safety-docker-client.ts +++ b/scripts/e2e/commitments-safety-docker-client.ts @@ -15,7 +15,7 @@ import { } from "../../dist/commitments/store.js"; type CommitmentProbeResult = { - due: Array>; + commitments: Array>; }; function assert(condition: unknown, message: string): asserts condition { @@ -132,8 +132,8 @@ async function runPackagedDoctor(stateDir: string): Promise { ); } -function runDueCommitmentProbe(stateDir: string, nowMs: number): CommitmentProbeResult { - const result = spawnSync("tsx", [process.argv[1]!, "--probe-due", String(nowMs)], { +function runCommitmentStoreProbe(stateDir: string, nowMs: number): CommitmentProbeResult { + const result = spawnSync("tsx", [process.argv[1]!, "--probe-list", String(nowMs)], { cwd: process.cwd(), env: { ...process.env, @@ -160,8 +160,8 @@ async function verifyDoctorImportAndRuntimeIsolation() { "utf8", ); - const beforeDoctor = runDueCommitmentProbe(stateDir, nowMs); - assert(beforeDoctor.due.length === 0, "runtime imported legacy JSON without doctor"); + const beforeDoctor = runCommitmentStoreProbe(stateDir, nowMs); + assert(beforeDoctor.commitments.length === 0, "runtime imported legacy JSON without doctor"); await fs.access(sourcePath); await runPackagedDoctor(stateDir); @@ -176,24 +176,19 @@ async function verifyDoctorImportAndRuntimeIsolation() { } }); - const { due } = runDueCommitmentProbe(stateDir, nowMs); - assert(due.length === 1, `unexpected imported due count ${due.length}`); - assert(!("sourceUserText" in due[0]), "legacy source user text surfaced after import"); + const { commitments } = runCommitmentStoreProbe(stateDir, nowMs); + assert(commitments.length === 1, `unexpected imported commitment count ${commitments.length}`); + assert(!("sourceUserText" in commitments[0]), "legacy source user text surfaced after import"); assert( - !("sourceAssistantText" in due[0]), + !("sourceAssistantText" in commitments[0]), "legacy source assistant text surfaced after import", ); }); } -async function runDueCommitmentProbeMode(nowMs: number): Promise { - const due = await listDueCommitmentsForSession({ - cfg: { commitments: { enabled: true } }, - agentId: "main", - sessionKey: "agent:main:qa-channel:commitments", - nowMs, - }); - console.log(JSON.stringify({ due })); +async function runCommitmentStoreProbeMode(nowMs: number): Promise { + const commitments = await listCommitments({ nowMs }); + console.log(JSON.stringify({ commitments })); } async function verifyExpiryTransition() { @@ -243,10 +238,10 @@ async function verifyExpiryTransition() { }); } -if (process.argv[2] === "--probe-due") { +if (process.argv[2] === "--probe-list") { const nowMs = Number(process.argv[3]); assert(Number.isFinite(nowMs), "commitment probe requires a finite timestamp"); - await runDueCommitmentProbeMode(nowMs); + await runCommitmentStoreProbeMode(nowMs); } else { await verifyExtractionRemainsRetired(); await verifyDoctorImportAndRuntimeIsolation(); diff --git a/test/scripts/docker-e2e-clients.test.ts b/test/scripts/docker-e2e-clients.test.ts index 81a9aaeb79e0..5f17b0617018 100644 --- a/test/scripts/docker-e2e-clients.test.ts +++ b/test/scripts/docker-e2e-clients.test.ts @@ -14,7 +14,7 @@ describe("Docker E2E client scripts", () => { expect(source).toContain("../../dist/commitments/runtime.test-support.js"); expect(source).toContain("../../dist/commitments/store.js"); expect(source).toContain('spawnSync("tsx"'); - expect(source).toContain("--probe-due"); + expect(source).toContain("--probe-list"); expect(source).not.toContain("../../dist/state/openclaw-state-db.js"); expect(source).toContain("verifyExtractionRemainsRetired()"); expect(source).toContain("verifyDoctorImportAndRuntimeIsolation()");