skopix 2.0.82 → 2.0.84

package/cli/commands/dashboard.js

@@ -1022,6 +1022,10 @@ export async function dashboardCommand(options) {
           const userEnv = await resolveUserSecretsEnv(currentUser && currentUser.id, teamMode);
           Object.assign(process.env, userEnv || {});
           const result = await processRecording({ steps, testName, url, provider: provider || 'gemini' });
+          // Match processed steps against library — substitute known elements
+          const matched = await matchStepsToLibrary(suitesDir, result.steps || []);
+          result.steps = matched.steps;
+          result.matchedCount = matched.count;
           sendJSON(res, 200, result);
         } catch (err) { sendJSON(res, 500, { error: err.message }); }
         return;
@@ -1572,6 +1576,8 @@ export async function dashboardCommand(options) {
         const data = JSON.parse(body);
         try {
           const result = await updateTest(suitesDir, scope, testId, data);
+          // Extract any new steps to pending review queue
+          extractStepsToLibrary(suitesDir, data.steps || [], data.name || testId).catch(() => {});
           sendJSON(res, 200, result);
         } catch (err) {
           sendJSON(res, 400, { error: err.message });
@@ -3865,6 +3871,47 @@ async function syncIssuesStatus() {
 // STEP LIBRARY — persistent store of reusable UI interactions
 // ═══════════════════════════════════════════════════════════════
 
+// Match recorded steps against library — substitute known elements automatically
+async function matchStepsToLibrary(suitesDir, steps) {
+  const library = await listLibrarySteps(suitesDir);
+  if (!library.length || !steps.length) return { steps, count: 0 };
+
+  let count = 0;
+  const matched = steps.map(step => {
+    const sSel = (step.stableSelector || step.selector || '').toLowerCase();
+    if (!sSel) return step;
+
+    // Find matching library element
+    const match = library.find(lib => {
+      const lSel = (lib.stableSelector || lib.selector || '').toLowerCase();
+      if (!lSel) return false;
+      // Exact match
+      if (sSel === lSel) return true;
+      // One contains the other (handles i.fa-plus vs a[title="x"] i.fa-plus)
+      if (sSel.includes(lSel) || lSel.includes(sSel)) return true;
+      // Shared key tokens
+      const keyTokens = lSel.match(/\.[a-z][a-z0-9_-]+|\[[\w-]+=["'][^"']+["']\]|#[a-z][a-z0-9_-]+/g) || [];
+      const strongTokens = keyTokens.filter(t => t.length > 8);
+      if (strongTokens.length > 0 && strongTokens.some(tok => sSel.includes(tok))) return true;
+      return false;
+    });
+
+    if (match) {
+      count++;
+      return {
+        ...step,
+        stableSelector: match.stableSelector || match.selector,
+        selector: match.selector || match.stableSelector,
+        description: match.name || step.description,
+        libraryId: match.id,
+      };
+    }
+    return step;
+  });
+
+  return { steps: matched, count };
+}
+
 const FOLDERS_FILE = () => path.join(os.homedir(), '.skopix', 'folders.yaml');
 
 async function getFolders() {

package/core/llm.js

@@ -509,6 +509,13 @@ export async function processRecording({ steps, testName, url, provider, apiKey,
       type: s.element.type,
       text: s.element.text,
       classes: s.element.classes,
+      title: s.element.title || null,
+      ariaLabel: s.element.ariaLabel || null,
+      piTestId: s.element.piTestId || null,
+      dataTestId: s.element.dataTestId || null,
+      parentTitle: s.element.parentTitle || null,
+      parentTestId: s.element.parentTestId || null,
+      parentClasses: s.element.parentClasses || null,
     } : null,
     value: s.action === 'type' && s.isPassword ? '[password - use process.env.TEST_PASSWORD]' : (s.value || null),
     isPassword: s.isPassword || false,
@@ -547,6 +554,8 @@ export async function processRecording({ steps, testName, url, provider, apiKey,
     + '\n\nYour jobs:\n'
     + '1. For each step, write a STABLE SELECTOR. Priority order:\n'
     + '   - data-testid, data-test, data-cy, data-qa, pi-test-identifier attributes use [attr="value"]\n'
+    + '   - MOST IMPORTANT: If the element data contains piTestId, use [pi-test-identifier="VALUE"] as the selector — this is the most reliable selector possible. Always check piTestId first.\n'
+    + '   - If parentTestId exists and piTestId does not, use [pi-test-identifier="parentTestId"] as anchor\n'
     + '   - title attribute — if the element has a title, use a[title="..."] or button[title="..."] — this is VERY reliable\n'
     + '   - Unique meaningful id (NOT random/generated IDs) use #id\n'
     + '   - Semantic selector e.g. button:has-text("Login"), input[name="email"]\n'

package/package.json

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