From 5099bbdb53fb3ed2c066e121f3c67c64a811ea19 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 1 May 2026 22:41:01 +0000 Subject: [PATCH 1/2] Initial plan From c1d458eb6d66148156ed0929d78b8dc554519a76 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 1 May 2026 22:44:51 +0000 Subject: [PATCH 2/2] fix(mcp): skip image in MCP response when filename is explicitly provided for browser_take_screenshot Agent-Logs-Url: https://github.com/microsoft/playwright/sessions/c5b0b096-f441-411a-9107-44373adcb55f Co-authored-by: yury-s <9798949+yury-s@users.noreply.github.com> --- packages/playwright-core/src/tools/backend/screenshot.ts | 3 ++- tests/mcp/screenshot.spec.ts | 5 ----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/playwright-core/src/tools/backend/screenshot.ts b/packages/playwright-core/src/tools/backend/screenshot.ts index 71342abc9e0c0..d2aeea504a2d3 100644 --- a/packages/playwright-core/src/tools/backend/screenshot.ts +++ b/packages/playwright-core/src/tools/backend/screenshot.ts @@ -67,7 +67,8 @@ const screenshot = defineTabTool({ response.addCode(`await page.screenshot(${formatObject({ ...options, path: resolvedFile.relativeName })});`); await response.addFileResult(resolvedFile, data); - await response.registerImageResult(data, fileType); + if (!params.filename) + await response.registerImageResult(data, fileType); } }); diff --git a/tests/mcp/screenshot.spec.ts b/tests/mcp/screenshot.spec.ts index bbe3c74d7991c..faa0d7d5eec73 100644 --- a/tests/mcp/screenshot.spec.ts +++ b/tests/mcp/screenshot.spec.ts @@ -238,11 +238,6 @@ test('browser_take_screenshot (filename: "output.png")', async ({ client, server text: expect.stringContaining(`output.png`), type: 'text', }, - { - data: expect.any(String), - mimeType: 'image/png', - type: 'image', - }, ], });