Fix custom locator registration timing and format issues

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

Author: Copilot

lib/helper/Playwright.js

@@ -483,49 +483,6 @@ class Playwright extends Helper {
       await playwright.selectors.register('__value', createValueEngine)
       await playwright.selectors.register('__disabled', createDisabledEngine)
       if (process.env.testIdAttribute) await playwright.selectors.setTestIdAttribute(process.env.testIdAttribute)
-      
-      // Register all custom locator strategies from the global registry
-      for (const [strategyName, strategyFunction] of globalCustomLocatorStrategies.entries()) {
-        if (!registeredCustomLocatorStrategies.has(strategyName)) {
-          try {
-            // Create a selector engine as JavaScript code string
-            const selectorEngineCode = `
-              (() => ({
-                create(root, target) {
-                  return null;
-                },
-                query(root, selector) {
-                  try {
-                    const strategyFunction = ${strategyFunction.toString()};
-                    const result = strategyFunction(selector, root);
-                    return Array.isArray(result) ? result[0] : result;
-                  } catch (error) {
-                    console.warn('Error in custom locator "${strategyName}":', error);
-                    return null;
-                  }
-                },
-                queryAll(root, selector) {
-                  try {
-                    const strategyFunction = ${strategyFunction.toString()};
-                    const result = strategyFunction(selector, root);
-                    return Array.isArray(result) ? result : result ? [result] : [];
-                  } catch (error) {
-                    console.warn('Error in custom locator "${strategyName}":', error);
-                    return [];
-                  }
-                }
-              }))()
-            `;
-
-            await playwright.selectors.register(strategyName, { content: selectorEngineCode })
-            registeredCustomLocatorStrategies.add(strategyName)
-          } catch (error) {
-            if (!error.message.includes('already registered')) {
-              console.warn(`Failed to register global custom locator strategy '${strategyName}':`, error)
-            }
-          }
-        }
-      }
     } catch (e) {
       console.warn(e)
     }
@@ -915,10 +872,56 @@ class Playwright extends Helper {
       this.debugSection('Url', target.url())
     })
 
+    // Register custom locator strategies after browser is started
+    await this._ensureCustomLocatorsRegistered()
+
     this.isRunning = true
     return this.browser
   }
 
+  async _ensureCustomLocatorsRegistered() {
+    // Register all custom locator strategies from the global registry
+    for (const [strategyName, strategyFunction] of globalCustomLocatorStrategies.entries()) {
+      if (!registeredCustomLocatorStrategies.has(strategyName)) {
+        try {
+          // Create a selector engine factory function like createValueEngine
+          const createCustomEngine = () => {
+            return {
+              create(root, target) {
+                return null;
+              },
+              query(root, selector) {
+                try {
+                  const result = strategyFunction(selector, root);
+                  return Array.isArray(result) ? result[0] : result;
+                } catch (error) {
+                  console.warn(`Error in custom locator "${strategyName}":`, error);
+                  return null;
+                }
+              },
+              queryAll(root, selector) {
+                try {
+                  const result = strategyFunction(selector, root);
+                  return Array.isArray(result) ? result : result ? [result] : [];
+                } catch (error) {
+                  console.warn(`Error in custom locator "${strategyName}":`, error);
+                  return [];
+                }
+              }
+            };
+          };
+
+          await playwright.selectors.register(strategyName, createCustomEngine)
+          registeredCustomLocatorStrategies.add(strategyName)
+        } catch (error) {
+          if (!error.message.includes('already registered')) {
+            console.warn(`Failed to register custom locator strategy '${strategyName}':`, error)
+          }
+        }
+      }
+    }
+  }
+
   _lookupCustomLocator(customStrategy) {
     if (typeof this.customLocatorStrategies !== 'object' || this.customLocatorStrategies === null) {
       return null
@@ -1345,6 +1348,9 @@ class Playwright extends Helper {
    * ```
    */
   async _locate(locator) {
+    // Ensure custom locators are registered before any locator operations
+    await this._ensureCustomLocatorsRegistered()
+    
     const context = await this._getContext()
 
     if (this.frame) return findElements(this.frame, locator)
@@ -1376,6 +1382,9 @@ class Playwright extends Helper {
    * ```
    */
   async _locateElement(locator) {
+    // Ensure custom locators are registered before any locator operations
+    await this._ensureCustomLocatorsRegistered()
+    
     const context = await this._getContext()
     return findElement(context, locator)
   }