skopix 2.0.79 → 2.0.81

package/cli/commands/agent.js

@@ -406,6 +406,8 @@ export async function agentCommand(options) {
         }
       }
       if (!clicked) throw new Error('Could not click: ' + selectors.join(', '));
+      // Wait for any navigation triggered by the click
+      await page.waitForLoadState('domcontentloaded', { timeout: 5000 }).catch(() => {});
       await page.waitForTimeout(500);
 
     } else if (step.action === 'type') {

package/cli/commands/dashboard.js

@@ -3288,6 +3288,7 @@ function startReplay(test, setupTest, activeRuns, reportsDir, currentUser, env)
             if (!clicked) throw new Error('Could not click element. Tried: ' + selectors.join(', '));
             const descLower = (step.description||'').toLowerCase(), selLower = selectors.join(' ').toLowerCase();
             const isSave = descLower.includes('save')||descLower.includes('submit')||selLower.includes('fa-save')||selLower.includes('fa-floppy');
+            await page.waitForLoadState('domcontentloaded', { timeout: 5000 }).catch(() => {});
             await page.waitForTimeout(isSave ? 2000 : 500);
             broadcast({ type: 'stdout', text: '    \u2713 Clicked' + (isSave?' (waited for save)':'') });
 
@@ -3901,28 +3902,78 @@ async function saveSavedUrls(urls) {
 }
 // Internal helper — sync selectors across all tests
 async function syncSelectorsInternal(suitesDir, fromSelectors, toSelector, toName) {
-  const allTests = await listAllTests(suitesDir);
+  const files = await fs.readdir(suitesDir);
+  const suiteFiles = files.filter(f => f.endsWith('.suite.yaml') || f.endsWith('.suite.yml'));
   let updated = 0;
   const fromLower = fromSelectors.map(s => s.toLowerCase());
   const allFromTokens = fromLower.flatMap(sel =>
     (sel.match(/\.[a-z][a-z0-9_-]+|\[[\w-]+=["'][^"']+["']\]|#[a-z][a-z0-9_-]+/g) || []).filter(t => t.length > 5)
   );
-  for (const test of allTests) {
-    if (!test.steps || !test.steps.length) continue;
+
+  function matchesSel(sSel) {
+    if (!sSel) return false;
+    const sLower = sSel.toLowerCase();
+    if (fromLower.includes(sLower)) return true;
+    if (allFromTokens.some(tok => sLower.includes(tok))) return true;
+    return fromLower.some(fSel => stepSimilarity({ selector: sLower }, { selector: fSel }) >= 0.7);
+  }
+
+  function syncSteps(steps) {
+    if (!Array.isArray(steps)) return { steps, changed: false };
     let changed = false;
-    const newSteps = test.steps.map(step => {
-      const sSel = (step.stableSelector||step.selector||'').toLowerCase();
-      const isMatch = fromLower.includes(sSel) ||
-        allFromTokens.some(tok => sSel.includes(tok)) ||
-        fromLower.some(fSel => stepSimilarity({ selector: sSel }, { selector: fSel }) >= 0.7);
-      if (isMatch && sSel) {
+    const newSteps = steps.map(step => {
+      const sel = step.stableSelector || step.selector || '';
+      if (matchesSel(sel)) {
         changed = true;
         updated++;
         return { ...step, stableSelector: toSelector, selector: toSelector, description: toName || step.description };
       }
       return step;
     });
-    if (changed) await updateTest(suitesDir, test.scope, test.id, { ...test, steps: newSteps });
+    return { steps: newSteps, changed };
+  }
+
+  for (const file of suiteFiles) {
+    const filePath = path.join(suitesDir, file);
+    try {
+      const content = await fs.readFile(filePath, 'utf8');
+      const data = yaml.parse(content);
+      if (!data || !Array.isArray(data.tests)) continue;
+      let fileChanged = false;
+
+      data.tests = data.tests.map(test => {
+        let testChanged = false;
+        // Sync structured steps
+        if (Array.isArray(test.steps) && test.steps.length > 0) {
+          const { steps: newSteps, changed } = syncSteps(test.steps);
+          if (changed) { test = { ...test, steps: newSteps }; testChanged = true; }
+        }
+        // Sync playwrightJs code string
+        if (test.playwrightJs && typeof test.playwrightJs === 'string') {
+          let newJs = test.playwrightJs;
+          for (const oldSel of fromSelectors) {
+            if (newJs.includes(oldSel)) {
+              newJs = newJs.split('"' + oldSel + '"').join('"' + toSelector + '"');
+              newJs = newJs.split("'" + oldSel + "'").join("'" + toSelector + "'");
+              // Update comment descriptions
+              if (toName) {
+                fromSelectors.forEach(s => {});  // placeholder — descriptions in comments are harder to match
+              }
+              testChanged = true;
+            }
+          }
+          if (testChanged) test = { ...test, playwrightJs: newJs };
+        }
+        if (testChanged) fileChanged = true;
+        return test;
+      });
+
+      if (fileChanged) {
+        await fs.writeFile(filePath, yaml.stringify(data));
+      }
+    } catch (err) {
+      process.stderr.write('[sync] error processing ' + file + ': ' + err.message + '\n');
+    }
   }
   return updated;
 }

package/package.json

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