@machina.ai/cell-cli-core 1.36.0-rc1 → 1.38.1-rc2

package/dist/src/agents/browser/browserManager.d.ts

@@ -18,6 +18,14 @@
 import { Client } from '@modelcontextprotocol/sdk/client/index.js';
 import type { Tool as McpTool } from '@modelcontextprotocol/sdk/types.js';
 import type { Config } from '../../config/config.js';
+/**
+ * Typed error for domain restriction violations.
+ * Thrown when a navigation tool targets a domain not in allowedDomains.
+ * Caught by mcpToolWrapper to terminate the agent immediately.
+ */
+export declare class DomainNotAllowedError extends Error {
+    constructor(message: string);
+}
 /**
  * Content item from an MCP tool call response.
  * Can be text or image (for take_screenshot).
@@ -48,9 +56,67 @@ export interface McpToolCallResult {
  */
 export declare class BrowserManager {
     private config;
+    private static instances;
+    /**
+     * Maximum number of parallel browser instances allowed in isolated mode.
+     * Prevents unbounded resource consumption from concurrent browser_agent calls.
+     */
+    static readonly MAX_PARALLEL_INSTANCES = 5;
+    /**
+     * Returns the cache key for a given config.
+     * Uses `sessionMode:profilePath` so different profiles get separate instances.
+     */
+    private static getInstanceKey;
+    /**
+     * Returns an existing BrowserManager for the current config's session mode
+     * and profile, or creates a new one.
+     *
+     * Concurrency rules:
+     * - **persistent / existing mode**: Only one instance is allowed at a time.
+     *   If the instance is already in-use, an error is thrown instructing the
+     *   caller to run browser tasks sequentially.
+     * - **isolated mode**: Parallel instances are allowed up to
+     *   MAX_PARALLEL_INSTANCES. Each isolated instance gets its own temp profile.
+     */
+    static getInstance(config: Config): BrowserManager;
+    /**
+     * Closes all cached BrowserManager instances and clears the cache.
+     * Called on /clear commands and CLI exit.
+     */
+    static resetAll(): Promise<void>;
+    /**
+     * Alias for resetAll — used by CLI exit cleanup for clarity.
+     */
+    static closeAll(): Promise<void>;
     private rawMcpClient;
     private mcpTransport;
     private discoveredTools;
+    private disconnected;
+    private isClosing;
+    private connectionPromise;
+    /**
+     * Whether this instance is currently acquired by an active invocation.
+     * Used by getInstance() to avoid handing the same browser to concurrent
+     * browser_agent calls.
+     */
+    private inUse;
+    /**
+     * Marks this instance as in-use. Call this when starting a browser agent
+     * invocation so concurrent calls get a separate instance.
+     */
+    acquire(): void;
+    /**
+     * Marks this instance as available for reuse. Call this in the finally
+     * block of a browser agent invocation.
+     */
+    release(): void;
+    /**
+     * Returns whether this instance is currently acquired by an active invocation.
+     */
+    isAcquired(): boolean;
+    /** State for action rate limiting */
+    private actionCounter;
+    private readonly maxActionsPerTask;
     /**
      * Whether to inject the automation overlay.
      * Always false in headless mode (no visible window to decorate).
@@ -74,18 +140,32 @@ export declare class BrowserManager {
      * @param toolName The name of the tool to call
      * @param args Arguments to pass to the tool
      * @param signal Optional AbortSignal to cancel the call
+     * @param isInternal Determine if the tool is for internal execution
      * @returns The result from the MCP server
      */
-    callTool(toolName: string, args: Record<string, unknown>, signal?: AbortSignal): Promise<McpToolCallResult>;
+    callTool(toolName: string, args: Record<string, unknown>, signal?: AbortSignal, isInternal?: boolean): Promise<McpToolCallResult>;
     /**
      * Safely maps a raw MCP SDK callTool response to our typed McpToolCallResult
      * without using unsafe type assertions.
      */
     private toResult;
+    /**
+     * Returns whether the MCP client is currently connected and healthy.
+     */
+    isConnected(): boolean;
     /**
      * Ensures browser and MCP client are connected.
+     * If a previous connection was lost (e.g., user closed the browser),
+     * this will reconnect with exponential backoff (up to MAX_RECONNECT_RETRIES).
+     *
+     * Concurrent callers share a single in-flight connection promise so that
+     * two subagents racing at startup do not trigger duplicate connectMcp() calls.
      */
     ensureConnection(): Promise<void>;
+    /**
+     * Connects to chrome-devtools-mcp with exponential backoff retry.
+     */
+    private connectWithRetry;
     /**
      * Closes browser and cleans up connections.
      * The browser process is managed by chrome-devtools-mcp, so closing
@@ -103,6 +183,12 @@ export declare class BrowserManager {
      * BrowserManager instance.
      */
     private connectMcp;
+    /**
+     * Classifies a connection error message into a known error type.
+     * Shared between connectMcp error recording and createConnectionError
+     * to ensure consistent error categorization across the browser agent.
+     */
+    private static classifyConnectionError;
     /**
      * Creates an Error with context-specific remediation based on the actual
      * error message and the current sessionMode.
@@ -120,12 +206,7 @@ export declare class BrowserManager {
      */
     private checkNavigationRestrictions;
     /**
-     * Registers a fallback notification handler on the MCP client to
-     * automatically re-inject the input blocker after any server-side
-     * notification (e.g. page navigation, resource updates).
-     *
-     * This covers ALL navigation types (link clicks, form submissions,
-     * history navigation) — not just explicit navigate_page tool calls.
+     * Checks whether a hostname matches any pattern in the allowed domains list.
      */
-    private registerInputBlockerHandler;
+    private isDomainAllowed;
 }