skopix 2.0.19 → 2.0.21

package/cli/commands/agent.js

@@ -361,9 +361,10 @@ export async function agentCommand(options) {
       await page.waitForTimeout(800);
 
     } else if (step.action === 'click') {
-      await page.waitForTimeout(100);
+      await page.waitForTimeout(300);
       let clicked = false;
       const selectors = [step.stableSelector, step.selector].filter(Boolean).map(sanitiseSelector);
+      // 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) {
@@ -372,56 +373,28 @@ export async function agentCommand(options) {
             const count = await page.locator(s).count();
             if (count > 1) {
               let bi = 0, bd = Infinity;
-              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 {} }
+              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 {}
         }
       }
-      if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().click({ timeout: 5000 }); clicked = true; } catch {} } }
-      if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().scrollIntoViewIfNeeded({ timeout: 2000 }).catch(() => {}); await page.locator(s).first().click({ timeout: 5000 }); clicked = true; } catch {} } }
+      // 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 {} } }
-      if (!clicked) {
-        // Last resort — JS dispatch for framework-managed elements
-        for (const s of selectors) {
-          if (clicked) break;
-          try {
-            await page.locator(s).first().evaluate(el => {
-              el.dispatchEvent(new MouseEvent('mousedown', { bubbles: true, cancelable: true }));
-              el.dispatchEvent(new MouseEvent('mouseup', { bubbles: true, cancelable: true }));
-              el.dispatchEvent(new MouseEvent('click', { bubbles: true, cancelable: true }));
-            });
-            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(', '));
-
-      // Special handling for Bootstrap dropdowns — wait for menu to appear
-      try {
-        const el = page.locator(selectors[0]).first();
-        const isDropdown = await el.evaluate(e => e.hasAttribute('data-toggle') && e.getAttribute('data-toggle') === 'dropdown').catch(() => false);
-        if (isDropdown) {
-          // Try jQuery dropdown toggle first (Bootstrap 3)
-          const toggled = await page.evaluate((sel) => {
-            try {
-              const el = document.querySelector(sel);
-              if (el && window.$) { window.$(el).dropdown('toggle'); return true; }
-            } catch {}
-            return false;
-          }, selectors[0]).catch(() => false);
-          if (!toggled) {
-            // Fallback: dispatch click on the element
-            await el.evaluate(e => e.click());
-          }
-          await page.waitForSelector('.dropdown-menu', { state: 'visible', timeout: 3000 }).catch(() => {});
-          await page.waitForTimeout(400);
-        }
-      } catch {}
-
-      await page.waitForTimeout(400);
+      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.19",
+  "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": {