@cutleryapp/agent 1.0.42 → 1.0.43

package/dist/mcp-executor.js

@@ -448,10 +448,23 @@ class TestExecutor {
                     }
                 }
                 catch (err) {
-                    // Log the error — do NOT call AI here, it's too slow and usually can't recover either
-                    console.log(`  ⚠️  Step failed: ${err.message.split('\n')[0]}`);
-                    stepError = err.message;
-                    result.success = false;
+                    // AI last-resort recovery — only when Playwright strategies all failed
+                    if (process.env.OPENAI_API_KEY) {
+                        console.log(`  🤖 Playwright failed, trying AI: ${err.message.split('\n')[0]}`);
+                        try {
+                            await aiSingleShot(page, raw);
+                        }
+                        catch (aiErr) {
+                            console.log(`  ⚠️  AI recovery also failed: ${aiErr.message.split('\n')[0]}`);
+                            stepError = err.message;
+                            result.success = false;
+                        }
+                    }
+                    else {
+                        console.log(`  ⚠️  Step failed: ${err.message.split('\n')[0]}`);
+                        stepError = err.message;
+                        result.success = false;
+                    }
                 }
                 // Screenshot on failure, on the last step, or every 5 steps — not every step
                 let screenshotB64 = "";
@@ -1249,39 +1262,27 @@ async function tryFill(page, label, value) {
     const labelRe = new RegExp(escapeRegex(label), "i");
     const variants = labelVariants(label);
     const attrContains = (attr) => variants.map(v => `input[${attr}*="${cssEscape(v)}" i], textarea[${attr}*="${cssEscape(v)}" i], [contenteditable="true"][${attr}*="${cssEscape(v)}" i]`).join(", ");
-    function withTimeout(fn, ms) {
-        return Promise.race([fn(), new Promise((_, r) => setTimeout(() => r(new Error('timeout')), ms))]);
-    }
-    // Phase 1: race the 3 highest-coverage strategies simultaneously (500ms cap)
-    // These cover labelled inputs, placeholder inputs, and role-textbox — 95%+ of forms
-    const phase1 = [
-        () => page.getByLabel(labelRe).first().fill(value),
-        () => page.getByPlaceholder(labelRe).first().fill(value),
-        () => page.getByRole("textbox", { name: labelRe }).first().fill(value),
-    ];
-    try {
-        await Promise.any(phase1.map(fn => withTimeout(fn, 500)));
-        return;
-    }
-    catch { /* all 3 failed, try phase 2 */ }
-    // Phase 2: attribute-based selectors covering name/id/data-test variants (300ms each)
-    const exactAttrs = variants.flatMap(v => [
-        `input[name="${cssEscape(v)}" i]`, `input[id="${cssEscape(v)}" i]`,
-        `textarea[name="${cssEscape(v)}" i]`, `textarea[id="${cssEscape(v)}" i]`,
-    ]).join(", ");
-    const phase2 = [
-        () => page.locator(exactAttrs).first().fill(value),
-        () => page.locator(`${attrContains("name")}, ${attrContains("id")}`).first().fill(value),
-        () => page.locator(attrContains("data-test")).first().fill(value),
-        () => page.locator(attrContains("aria-label")).first().fill(value),
-        () => page.locator(attrContains("placeholder")).first().fill(value),
+    // Sequential strategies — do NOT run concurrently (concurrent fills can cross-contaminate fields)
+    // Each uses a short timeout since if the element exists, Playwright resolves near-instantly.
+    const strategies = [
+        ['getByLabel', () => page.getByLabel(labelRe).first().fill(value)],
+        ['getByPlaceholder', () => page.getByPlaceholder(labelRe).first().fill(value)],
+        ['getByRole', () => page.getByRole("textbox", { name: labelRe }).first().fill(value)],
+        ['id/name exact', () => page.locator(variants.flatMap(v => [
+                `input[name="${cssEscape(v)}" i]`, `input[id="${cssEscape(v)}" i]`,
+                `textarea[name="${cssEscape(v)}" i]`, `textarea[id="${cssEscape(v)}" i]`,
+            ]).join(", ")).first().fill(value)],
+        ['id/name contains', () => page.locator(`${attrContains("name")}, ${attrContains("id")}`).first().fill(value)],
+        ['data-test', () => page.locator(attrContains("data-test")).first().fill(value)],
+        ['aria-label', () => page.locator(attrContains("aria-label")).first().fill(value)],
+        ['placeholder attr', () => page.locator(attrContains("placeholder")).first().fill(value)],
     ];
-    for (const fn of phase2) {
+    for (const [, fn] of strategies) {
         try {
-            await withTimeout(fn, 300);
+            await Promise.race([fn(), new Promise((_, r) => setTimeout(() => r(new Error('t')), 400))]);
             return;
         }
-        catch { /* next */ }
+        catch { /* next strategy */ }
     }
     throw new Error(`Could not find input field: "${label}"`);
 }

package/package.json

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