@empiricalrun/test-gen 0.38.9 → 0.38.11

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @empiricalrun/test-gen
 
+## 0.38.11
+
+### Patch Changes
+
+- fb4c866: fix: parent and child elements getting highlighted
+
+## 0.38.10
+
+### Patch Changes
+
+- e3f04e6: fix: add support for injecting script before agent action
+
 ## 0.38.9
 
 ### Patch Changes

package/browser-injected-scripts/annotate-elements.js

@@ -157,10 +157,27 @@ function annotateClickableElements(options = {}) {
       isClickable = true;
     }
 
+    // Tag-specific clickability
+    const focusableTags = [
+      "a",
+      "button",
+      "input",
+      "select",
+      "textarea",
+      "object",
+      "embed",
+      "label",
+      "details",
+    ];
+
     // Check for onclick attribute or listener
     if (
-      element.hasAttribute("onclick") ||
-      typeof element.onclick === "function"
+      (element.hasAttribute("onclick") ||
+      typeof element.onclick === "function") 
+      // check for parent element having same on click
+      && element.parentNode.onclick !== element.onclick
+      // parent element should not be a focusable tag like button
+      && focusableTags.indexOf(element.parentNode.tagName.toLowerCase()) === -1
     ) {
       isClickable = true;
     }
@@ -210,18 +227,7 @@ function annotateClickableElements(options = {}) {
       }
     }
 
-    // Tag-specific clickability
-    const focusableTags = [
-      "a",
-      "button",
-      "input",
-      "select",
-      "textarea",
-      "object",
-      "embed",
-      "label",
-      "details",
-    ];
+    
     if (focusableTags.includes(tagName)) {
       switch (tagName) {
         case "a":

package/dist/agent/master/run.js

@@ -189,9 +189,9 @@ async function createTestUsingMasterAgent({ task, page, testCase, options, scope
             page = actions.getStateVariables()[plannerResp.pageName];
             // update page in actions
             testGenPage.updatePage({ page, name: plannerResp.pageName });
-            // inject scripts in the updated
-            await (0, utils_2.injectPwLocatorGenerator)(testGenPage.pwPageInstance);
         }
+        // inject scripts in the page
+        await (0, utils_2.injectPwLocatorGenerator)(testGenPage.pwPageInstance);
         const buffer = await page.screenshot({
         //This is done to improve element annotation accuracy, anyways it doesn't annotate elements which are out of viewport
         // fullPage: true,

package/dist/browser-injected-scripts/annotate-elements.js

@@ -157,10 +157,27 @@ function annotateClickableElements(options = {}) {
       isClickable = true;
     }
 
+    // Tag-specific clickability
+    const focusableTags = [
+      "a",
+      "button",
+      "input",
+      "select",
+      "textarea",
+      "object",
+      "embed",
+      "label",
+      "details",
+    ];
+
     // Check for onclick attribute or listener
     if (
-      element.hasAttribute("onclick") ||
-      typeof element.onclick === "function"
+      (element.hasAttribute("onclick") ||
+      typeof element.onclick === "function") 
+      // check for parent element having same on click
+      && element.parentNode.onclick !== element.onclick
+      // parent element should not be a focusable tag like button
+      && focusableTags.indexOf(element.parentNode.tagName.toLowerCase()) === -1
     ) {
       isClickable = true;
     }
@@ -210,18 +227,7 @@ function annotateClickableElements(options = {}) {
       }
     }
 
-    // Tag-specific clickability
-    const focusableTags = [
-      "a",
-      "button",
-      "input",
-      "select",
-      "textarea",
-      "object",
-      "embed",
-      "label",
-      "details",
-    ];
+    
     if (focusableTags.includes(tagName)) {
       switch (tagName) {
         case "a":

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empiricalrun/test-gen",
-  "version": "0.38.9",
+  "version": "0.38.11",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"