From 7086b0d057197d415394c16827e2b5a239bb2d81 Mon Sep 17 00:00:00 2001
From: Raul Macarie <raul@macarie.me>
Date: Thu, 29 Jan 2026 18:49:07 +0100
Subject: [PATCH] fix(browser): don't take failure screenshot if tests fail
 because of `toMatchScreenshot`

---
 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 767555fbfb1b..31e66e0008f7 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) => {