chore: do not return fullPage screenshots to the LLM (#849)

src/tools/screenshot.ts

@@ -75,11 +75,16 @@ const screenshot = defineTabTool({
 
     const buffer = locator ? await locator.screenshot(options) : await tab.page.screenshot(options);
     response.addResult(`Took the ${screenshotTarget} screenshot and saved it as ${fileName}`);
+
+    // https://github.com/microsoft/playwright-mcp/issues/817
+    // Never return large images to LLM, saving them to the file system is enough.
+    if (!params.fullPage) {
       response.addImage({
         contentType: fileType === 'png' ? 'image/png' : 'image/jpeg',
         data: buffer
       });
     }
+  }
 });
 
 export default [

tests/screenshot.spec.ts

@@ -264,12 +264,7 @@ test('browser_take_screenshot (fullPage: true)', async ({ startClient, server },
       {
         text: expect.stringContaining('fullPage: true'),
         type: 'text',
-      },
+      }
-      {
-        data: expect.any(String),
-        mimeType: 'image/png',
-        type: 'image',
-      },
     ],
   });
 });