openclaw
diff --git a/‎src/cli/program/register.status-health-sessions.test.ts‎
Lines changed: 10 additions & 1 deletion b/‎src/cli/program/register.status-health-sessions.test.ts‎
Lines changed: 10 additions & 1 deletion
diff --git a/‎src/cli/program/register.status-health-sessions.ts‎
Lines changed: 2 additions & 0 deletions b/‎src/cli/program/register.status-health-sessions.ts‎
Lines changed: 2 additions & 0 deletions
diff --git a/‎src/commands/sessions-compact.test.ts‎
Lines changed: 10 additions & 0 deletions b/‎src/commands/sessions-compact.test.ts‎
Lines changed: 10 additions & 0 deletions
diff --git a/‎src/commands/sessions-compact.ts‎
Lines changed: 3 additions & 4 deletions b/‎src/commands/sessions-compact.ts‎
Lines changed: 3 additions & 4 deletions
@@ -332,9 +332,18 @@ describe("registerStatusHealthSessionsCommands", () => {
   });
 
   it("rejects other unsupported inherited parent list options for compact", async () => {
-    await runCli(["sessions", "--all-agents", "--limit", "25", "compact", "agent:work:main"]);
+    await runCli([
+      "sessions",
+      "--all-agents",
+      "--limit",
+      "25",
+      "--verbose",
+      "compact",
+      "agent:work:main",
+    ]);
 
     expect(runtime.error).toHaveBeenCalledWith(expect.stringContaining("--all-agents"));
+    expect(runtime.error).toHaveBeenCalledWith(expect.stringContaining("--verbose"));
     expect(runtime.exit).toHaveBeenCalledWith(1);
     expect(sessionsCompactCommand).not.toHaveBeenCalled();
   });
 
@@ -411,13 +411,15 @@ export function registerStatusHealthSessionsCommands(program: Command) {
             allAgents?: boolean;
             active?: string;
             limit?: string;
+            verbose?: boolean;
           }
         | undefined;
       const unsupportedParentOptions = [
         parentOpts?.store !== undefined ? "--store" : undefined,
         parentOpts?.allAgents ? "--all-agents" : undefined,
         parentOpts?.active !== undefined ? "--active" : undefined,
         parentOpts?.limit !== undefined ? "--limit" : undefined,
+        parentOpts?.verbose ? "--verbose" : undefined,
       ].filter((flag): flag is string => flag !== undefined);
       if (unsupportedParentOptions.length > 0) {
         const plural = unsupportedParentOptions.length > 1 ? "options" : "option";
 
@@ -80,6 +80,16 @@ describe("sessionsCompactCommand", () => {
     expect(joinedArgs(runtime.error)).toContain("summarize interrupted");
   });
 
+  it("exits non-zero when the gateway response omits explicit success", async () => {
+    callGatewayCli.mockResolvedValue({ key: "agent:main:main", compacted: false });
+    const runtime = createRuntime();
+
+    await sessionsCompactCommand({ key: "agent:main:main" }, runtime);
+
+    expect(runtime.exit).toHaveBeenCalledWith(1);
+    expect(joinedArgs(runtime.error)).toContain("Compaction failed");
+  });
+
   it("exits non-zero and surfaces the error when the RPC throws", async () => {
     callGatewayCli.mockRejectedValue(new Error("gateway unreachable"));
     const runtime = createRuntime();
 
@@ -99,10 +99,9 @@ export async function sessionsCompactCommand(
     return;
   }
 
-  // An explicit ok:false means the gateway accepted the request but the
-  // compaction itself did not complete (for example an interrupted summarize
-  // run). Surface it as a failure instead of letting the process exit 0.
-  const failed = result?.ok === false;
+  // Success is explicit. A malformed or version-skewed payload must not turn
+  // into the same exit-0 message as a genuine no-op compaction.
+  const failed = result?.ok !== true;
 
   if (opts.json) {
     writeRuntimeJson(runtime, result);