From a6135862c25abf0e38f8a08ab909692334d2f337 Mon Sep 17 00:00:00 2001
From: Christian Vidal Wolf <christianvidalwolf@gmail.com>
Date: Sat, 21 Feb 2026 17:59:46 +0100
Subject: [PATCH] fix: resolve primary_metric toUpperCase crash on legacy
 experiment detail

---
 components/ExperimentDetail.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/components/ExperimentDetail.tsx b/components/ExperimentDetail.tsx
index cf73015..22f2597 100644
--- a/components/ExperimentDetail.tsx
+++ b/components/ExperimentDetail.tsx
@@ -67,7 +67,7 @@ const ExperimentDetail: React.FC<ExperimentDetailProps> = ({ experimentId, onClo
   const chartData = useMemo(() => {
     if (!experiment || !salesData.length) return [];
 
-    const asinSet = new Set((experiment.asins || []).map(a => a.toUpperCase()));
+    const asinSet = new Set((experiment.asins || []).map(a => (a || '').toUpperCase()));
     const weeklyMap = new Map<string, any>();
 
     salesData.forEach(r => {
@@ -253,7 +253,7 @@ const ExperimentDetail: React.FC<ExperimentDetailProps> = ({ experimentId, onClo
                 <div className="bg-slate-800/30 border border-slate-700 rounded-xl p-4">
                   <div className="flex justify-between items-center mb-4">
                     <h3 className="text-sm font-semibold text-slate-300">
-                      Timeline Impact ({experiment.primary_metric.toUpperCase()})
+                      Timeline Impact ({(experiment.primary_metric || 'units').toUpperCase()})
                     </h3>
                   </div>
                   <div className="h-48 w-full">