@cutleryapp/agent 1.0.10 → 1.0.12

package/dist/mcp-executor.js

@@ -92,26 +92,22 @@ class TestExecutor {
                         }
                     }
                     else if (lower.includes("fill") || lower.includes("type") || lower.includes("enter")) {
-                        const quoted = raw.match(/(?:enter|fill|type)\s+"([^"]+)"\s+(?:in|into)\s+(?:the\s+)?(?:"([^"]+)"|(\w[\w\s]*?)\s*(?:field|input|box|area)?)\s*$/i);
-                        if (quoted) {
-                            const value = quoted[1];
-                            const quotedTarget = quoted[2];
-                            const bareLabel = quoted[3];
-                            // A token in quotes is usually a human-readable label
-                            // (e.g. `Fill "standard_user" in "Username"`), not a CSS
-                            // selector. Only treat it as a CSS selector when it actually
-                            // looks like one — otherwise resolve via tryFill so the
-                            // label can match placeholder/aria/data-test/label etc.
+                        // Support both quoted and unquoted formats:
+                        //   Fill "standard_user" in "Username"
+                        //   Fill standard_user in Username field
+                        // Split on first " in " / " into " to separate value from field
+                        const match = raw.match(/(?:enter|fill|type)\s+"([^"]+)"\s+(?:in|into)\s+(?:the\s+)?"?([^"]+?)"?\s*(?:field|input|box|area)?\s*$/i) ||
+                            raw.match(/(?:enter|fill|type)\s+(\S+)\s+(?:in|into)\s+(?:the\s+)?(.+?)\s*(?:field|input|box|area)?\s*$/i);
+                        if (match) {
+                            const value = match[1].trim();
+                            const fieldLabel = match[2].trim();
                             const looksLikeCss = (s) => /[#.\[\]:>]/.test(s);
-                            if (quotedTarget && looksLikeCss(quotedTarget)) {
-                                await page.waitForSelector(quotedTarget, { state: "visible", timeout: 5000 });
-                                await page.fill(quotedTarget, value);
+                            if (looksLikeCss(fieldLabel)) {
+                                await page.waitForSelector(fieldLabel, { state: "visible", timeout: 5000 });
+                                await page.fill(fieldLabel, value);
                             }
-                            else if (quotedTarget) {
-                                await tryFill(page, quotedTarget.trim(), value);
-                            }
-                            else if (bareLabel) {
-                                await tryFill(page, bareLabel.trim(), value);
+                            else {
+                                await tryFill(page, fieldLabel, value);
                             }
                         }
                     }
@@ -223,11 +219,12 @@ async function tryClick(page, nameRe, label) {
 }
 async function tryClickScoped(page, nameRe, target, scope) {
     const FAST = 3000;
-    // Find a container that contains the scope text, then click the target inside it
+    // 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("${scope}")`).last().getByRole('button', { name: nameRe }).first().click({ timeout: FAST }),
-        () => page.locator(`:has-text("${scope}")`).last().getByRole('link', { name: nameRe }).first().click({ timeout: FAST }),
-        () => page.locator(`:has-text("${scope}")`).last().getByText(nameRe).first().click({ timeout: FAST }),
+        () => 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 }),

package/package.json

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