skopix 2.0.78 → 2.0.80

package/cli/commands/dashboard.js

@@ -3901,28 +3901,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;
 }
@@ -3967,6 +4017,9 @@ async function approvePendingStep(suitesDir, id) {
   const { id: _id, addedAt: _a, ...stepData } = step;
   const added = await addLibraryStep(suitesDir, stepData);
   await savePendingSteps(pending.filter(s => s.id !== id));
+  // Sync test descriptions to use the approved name
+  const sel = stepData.stableSelector || stepData.selector || '';
+  if (sel) await syncSelectorsInternal(suitesDir, [sel.toLowerCase()], sel, stepData.name || stepData.description || '');
   return added;
 }
 
@@ -3980,6 +4033,8 @@ async function approveAllPending(suitesDir) {
   for (const step of pending) {
     const { id: _id, addedAt: _a, ...stepData } = step;
     await addLibraryStep(suitesDir, stepData);
+    const sel = stepData.stableSelector || stepData.selector || '';
+    if (sel) await syncSelectorsInternal(suitesDir, [sel.toLowerCase()], sel, stepData.name || stepData.description || '');
   }
   await savePendingSteps([]);
   return pending.length;

package/package.json

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