From 414e9c87cbc085b213cd036146ec8aaf0a93a178 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 14 Mar 2026 01:00:17 +0000
Subject: [PATCH] refactor: share browser console result formatting

---
 src/agents/tools/browser-tool.actions.ts | 47 +++++++++++-------------
 1 file changed, 21 insertions(+), 26 deletions(-)

diff --git a/src/agents/tools/browser-tool.actions.ts b/src/agents/tools/browser-tool.actions.ts
index 673585d16b3..52551b166a3 100644
--- a/src/agents/tools/browser-tool.actions.ts
+++ b/src/agents/tools/browser-tool.actions.ts
@@ -54,6 +54,25 @@ function formatTabsToolResult(tabs: unknown[]): AgentToolResult<unknown> {
   };
 }
 
+function formatConsoleToolResult(result: {
+  targetId?: string;
+  messages?: unknown[];
+}): AgentToolResult<unknown> {
+  const wrapped = wrapBrowserExternalJson({
+    kind: "console",
+    payload: result,
+    includeWarning: false,
+  });
+  return {
+    content: [{ type: "text" as const, text: wrapped.wrappedText }],
+    details: {
+      ...wrapped.safeDetails,
+      targetId: typeof result.targetId === "string" ? result.targetId : undefined,
+      messageCount: Array.isArray(result.messages) ? result.messages.length : undefined,
+    },
+  };
+}
+
 function isChromeStaleTargetError(profile: string | undefined, err: unknown): boolean {
   if (profile !== "chrome") {
     return false;
@@ -258,34 +277,10 @@ export async function executeConsoleAction(params: {
         targetId,
       },
     })) as { ok?: boolean; targetId?: string; messages?: unknown[] };
-    const wrapped = wrapBrowserExternalJson({
-      kind: "console",
-      payload: result,
-      includeWarning: false,
-    });
-    return {
-      content: [{ type: "text" as const, text: wrapped.wrappedText }],
-      details: {
-        ...wrapped.safeDetails,
-        targetId: typeof result.targetId === "string" ? result.targetId : undefined,
-        messageCount: Array.isArray(result.messages) ? result.messages.length : undefined,
-      },
-    };
+    return formatConsoleToolResult(result);
   }
   const result = await browserConsoleMessages(baseUrl, { level, targetId, profile });
-  const wrapped = wrapBrowserExternalJson({
-    kind: "console",
-    payload: result,
-    includeWarning: false,
-  });
-  return {
-    content: [{ type: "text" as const, text: wrapped.wrappedText }],
-    details: {
-      ...wrapped.safeDetails,
-      targetId: result.targetId,
-      messageCount: result.messages.length,
-    },
-  };
+  return formatConsoleToolResult(result);
 }
 
 export async function executeActAction(params: {