brave-real-browser-mcp-server 2.23.7 → 2.23.9

package/dist/handlers/interaction-handlers.js

@@ -44,15 +44,31 @@ export async function handleClick(args) {
                 strategyMessage = `\n🔄 Self-healing: Used ${strategy} fallback selector: ${usedSelector}`;
                 tracker.setProgress(50, `🔄 Using fallback selector: ${usedSelector}`);
             }
+            // Text-based strategies have special __TEXT__: prefix or are strategy-identified
+            // These have invalid CSS selectors, so we skip waitForSelector and use element directly
+            const isTextBasedStrategy = ['text-content', 'implicit-text-match', 'text', 'text-exact', 'text-partial', 'tag-text'].includes(strategy) ||
+                usedSelector.startsWith('__TEXT__:');
+            // Clean display selector for output (remove __TEXT__: prefix)
+            const displaySelector = usedSelector.startsWith('__TEXT__:')
+                ? usedSelector.replace('__TEXT__:', 'text:')
+                : usedSelector;
             try {
                 tracker.setProgress(60, '⏳ Waiting for element to be ready...');
-                // Wait for element to be ready
-                await pageInstance.waitForSelector(usedSelector, { timeout: 5000 });
+                // Only wait for selector if it's a valid CSS selector (not text-based)
+                if (!isTextBasedStrategy) {
+                    await pageInstance.waitForSelector(usedSelector, { timeout: 5000 });
+                }
                 // Check element visibility and interaction options
                 const boundingBox = await element.boundingBox();
                 if (!boundingBox) {
                     tracker.setProgress(70, '🔧 No bounding box, using JavaScript click...');
-                    await pageInstance.$eval(usedSelector, (el) => el.click());
+                    // For text-based strategies, use element.evaluate to click
+                    if (isTextBasedStrategy) {
+                        await element.evaluate((el) => el.click());
+                    }
+                    else {
+                        await pageInstance.$eval(usedSelector, (el) => el.click());
+                    }
                 }
                 else {
                     tracker.setProgress(70, '🖱️ Clicking element...');
@@ -60,12 +76,13 @@ export async function handleClick(args) {
                         tracker.setProgress(75, '🔄 Clicking and waiting for navigation...');
                         await Promise.all([
                             pageInstance.waitForNavigation({ waitUntil: 'networkidle2' }),
-                            pageInstance.click(usedSelector),
+                            element.click(), // Use element.click() for all strategies
                         ]);
                         tracker.setProgress(90, '✅ Navigation completed');
                     }
                     else {
-                        await pageInstance.click(usedSelector);
+                        // Use element.click() directly - works for all strategies
+                        await element.click();
                     }
                 }
                 tracker.complete('🎉 Click completed successfully');
@@ -82,7 +99,13 @@ export async function handleClick(args) {
                 tracker.setProgress(85, '⚠️ Click failed, trying JavaScript fallback...');
                 // Final fallback: JavaScript click
                 try {
-                    await pageInstance.$eval(usedSelector, (el) => el.click());
+                    // For text-based strategies, use element.evaluate instead of $eval
+                    if (isTextBasedStrategy) {
+                        await element.evaluate((el) => el.click());
+                    }
+                    else {
+                        await pageInstance.$eval(usedSelector, (el) => el.click());
+                    }
                     tracker.complete('🎉 Click completed via JavaScript fallback');
                     return {
                         content: [

package/dist/index.js

@@ -63,7 +63,7 @@ handleM3u8Parser, handleCookieManager,
 // Download tools
 handleFileDownloader, 
 // Enhanced streaming/download tools
-handleIframeHandler, handlePopupHandler, handleStreamExtractor, } from './handlers/advanced-tools.js';
+handleIframeHandler, handleStreamExtractor, } from './handlers/advanced-tools.js';
 // State for video recording
 const recorderState = new Map();
 debug('All modules loaded successfully');
@@ -259,10 +259,6 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 if (!page)
                     throw new Error('Browser not initialized. Call browser_init first.');
                 return { content: [{ type: 'text', text: JSON.stringify(await handleIframeHandler(page, args)) }] };
-            case TOOL_NAMES.POPUP_HANDLER:
-                if (!page)
-                    throw new Error('Browser not initialized. Call browser_init first.');
-                return { content: [{ type: 'text', text: JSON.stringify(await handlePopupHandler(page, args)) }] };
             case TOOL_NAMES.STREAM_EXTRACTOR:
                 if (!page)
                     throw new Error('Browser not initialized. Call browser_init first.');

package/dist/lsp/browser-automation-lsp.js

@@ -323,15 +323,6 @@ const TOOL_DEFINITIONS = {
             { name: 'selector', type: 'string', description: 'CSS selector of target iframe', required: false },
         ],
     },
-    popup_handler: {
-        name: 'popup_handler',
-        description: 'Handle popups, new tabs, and ad overlays during navigation',
-        category: 'Advanced',
-        parameters: [
-            { name: 'action', type: 'string', description: 'Popup handling action', required: false, enum: ['block', 'allow', 'close', 'switch', 'list', 'closeAll'] },
-            { name: 'autoCloseAds', type: 'boolean', description: 'Automatically close detected ad popups', required: false, default: true },
-        ],
-    },
 };
 // ============================================================
 // LSP SERVER

package/dist/self-healing-locators.js

@@ -145,7 +145,7 @@ export class SelfHealingLocators {
             if (element) {
                 return {
                     element,
-                    usedSelector: `${textSelector.tagName || '*'}[text*="${textSelector.text}"]`,
+                    usedSelector: `__TEXT__:${textSelector.tagName || '*'}:${textSelector.text}`,
                     strategy: 'text-content'
                 };
             }
@@ -171,7 +171,7 @@ export class SelfHealingLocators {
             if (textElement) {
                 return {
                     element: textElement,
-                    usedSelector: `//*[text()[contains(., "${primarySelector}")]]`,
+                    usedSelector: `__TEXT__:*:${primarySelector}`,
                     strategy: 'implicit-text-match'
                 };
             }

package/dist/tool-definitions.js

@@ -626,21 +626,6 @@ export const TOOLS = [
             },
         },
     },
-    {
-        name: 'popup_handler',
-        description: 'Handle popups, new tabs, and ad overlays during navigation',
-        inputSchema: {
-            type: 'object',
-            additionalProperties: false,
-            properties: {
-                action: { type: 'string', enum: ['block', 'allow', 'close', 'switch', 'list', 'closeAll'], description: 'Popup handling action' },
-                autoCloseAds: { type: 'boolean', description: 'Automatically close detected ad popups', default: true },
-                waitForTarget: { type: 'boolean', description: 'Wait for new tab/popup to open', default: false },
-                targetIndex: { type: 'number', description: 'Index of tab to switch to' },
-                timeout: { type: 'number', description: 'Timeout for waiting operations in ms', default: 10000 },
-            },
-        },
-    },
     {
         name: 'stream_extractor',
         description: 'Master tool: Extract direct download/stream URLs from any page with automatic redirect following, countdown waiting, and format detection',
@@ -698,7 +683,6 @@ export const TOOL_NAMES = {
     FILE_DOWNLOADER: 'file_downloader',
     // Enhanced tools
     IFRAME_HANDLER: 'iframe_handler',
-    POPUP_HANDLER: 'popup_handler',
     STREAM_EXTRACTOR: 'stream_extractor',
 };
 // Tool categories for organization

package/dist/unified-server.js

@@ -536,7 +536,6 @@ async function main() {
             console.error('   🍪 cookie_manager        - Manage cookies');
             console.error('   📥 file_downloader       - Download files');
             console.error('   🖼️  iframe_handler        - Handle iframes');
-            console.error('   🚫 popup_handler         - Handle popups/ads');
             console.error('');
             console.error('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
             console.error('');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brave-real-browser-mcp-server",
-  "version": "2.23.7",
+  "version": "2.23.9",
   "description": "🦁 MCP server for Brave Real Browser - NPM Workspaces Monorepo with anti-detection features, SSE streaming, and LSP compatibility",
   "type": "module",
   "main": "dist/index.js",
@@ -50,7 +50,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "latest",
     "@types/turndown": "latest",
-    "brave-real-browser": "^2.4.7",
+    "brave-real-browser": "^2.4.9",
     "turndown": "latest",
     "vscode-languageserver": "^9.0.1",
     "vscode-languageserver-textdocument": "^1.0.12"