fix feishu login qr rendering (#97087)

NianJiuZst · web-flow · commit 1089253ca97c · 2026-06-26T12:09:38.000-07:00
Co-authored-by: NianJiuZst &lt;180004567+users.noreply.github.com&gt;
diff --git a/extensions/feishu/src/app-registration.test.ts b/extensions/feishu/src/app-registration.test.ts
@@ -1,16 +1,21 @@
 // Feishu tests cover app registration plugin behavior.
 import { MAX_TIMER_TIMEOUT_MS } from "openclaw/plugin-sdk/number-runtime";
 import { afterEach, describe, expect, it, vi } from "vitest";
-import { beginAppRegistration, pollAppRegistration } from "./app-registration.js";
+import { beginAppRegistration, pollAppRegistration, printQrCode } from "./app-registration.js";
 
-const { fetchWithSsrFGuardMock } = vi.hoisted(() => ({
+const { fetchWithSsrFGuardMock, renderQrTerminalMock } = vi.hoisted(() => ({
   fetchWithSsrFGuardMock: vi.fn(),
+  renderQrTerminalMock: vi.fn(async () => "terminal-qr"),
 }));
 
 vi.mock("openclaw/plugin-sdk/ssrf-runtime", () => ({
   fetchWithSsrFGuard: fetchWithSsrFGuardMock,
 }));
 
+vi.mock("./qr-terminal.js", () => ({
+  renderQrTerminal: renderQrTerminalMock,
+}));
+
 function mockFeishuJson(payload: unknown) {
   fetchWithSsrFGuardMock.mockResolvedValueOnce({
     response: new Response(JSON.stringify(payload), { status: 200 }),
@@ -23,6 +28,7 @@ describe("Feishu app registration", () => {
     vi.useRealTimers();
     vi.restoreAllMocks();
     fetchWithSsrFGuardMock.mockReset();
+    renderQrTerminalMock.mockClear();
   });
 
   it("defaults unsafe begin polling lifetimes from provider responses", async () => {
@@ -59,4 +65,16 @@ describe("Feishu app registration", () => {
     await vi.runOnlyPendingTimersAsync();
     await expect(poll).resolves.toEqual({ status: "timeout" });
   });
+
+  it("prints scan-to-create QR codes with compact terminal rendering", async () => {
+    const writeSpy = vi.spyOn(process.stdout, "write").mockImplementation(() => true);
+
+    await printQrCode("https://accounts.feishu.cn/verify?device_code=long-device-code");
+
+    expect(renderQrTerminalMock).toHaveBeenCalledWith(
+      "https://accounts.feishu.cn/verify?device_code=long-device-code",
+      { small: true },
+    );
+    expect(writeSpy).toHaveBeenCalledWith("terminal-qr\n");
+  });
 });
diff --git a/extensions/feishu/src/app-registration.ts b/extensions/feishu/src/app-registration.ts
@@ -266,7 +266,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);
+  const output = await renderQrTerminal(url, { small: true });
   process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -266,7 +266,7 @@ export async function pollAppRegistration(params: {`
`266`	`266`	`* otherwise the pattern is corrupted and cannot be scanned.`
`267`	`267`	`*/`
`268`	`268`	`export async function printQrCode(url: string): Promise<void> {`
`269`		`- const output = await renderQrTerminal(url);`
	`269`	`+ const output = await renderQrTerminal(url, { small: true });`
`270`	`270`	process.stdout.write(output.endsWith("\n") ? output : `${output}\n`);
`271`	`271`	`}`
`272`	`272`