fix: preserve ClickClack nonce lookup causes (#105775)

shakkernerd · shakkernerd · commit 72b42f0ed751 · 2026-07-13T19:57:34.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -34,8 +34,7 @@ Docs: https://docs.openclaw.ai
 ### Fixes
 
 - **Gateway in-process restarts:** clear stale SIGUSR1 restart state and resume prepared host suspensions before rebuilding runtime admission, preventing restart cooldowns or paused scheduling from leaking into the next lifecycle.
-- **ClickClack durable media delivery:** route media replies through required delivery, reuse owner-scoped upload and message nonces across retries, repair persisted attachment state without rereading source media, and fail closed when an older ClickClack server cannot prove an unknown send. Thanks @shakkernerd.
-- **ClickClack model reply budgets:** remove the channel-level token cap and use the selected provider and model's runtime output budget so short compatibility limits no longer suppress valid replies. Thanks @shakkernerd.
+- **ClickClack durable media delivery:** route media replies through required delivery, reuse owner-scoped upload and message nonces across retries, repair persisted attachment state without rereading source media, fail closed when an older ClickClack server cannot prove an unknown send, and use the selected provider and model's runtime output budget instead of a channel-level token cap. Thanks @jjjhenriksen and @shakkernerd.
 - **Deepgram realtime custom endpoints:** validate Voice Call streaming base URLs with secret-safe errors, preserve explicit `ws://` and `wss://` endpoints, and map HTTP schemes to their matching WebSocket transport for dedicated and self-hosted deployments. (#105334) Thanks @dwc1997.
 - **macOS remote node readiness:** take the main-session key from the node hello snapshot instead of opening an operator connection during node admission, preventing remote tunnel recovery from leaving Computer Use and node exec stuck in lifecycle transition.
 - **Claude CLI context budgets:** honor Anthropic model and per-agent `contextTokens` limits by passing the effective limit to Claude Code's native auto-compactor and persisting the same prepared budget in OpenClaw session state. Fixes #80933. (#93198) Thanks @mushuiyu886.
diff --git a/extensions/clickclack/src/http-client.test.ts b/extensions/clickclack/src/http-client.test.ts
@@ -410,7 +410,10 @@ describe("ClickClack HTTP client", () => {
     ).resolves.toBeUndefined();
     await expect(
       client.findUploadByNonce({ workspaceId: "wsp_1", nonce: "unsupported" }),
-    ).rejects.toThrow("does not support durable upload nonce lookup");
+    ).rejects.toMatchObject({
+      message: expect.stringContaining("does not support durable upload nonce lookup"),
+      cause: expect.objectContaining({ status: 404 }),
+    });
     await expect(
       client.findUploadByNonce({ workspaceId: "wsp_1", nonce: "broken" }),
     ).rejects.toThrow("ClickClack 503");
@@ -466,7 +469,10 @@ describe("ClickClack HTTP client", () => {
     ).resolves.toBeUndefined();
     await expect(
       client.findMessageByNonce({ workspaceId: "wsp_1", nonce: "unsupported" }),
-    ).rejects.toThrow("does not support durable message nonce lookup");
+    ).rejects.toMatchObject({
+      message: expect.stringContaining("does not support durable message nonce lookup"),
+      cause: expect.objectContaining({ status: 404 }),
+    });
     await expect(
       client.findMessageByNonce({ workspaceId: "wsp_1", nonce: "broken" }),
     ).rejects.toThrow("ClickClack 503");
diff --git a/extensions/clickclack/src/http-client.ts b/extensions/clickclack/src/http-client.ts
@@ -218,7 +218,9 @@ export function createClickClackClient(options: ClientOptions) {
           if (error.headers.get("X-ClickClack-Message-Nonce") === "supported") {
             return undefined;
           }
-          throw new Error("ClickClack server does not support durable message nonce lookup");
+          throw new Error("ClickClack server does not support durable message nonce lookup", {
+            cause: error,
+          });
         }
         throw error;
       }
@@ -312,7 +314,9 @@ export function createClickClackClient(options: ClientOptions) {
           if (error.headers.get("X-ClickClack-Upload-Nonce") === "supported") {
             return undefined;
           }
-          throw new Error("ClickClack server does not support durable upload nonce lookup");
+          throw new Error("ClickClack server does not support durable upload nonce lookup", {
+            cause: error,
+          });
         }
         throw error;
       }

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,9 @@ export function createClickClackClient(options: ClientOptions) {`
`218`	`218`	`if (error.headers.get("X-ClickClack-Message-Nonce") === "supported") {`
`219`	`219`	`return undefined;`
`220`	`220`	`}`
`221`		`- throw new Error("ClickClack server does not support durable message nonce lookup");`
	`221`	`+ throw new Error("ClickClack server does not support durable message nonce lookup", {`
	`222`	`+ cause: error,`
	`223`	`+ });`
`222`	`224`	`}`
`223`	`225`	`throw error;`
`224`	`226`	`}`
`@@ -312,7 +314,9 @@ export function createClickClackClient(options: ClientOptions) {`
`312`	`314`	`if (error.headers.get("X-ClickClack-Upload-Nonce") === "supported") {`
`313`	`315`	`return undefined;`
`314`	`316`	`}`
`315`		`- throw new Error("ClickClack server does not support durable upload nonce lookup");`
	`317`	`+ throw new Error("ClickClack server does not support durable upload nonce lookup", {`
	`318`	`+ cause: error,`
	`319`	`+ });`
`316`	`320`	`}`
`317`	`321`	`throw error;`
`318`	`322`	`}`