@browserless.io/browserless 2.7.0 → 2.8.0

package/static/function/client.js

@@ -21466,7 +21466,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.mouse} to click in the center of the
+       * needed, and then uses {@link Page.mouse} to click in the center of the
        * element. If there's no element matching `selector`, the method throws an
        * error.
        *
@@ -21515,7 +21515,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.mouse}
+       * needed, and then uses {@link Page.mouse}
        * to hover over the center of the element.
        * If there's no element matching `selector`, the method throws an error.
        * @param selector - A
@@ -21562,7 +21562,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.touchscreen}
+       * needed, and then uses {@link Page.touchscreen}
        * to tap in the center of the element.
        * If there's no element matching `selector`, the method throws an error.
        * @param selector - A
@@ -23755,7 +23755,7 @@
     for (const [name2, jsValue] of Object.entries(replacements)) {
       value = value.replace(
         new RegExp(`PLACEHOLDER\\(\\s*(?:'${name2}'|"${name2}")\\s*\\)`, "g"),
-        // Wrapping this ensures tersers that accidently inline PLACEHOLDER calls
+        // Wrapping this ensures tersers that accidentally inline PLACEHOLDER calls
         // are still valid. Without, we may get calls like ()=>{...}() which is
         // not valid.
         `(${jsValue})`
@@ -24108,7 +24108,16 @@
       role
     });
     return nodes.filter((node) => {
-      return !node.role || !NON_ELEMENT_NODE_ROLES.has(node.role.value);
+      if (node.ignored) {
+        return false;
+      }
+      if (!node.role) {
+        return false;
+      }
+      if (NON_ELEMENT_NODE_ROLES.has(node.role.value)) {
+        return false;
+      }
+      return true;
     });
   };
   var isKnownAttribute = (attribute) => {
@@ -24883,7 +24892,7 @@
       }
       /**
        * This method scrolls element into view if needed, and then
-       * uses {@link Page} to hover over the center of the element.
+       * uses {@link Page.mouse} to hover over the center of the element.
        * If the element is detached from DOM, the method throws an error.
        */
       async hover() {
@@ -24893,7 +24902,7 @@
       }
       /**
        * This method scrolls element into view if needed, and then
-       * uses {@link Page | Page.mouse} to click in the center of the element.
+       * uses {@link Page.mouse} to click in the center of the element.
        * If the element is detached from DOM, the method throws an error.
        */
       async click(options = {}) {
@@ -28092,6 +28101,7 @@ ${sourceUrlComment}
     // frameID -> childFrameIDs
     #childIds = /* @__PURE__ */ new Map();
     #mainFrame;
+    #isMainFrameStale = false;
     #waitRequests = /* @__PURE__ */ new Map();
     getMainFrame() {
       return this.#mainFrame;
@@ -28124,8 +28134,9 @@ ${sourceUrlComment}
           this.#childIds.set(frame._parentId, /* @__PURE__ */ new Set());
         }
         this.#childIds.get(frame._parentId).add(frame._id);
-      } else if (!this.#mainFrame) {
+      } else if (!this.#mainFrame || this.#isMainFrameStale) {
         this.#mainFrame = frame;
+        this.#isMainFrameStale = false;
       }
       this.#waitRequests.get(frame._id)?.forEach((request) => {
         return request.resolve(frame);
@@ -28137,7 +28148,7 @@ ${sourceUrlComment}
       if (frame._parentId) {
         this.#childIds.get(frame._parentId)?.delete(frame._id);
       } else {
-        this.#mainFrame = void 0;
+        this.#isMainFrameStale = true;
       }
     }
     childFrames(frameId) {

package/static/function/index.html

@@ -21474,7 +21474,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.mouse} to click in the center of the
+       * needed, and then uses {@link Page.mouse} to click in the center of the
        * element. If there's no element matching `selector`, the method throws an
        * error.
        *
@@ -21523,7 +21523,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.mouse}
+       * needed, and then uses {@link Page.mouse}
        * to hover over the center of the element.
        * If there's no element matching `selector`, the method throws an error.
        * @param selector - A
@@ -21570,7 +21570,7 @@
       }
       /**
        * This method fetches an element with `selector`, scrolls it into view if
-       * needed, and then uses {@link Page | Page.touchscreen}
+       * needed, and then uses {@link Page.touchscreen}
        * to tap in the center of the element.
        * If there's no element matching `selector`, the method throws an error.
        * @param selector - A
@@ -23763,7 +23763,7 @@
     for (const [name2, jsValue] of Object.entries(replacements)) {
       value = value.replace(
         new RegExp(`PLACEHOLDER\\(\\s*(?:'${name2}'|"${name2}")\\s*\\)`, "g"),
-        // Wrapping this ensures tersers that accidently inline PLACEHOLDER calls
+        // Wrapping this ensures tersers that accidentally inline PLACEHOLDER calls
         // are still valid. Without, we may get calls like ()=>{...}() which is
         // not valid.
         `(${jsValue})`
@@ -24116,7 +24116,16 @@
       role
     });
     return nodes.filter((node) => {
-      return !node.role || !NON_ELEMENT_NODE_ROLES.has(node.role.value);
+      if (node.ignored) {
+        return false;
+      }
+      if (!node.role) {
+        return false;
+      }
+      if (NON_ELEMENT_NODE_ROLES.has(node.role.value)) {
+        return false;
+      }
+      return true;
     });
   };
   var isKnownAttribute = (attribute) => {
@@ -24891,7 +24900,7 @@
       }
       /**
        * This method scrolls element into view if needed, and then
-       * uses {@link Page} to hover over the center of the element.
+       * uses {@link Page.mouse} to hover over the center of the element.
        * If the element is detached from DOM, the method throws an error.
        */
       async hover() {
@@ -24901,7 +24910,7 @@
       }
       /**
        * This method scrolls element into view if needed, and then
-       * uses {@link Page | Page.mouse} to click in the center of the element.
+       * uses {@link Page.mouse} to click in the center of the element.
        * If the element is detached from DOM, the method throws an error.
        */
       async click(options = {}) {
@@ -28100,6 +28109,7 @@ ${sourceUrlComment}
     // frameID -> childFrameIDs
     #childIds = /* @__PURE__ */ new Map();
     #mainFrame;
+    #isMainFrameStale = false;
     #waitRequests = /* @__PURE__ */ new Map();
     getMainFrame() {
       return this.#mainFrame;
@@ -28132,8 +28142,9 @@ ${sourceUrlComment}
           this.#childIds.set(frame._parentId, /* @__PURE__ */ new Set());
         }
         this.#childIds.get(frame._parentId).add(frame._id);
-      } else if (!this.#mainFrame) {
+      } else if (!this.#mainFrame || this.#isMainFrameStale) {
         this.#mainFrame = frame;
+        this.#isMainFrameStale = false;
       }
       this.#waitRequests.get(frame._id)?.forEach((request) => {
         return request.resolve(frame);
@@ -28145,7 +28156,7 @@ ${sourceUrlComment}
       if (frame._parentId) {
         this.#childIds.get(frame._parentId)?.delete(frame._id);
       } else {
-        this.#mainFrame = void 0;
+        this.#isMainFrameStale = true;
       }
     }
     childFrames(frameId) {