@cutleryapp/agent 1.0.13 → 1.0.15

package/dist/mcp-executor.js

@@ -80,7 +80,7 @@ class TestExecutor {
                         let label = labelMatch?.[1]?.trim();
                         if (label) {
                             // Split "Add to cart under Sauce Labs Bike Light product" into target + scope
-                            const scopeMatch = label.match(/^(.+?)\s+(?:under|inside|within|in the)\s+(.+)$/i);
+                            const scopeMatch = label.match(/^(.+?)\s+(?:under|inside|within|in the|in)\s+(.+)$/i);
                             const target = scopeMatch ? scopeMatch[1].trim() : label;
                             const scope = scopeMatch ? scopeMatch[2].trim() : null;
                             const nameRe = new RegExp(escapeRegex(target), 'i');
@@ -221,21 +221,54 @@ async function tryClickScoped(page, nameRe, target, scope) {
     const FAST = 3000;
     // Strip trailing generic nouns that won't appear verbatim on the page
     const cleanScope = scope.replace(/\s+(?:product|item|section|card|row|container|element|button|link|area|panel|block)$/i, '').trim();
-    const strategies = [
-        () => page.locator(`:has-text("${cleanScope}")`).last().getByRole('button', { name: nameRe }).first().click({ timeout: FAST }),
-        () => page.locator(`:has-text("${cleanScope}")`).last().getByRole('link', { name: nameRe }).first().click({ timeout: FAST }),
-        () => page.locator(`:has-text("${cleanScope}")`).last().getByText(nameRe).first().click({ timeout: FAST }),
-        // Fallback: ignore scope and click anywhere
-        () => page.getByRole('button', { name: nameRe }).first().click({ timeout: FAST }),
-        () => page.getByText(nameRe).first().click({ timeout: FAST }),
+    // Use card/item container selectors — these are tight enough to contain the button
+    // but not so deep that they exclude it. Filter by scope text, then click target within.
+    const containerSelectors = [
+        '[class*="item"]',
+        '[class*="card"]',
+        '[class*="product"]',
+        'li',
+        'article',
+        'tr',
+        '[role="listitem"]',
+        '[role="row"]',
     ];
-    for (const fn of strategies) {
+    for (const containerSel of containerSelectors) {
         try {
-            await fn();
-            return true;
+            const container = page.locator(containerSel).filter({ hasText: cleanScope });
+            const count = await container.count();
+            if (count === 0)
+                continue;
+            // Try button first, then any clickable element
+            try {
+                await container.first().getByRole('button', { name: nameRe }).first().click({ timeout: FAST });
+                return true;
+            }
+            catch { /* try next */ }
+            try {
+                await container.first().getByRole('link', { name: nameRe }).first().click({ timeout: FAST });
+                return true;
+            }
+            catch { /* try next */ }
+            try {
+                await container.first().getByText(nameRe).first().click({ timeout: FAST });
+                return true;
+            }
+            catch { /* try next */ }
         }
-        catch { /* try next */ }
+        catch { /* try next container */ }
+    }
+    // Fallback: ignore scope
+    try {
+        await page.getByRole('button', { name: nameRe }).first().click({ timeout: FAST });
+        return true;
+    }
+    catch { }
+    try {
+        await page.getByText(nameRe).first().click({ timeout: FAST });
+        return true;
     }
+    catch { }
     return false;
 }
 async function tryFill(page, label, value) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@cutleryapp/agent",
-    "version": "1.0.13",
+    "version": "1.0.15",
     "description": "Local agent that connects your machine to the Cutlery QA platform and runs UI tests via Playwright",
     "main": "dist/cli.js",
     "bin": {