From 37973f08765748aaf034c934c9470441a50aec0f Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 20 Jun 2025 09:23:29 +0000
Subject: [PATCH] Fix misleading browser_close message by adding resultOverride

Co-authored-by: Skn0tt <14912729+Skn0tt@users.noreply.github.com>
---
 src/tools/common.ts  | 6 ++++++
 tests/launch.spec.ts | 2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/tools/common.ts b/src/tools/common.ts
index 8a16c35..1b6191b 100644
--- a/src/tools/common.ts
+++ b/src/tools/common.ts
@@ -34,6 +34,12 @@ const close = defineTool({
       code: [`await page.close()`],
       captureSnapshot: false,
       waitForNetwork: false,
+      resultOverride: {
+        content: [{
+          type: 'text',
+          text: 'Browser closed successfully.',
+        }],
+      },
     };
   },
 });
diff --git a/tests/launch.spec.ts b/tests/launch.spec.ts
index 25a24e5..47742ef 100644
--- a/tests/launch.spec.ts
+++ b/tests/launch.spec.ts
@@ -29,7 +29,7 @@ test('test reopen browser', async ({ startClient, server, mcpMode }) => {
 
   expect(await client.callTool({
     name: 'browser_close',
-  })).toContainTextContent('No open pages available');
+  })).toContainTextContent('Browser closed successfully.');
 
   expect(await client.callTool({
     name: 'browser_navigate',