fix: skip _finishTest and _cleanup for ALL helpers when browser restarts

Previous fix only checked individual helpers, but non-browser helpers like
JSONResponse and ScreenshotSessionHelper also don't need cleanup when
browser restarts. Now checks if ANY helper has restart enabled and
skips _finishTest/_cleanup for all helpers in that case.

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

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

Author: DavertMik

lib/listener/helpers.js

@@ -73,27 +73,25 @@ export default function () {
   })
 
   event.dispatcher.on(event.all.result, () => {
-    // Skip _finishTest for helpers that restart browser/context to avoid double cleanup
+    // Skip _finishTest for all helpers if any browser helper restarts to avoid double cleanup
+    const hasBrowserRestart = Object.values(helpers).some(helper => (helper.config && helper.config.restart) || (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context')))
+
     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)
-        }
+      if (helper._finishTest && !hasBrowserRestart) {
+        recorder.add(`hook ${key}._finishTest()`, () => helper._finishTest(), true, false)
       }
     })
   })
 
   event.dispatcher.on(event.all.after, () => {
-    // Skip _cleanup for helpers that restart browser/context to avoid double cleanup
+    // Skip _cleanup for all helpers if any browser helper restarts to avoid double cleanup
+    const hasBrowserRestart = Object.values(helpers).some(helper => (helper.config && helper.config.restart) || (helper.options && (helper.options.restart === 'browser' || helper.options.restart === 'context')))
+
     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)
-        }
+      if (helper._cleanup && !hasBrowserRestart) {
+        recorder.add(`hook ${key}._cleanup()`, () => helper._cleanup(), true, false)
       }
     })
   })