Changes before error encountered

Co-authored-by: kobenguyent <7845001+kobenguyent@users.noreply.github.com>

Author: Copilot

lib/helper/Puppeteer.js

@@ -2729,51 +2729,17 @@ class Puppeteer extends Helper {
 module.exports = Puppeteer
 
 /**
- * Build locator string for Puppeteer's Locator API
- * Similar to Playwright's buildLocatorString function for consistency
- * @param {Locator} locator - CodeceptJS Locator object
- * @returns {string} Locator string compatible with Puppeteer's locator() method
- */
-function buildLocatorString(locator) {
-  if (locator.isCustom()) {
-    return `${locator.type}=${locator.value}`
-  }
-  if (locator.isXPath()) {
-    return `xpath=${locator.value}`
-  }
-  return locator.simplify()
-}
-
-/**
- * Find elements using Puppeteer's modern Locator API with ElementHandle fallback
- * This provides better reliability and waiting behavior while maintaining backward compatibility
+ * Find elements using Puppeteer's native element discovery methods
+ * Note: Unlike Playwright, Puppeteer's Locator API doesn't have .all() method for multiple elements
  * @param {Page|Frame|ElementHandle} matcher - Puppeteer context to search within
  * @param {Object|string} locator - Locator specification
- * @returns {Promise<ElementHandle[]>} Array of ElementHandle objects for compatibility
+ * @returns {Promise<ElementHandle[]>} Array of ElementHandle objects
  */
 async function findElements(matcher, locator) {
   if (locator.react) return findReactElements.call(this, locator)
   locator = new Locator(locator, 'css')
 
-  // Use Locator API for better reliability, then convert to ElementHandles for compatibility
-  if (matcher.locator) {
-    const locatorElement = matcher.locator(buildLocatorString(locator))
-    const handles = await locatorElement.all()
-    // Convert Locator elements to ElementHandles for backward compatibility
-    return Promise.all(
-      handles.map(async handle => {
-        // For Puppeteer Locators, we can get ElementHandle using waitHandle()
-        try {
-          return await handle.waitHandle()
-        } catch (e) {
-          // Fallback for edge cases
-          return handle
-        }
-      }),
-    )
-  }
-
-  // Fallback to legacy approach if Locator not available
+  // Use proven legacy approach - Puppeteer Locator API doesn't have .all() method
   if (!locator.isXPath()) return matcher.$$(locator.simplify())
   // puppeteer version < 19.4.0 is no longer supported. This one is backward support.
   if (puppeteer.default?.defaultBrowserRevision) {
@@ -2783,29 +2749,17 @@ async function findElements(matcher, locator) {
 }
 
 /**
- * Find a single element using Puppeteer's modern Locator API with ElementHandle fallback
+ * Find a single element using Puppeteer's native element discovery methods
+ * Note: Puppeteer Locator API doesn't have .first() method like Playwright
  * @param {Page|Frame|ElementHandle} matcher - Puppeteer context to search within
  * @param {Object|string} locator - Locator specification
- * @returns {Promise<ElementHandle>} Single ElementHandle object for compatibility
+ * @returns {Promise<ElementHandle>} Single ElementHandle object
  */
 async function findElement(matcher, locator) {
   if (locator.react) return findReactElements.call(this, locator)
   locator = new Locator(locator, 'css')
 
-  // Use Locator API for better reliability, then get first ElementHandle for compatibility
-  if (matcher.locator) {
-    const locatorElement = matcher.locator(buildLocatorString(locator))
-    const handle = await locatorElement.first()
-    // Convert to ElementHandle for backward compatibility
-    try {
-      return await handle.waitHandle()
-    } catch (e) {
-      // Fallback for edge cases
-      return handle
-    }
-  }
-
-  // Fallback to legacy approach if Locator not available
+  // Use proven legacy approach - Puppeteer Locator API doesn't have .first() method
   if (!locator.isXPath()) {
     const elements = await matcher.$$(locator.simplify())
     return elements[0]