fix: Puppeteer grabAttributeFrom and XPath click issues

- Remove jsonValue() call in grabAttributeFromAll (Puppeteer 24.x returns values directly)
- Fix XPath selector to work with all matcher types (Page/Frame/ElementHandle)
- Use try/catch for ::-p-xpath() selector with fallback for ElementHandles

Author: kobenguyent

lib/helper/Puppeteer.js

@@ -2082,7 +2082,7 @@ class Puppeteer extends Helper {
     const array = []
     for (let index = 0; index < els.length; index++) {
       const a = await this._evaluateHandeInContext((el, attr) => el[attr] || el.getAttribute(attr), els[index], attr)
-      array.push(await a.jsonValue())
+      array.push(a)
     }
     return array
   }
@@ -2887,12 +2887,16 @@ async function findElements(matcher, locator) {
   }
 
   // For Puppeteer 24.x+, $x method was removed
-  // Use ::-p-xpath() selector syntax for Page/Frame
-  const matcherType = matcher.constructor?.name
-  if (matcherType === 'CdpPage' || matcherType === 'CdpFrame') {
-    // matcher is Page or Frame - use ::-p-xpath selector
+  // Use ::-p-xpath() selector syntax
+  // Check if matcher has $$ method (Page, Frame, or ElementHandle)
+  if (matcher && typeof matcher.$$ === 'function') {
     const xpathSelector = `::-p-xpath(${locator.value})`
-    return matcher.$$(xpathSelector)
+    try {
+      return await matcher.$$(xpathSelector)
+    } catch (e) {
+      // XPath selector may not work on ElementHandle, fall through to evaluate method
+      this.debug && this.debug(`XPath selector failed on ${matcher.constructor?.name}: ${e.message}`)
+    }
   }
 
   // ElementHandles don't support XPath directly  // Search within the element by making XPath relative