removed screenshot capabilities for now.

Author: bhouston

BROWSER_AUTOMATION.md

@@ -8,7 +8,6 @@ This document describes the browser automation capabilities implemented in the p
 
 - Browser session management
 - Page control and interaction
-- Screenshot capture
 - Resource cleanup and management
 - Error handling
 - Type-safe API
@@ -30,12 +29,6 @@ Manages page interactions:
 - State management
 - Event handling
 
-### ScreenshotManager
-Provides screenshot capabilities:
-- Full page captures
-- Element-specific captures
-- Screenshot storage and management
-
 ## Installation & Setup
 
 ```bash
@@ -69,14 +62,6 @@ await page.click('#submit-button');
 await page.type('#search-input', 'search term');
 ```
 
-### Taking Screenshots
-```typescript
-await page.screenshot({
-  path: './screenshot.png',
-  fullPage: true
-});
-```
-
 ## Error Handling
 
 The system implements comprehensive error handling:
@@ -117,7 +102,6 @@ Main class providing browser automation capabilities.
 #### Methods
 - `newPage()`: Creates a new page session
 - `close()`: Closes all browser sessions
-- `screenshot()`: Captures screenshots
 - `evaluate()`: Evaluates JavaScript in the page context
 
 ### PageController
@@ -129,13 +113,6 @@ Handles page-specific operations.
 - `type(selector: string, text: string)`: Types text
 - `waitForSelector(selector: string)`: Waits for element
 
-### ScreenshotManager
-Manages screenshot operations.
-
-#### Methods
-- `capture(options: ScreenshotOptions)`: Takes screenshot
-- `saveToFile(path: string)`: Saves screenshot to file
-
 ## Future Enhancements
 
 The following features are planned for future releases:
@@ -193,4 +170,4 @@ Please see CONTRIBUTING.md for guidelines on contributing to this project.
 
 ## License
 
-This project is licensed under the MIT License - see the LICENSE file for details.
+This project is licensed under the MIT License - see the LICENSE file for details

src/tools/browser/BrowserAutomation.ts

@@ -1,6 +1,5 @@
 import { BrowserManager } from "./browser-manager.js";
 import { PageController } from "./page-controller.js";
-import { ScreenshotController } from "./screenshot.js";
 
 export class BrowserAutomation {
   private static instance: BrowserAutomation;
@@ -20,12 +19,10 @@ export class BrowserAutomation {
   async createSession(headless: boolean = false) {
     const session = await this.browserManager.createSession({ headless });
     const pageController = new PageController(session.page);
-    const screenshotManager = new ScreenshotController(session.page);
 
     return {
       sessionId: session.id,
       pageController,
-      screenshotManager,
       close: () => this.browserManager.closeSession(session.id),
     };
   }

src/tools/browser/browseMessage.ts

@@ -1,24 +1,16 @@
 import { Tool } from '../../core/types.js';
 import { z } from 'zod';
 import { zodToJsonSchema } from 'zod-to-json-schema';
-import { browserSessions, type BrowserAction, SelectorType, type ScreenshotOptions } from './types.js';
-
-// Schema for browser action options
-const screenshotOptionsSchema = z.object({
-  path: z.string().optional(),
-  fullPage: z.boolean().optional(),
-  type: z.enum(['png', 'jpeg']).optional(),
-  quality: z.number().min(0).max(100).optional(),
-}).optional();
+import { browserSessions, type BrowserAction, SelectorType } from './types.js';
 
 // Schema for browser action
 const browserActionSchema = z.object({
-  type: z.enum(['goto', 'click', 'type', 'wait', 'screenshot', 'content', 'close']),
+  type: z.enum(['goto', 'click', 'type', 'wait', 'content', 'close']),
   url: z.string().url().optional(),
   selector: z.string().optional(),
   selectorType: z.nativeEnum(SelectorType).optional(),
   text: z.string().optional(),
-  options: screenshotOptionsSchema,
+  options: z.object({}).optional(),
 }).describe('Browser action to perform');
 
 // Main parameter schema
@@ -32,7 +24,6 @@ const parameterSchema = z.object({
 const returnSchema = z.object({
   status: z.string(),
   content: z.string().optional(),
-  screenshot: z.string().optional(),
   error: z.string().optional(),
 });
 
@@ -110,16 +101,6 @@ export const browseMessageTool: Tool<Parameters, ReturnType> = {
           return { status: 'success' };
         }
 
-        case 'screenshot': {
-          const screenshotBuffer = await page.screenshot({
-            ...action.options,
-            type: 'png',
-          });
-          const screenshotBase64 = screenshotBuffer.toString('base64');
-          logger.verbose('Screenshot captured successfully');
-          return { status: 'success', screenshot: screenshotBase64 };
-        }
-
         case 'content': {
           const content = await page.content();
           logger.verbose('Page content retrieved successfully');

src/tools/browser/screenshot.ts

@@ -23,7 +23,6 @@ export enum BrowserErrorCode {
   UNKNOWN = "UNKNOWN",
   SELECTOR_INVALID = "SELECTOR_INVALID",
   ELEMENT_NOT_FOUND = "ELEMENT_NOT_FOUND",
-  SCREENSHOT_FAILED = "SCREENSHOT_FAILED",
 }
 
 // Browser error class
@@ -54,15 +53,6 @@ export interface SelectorOptions {
   visible?: boolean;
 }
 
-// Screenshot options
-export interface ScreenshotOptions {
-  path?: string;
-  fullPage?: boolean;
-  type?: "png" | "jpeg";
-  quality?: number;
-  scale?: number;
-}
-
 // Global map to store browser sessions
 export const browserSessions: Map<string, BrowserSession> = new Map();
 
@@ -77,6 +67,5 @@ export type BrowserAction =
       selectorType?: SelectorType;
     }
   | { type: "wait"; selector: string; selectorType?: SelectorType }
-  | { type: "screenshot"; options?: ScreenshotOptions }
   | { type: "content" }
-  | { type: "close" };
+  | { type: "close" };