From f25c399a276af97e137dacb5cd2c66f3a8fafc8d Mon Sep 17 00:00:00 2001
From: Ashton Anderson <ashton@cs.toronto.edu>
Date: Thu, 26 Feb 2026 02:30:46 -0500
Subject: [PATCH] fix: avoid uci flash in blunder meter moves

---
 src/components/Analysis/BoardChrome.tsx | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/src/components/Analysis/BoardChrome.tsx b/src/components/Analysis/BoardChrome.tsx
index daa117bc..ccde6311 100644
--- a/src/components/Analysis/BoardChrome.tsx
+++ b/src/components/Analysis/BoardChrome.tsx
@@ -1,3 +1,4 @@
+import { useEffect, useRef } from 'react'
 import { motion, type MotionValue } from 'framer-motion'
 import type { ColorSanMapping, BlunderMeterResult } from 'src/types'
 
@@ -350,6 +351,21 @@ export const AnalysisCompactBlunderMeter: React.FC<
   variant = 'mobile',
 }) => {
   const isDesktop = variant === 'desktop'
+  const sanLabelCacheRef = useRef<Record<string, string>>({})
+
+  useEffect(() => {
+    if (!colorSanMapping) return
+
+    for (const [move, entry] of Object.entries(colorSanMapping)) {
+      if (entry?.san) {
+        sanLabelCacheRef.current[move] = entry.san
+      }
+    }
+  }, [colorSanMapping])
+
+  const getDisplayMoveLabel = (move: string) =>
+    colorSanMapping?.[move]?.san ?? sanLabelCacheRef.current[move] ?? '...'
+
   const getTopCategoryMoves = (
     moves: { move: string; probability: number }[],
   ) => {
@@ -360,7 +376,7 @@ export const AnalysisCompactBlunderMeter: React.FC<
       .map((entry) => ({
         move: entry.move,
         probability: Math.round(entry.probability),
-        label: colorSanMapping?.[entry.move]?.san || entry.move,
+        label: getDisplayMoveLabel(entry.move),
       }))
 
     if (!playedMove || topMoves.some((entry) => entry.move === playedMove)) {
@@ -377,8 +393,7 @@ export const AnalysisCompactBlunderMeter: React.FC<
       {
         move: playedMoveEntry.move,
         probability: Math.round(playedMoveEntry.probability),
-        label:
-          colorSanMapping?.[playedMoveEntry.move]?.san || playedMoveEntry.move,
+        label: getDisplayMoveLabel(playedMoveEntry.move),
       },
     ]
   }