fix(bench): Best/Worst headline respects higher_is_better

src/app/alternatives/[slug]/page.tsx

@@ -250,15 +250,15 @@ export default async function AlternativePage({
           <dl className="mt-10 grid grid-cols-2 sm:flex sm:flex-wrap items-baseline gap-x-8 gap-y-3 border-y border-rule py-4">
             <SummaryStat
               label="Best"
-              value={`${fmtValue(fieldMin, bench.unit)}${unitSuffix(bench.unit)}`}
+              value={`${fmtValue(bench.higherIsBetter ? fieldMax : fieldMin, bench.unit)}${unitSuffix(bench.unit)}`}
             />
             <SummaryStat
               label="Median"
               value={`${fmtValue(fieldMedian, bench.unit)}${unitSuffix(bench.unit)}`}
             />
             <SummaryStat
               label="Worst"
-              value={`${fmtValue(fieldMax, bench.unit)}${unitSuffix(bench.unit)}`}
+              value={`${fmtValue(bench.higherIsBetter ? fieldMin : fieldMax, bench.unit)}${unitSuffix(bench.unit)}`}
             />
             <SummaryStat
               label="Spread"

src/components/benchmark-body.tsx

@@ -197,7 +197,7 @@
   // L1/L2 layer counts. When both > 0 the bench mixes L1 and L2 chains
   // and we render a top-level Layer toggle that filters the entire page
   // (chart + summary + ledger) to one layer at a time. Default is L1.
   const layerCounts = useMemo(() => {
     let l1 = 0;
     let l2 = 0;
     for (const r of benchmark.results) {
@@ -212,7 +212,7 @@
   // hasLayerSplit is false the original benchmark is returned untouched
   // so non-layer benches keep their existing behavior. The chart, the
   // summary stats and the ledger all read from `viewBenchmark`.
   const viewBenchmark = useMemo(() => {
     if (!hasLayerSplit) return benchmark;
     return {
       ...benchmark,
@@ -230,7 +230,7 @@
   // they always saw.
   const allowedViews = viewsForBenchmark(viewBenchmark);
   const defaultView = defaultViewFor(viewBenchmark);
   const [view, setView, viewMounted] = useViewPreference(
     viewBenchmark.slug,
     defaultView,
     allowedViews,
@@ -241,7 +241,7 @@
   // hidden when they switch to distribution or donut - the model is
   // "this is the field of providers the reader chose to focus on",
   // not "what each view chose to drop". Resets on bench navigation.
   const [excluded, setExcluded] = useState<Set<string>>(() => new Set());
   const toggleExclude = (slug: string) =>
     setExcluded((prev) => {
       const next = new Set(prev);
@@ -257,22 +257,22 @@
   // being split between the dimension row and the chart toolbar.
   const chartRegions = chartOnlyRegions(benchmark);
   const showChartRegionRow = regionOptions.length === 0 && chartRegions.length > 1;
   const [chartRegion, setChartRegion] = useState<string>("all");
 
   // Active companion-metric panel. null = main spec metric (default chart
   // data, default unit, default header). When a panel id is set, the chart
   // pulls its per-provider series from panel.seriesByProvider, swaps the
   // header label to panel.label, and the Y-axis unit to panel.unit.
   const [activePanelId, setActivePanelId] = useState<string | null>(null);
   // Single Top-N value shared across every chart view AND the ledger
   // so a reader who picks "Top 5" sees the same 5 providers in every
   // surface. Each chart still computes its own option set off its own
   // post-filter cohort, but the active value is parent-controlled.
   const [topN, setTopN] = useState<number | null>(null);
   const topNControl = useMemo(() => ({ topN, setTopN }), [topN]);
   const activePanel =
     benchmark.metricPanels?.find((p) => p.id === activePanelId) ?? null;
   const chartRegionOptions: ChainOption[] = useMemo(
     () => [
       { value: "all", label: "All" },
       ...chartRegions.map((r) => ({ value: r, label: REGION_DISPLAY[r] ?? r })),
@@ -359,31 +359,39 @@
         </div>
       )}
 
-      {!isDraft && benchmark.unit !== "count" && (
-        <dl className="mt-10 card rounded-xl grid grid-cols-2 sm:flex sm:flex-wrap divide-y divide-x sm:divide-y-0 divide-rule overflow-hidden">
-          <SummaryStat
-            label="Best"
-            value={`${fmtValue(fieldMin, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
-          />
-          <SummaryStat
-            label="Median"
-            value={`${fmtValue(fieldMedian, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
-          />
-          <SummaryStat
-            label="Worst"
-            value={`${fmtValue(fieldMax, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
-          />
-          <SummaryStat
-            label="Spread"
-            value={tailSpread > 0 ? `${tailSpread.toFixed(1)}×` : "-"}
-            hint={
-              tailSpread > 0
-                ? `${fmtUnit(tailMin, benchmark.unit)} → ${fmtUnit(tailMax, benchmark.unit)}`
-                : undefined
-            }
-          />
-        </dl>
-      )}
+      {!isDraft && benchmark.unit !== "count" && (() => {
+        // For higher-is-better benches (e.g. HL frontends USD revenue), the
+        // "best" headline is the max value, not the min. Latency benches keep
+        // the original min=best mapping.
+        const higherIsBetter = benchmark.higherIsBetter === true;
+        const bestValue = higherIsBetter ? fieldMax : fieldMin;
+        const worstValue = higherIsBetter ? fieldMin : fieldMax;
+        return (
+          <dl className="mt-10 card rounded-xl grid grid-cols-2 sm:flex sm:flex-wrap divide-y divide-x sm:divide-y-0 divide-rule overflow-hidden">
+            <SummaryStat
+              label="Best"
+              value={`${fmtValue(bestValue, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
+            />
+            <SummaryStat
+              label="Median"
+              value={`${fmtValue(fieldMedian, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
+            />
+            <SummaryStat
+              label="Worst"
+              value={`${fmtValue(worstValue, benchmark.unit)}${unitSuffix(benchmark.unit)}`}
+            />
+            <SummaryStat
+              label="Spread"
+              value={tailSpread > 0 ? `${tailSpread.toFixed(1)}×` : "-"}
+              hint={
+                tailSpread > 0
+                  ? `${fmtUnit(tailMin, benchmark.unit)} → ${fmtUnit(tailMax, benchmark.unit)}`
+                  : undefined
+              }
+            />
+          </dl>
+        );
+      })()}
 
       {!isDraft && (
         <>