skopix 2.0.20 → 2.0.21

package/cli/commands/agent.js

@@ -361,10 +361,11 @@ export async function agentCommand(options) {
       await page.waitForTimeout(800);
 
     } else if (step.action === 'click') {
+      await page.waitForTimeout(300);
       let clicked = false;
       const selectors = [step.stableSelector, step.selector].filter(Boolean).map(sanitiseSelector);
-      // Try position-matched click first (for elements with coordinates)
-      if (step.elementX || step.clickX) {
+      // Strategy 1: coordinate-based (best for duplicate elements)
+      if (!clicked && (step.elementX || step.clickX)) {
         const tx = step.elementX || step.clickX, ty = step.elementY || step.clickY;
         for (const s of selectors) {
           if (clicked) break;
@@ -375,17 +376,25 @@ export async function agentCommand(options) {
               for (let i = 0; i < count; i++) { try { const box = await page.locator(s).nth(i).boundingBox({ timeout: 2000 }); if (!box) continue; const d = Math.sqrt(Math.pow(box.x+box.width/2-tx,2)+Math.pow(box.y+box.height/2-ty,2)); if (d < bd) { bd=d; bi=i; } } catch {} }
               await page.locator(s).nth(bi).click({ timeout: 5000 }); clicked = true;
             } else if (count === 1) {
+              await page.locator(s).first().waitFor({ state: 'visible', timeout: 3000 });
               await page.locator(s).first().click({ timeout: 5000 }); clicked = true;
             }
           } catch {}
         }
       }
-      // Simple click fallback
-      if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().click({ timeout: 5000 }); clicked = true; } catch {} } }
-      // Force click (for off-screen elements)
+      // Strategy 2: normal click
+      if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().waitFor({ state: 'visible', timeout: 3000 }); await page.locator(s).first().click({ timeout: 5000 }); clicked = true; } catch {} } }
+      // Strategy 3: force click
       if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().click({ force: true, timeout: 5000 }); clicked = true; } catch {} } }
+      // Strategy 4: click clickable ancestor (for icons/SVGs)
+      if (!clicked && step.element) {
+        const tag = (step.element.tag || '').toLowerCase();
+        if (['i', 'svg', 'path', 'span', 'img'].includes(tag)) {
+          for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().locator('xpath=ancestor-or-self::*[self::a or self::button or @role="button"][1]').first().click({ timeout: 5000 }); clicked = true; } catch {} }
+        }
+      }
       if (!clicked) throw new Error('Could not click: ' + selectors.join(', '));
-      await page.waitForTimeout(200);
+      await page.waitForTimeout(500);
 
     } else if (step.action === 'type') {
       await page.locator(sel).first().click({ timeout: 5000 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skopix",
-  "version": "2.0.20",
+  "version": "2.0.21",
   "description": "Browser-based QA tool — record tests by using your app, replay them deterministically, generate Playwright code automatically",
   "main": "cli/index.js",
   "bin": {