From f958e311d23da644b94e5278ae22e6d246a5d8a2 Mon Sep 17 00:00:00 2001
From: Gustavo Madeira Santana <gumadeiras@gmail.com>
Date: Tue, 14 Apr 2026 16:37:40 -0400
Subject: [PATCH] tests: fix Feishu card action runtime mock typing

---
 .../src/monitor.card-action.lifecycle.test.ts   | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/extensions/feishu/src/monitor.card-action.lifecycle.test.ts b/extensions/feishu/src/monitor.card-action.lifecycle.test.ts
index 58f408bd03d..2d0c9be31c1 100644
--- a/extensions/feishu/src/monitor.card-action.lifecycle.test.ts
+++ b/extensions/feishu/src/monitor.card-action.lifecycle.test.ts
@@ -32,7 +32,7 @@ const {
 } = getFeishuLifecycleTestMocks();
 
 let _handlers: Record<string, (data: unknown) => Promise<void>> = {};
-let lastRuntimeError: ReturnType<typeof vi.fn> | null = null;
+let lastRuntime: ReturnType<typeof createRuntimeEnv> | null = null;
 const originalStateDir = process.env.OPENCLAW_STATE_DIR;
 const lifecycleConfig = createFeishuLifecycleConfig({
   accountId: "acct-card",
@@ -95,14 +95,13 @@ function createCardActionEvent(params: {
 }
 
 async function setupLifecycleMonitor() {
-  const runtime = createRuntimeEnv();
-  lastRuntimeError = runtime.error;
+  lastRuntime = createRuntimeEnv();
   return setupFeishuLifecycleHandler({
     createEventDispatcherMock,
     onRegister: (registered) => {
       _handlers = registered;
     },
-    runtime,
+    runtime: lastRuntime,
     cfg: lifecycleConfig,
     account: lifecycleAccount,
     handlerKey: "card.action.trigger",
@@ -115,7 +114,7 @@ describe("Feishu card-action lifecycle", () => {
     vi.useRealTimers();
     vi.clearAllMocks();
     _handlers = {};
-    lastRuntimeError = null;
+    lastRuntime = null;
     resetProcessedFeishuCardActionTokensForTests();
     setFeishuLifecycleStateDir("openclaw-feishu-card-action");
 
@@ -172,7 +171,7 @@ describe("Feishu card-action lifecycle", () => {
       createFeishuReplyDispatcherMock,
     });
 
-    expect(lastRuntimeError).not.toHaveBeenCalled();
+    expect(lastRuntime?.error).not.toHaveBeenCalled();
     expect(dispatchReplyFromConfigMock).toHaveBeenCalledTimes(1);
     expect(createFeishuReplyDispatcherMock).toHaveBeenCalledTimes(1);
     expect(createFeishuReplyDispatcherMock).toHaveBeenCalledWith(
@@ -213,10 +212,10 @@ describe("Feishu card-action lifecycle", () => {
       handler: onCardAction,
       event,
       dispatchReplyFromConfigMock,
-      runtimeErrorMock: lastRuntimeError as ReturnType<typeof vi.fn>,
+      runtimeErrorMock: lastRuntime?.error as ReturnType<typeof vi.fn>,
     });
 
-    expect(lastRuntimeError).toHaveBeenCalledTimes(1);
+    expect(lastRuntime?.error).toHaveBeenCalledTimes(1);
     expect(dispatchReplyFromConfigMock).toHaveBeenCalledTimes(1);
     expectFeishuReplyDispatcherSentFinalReplyOnce({ createFeishuReplyDispatcherMock });
   });
@@ -252,7 +251,7 @@ describe("Feishu card-action lifecycle", () => {
       },
     });
 
-    expect(lastRuntimeError).toHaveBeenCalledWith(
+    expect(lastRuntime?.error).toHaveBeenCalledWith(
       "feishu[acct-card]: ignoring malformed card action payload",
     );
     expect(dispatchReplyFromConfigMock).not.toHaveBeenCalled();