fix: skip _finishTest and _cleanup when browser restart is enabled

Prevents double cleanup and hanging when BROWSER_RESTART=browser or
BROWSER_RESTART=context is used. The cleanup already happens in _after()
and _afterSuite() hooks, so _finishTest() and _cleanup() are redundant
and cause the process to hang trying to clean up an already closed browser.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: DavertMik

lib/listener/helpers.js

@@ -73,10 +73,28 @@ export default function () {
   })
 
   event.dispatcher.on(event.all.result, () => {
-    runAsyncHelpersHook('_finishTest', {}, true)
+    // Skip _finishTest for helpers that restart browser/context to avoid double cleanup
+    Object.keys(helpers).forEach(key => {
+      const helper = helpers[key]
+      if (helper._finishTest) {
+        const shouldSkip = (helper.config && helper.config.restart) || (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context'))
+        if (!shouldSkip) {
+          recorder.add(`hook ${key}._finishTest()`, () => helper._finishTest(), true, false)
+        }
+      }
+    })
   })
 
   event.dispatcher.on(event.all.after, () => {
-    runAsyncHelpersHook('_cleanup', {}, true)
+    // Skip _cleanup for helpers that restart browser/context to avoid double cleanup
+    Object.keys(helpers).forEach(key => {
+      const helper = helpers[key]
+      if (helper._cleanup) {
+        const shouldSkip = (helper.config && helper.config.restart) || (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context'))
+        if (!shouldSkip) {
+          recorder.add(`hook ${key}._cleanup()`, () => helper._cleanup(), true, false)
+        }
+      }
+    })
   })
 }