fix(exec): describe dashboard URL handoff

steipete · steipete · commit 08f442558a84 · 2026-07-10T15:14:49.000-07:00
diff --git a/src/agents/bash-tools.exec-host-shared.test.ts b/src/agents/bash-tools.exec-host-shared.test.ts
@@ -637,7 +637,9 @@ describe("buildExecApprovalPendingToolResult", () => {
       nodeId: "node-mac-1",
     });
     const text = result.content.find((part) => part.type === "text")?.text ?? "";
-    expect(text).toContain("Open the approval inbox with `openclaw dashboard --no-open`.");
+    expect(text).toContain(
+      "Print the Control UI URL with `openclaw dashboard --no-open`, open it in a browser, then use the approval inbox.",
+    );
     expect(text).toContain(
       "Inspect the node's effective exec policy with `openclaw approvals get --node node-mac-1`.",
     );
diff --git a/src/agents/embedded-agent-subscribe.handlers.tools.test.ts b/src/agents/embedded-agent-subscribe.handlers.tools.test.ts
@@ -2187,7 +2187,9 @@ describe("handleToolExecutionEnd exec approval prompts", () => {
       "tool result text",
     );
     expect(text).toContain("no interactive approval client is currently available");
-    expect(text).toContain("Open the approval inbox with `openclaw dashboard --no-open`.");
+    expect(text).toContain(
+      "Print the Control UI URL with `openclaw dashboard --no-open`, open it in a browser, then use the approval inbox.",
+    );
     expect(text).toContain(
       "Inspect the node's effective exec policy with `openclaw approvals get --node node-mac-1`.",
     );
diff --git a/src/infra/exec-approval-reply.test.ts b/src/infra/exec-approval-reply.test.ts
@@ -116,20 +116,21 @@ describe("exec approval reply helpers", () => {
     );
   });
 
-  it("distinguishes the node approval inbox from policy inspection", () => {
+  it("distinguishes node approval-inbox access from policy inspection", () => {
     const text = buildExecApprovalUnavailableReplyPayload({
       reason: "no-approval-route",
       host: "node",
       nodeId: "mac-1",
     }).text;
 
-    expect(text).toContain("Open the approval inbox with `openclaw dashboard --no-open`.");
     expect(text).toContain(
-      "Inspect the node's effective exec policy with `openclaw approvals get --node mac-1`.",
+      "Print the Control UI URL with `openclaw dashboard --no-open`, open it in a browser, then use the approval inbox.",
     );
-    expect(text).not.toContain(
-      "approval inbox with `openclaw dashboard --no-open` or `openclaw approvals get",
+    expect(text).toContain(
+      "Inspect the node's effective exec policy with `openclaw approvals get --node mac-1`.",
     );
+    expect(text).not.toContain("`openclaw dashboard --no-open` or `openclaw approvals get");
+    expect(text).not.toContain("Open the approval inbox with");
     expect(text).not.toContain("exec-approvals list");
   });
 
diff --git a/src/infra/exec-approval-reply.ts b/src/infra/exec-approval-reply.ts
@@ -90,7 +90,8 @@ function buildGenericNativeExecApprovalFallbackText(params?: {
   const clients = resolveNativeExecApprovalClientList({
     excludeChannel: params?.excludeChannel,
   });
-  let manualRecovery = "Open the approval inbox with `openclaw dashboard --no-open`.";
+  let manualRecovery =
+    "Print the Control UI URL with `openclaw dashboard --no-open`, open it in a browser, then use the approval inbox.";
   if (params?.host === "node") {
     const nodeId = normalizeOptionalString(params.nodeId) ?? "<id|name|ip>";
     manualRecovery += ` Inspect the node's effective exec policy with \`openclaw approvals get --node ${nodeId}\`.`;
