From 63d792933350bef91f5688a2b4392e339b2ed370 Mon Sep 17 00:00:00 2001
From: Hiroshi Ogawa <4232207+hi-ogawa@users.noreply.github.com>
Date: Wed, 21 Oct 2026 17:21:28 +0900
Subject: [PATCH] refactor(browser): clarify failure screenshot naming

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 packages/browser/src/client/tester/runner.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/packages/browser/src/client/tester/runner.ts b/packages/browser/src/client/tester/runner.ts
index 6ba4a8fc7..0fe96a6d8 100644
--- a/packages/browser/src/client/tester/runner.ts
+++ b/packages/browser/src/client/tester/runner.ts
@@ -464,8 +464,9 @@ async function updateTestFilesLocations(files: File[], sourceMaps: Map<string, a
 
 function getFailureScreenshotName(test: Test, retry: number, repeats: number) {
   const name = test.fullTestName.replace(/\W/g, '-')
-  const retryCount = typeof test.retry === 'number' ? test.retry : (test.retry?.count ?? 0)
-  if (!retryCount && !test.repeats) {
+  const maxRetries = typeof test.retry === 'number' ? test.retry : (test.retry?.count ?? 0)
+  // tests that can run more than once get a suffix on every attempt, including the first
+  if (!maxRetries && !test.repeats) {
     return name
   }
   return `${name}-${repeats}-${retry}`
-- 
2.54.0