skopix 2.0.20 → 2.0.22

package/cli/commands/agent.js

@@ -254,6 +254,14 @@ export async function agentCommand(options) {
         deviceScaleFactor: 1,
         ...(wantReport ? { recordVideo: { dir: sessionDir, size: { width: 1280, height: 800 } } } : {}),
       });
+      // Set zoom via initScript so it persists across page loads and route changes
+      if (browserZoom !== 1) {
+        await ctx.addInitScript((zoom) => {
+          const applyZ = () => { if (document.documentElement) document.documentElement.style.zoom = zoom; };
+          applyZ();
+          document.addEventListener('DOMContentLoaded', applyZ);
+        }, String(browserZoom));
+      }
       const page = await ctx.newPage();
       const applyZoom = async () => {
         if (browserZoom !== 1) {
@@ -272,6 +280,9 @@ export async function agentCommand(options) {
       // Apply zoom now if no setup test (otherwise applied at setup→main transition)
       if (!setupTest) await applyZoom();
 
+      // Listen for future navigations and re-apply zoom
+      page.on('load', () => { applyZoom().catch(() => {}); });
+
       send({ type: 'stdout', text: '◆ Replaying ' + allSteps.length + ' steps on ' + os.hostname() });
 
       for (const step of allSteps) {
@@ -358,13 +369,15 @@ export async function agentCommand(options) {
         try { const ro = new URL(navUrl).origin; const to = new URL(test.url).origin; if (ro !== to) navUrl = navUrl.replace(ro, to); } catch {}
       }
       await page.goto(navUrl, { waitUntil: 'domcontentloaded', timeout: 15000 });
+      await applyZoom();
       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 +388,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.22",
   "description": "Browser-based QA tool — record tests by using your app, replay them deterministically, generate Playwright code automatically",
   "main": "cli/index.js",
   "bin": {