skopix 2.0.16 → 2.0.17

package/core/llm.js

@@ -553,6 +553,7 @@ export async function processRecording({ steps, testName, url, provider, apiKey,
     + '   - 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'
     + '   - 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,7 +87,7 @@ export class RecordingSession {
       }
 
       function getElementInfo(el) {
-        return {
+        const info = {
           tag: el.tagName.toLowerCase(),
           id: el.id || null,
           name: el.name || null,
@@ -96,6 +96,17 @@ export class RecordingSession {
           selector: getSelector(el),
           classes: el.className ? el.className.toString().trim().slice(0, 100) : null,
         };
+        // 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;
+        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.parentTestId = p.getAttribute('pi-test-identifier') || p.getAttribute('data-testid') || null;
+        }
+        return info;
       }
 
       // ─── Action listeners ─────────────────────────────────────────────────

package/package.json

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