skopix 2.0.15 → 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.15",
+  "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": {

package/web/app/index.html

@@ -2240,9 +2240,9 @@ body.viewer-mode .saved-test-row { cursor: default !important; }
         <div style="display:flex;align-items:center;gap:8px">
           <label style="font-family:var(--mono);font-size:12px;color:var(--muted);white-space:nowrap">Browser zoom:</label>
           <select class="form-select" id="re-browser-zoom" style="padding:5px 8px;font-size:12px;width:90px">
-            <option value="1">100%</option>
+            <option value="1" selected>100%</option>
             <option value="0.9">90%</option>
-            <option value="0.8" selected>80%</option>
+            <option value="0.8">80%</option>
             <option value="0.75">75%</option>
             <option value="0.67">67%</option>
           </select>
@@ -5467,7 +5467,7 @@ async function confirmAddToSuite() {
 }
 
 // ── RECORDED TEST EDITOR ────────────────────────────────────────────────────
-let reEditorState = { scope: null, testId: null, steps: [], testName: '', url: '', reusable: false, setup: '', credentials: '', tags: [], generateReport: true, browserZoom: 0.8 };
+let reEditorState = { scope: null, testId: null, steps: [], testName: '', url: '', reusable: false, setup: '', credentials: '', tags: [], generateReport: true, browserZoom: 1 };
 let reAssertInsertAfter = -1;
 
 async function openRecordedTestEditor(scope, testId) {
@@ -5475,7 +5475,7 @@ async function openRecordedTestEditor(scope, testId) {
     const res = await fetch(API_BASE + '/api/test/' + encodeURIComponent(scope) + '/' + encodeURIComponent(testId));
     const test = await res.json();
     if (!res.ok) { showToast(test.error || 'Failed to load test'); return; }
-    reEditorState = { scope, testId, steps: JSON.parse(JSON.stringify(test.steps || [])), testName: test.name || '', url: test.url || '', reusable: !!test.reusable, setup: test.setup || '', credentials: test.credentials || '', tags: test.tags || [], generateReport: test.generateReport !== false, browserZoom: test.browserZoom || 0.8 };
+    reEditorState = { scope, testId, steps: JSON.parse(JSON.stringify(test.steps || [])), testName: test.name || '', url: test.url || '', reusable: !!test.reusable, setup: test.setup || '', credentials: test.credentials || '', tags: test.tags || [], generateReport: test.generateReport !== false, browserZoom: test.browserZoom || 1 };
     document.getElementById('re-name').value = reEditorState.testName;
     document.getElementById('re-url').value = reEditorState.url;
     document.getElementById('re-reusable').checked = reEditorState.reusable;
@@ -5737,7 +5737,7 @@ async function saveRecordedEdits() {
   const url = document.getElementById('re-url').value.trim();
   const reusable = document.getElementById('re-reusable').checked;
   const generateReport = document.getElementById('re-generate-report')?.checked !== false;
-  const browserZoom = parseFloat(document.getElementById('re-browser-zoom')?.value || '0.8');
+  const browserZoom = parseFloat(document.getElementById('re-browser-zoom')?.value || '1');
   const setup = document.getElementById('re-setup').value || null;
   const credentials = document.getElementById('re-credentials')?.value || '';
   const tags = (document.getElementById('re-tags')?.value || '').split(',').map(t => t.trim()).filter(Boolean);