brave-real-browser-mcp-server 2.23.8 → 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/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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brave-real-browser-mcp-server",
-  "version": "2.23.8",
+  "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.8",
+    "brave-real-browser": "^2.4.9",
     "turndown": "latest",
     "vscode-languageserver": "^9.0.1",
     "vscode-languageserver-textdocument": "^1.0.12"