From 83ca025474066287e21a6e20bb2cdb0f96401825 Mon Sep 17 00:00:00 2001
From: Raul Macarie <raul@macarie.me>
Date: Mon, 2 Feb 2026 14:58:34 +0100
Subject: [PATCH] fix(browser): don't take failure screenshot if tests have
 artifacts created by `toMatchScreenshot` (#9552)

---
 packages/browser/src/client/tester/runner.ts | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/packages/browser/src/client/tester/runner.ts b/packages/browser/src/client/tester/runner.ts
index 767555fbf..31e66e000 100644
--- a/packages/browser/src/client/tester/runner.ts
+++ b/packages/browser/src/client/tester/runner.ts
@@ -160,7 +160,15 @@ export function createBrowserRunner(
     }
 
     onTaskFinished = async (task: Task) => {
-      if (this.config.browser.screenshotFailures && document.body.clientHeight > 0 && task.result?.state === 'fail') {
+      if (
+        this.config.browser.screenshotFailures
+        && document.body.clientHeight > 0
+        && task.result?.state === 'fail'
+        && task.type === 'test'
+        && task.artifacts.every(
+          artifact => artifact.type !== 'internal:toMatchScreenshot',
+        )
+      ) {
         const screenshot = await page.screenshot({
           timeout: this.config.browser.providerOptions?.actionTimeout ?? 5_000,
         } as any /** TODO */).catch((err) => {
-- 
2.54.0