From 659f540270a0035ee5ee0dc43d69b06327269c17 Mon Sep 17 00:00:00 2001
From: Brandon Walter <51866976+echoparkbaby@users.noreply.github.com>
Date: Sat, 9 May 2026 12:33:26 -0700
Subject: [PATCH] fix: drop redundant stage suffix from Burn-In failed chip
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Per user: '(LONG SMART)' was redundant since the LONG SMART column
already shows FAILED. Same for short SMART and surface_validate
(the dominant case — the drawer shows per-stage Reason for digging).

Suffix kept for precheck / final_check since those are rare enough
that the hint is genuinely helpful.
---
 app/templates/components/drives_table.html | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/templates/components/drives_table.html b/app/templates/components/drives_table.html
index 907a827..26fada7 100644
--- a/app/templates/components/drives_table.html
+++ b/app/templates/components/drives_table.html
@@ -46,7 +46,13 @@
   {%- elif bi.state == 'passed' -%}
     <span class="chip chip-passed">Passed</span>
   {%- elif bi.state == 'failed' -%}
-    <span class="chip chip-failed">Failed{% if bi.stage_name %} ({{ bi.stage_name | replace('_',' ') }}){% endif %}</span>
+    {# Suppress the stage suffix for SMART + surface_validate stages.
+       SMART has its own columns, and surface_validate is the dominant
+       case so a redundant suffix just adds visual noise. The drawer
+       shows the per-stage Reason for any digging. Keep the suffix for
+       precheck / final_check since those are rare enough that the hint
+       is helpful. #}
+    <span class="chip chip-failed">Failed{% if bi.stage_name and bi.stage_name not in ('short_smart', 'long_smart', 'surface_validate') %} ({{ bi.stage_name | replace('_',' ') }}){% endif %}</span>
   {%- elif bi.state == 'cancelled' -%}
     <span class="chip chip-aborted">Cancelled</span>
   {%- elif bi.state == 'unknown' -%}