fix(dashboard): name URL fragment key and suppress redundant --no-open hint

praveen9354 · praveen9354 · commit f6d0f71f8131 · 2026-04-30T15:31:19.000Z
diff --git a/src/commands/dashboard.links.test.ts b/src/commands/dashboard.links.test.ts
@@ -183,6 +183,8 @@ describe("dashboardCommand", () => {
 
     // UX: user must be pointed to where their token lives so they can self-recover.
     expect(allLogs).toMatch(/OPENCLAW_GATEWAY_TOKEN/);
+    // UX: hint must name the URL fragment key so the user knows the syntax.
+    expect(allLogs).toContain("key `token`");
   });
 
   it("respects --no-open and tells user token URL is in clipboard", async () => {
@@ -198,12 +200,25 @@ describe("dashboardCommand", () => {
     );
   });
 
-  it("respects --no-open with plain URL hint when clipboard fails", async () => {
+  it("respects --no-open and falls through to manual-auth hint when clipboard fails (token configured)", async () => {
     mockSnapshot("abc");
     copyToClipboardMock.mockResolvedValue(false);
 
     await dashboardCommand(runtime, { noOpen: true });
 
+    // Redundant fallback hint is suppressed when the manual-auth hint speaks.
+    expect(runtime.log).not.toHaveBeenCalledWith(
+      "Browser launch disabled (--no-open). Use the URL above.",
+    );
+    expect(runtime.log).toHaveBeenCalledWith(expect.stringContaining("OPENCLAW_GATEWAY_TOKEN"));
+  });
+
+  it("respects --no-open with plain URL hint when clipboard fails and no token is configured", async () => {
+    mockSnapshot("");
+    copyToClipboardMock.mockResolvedValue(false);
+
+    await dashboardCommand(runtime, { noOpen: true });
+
     expect(runtime.log).toHaveBeenCalledWith(
       "Browser launch disabled (--no-open). Use the URL above.",
     );
diff --git a/src/commands/dashboard.ts b/src/commands/dashboard.ts
@@ -86,15 +86,18 @@ export async function dashboardCommand(
         : "Browser launch disabled (--no-open). Use the URL above.";
   }
 
+  const fallbackToManualAuth = !copied && !opened && includeTokenInUrl;
+  const suppressNoOpenHint = options.noOpen === true && fallbackToManualAuth;
+
   if (opened) {
     runtime.log("Opened in your browser. Keep that tab to control OpenClaw.");
-  } else if (hint) {
+  } else if (hint && !suppressNoOpenHint) {
     runtime.log(hint);
   }
 
-  if (!copied && !opened && includeTokenInUrl) {
+  if (fallbackToManualAuth) {
     runtime.log(
-      "Token auto-auth not delivered. Append your gateway token (from OPENCLAW_GATEWAY_TOKEN or gateway.auth.token) as a URL fragment to authenticate.",
+      "Token auto-auth not delivered. Append your gateway token (from OPENCLAW_GATEWAY_TOKEN or gateway.auth.token) as a URL fragment with key `token` to authenticate.",
     );
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -86,15 +86,18 @@ export async function dashboardCommand(`
`86`	`86`	`: "Browser launch disabled (--no-open). Use the URL above.";`
`87`	`87`	`}`
`88`	`88`
	`89`	`+ const fallbackToManualAuth = !copied && !opened && includeTokenInUrl;`
	`90`	`+ const suppressNoOpenHint = options.noOpen === true && fallbackToManualAuth;`
	`91`	`+`
`89`	`92`	`if (opened) {`
`90`	`93`	`runtime.log("Opened in your browser. Keep that tab to control OpenClaw.");`
`91`		`- } else if (hint) {`
	`94`	`+ } else if (hint && !suppressNoOpenHint) {`
`92`	`95`	`runtime.log(hint);`
`93`	`96`	`}`
`94`	`97`
`95`		`- if (!copied && !opened && includeTokenInUrl) {`
	`98`	`+ if (fallbackToManualAuth) {`
`96`	`99`	`runtime.log(`
`97`		`- "Token auto-auth not delivered. Append your gateway token (from OPENCLAW_GATEWAY_TOKEN or gateway.auth.token) as a URL fragment to authenticate.",`
	`100`	+ "Token auto-auth not delivered. Append your gateway token (from OPENCLAW_GATEWAY_TOKEN or gateway.auth.token) as a URL fragment with key `token` to authenticate.",
`98`	`101`	`);`
`99`	`102`	`}`
`100`	`103`	`}`