fix(Playwright): Handle selector registration in worker processes

- Wrap __value and __disabled selector registration in try-catch
- Handle 'already registered' errors gracefully
- Add page/context validation before storage cleanup
- Skip storage cleanup if no active page/context exists

These changes prevent errors when running tests in worker processes where
Playwright's selector registration state is shared globally across workers.

Author: kobenguyent

lib/helper/Playwright.js

@@ -506,11 +506,30 @@ class Playwright extends Helper {
 
     // register an internal selector engine for reading value property of elements in a selector
     try {
-      if (!defaultSelectorEnginesInitialized) {
+      // Always wrap in try-catch since selectors might be registered globally across workers
+      try {
         await playwright.selectors.register('__value', createValueEngine)
-        await playwright.selectors.register('__disabled', createDisabledEngine)
-        if (process.env.testIdAttribute) await playwright.selectors.setTestIdAttribute(process.env.testIdAttribute)
         defaultSelectorEnginesInitialized = true
+      } catch (e) {
+        if (!e.message.includes('already registered')) {
+          throw e
+        }
+        // Selector already registered globally, mark as initialized
+        defaultSelectorEnginesInitialized = true
+      }
+      try {
+        await playwright.selectors.register('__disabled', createDisabledEngine)
+      } catch (e) {
+        if (!e.message.includes('already registered')) {
+          throw e
+        }
+      }
+      if (process.env.testIdAttribute) {
+        try {
+          await playwright.selectors.setTestIdAttribute(process.env.testIdAttribute)
+        } catch (e) {
+          // Ignore if already set
+        }
       }
 
       // Register all custom locator strategies from the global registry
@@ -651,7 +670,20 @@ class Playwright extends Helper {
           }
         }
         this.debugSection('New Session', JSON.stringify(this.contextOptions))
-        this.browserContext = await this.browser.newContext(this.contextOptions) // Adding the HTTPSError ignore in the context so that we can ignore those errors
+        try {
+          this.browserContext = await this.browser.newContext(this.contextOptions) // Adding the HTTPSError ignore in the context so that we can ignore those errors
+        } catch (e) {
+          // In worker processes, selectors might already be registered globally
+          // causing newContext to fail. Try again or use the error
+          if (e.message && e.message.includes('already registered')) {
+            // Selectors already registered, try creating context anyway
+            // This happens in worker processes where Playwright state is shared
+            this.debugSection('Session', 'Selectors already registered, retrying context creation')
+            this.browserContext = await this.browser.newContext(this.contextOptions)
+          } else {
+            throw e
+          }
+        }
       }
     }
 
@@ -4775,6 +4807,11 @@ async function refreshContextSession() {
   }
 
   try {
+    if (!this.page || !this.browserContext) {
+      this.debugSection('Session', 'Skipping storage cleanup - no active page/context')
+      return
+    }
+    
     const currentUrl = await this.grabCurrentUrl()
 
     if (currentUrl.startsWith('http')) {