fix(media): default terminal QR to full-block output (#77820)

neeravmakwana · cursoragent · neeravmakwana · commit 37defed085aa · 2026-05-05T08:57:10.000-04:00
Avoid node-qrcode compact (small) terminal mode, which emits a dense
ANSI final row that breaks scanning on some terminals.

Covers WhatsApp/Feishu login flows and the pairing QR CLI path.

Co-authored-by: Cursor &lt;cursoragent@cursor.com&gt;
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -69,6 +69,7 @@ Docs: https://docs.openclaw.ai
 
 ### Fixes
 
+- CLI/media: render terminal QR codes with full-block characters by default so the bundled `qrcode` terminal renderer does not emit a pathologically dense ANSI final row in compact half-block mode that breaks scanning in some terminals. Fixes #77820. Thanks @KrasimirKralev.
 - TUI/sessions: bound the session picker to recent rows and use exact lookup-style refreshes for the active session, so dusty stores no longer make TUI hydrate weeks-old transcripts before becoming responsive. Thanks @vincentkoc.
 - Doctor/gateway: report recent supervisor restart handoffs in `openclaw doctor --deep`, using the installed service environment when available so service-managed clean exits are visible in guided diagnostics. Thanks @shakkernerd.
 - Gateway/status: show recent supervisor restart handoffs in `openclaw gateway status --deep`, including JSON details, so clean service-managed restarts are reported as restart handoffs instead of opaque stopped-service diagnostics. Thanks @shakkernerd.
diff --git a/extensions/feishu/src/app-registration.ts b/extensions/feishu/src/app-registration.ts
@@ -252,7 +252,7 @@ export async function pollAppRegistration(params: {
  * otherwise the pattern is corrupted and cannot be scanned.
  */
 export async function printQrCode(url: string): Promise<void> {
-  const output = await renderQrTerminal(url, { small: true });
+  const output = await renderQrTerminal(url);
   process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
 }
 
diff --git a/extensions/whatsapp/src/login.coverage.test.ts b/extensions/whatsapp/src/login.coverage.test.ts
@@ -147,8 +147,8 @@ describe("loginWeb coverage", () => {
     );
     expect(runtime.log).toHaveBeenCalledWith("terminal:initial-qr");
     expect(runtime.log).toHaveBeenCalledWith("terminal:restart-qr");
-    expect(renderQrTerminalMock).toHaveBeenCalledWith("initial-qr", { small: true });
-    expect(renderQrTerminalMock).toHaveBeenCalledWith("restart-qr", { small: true });
+    expect(renderQrTerminalMock).toHaveBeenCalledWith("initial-qr");
+    expect(renderQrTerminalMock).toHaveBeenCalledWith("restart-qr");
   });
 
   it("clears creds and throws when logged out", async () => {
diff --git a/extensions/whatsapp/src/login.ts b/extensions/whatsapp/src/login.ts
@@ -22,7 +22,7 @@ export async function loginWeb(
   const restoredFromBackup = await restoreCredsFromBackupIfNeeded(account.authDir);
   const onQr = (qr: string) => {
     runtime.log("Open the WhatsApp app, go to Linked Devices, then scan this QR:");
-    void renderQrTerminal(qr, { small: true })
+    void renderQrTerminal(qr)
       .then((output) => {
         runtime.log(output.endsWith("\n") ? output.slice(0, -1) : output);
       })
diff --git a/extensions/whatsapp/src/session.ts b/extensions/whatsapp/src/session.ts
@@ -119,7 +119,7 @@ async function safeSaveCreds(
 }
 
 async function printTerminalQr(qr: string): Promise<void> {
-  const output = await renderQrTerminal(qr, { small: true });
+  const output = await renderQrTerminal(qr);
   process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
 }
 
diff --git a/src/cli/qr-cli.ts b/src/cli/qr-cli.ts
@@ -25,7 +25,7 @@ type QrCliOptions = {
 };
 
 function renderQrAscii(data: string): Promise<string> {
-  return renderQrTerminal(data, { small: true });
+  return renderQrTerminal(data);
 }
 function readDevicePairPublicUrlFromConfig(cfg: OpenClawConfig): string | undefined {
   const value = cfg.plugins?.entries?.["device-pair"]?.config?.["publicUrl"];
diff --git a/src/media/qr-terminal.render.test.ts b/src/media/qr-terminal.render.test.ts
@@ -0,0 +1,20 @@
+import { describe, expect, it } from "vitest";
+import { renderQrTerminal } from "./qr-terminal.ts";
+
+describe("renderQrTerminal (real qrcode runtime)", () => {
+  it("keeps per-row ANSI sequence counts in line with typical rows", async () => {
+    const sample = "https://wa.me/login/2@SAMPLE-TOKEN-1234567890ABCDEF";
+    const rendered = await renderQrTerminal(sample);
+    const ansiSgr = new RegExp(`${String.fromCharCode(0x1b)}\\[[0-9;]*m`, "g");
+    const escCounts = rendered
+      .split(/\r?\n/)
+      .map((line) => (line.match(ansiSgr) ?? []).length)
+      .filter((count) => count > 0);
+    expect(escCounts.length).toBeGreaterThan(0);
+    const sorted = escCounts.toSorted((a, b) => a - b);
+    const median = sorted[Math.floor(sorted.length / 2)] ?? 0;
+    const max = Math.max(...escCounts);
+    expect(median).toBeGreaterThan(0);
+    expect(max).toBeLessThanOrEqual(median * 6);
+  });
+});
diff --git a/src/media/qr-terminal.test.ts b/src/media/qr-terminal.test.ts
@@ -16,7 +16,7 @@ describe("renderQrTerminal", () => {
   it("delegates terminal rendering to qrcode", async () => {
     await expect(renderQrTerminal("openclaw")).resolves.toBe("ASCII-QR");
     expect(toString).toHaveBeenCalledWith("openclaw", {
-      small: true,
+      small: false,
       type: "terminal",
     });
   });
diff --git a/src/media/qr-terminal.ts b/src/media/qr-terminal.ts
@@ -6,7 +6,7 @@ export async function renderQrTerminal(
 ): Promise<string> {
   const qrCode = await loadQrCodeRuntime();
   return await qrCode.toString(normalizeQrText(input), {
-    small: opts.small ?? true,
+    small: opts.small ?? false,
     type: "terminal",
   });
 }

Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,7 @@ export async function pollAppRegistration(params: {`
`252`	`252`	`* otherwise the pattern is corrupted and cannot be scanned.`
`253`	`253`	`*/`
`254`	`254`	`export async function printQrCode(url: string): Promise<void> {`
`255`		`- const output = await renderQrTerminal(url, { small: true });`
	`255`	`+ const output = await renderQrTerminal(url);`
`256`	`256`	process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
`257`	`257`	`}`
`258`	`258`
Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ async function safeSaveCreds(`
`119`	`119`	`}`
`120`	`120`
`121`	`121`	`async function printTerminalQr(qr: string): Promise<void> {`
`122`		`- const output = await renderQrTerminal(qr, { small: true });`
	`122`	`+ const output = await renderQrTerminal(qr);`
`123`	`123`	process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
`124`	`124`	`}`
`125`	`125`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ type QrCliOptions = {`
`25`	`25`	`};`
`26`	`26`
`27`	`27`	`function renderQrAscii(data: string): Promise<string> {`
`28`		`- return renderQrTerminal(data, { small: true });`
	`28`	`+ return renderQrTerminal(data);`
`29`	`29`	`}`
`30`	`30`	`function readDevicePairPublicUrlFromConfig(cfg: OpenClawConfig): string \| undefined {`
`31`	`31`	`const value = cfg.plugins?.entries?.["device-pair"]?.config?.["publicUrl"];`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ export async function renderQrTerminal(`
`6`	`6`	`): Promise<string> {`
`7`	`7`	`const qrCode = await loadQrCodeRuntime();`
`8`	`8`	`return await qrCode.toString(normalizeQrText(input), {`
`9`		`- small: opts.small ?? true,`
	`9`	`+ small: opts.small ?? false,`
`10`	`10`	`type: "terminal",`
`11`	`11`	`});`
`12`	`12`	`}`