skopix 2.0.16 → 2.0.18

package/cli/commands/agent.js

@@ -384,7 +384,32 @@ export async function agentCommand(options) {
       }
       if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().scrollIntoViewIfNeeded({ timeout: 3000 }).catch(() => {}); await page.locator(s).first().click({ timeout: 5000 }); clicked = true; } catch {} } }
       if (!clicked) { for (const s of selectors) { if (clicked) break; try { await page.locator(s).first().click({ force: true, timeout: 5000 }); clicked = true; } catch {} } }
+      if (!clicked) {
+        // Last resort — JS dispatch for framework-managed elements
+        for (const s of selectors) {
+          if (clicked) break;
+          try {
+            await page.locator(s).first().evaluate(el => {
+              el.dispatchEvent(new MouseEvent('mousedown', { bubbles: true, cancelable: true }));
+              el.dispatchEvent(new MouseEvent('mouseup', { bubbles: true, cancelable: true }));
+              el.dispatchEvent(new MouseEvent('click', { bubbles: true, cancelable: true }));
+            });
+            clicked = true;
+          } catch {}
+        }
+      }
       if (!clicked) throw new Error('Could not click: ' + selectors.join(', '));
+
+      // Special handling for Bootstrap dropdowns — wait for menu to appear
+      try {
+        const el = await page.locator(selectors[0]).first();
+        const isDropdown = await el.evaluate(e => e.hasAttribute('data-toggle') && e.getAttribute('data-toggle') === 'dropdown').catch(() => false);
+        if (isDropdown) {
+          await page.waitForSelector('.dropdown-menu:visible, .dropdown.open .dropdown-menu', { timeout: 3000 }).catch(() => {});
+          await page.waitForTimeout(300);
+        }
+      } catch {}
+
       await page.waitForTimeout(400);
 
     } else if (step.action === 'type') {

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.18",
   "description": "Browser-based QA tool — record tests by using your app, replay them deterministically, generate Playwright code automatically",
   "main": "cli/index.js",
   "bin": {