fix: add role locator support and XPath handling to Puppeteer findElements/findElement

- Add role locator check in findElements to call findByRole (like Playwright)
- Add role locator check in findElement to call findByRole
- Fix XPath handling in findElement to use findElements (removes $x dependency)
- Fixes 6 failing Puppeteer tests:
  - role locators with text filter (exact match case-sensitive)
  - role locator combinations (checkbox state)
  - grab elements by role
  - waitForClickable by XPath (3 tests)

All role locator tests now pass with proper text filtering and exact matching.

Author: kobenguyent

lib/helper/Puppeteer.js

@@ -2877,6 +2877,9 @@ async function findElements(matcher, locator) {
   if (isReactLocator) return findReactElements.call(this, locator)
 
   locator = new Locator(locator, 'css')
+  
+  // Check if locator is a role locator and call findByRole
+  if (locator.isRole()) return findByRole.call(this, matcher, locator)
 
   // Use proven legacy approach - Puppeteer Locator API doesn't have .all() method
   if (!locator.isXPath()) return matcher.$$(locator.simplify())
@@ -2934,18 +2937,22 @@ async function findElements(matcher, locator) {
 async function findElement(matcher, locator) {
   if (locator.react) return findReactElements.call(this, locator)
   locator = new Locator(locator, 'css')
+  
+  // Check if locator is a role locator and call findByRole
+  if (locator.isRole()) {
+    const elements = await findByRole.call(this, matcher, locator)
+    return elements[0]
+  }
 
   // Use proven legacy approach - Puppeteer Locator API doesn't have .first() method
   if (!locator.isXPath()) {
     const elements = await matcher.$$(locator.simplify())
     return elements[0]
   }
-  // puppeteer version < 19.4.0 is no longer supported. This one is backward support.
-  if (puppeteer.default?.defaultBrowserRevision) {
-    const elements = await matcher.$$(`xpath/${locator.value}`)
-    return elements[0]
-  }
-  const elements = await matcher.$x(locator.value)
+  
+  // For XPath in Puppeteer 24.x+, use the same approach as findElements
+  // $x method was removed, so we use ::-p-xpath() or fallback
+  const elements = await findElements.call(this, matcher, locator)
   return elements[0]
 }