From 9ad44fd3f6716b2728fbecf90e226fc58e688010 Mon Sep 17 00:00:00 2001
From: stroblme <stroblme@posteo.de>
Date: Tue, 25 Aug 2026 22:13:32 +0200
Subject: [PATCH] Draw the edge sparkline only for a value it can plot

The popover showed a non-numeric value twice: only numbers are recorded,
so MessageSparkline fell through to a collapsed ValuePreview and the
inspector rendered its own expanded one below it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013Gf7WaExcJ9bs3kfJXB3nK
---
 frontend/src/components/Flow/EdgeInspector.tsx | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/Flow/EdgeInspector.tsx b/frontend/src/components/Flow/EdgeInspector.tsx
index f2aa90e..53c2b3e 100644
--- a/frontend/src/components/Flow/EdgeInspector.tsx
+++ b/frontend/src/components/Flow/EdgeInspector.tsx
@@ -132,15 +132,18 @@ export function EdgeInspector({
         </div>
 
         {/* The same curve the node panel draws for this message: one value
-            says little, how it has been moving says the rest. */}
-        {live === undefined ? null : (
+            says little, how it has been moving says the rest. Only for a
+            number, because only numbers are recorded — for anything else the
+            sparkline falls through to its own copy of the value, which the
+            popover already shows below. */}
+        {typeof live?.value === "number" ? (
           <div className="mt-2">
             <MessageSparkline
               flow={flow}
               name={displayName(flow, edge.message)}
             />
           </div>
-        )}
+        ) : null}
 
         {/* Several nodes can publish one message, and so can a dashboard or
             another flow — so the value alone does not say what caused it. */}