skopix 2.0.80 → 2.0.82

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)':'') });
 

package/core/llm.js

@@ -547,13 +547,15 @@ 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'
+    + '   - 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'
     + '   - Role + text e.g. [role="button"]:has-text("Submit")\n'
     + '   - Class-based selector for well-named classes e.g. .chart-container, .save-btn\n'
     + '   - Fall back to the original selector if nothing better\n'
     + '   CRITICAL RULES FOR SELECTORS:\n'
-    + '   - For icon elements (<i>, <span> with fa/icon classes): use the parent element as the anchor if it has a meaningful class/attribute, e.g. .chart-panel__panel-heading__panel-button i, [aria-label="Options"] i. NEVER generate a bare i.fa-something if there are multiple on the page — always scope it to its parent container\n'
+    + '   - ONLY use attributes that ACTUALLY EXIST on the element from the captured data. NEVER invent aria-label, title, or other attributes that are not in the element info.\n'
+    + '   - For icon elements (<i>, <span> with fa/icon classes): use the PARENT title or pi-test-identifier as the anchor e.g. a[title="Create new chart"] or [pi-test-identifier="x"] i. Check parentTitle and parentTestId fields.\n'
     + '   - NEVER use IDs that look randomly generated (e.g. #highcharts-abc123-58, #ng-view-1, anything with random hex/numbers)\n'
     + '   - NEVER use :nth-child or :nth-of-type positional selectors\n'
     + '   - For chart/visualization containers: use class-based selectors like .highcharts-container, .chart-wrapper, [class*="chart"]\n'

package/core/recorder.js

@@ -87,23 +87,32 @@ export class RecordingSession {
       }
 
       function getElementInfo(el) {
+        const piTestId = el.getAttribute('pi-test-identifier') || null;
+        const title = el.getAttribute('title') || null;
+        const ariaLabel = el.getAttribute('aria-label') || null;
+        const dataTestId = el.getAttribute('data-testid') || el.getAttribute('data-test-id') || null;
         const info = {
           tag: el.tagName.toLowerCase(),
           id: el.id || null,
           name: el.name || null,
           type: el.type || null,
-          text: (el.innerText || el.value || el.placeholder || el.getAttribute('aria-label') || '').trim().slice(0, 80),
+          text: (el.innerText || el.value || el.placeholder || ariaLabel || title || '').trim().slice(0, 80),
           selector: getSelector(el),
           classes: el.className ? el.className.toString().trim().slice(0, 100) : null,
+          title,
+          ariaLabel,
+          piTestId,
+          dataTestId,
         };
         // For icon elements with no meaningful ID/text, capture parent context for better selector generation
-        const isIcon = ['i', 'span', 'svg'].includes(info.tag) && !info.id && !info.text;
+        const isIcon = ['i', 'span', 'svg'].includes(info.tag) && !info.id && !info.text && !piTestId;
         if (isIcon && el.parentElement) {
           const p = el.parentElement;
           info.parentTag = p.tagName.toLowerCase();
           info.parentClasses = p.className ? p.className.toString().trim().slice(0, 100) : null;
           info.parentSelector = getSelector(p);
           info.parentAriaLabel = p.getAttribute('aria-label') || null;
+          info.parentTitle = p.getAttribute('title') || null;
           info.parentTestId = p.getAttribute('pi-test-identifier') || p.getAttribute('data-testid') || null;
         }
         return info;

package/package.json

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