fix(cli): clear suppressed delivery payloads

steipete · steipete · commit c515d17f62d1 · 2026-05-10T14:00:23.000+01:00
diff --git a/src/agents/command/delivery.test.ts b/src/agents/command/delivery.test.ts
@@ -582,7 +582,7 @@ describe("normalizeAgentCommandReplyPayloads", () => {
     });
   });
 
-  it("surfaces no-payload deliveryStatus without changing the legacy success boolean", async () => {
+  it("marks no-payload deliveryStatus as terminal delivery success", async () => {
     const delivered = await deliverAgentCommandResult({
       cfg: {} as OpenClawConfig,
       deps: {} as CliDeps,
@@ -599,7 +599,7 @@ describe("normalizeAgentCommandReplyPayloads", () => {
       result: createResult(),
     });
 
-    expect(delivered.deliverySucceeded).toBeUndefined();
+    expect(delivered.deliverySucceeded).toBe(true);
     expect(delivered.deliveryStatus).toMatchObject({
       requested: true,
       attempted: false,
@@ -628,7 +628,7 @@ describe("normalizeAgentCommandReplyPayloads", () => {
     });
 
     expect(delivered.payloads).toEqual([]);
-    expect(delivered.deliverySucceeded).toBeUndefined();
+    expect(delivered.deliverySucceeded).toBe(true);
     expect(delivered.deliveryStatus).toMatchObject({
       requested: true,
       attempted: false,
diff --git a/src/agents/command/delivery.ts b/src/agents/command/delivery.ts
@@ -506,10 +506,12 @@ export async function deliverAgentCommandResult(params: {
   const deliveryPayloads = projectOutboundPayloadPlanForOutbound(outboundPayloadPlan);
   if (deliveryPayloads.length === 0) {
     deliveryStatus = deliver ? (deliveryStatus ?? noVisiblePayloadStatus()) : undefined;
+    const deliverySucceeded = deliveryStatus?.succeeded === true ? true : undefined;
     emitJsonEnvelope(deliveryStatus);
     return {
       payloads: normalizedPayloads,
       meta: resultMeta,
+      ...(deliverySucceeded !== undefined ? { deliverySucceeded } : {}),
       ...(deliveryStatus ? { deliveryStatus } : {}),
     };
   }
