@cutleryapp/agent 1.0.41 → 1.0.43

package/dist/mcp-executor.js

@@ -175,7 +175,14 @@ class TestExecutor {
                                 await page.fill(fieldLabel, value);
                             }
                             else {
-                                await tryFill(page, fieldLabel, value);
+                                // Check if it's a date/birth field and use date-aware fill
+                                const isDateField = /date|birth|dob/i.test(fieldLabel);
+                                if (isDateField) {
+                                    await tryFillDate(page, fieldLabel, value);
+                                }
+                                else {
+                                    await tryFill(page, fieldLabel, value);
+                                }
                             }
                             handled = true;
                         }
@@ -441,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 = "";
@@ -1176,43 +1196,93 @@ async function tryClickScoped(page, nameRe, target, scope) {
     catch { }
     return false;
 }
+/** Parse a date string in any common format and return {day, month, year} */
+function parseDate(value) {
+    // Formats: DD/MM/YYYY, MM/DD/YYYY, YYYY-MM-DD, DD-MM-YYYY, DD Mon YYYY, etc.
+    const patterns = [
+        /^(\d{1,2})[\/\-](\d{1,2})[\/\-](\d{4})$/, // DD/MM/YYYY or MM/DD/YYYY
+        /^(\d{4})[\/\-](\d{1,2})[\/\-](\d{1,2})$/, // YYYY-MM-DD
+    ];
+    for (const p of patterns) {
+        const m = value.match(p);
+        if (m) {
+            const [, a, b, c] = m;
+            // Heuristic: if first group is 4 digits → YYYY-MM-DD
+            const [year, month, day] = a.length === 4 ? [a, b, c] : [c, b, a];
+            const iso = `${year}-${month.padStart(2, '0')}-${day.padStart(2, '0')}`;
+            return { iso, day: day.padStart(2, '0'), month: month.padStart(2, '0'), year };
+        }
+    }
+    return null;
+}
+/** Fill a date field — handles input[type=date], calendar pickers, and plain text inputs */
+async function tryFillDate(page, label, value) {
+    const labelRe = new RegExp(escapeRegex(label), 'i');
+    const parsed = parseDate(value);
+    // Strategy 1: input[type="date"] — needs YYYY-MM-DD format
+    if (parsed) {
+        const dateInputSelectors = [
+            `input[type="date"]`,
+            `input[type="datetime-local"]`,
+        ];
+        for (const sel of dateInputSelectors) {
+            try {
+                const loc = page.locator(sel).first();
+                if (await loc.isVisible({ timeout: 400 })) {
+                    await loc.fill(parsed.iso, { timeout: 1000 });
+                    return;
+                }
+            }
+            catch { /* try next */ }
+        }
+    }
+    // Strategy 2: plain text input — type the value as-is, dismiss any calendar that opens
+    try {
+        const input = await Promise.any([
+            page.getByLabel(labelRe).first().elementHandle({ timeout: 500 }),
+            page.getByPlaceholder(labelRe).first().elementHandle({ timeout: 500 }),
+            page.getByRole('textbox', { name: labelRe }).first().elementHandle({ timeout: 500 }),
+        ]);
+        if (input) {
+            await input.click();
+            await page.keyboard.press('Escape'); // close any calendar that opened
+            await page.waitForTimeout(150);
+            await input.click({ clickCount: 3 }); // select all existing text
+            await page.keyboard.type(value, { delay: 30 });
+            await page.keyboard.press('Escape'); // close calendar again after typing
+            await page.keyboard.press('Tab'); // commit the value
+            return;
+        }
+    }
+    catch { /* fall through */ }
+    // Strategy 3: fallback to regular fill
+    await tryFill(page, label, value);
+}
 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.41",
+    "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": {