From b8dc33f1803c17dd3dd66febc8a9d277db8deee9 Mon Sep 17 00:00:00 2001 From: ajmallesh Date: Wed, 22 Oct 2025 12:15:47 -0700 Subject: [PATCH] chore: remove screenshot saving from Playwright MCP instances MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove unnecessary screenshot storage to reduce file I/O and disk usage: - Removed screenshot directory creation - Removed --output-dir flag from Playwright MCP setup - Agents can still take screenshots, but they won't persist to disk Screenshots were not being used by any part of Shannon for analysis or reporting, making their storage unnecessary overhead. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- src/setup/environment.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/setup/environment.js b/src/setup/environment.js index b63240d..dbee8f7 100644 --- a/src/setup/environment.js +++ b/src/setup/environment.js @@ -21,21 +21,16 @@ export async function setupMCP(sourceDir) { } } - // Ensure screenshot directories exist - await fs.ensureDir(path.join(sourceDir, 'screenshots')); - // Create 5 isolated instances sequentially to avoid config conflicts for (let i = 1; i <= 5; i++) { const instanceName = `playwright-agent${i}`; - const screenshotDir = path.join(sourceDir, 'screenshots', instanceName); const userDataDir = `/tmp/${instanceName}`; - // Ensure both directories exist - await fs.ensureDir(screenshotDir); + // Ensure user data directory exists await fs.ensureDir(userDataDir); try { - await $`claude mcp add ${instanceName} --scope user -- npx @playwright/mcp@latest --isolated --user-data-dir ${userDataDir} --output-dir ${screenshotDir}`; + await $`claude mcp add ${instanceName} --scope user -- npx @playwright/mcp@latest --isolated --user-data-dir ${userDataDir}`; console.log(chalk.green(` ✅ ${instanceName} configured`)); } catch (error) { if (error.message?.includes('already exists')) {