From 3f3c6e594dbff583f71ffe0aa45883252680ad2b Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 13:35:03 +0000
Subject: [PATCH 1/6] Overhaul medication top-4 boxes with a smart content-fit
 rule
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The four top boxes on the medication detail page (and several search cells)
crammed full clinical sentences into a format built for short, glanceable
values. Introduce one shared rule — constrained surfaces show curated crisp
tokens; full content stays one tap away in the sections/reference — and apply
it across the detail page and prescribing search without removing any content.

Top-4 boxes: replace the verbose derived tiles (medicationDetailTiles) with
medicationHeroMetrics, driven by each record's already-curated ~4-item stats
array (Max Dose · half-life · drug-specific risk/caution flags). Max Dose
leads and renders as the primary/clinical tile (its flag:"hi" marks ceiling
importance, not danger) so red stays reserved for genuine safety signals; tone
comes from medicationStatTone. A clamped one-line indication subtitle under
the header preserves the "what it treats" orientation.

Content-fit rule: add a shared shortValue() helper (first clinical clause +
word-boundary cap, generalising the formulationShortLabel idiom); crisp
medicationIndication to a single clause; clamp the verbose search cells
(indication, prescribing action, dose) so rows stay uniform; and collapse long
quick-reference rows with a chevron using the differential clamp-on-collapse
pattern (line-clamp-2 + group-open:line-clamp-none). Retire the now-duplicate
sidebar "Key stats" card.

Tests cover shortValue capping/clause preservation and medicationHeroMetrics
ordering, capping, tone, and backfill. Verified via lint, typecheck, vitest,
prettier, type/icon-scale + brand checks, and Chromium screenshots across
five drug classes in light/dark/mobile (no horizontal overflow).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 .../medication-prescribing-workspace.tsx      |  12 +-
 .../medication-record-page.tsx                | 258 ++++++++++--------
 src/lib/medications.ts                        | 100 +++++--
 tests/medications.test.ts                     | 109 ++++++++
 4 files changed, 323 insertions(+), 156 deletions(-)

diff --git a/src/components/clinical-dashboard/medication-prescribing-workspace.tsx b/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
index e9fea8a11..3c818f3aa 100644
--- a/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
+++ b/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
@@ -535,7 +535,7 @@ function MedicationResults({
                       <span className="block break-words text-base-minus font-semibold text-[color:var(--text-heading)]">
                         <HighlightedName text={result.name} term={query} />
                       </span>
-                      <span className="block break-words text-xs font-medium text-[color:var(--text-muted)]">
+                      <span className="line-clamp-1 break-words text-xs font-medium text-[color:var(--text-muted)]">
                         {result.indication}
                       </span>
                       {showMatchBadge || result.match !== "Exact clinical fit" || row.badges.length > 0 ? (
@@ -548,13 +548,13 @@ function MedicationResults({
                       ) : null}
                     </div>
                   </div>
-                  <span className="nums text-sm-minus font-semibold text-[color:var(--text-heading)]">
+                  <span className="nums line-clamp-2 text-sm-minus font-semibold text-[color:var(--text-heading)]">
                     {result.dose}
                   </span>
                   <DoseCeiling value={result.ceiling} />
                   <span className="flex min-w-0 items-start gap-1.5 text-sm-minus font-medium leading-[1.4] text-[color:var(--text-heading)]">
                     <ActionToneIcon tone={result.actionTone} className="mt-0.5 h-3.5 w-3.5" />
-                    <span className="min-w-0 break-words">{result.action}</span>
+                    <span className="line-clamp-2 min-w-0 break-words">{result.action}</span>
                   </span>
                   {result.href ? (
                     <ChevronRight
@@ -616,7 +616,9 @@ function MedicationResults({
                   <p className="line-clamp-2 break-words text-base-minus font-semibold leading-5 text-[color:var(--text-heading)]">
                     <HighlightedName text={result.name} term={query} />
                   </p>
-                  <p className="break-words text-xs font-medium text-[color:var(--text-muted)]">{result.indication}</p>
+                  <p className="line-clamp-1 break-words text-xs font-medium text-[color:var(--text-muted)]">
+                    {result.indication}
+                  </p>
                 </div>
                 {showMatchBadge || result.match !== "Exact clinical fit" || row.badges.length > 0 ? (
                   <div className="flex max-w-full flex-wrap items-center gap-1.5">
@@ -630,7 +632,7 @@ function MedicationResults({
                   <span className="nums break-words">{result.dose}</span>
                   <DoseCeiling value={result.ceiling} />
                 </div>
-                <p className="break-words text-pretty text-xs leading-[1.45] text-[color:var(--text-muted)]">
+                <p className="line-clamp-2 break-words text-pretty text-xs leading-[1.45] text-[color:var(--text-muted)]">
                   <ActionToneIcon tone={result.actionTone} className="mr-1 inline-block h-3.5 w-3.5 align-[-0.15em]" />
                   {result.action}
                 </p>
diff --git a/src/components/clinical-dashboard/medication-record-page.tsx b/src/components/clinical-dashboard/medication-record-page.tsx
index dc1aa5526..ee6653ff6 100644
--- a/src/components/clinical-dashboard/medication-record-page.tsx
+++ b/src/components/clinical-dashboard/medication-record-page.tsx
@@ -14,14 +14,15 @@ import {
   Gauge,
   Lock,
   Pill,
+  ShieldAlert,
   ShieldCheck,
-  Sparkles,
+  Timer,
   type LucideIcon,
 } from "lucide-react";
 import Link from "next/link";
 import { useMemo, useRef, useState, type CSSProperties, type KeyboardEvent as ReactKeyboardEvent } from "react";
 
-import { BadgeCluster, clinicalBadgeToneClass } from "@/components/clinical-dashboard/clinical-badge";
+import { BadgeCluster } from "@/components/clinical-dashboard/clinical-badge";
 import { MedicationConsiderations } from "@/components/clinical-dashboard/medication-considerations";
 import { PatientProfilePanel } from "@/components/clinical-dashboard/patient-profile-panel";
 import { useMedicationDetail } from "@/components/clinical-dashboard/use-medication-catalog";
@@ -30,11 +31,27 @@ import {
   medicationAccessFields,
   medicationIdentityBadges,
   medicationRowBadges,
-  medicationStatTone,
   type MedicationGovernance,
 } from "@/lib/medication-badges";
-import { medicationDetailTiles, type MedicationRecord, type MedicationSection } from "@/lib/medications";
-import { cn, EmptyState, LoadingPanel, pageContainer, toneDanger } from "@/components/ui-primitives";
+import {
+  medicationHeroMetrics,
+  medicationIndication,
+  type MedicationHeroMetric,
+  type MedicationQuickRow,
+  type MedicationRecord,
+  type MedicationSection,
+} from "@/lib/medications";
+import type { SemanticTone } from "@/lib/semantic-tone";
+import {
+  cn,
+  EmptyState,
+  LoadingPanel,
+  pageContainer,
+  toneDanger,
+  toneInfo,
+  toneSuccess,
+  toneWarning,
+} from "@/components/ui-primitives";
 
 const sectionIcons: Record<string, LucideIcon> = {
   dose: CalendarDays,
@@ -74,83 +91,79 @@ function medicationAccentStyle(accent: string | undefined): CSSProperties {
   } as CSSProperties;
 }
 
-// Icon + categorical chip per detail tile (index-aligned with medicationDetailTiles:
-// Prescribing answer / Dosing / Dose ceiling / Avoid). The danger tile is toned
-// separately below.
-const detailTileDecor: Array<{ icon: LucideIcon; chip: string }> = [
-  {
-    icon: Sparkles,
-    chip: "border-[color:var(--type-source-border)] bg-[color:var(--type-source-soft)] text-[color:var(--type-source)]",
+// Tone-driven hero metric tile. Colour comes only from the metric's semantic tone
+// (medicationStatTone, via medicationHeroMetrics) so it honours the #659 contract:
+// green = success, amber = caution, red = safety, teal = primary/evidence. The
+// value stays in a high-contrast heading colour on every tone so text never sits
+// on a same-hue wash (the readability lesson from #659) — the border, soft fill,
+// label and icon chip carry the colour.
+const heroToneTile: Record<SemanticTone, { card: string; chip: string; label: string; value: string }> = {
+  clinical: {
+    card: "border-[color:var(--clinical-accent-border)] bg-[color:var(--clinical-accent-soft)]",
+    chip: "border-[color:var(--clinical-accent)]/25 bg-[color:var(--surface)] text-[color:var(--clinical-accent)]",
+    label: "text-[color:var(--clinical-accent)]",
+    value: "text-[color:var(--text-heading)]",
   },
-  {
-    icon: CalendarDays,
-    chip: "border-[color:var(--type-document-border)] bg-[color:var(--type-document-soft)] text-[color:var(--type-document)]",
+  danger: {
+    card: toneDanger,
+    chip: "border-[color:var(--danger)]/25 bg-[color:var(--surface)] text-[color:var(--danger)]",
+    label: "text-[color:var(--danger)]",
+    value: "text-[color:var(--danger-text)]",
   },
-  {
-    icon: Gauge,
-    chip: "border-[color:var(--type-service-border)] bg-[color:var(--type-service-soft)] text-[color:var(--type-service)]",
+  warning: {
+    card: toneWarning,
+    chip: "border-[color:var(--warning)]/25 bg-[color:var(--surface)] text-[color:var(--warning)]",
+    label: "text-[color:var(--warning)]",
+    value: "text-[color:var(--text-heading)]",
   },
-  { icon: Ban, chip: "border-[color:var(--danger)]/25 bg-[color:var(--danger-soft)] text-[color:var(--danger)]" },
-];
-
-function DetailTile({
-  label,
-  value,
-  meta,
-  danger = false,
-  icon: Icon,
-  chip,
-}: {
-  label: string;
-  value: string;
-  meta?: string;
-  danger?: boolean;
-  icon: LucideIcon;
-  chip: string;
-}) {
+  success: {
+    card: toneSuccess,
+    chip: "border-[color:var(--success)]/25 bg-[color:var(--surface)] text-[color:var(--success)]",
+    label: "text-[color:var(--success)]",
+    value: "text-[color:var(--text-heading)]",
+  },
+  info: {
+    card: toneInfo,
+    chip: "border-[color:var(--info-border)] bg-[color:var(--surface)] text-[color:var(--info)]",
+    label: "text-[color:var(--info)]",
+    value: "text-[color:var(--text-heading)]",
+  },
+  neutral: {
+    card: "border-[color:var(--border)] bg-[color:var(--surface-raised)]",
+    chip: "border-[color:var(--border)] bg-[color:var(--surface-subtle)] text-[color:var(--text-muted)]",
+    label: "text-[color:var(--text-muted)]",
+    value: "text-[color:var(--text-heading)]",
+  },
+};
+
+// Icon keyed to the metric's meaning: a gauge for the dose ceiling, a timer for
+// time-based metrics (half-life / onset / duration), and a tone-appropriate shield
+// or alert for everything else (risk & caution flags). Rendered directly (rather
+// than assigning the component to a local) so it stays a static component.
+function HeroMetricIcon({ metric }: { metric: MedicationHeroMetric }) {
+  const label = metric.label.toLowerCase();
+  const iconClass = "h-3.5 w-3.5";
+  if (/dose|ceiling|\bmax\b/.test(label)) return <Gauge className={iconClass} aria-hidden="true" />;
+  if (/half-life|onset|duration|timing|freq/.test(label)) return <Timer className={iconClass} aria-hidden="true" />;
+  if (metric.tone === "danger") return <ShieldAlert className={iconClass} aria-hidden="true" />;
+  if (metric.tone === "warning") return <TriangleAlert className={iconClass} aria-hidden="true" />;
+  if (metric.tone === "success") return <ShieldCheck className={iconClass} aria-hidden="true" />;
+  return <Activity className={iconClass} aria-hidden="true" />;
+}
+
+function DetailTile({ metric }: { metric: MedicationHeroMetric }) {
+  const tone = heroToneTile[metric.tone];
   return (
-    <div
-      className={cn(
-        "rounded-lg border p-3 shadow-[var(--shadow-inset)]",
-        danger ? toneDanger : "border-[color:var(--border)] bg-[color:var(--surface-raised)]",
-      )}
-    >
+    <div className={cn("rounded-lg border p-3 shadow-[var(--shadow-inset)]", tone.card)}>
       <div className="flex items-center gap-2">
-        <span
-          className={cn(
-            "grid h-6 w-6 shrink-0 place-items-center rounded-md border",
-            danger ? "border-[color:var(--danger)]/25 bg-[color:var(--surface)] text-[color:var(--danger)]" : chip,
-          )}
-        >
-          <Icon className="h-3.5 w-3.5" aria-hidden="true" />
+        <span className={cn("grid h-6 w-6 shrink-0 place-items-center rounded-md border", tone.chip)}>
+          <HeroMetricIcon metric={metric} />
         </span>
-        <p
-          className={cn(
-            "text-2xs font-semibold uppercase tracking-[0.08em]",
-            danger ? "text-[color:var(--danger)]" : "text-[color:var(--text-muted)]",
-          )}
-        >
-          {label}
+        <p className={cn("text-2xs font-semibold uppercase leading-tight tracking-[0.08em]", tone.label)}>
+          {metric.label}
         </p>
       </div>
-      <p
-        className={cn(
-          "mt-1.5 text-sm-minus font-semibold leading-5",
-          danger ? "text-[color:var(--danger-text)]" : "text-[color:var(--text-heading)]",
-        )}
-      >
-        {value}
-      </p>
-      {meta ? (
-        <p
-          className={cn(
-            "mt-0.5 text-2xs font-semibold uppercase tracking-[0.06em]",
-            danger ? "text-[color:var(--danger-text)]" : "text-[color:var(--text-muted)]",
-          )}
-        >
-          {meta}
-        </p>
-      ) : null}
+      <p className={cn("mt-1.5 text-sm-minus font-semibold leading-5", tone.value)}>{metric.value}</p>
     </div>
   );
 }
@@ -277,6 +290,45 @@ function SidebarCard({ title, icon: Icon, children }: { title: string; icon: Luc
   );
 }
 
+// Quick-reference values are the verbose ones (≈164 chars median, up to ~560) and
+// would otherwise make the fixed-width sidebar run very tall. Long values collapse
+// to two lines with a chevron affordance and expand in place on tap — the same
+// clamp-on-collapse pattern the differential sections use (line-clamp-2 +
+// group-open:line-clamp-none inside a <details>) — so nothing is removed, just
+// tucked one tap away. Short values render as a plain row with no toggle.
+function QuickRefRow({ row }: { row: MedicationQuickRow }) {
+  const value = row.value.replace(/\*\*/g, "");
+  const label = (
+    <p className="text-2xs font-semibold uppercase tracking-[0.08em] text-[color:var(--text-muted)]">{row.label}</p>
+  );
+
+  if (value.length <= 110) {
+    return (
+      <div className="px-3 py-2.5">
+        {label}
+        <p className="mt-1 text-xs leading-5 text-[color:var(--text-heading)]">{value}</p>
+      </div>
+    );
+  }
+
+  return (
+    <details className="group">
+      <summary className="flex cursor-pointer list-none flex-col px-3 py-2.5 [&::-webkit-details-marker]:hidden">
+        <span className="flex items-center justify-between gap-2">
+          {label}
+          <ChevronDown
+            className="h-3.5 w-3.5 shrink-0 text-[color:var(--text-soft)] transition group-open:rotate-180"
+            aria-hidden="true"
+          />
+        </span>
+        <span className="mt-1 line-clamp-2 text-xs leading-5 text-[color:var(--text-heading)] group-open:line-clamp-none">
+          {value}
+        </span>
+      </summary>
+    </details>
+  );
+}
+
 function MedicationAccessPanel({ record }: { record: MedicationRecord }) {
   const badges = useMemo(() => medicationAccessBadges(record), [record]);
   const fields = useMemo(() => medicationAccessFields(record), [record]);
@@ -314,8 +366,9 @@ function MedicationRecordDetail({
   record: MedicationRecord;
   governance?: MedicationGovernance;
 }) {
-  const tiles = useMemo(() => medicationDetailTiles(record), [record]);
+  const metrics = useMemo(() => medicationHeroMetrics(record), [record]);
   const badges = useMemo(() => medicationIdentityBadges(record, governance), [record, governance]);
+  const indication = useMemo(() => medicationIndication(record), [record]);
   const [activeTab, setActiveTab] = useState<MedicationTabId>("summary");
 
   const sectionsByTab = useMemo(() => {
@@ -359,26 +412,20 @@ function MedicationRecordDetail({
                     </>
                   ) : null}
                 </p>
+                {indication ? (
+                  <p className="mt-1 line-clamp-1 text-sm-minus leading-5 text-[color:var(--text-muted)]">
+                    {indication}
+                  </p>
+                ) : null}
                 <BadgeCluster items={badges} limit={5} showOverflowCount className="mt-2" />
               </div>
             </div>
           </section>
 
           <section className="grid grid-cols-2 gap-2.5 xl:grid-cols-4">
-            {tiles.map((tile, index) => {
-              const decor = detailTileDecor[index] ?? detailTileDecor[detailTileDecor.length - 1];
-              return (
-                <DetailTile
-                  key={tile.label}
-                  label={tile.label}
-                  value={tile.value}
-                  meta={tile.meta}
-                  danger={tile.danger}
-                  icon={tile.danger ? Ban : decor.icon}
-                  chip={decor.chip}
-                />
-              );
-            })}
+            {metrics.map((metric) => (
+              <DetailTile key={metric.label} metric={metric} />
+            ))}
           </section>
 
           <section className="space-y-2.5">
@@ -414,43 +461,12 @@ function MedicationRecordDetail({
           <SidebarCard title="Quick reference" icon={BookOpen}>
             <div className="divide-y divide-[color:var(--border)]">
               {record.quick.map((row) => (
-                <div key={row.label} className="px-3 py-2.5">
-                  <p className="text-2xs font-semibold uppercase tracking-[0.08em] text-[color:var(--text-muted)]">
-                    {row.label}
-                  </p>
-                  <p className="mt-1 text-xs leading-5 text-[color:var(--text-heading)]">
-                    {row.value.replace(/\*\*/g, "")}
-                  </p>
-                </div>
+                <QuickRefRow key={row.label} row={row} />
               ))}
             </div>
           </SidebarCard>
 
           <MedicationAccessPanel record={record} />
-
-          {record.stats.length ? (
-            <SidebarCard title="Key stats" icon={Gauge}>
-              <div className="grid grid-cols-2 gap-2 p-3">
-                {record.stats.map((stat) => {
-                  const tone = medicationStatTone(stat);
-                  return (
-                    <div
-                      key={stat.label}
-                      className={cn(
-                        "rounded-md border bg-[color:var(--surface-subtle)] p-2",
-                        clinicalBadgeToneClass(tone),
-                      )}
-                    >
-                      <p className="text-2xs font-semibold uppercase tracking-[0.08em] text-[color:var(--text-muted)]">
-                        {stat.label}
-                      </p>
-                      <p className="mt-1 text-sm-minus font-semibold">{stat.value}</p>
-                    </div>
-                  );
-                })}
-              </div>
-            </SidebarCard>
-          ) : null}
         </aside>
       </div>
     </div>
diff --git a/src/lib/medications.ts b/src/lib/medications.ts
index d5ad31e2e..17a029620 100644
--- a/src/lib/medications.ts
+++ b/src/lib/medications.ts
@@ -1,4 +1,6 @@
 import { normalizeSearchText, rankCatalogRecords } from "@/lib/catalog-search";
+import { medicationStatTone } from "@/lib/medication-badges";
+import type { SemanticTone } from "@/lib/semantic-tone";
 
 export type MedicationPatientMetadata = {
   factors?: string[];
@@ -139,12 +141,30 @@ export function medicationSearchText(record: MedicationRecord) {
 }
 
 export function medicationIndication(record: MedicationRecord) {
-  return (
+  const raw =
     firstRowValue(record, "ind", "primary") ||
     firstRowValue(record, "summary", "overview") ||
     record.subclass ||
-    record.category
-  );
+    record.category;
+  // Emit a single clinical clause so constrained surfaces (search rows, cross-mode
+  // subtitles, the detail header) get one crisp line; the full indication list
+  // still lives in the record's indication/summary sections.
+  return firstClinicalSentence(raw.replace(/\*\*/g, "")) || raw;
+}
+
+// Shared "short & sharp" clamp for constrained surfaces: take the first clinical
+// clause (protecting decimals/abbreviations, see firstClinicalSentence), strip
+// markdown bold, then cap on a word boundary with an ellipsis. Generalises the
+// slice(0, N)… idiom already used by formulationShortLabel (medication-badges.ts)
+// so every glanceable cell/tile trims content the same way. Full text is always
+// preserved in the deep-content sections/reference — this only shapes previews.
+export function shortValue(text: string, cap = 24): string {
+  const clause = firstClinicalSentence((text ?? "").replace(/\*\*/g, "").trim());
+  if (clause.length <= cap) return clause;
+  const slice = clause.slice(0, cap);
+  const boundary = slice.lastIndexOf(" ");
+  const head = (boundary > cap * 0.6 ? slice.slice(0, boundary) : slice).replace(/[\s,;:]+$/, "");
+  return `${head}…`;
 }
 
 // Take the first sentence without mangling decimals ("1.5 mg") or common
@@ -234,7 +254,7 @@ export function medicationToSearchResult(match: MedicationSearchMatch): Medicati
     name: medication.name,
     indication: medicationIndication(medication),
     match: score >= 12 ? "Exact clinical fit" : score >= 6 ? "Good clinical fit" : "Related match",
-    dose: medicationUsualDose(medication),
+    dose: shortValue(medicationUsualDose(medication), 44),
     ceiling: medicationCeiling(medication),
     action: action.text,
     actionTone: action.tone,
@@ -295,31 +315,51 @@ export function rankMedicationRecords(
 
 export { medicationIdentityBadges } from "@/lib/medication-badges";
 
-export function medicationDetailTiles(record: MedicationRecord) {
+export type MedicationHeroMetric = {
+  label: string;
+  value: string;
+  tone: SemanticTone;
+};
+
+// "Max Dose" (and a bare "Dose") carry flag:"hi" to mark the prescribing
+// ceiling's importance, not a safety stop — render it as the primary/clinical
+// metric so red stays reserved for genuine risk signals (contraindications,
+// toxicity, teratogenicity), matching the #659 colour contract.
+function heroMetricTone(stat: MedicationStat): SemanticTone {
+  if (/^(?:max\s+)?dose$/i.test(stat.label.trim())) return "clinical";
+  return medicationStatTone(stat);
+}
+
+// The top-of-page metric tiles. Every record already carries a curated ~4-item
+// `stats` array of short, glanceable values (median 8 chars) — Max Dose plus
+// half-life and drug-specific risk/caution flags — so those ARE the "max dose ·
+// duration · cautions, short and sharp". We lead with Max Dose (the most-cited
+// ceiling), cap each value with shortValue, and backfill from crisp derived
+// tokens only for the rare record with fewer than four stats. Verbose derived
+// sentences (indication/dosing/contraindications) are not tiled here — they live
+// in the header subtitle and the detail sections instead.
+export function medicationHeroMetrics(record: MedicationRecord): MedicationHeroMetric[] {
+  const isMaxDose = (stat: MedicationStat) => /^(?:max\s+)?dose$/i.test(stat.label.trim());
+  const ordered = [...record.stats.filter(isMaxDose), ...record.stats.filter((stat) => !isMaxDose(stat))];
+  const metrics: MedicationHeroMetric[] = ordered.slice(0, 4).map((stat) => ({
+    label: stat.label,
+    value: shortValue(stat.value),
+    tone: heroMetricTone(stat),
+  }));
+  if (metrics.length >= 4) return metrics;
+
   const usualDose = medicationUsualDose(record);
-  const ceiling = medicationCeiling(record);
-  const avoid = medicationAction(record);
-  return [
-    {
-      label: "Prescribing answer",
-      value: medicationIndication(record).split(".")[0] ?? record.name,
-      meta: record.subclass || record.category,
-    },
-    {
-      label: "Dosing",
-      value: usualDose,
-      meta: record.stats[0]?.label ?? "Usual dose",
-    },
-    {
-      label: "Dose ceiling",
-      value: ceiling,
-      meta: "MAX",
-    },
-    {
-      label: "Avoid",
-      value: avoid?.split(",")[0] ?? "Review contraindications",
-      meta: record.schedule === "S8" ? "Controlled" : "Safety",
-      danger: true,
-    },
-  ];
+  const backfills: MedicationHeroMetric[] = [
+    { label: "Usual dose", value: shortValue(usualDose), tone: "clinical" },
+    record.schedule ? { label: "Schedule", value: record.schedule, tone: "neutral" } : null,
+    record.category ? { label: "Category", value: shortValue(record.category), tone: "neutral" } : null,
+  ].filter((metric): metric is MedicationHeroMetric => metric !== null);
+
+  for (const fill of backfills) {
+    if (metrics.length >= 4) break;
+    if (metrics.some((metric) => metric.label.toLowerCase() === fill.label.toLowerCase())) continue;
+    if (!fill.value || fill.value === "See dosing" || fill.value === "See reference") continue;
+    metrics.push(fill);
+  }
+  return metrics;
 }
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index 0ac77dcc2..fc8750777 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -4,8 +4,11 @@ import { getMedicationRecord, loadMedicationSnapshot } from "@/lib/medication-sn
 import {
   firstClinicalSentence,
   medicationActionDetail,
+  medicationHeroMetrics,
+  medicationIndication,
   medicationToSearchResult,
   rankMedicationRecords,
+  shortValue,
   type MedicationRecord,
 } from "@/lib/medications";
 
@@ -209,3 +212,109 @@ describe("medication action tone", () => {
     expect(firstClinicalSentence("No trailing period")).toBe("No trailing period");
   });
 });
+
+describe("shortValue", () => {
+  it("returns short text unchanged and strips markdown bold", () => {
+    expect(shortValue("200 mg/day")).toBe("200 mg/day");
+    expect(shortValue("**6 g/day**")).toBe("6 g/day");
+    expect(shortValue("")).toBe("");
+  });
+
+  it("caps long text on a word boundary with a trailing ellipsis and no dangling punctuation", () => {
+    const out = shortValue("PO 2 tablets (666 mg) three times daily with meals", 24);
+    expect(out.endsWith("…")).toBe(true);
+    expect(out.length).toBeLessThanOrEqual(25); // cap + the ellipsis char
+    expect(out).not.toMatch(/[\s,;:]…$/); // no space/punctuation immediately before the ellipsis
+  });
+
+  it("keeps only the first clinical clause, protecting decimals and abbreviations", () => {
+    expect(shortValue("Start 1.5 mg NOCTE. Titrate weekly.", 40)).toBe("Start 1.5 mg NOCTE");
+  });
+});
+
+describe("medicationHeroMetrics", () => {
+  it("leads with Max Dose as the primary/clinical tile and keeps the other stats in order", () => {
+    const record = buildRecord({
+      stats: [
+        { label: "Half-life", value: "26 h", cls: "", flag: "" },
+        { label: "GI Upset", value: "COMMON", cls: "warn", flag: "" },
+        { label: "Max Dose", value: "200 mg/day", cls: "hi", flag: "hi" },
+        { label: "Post-MI Safe", value: "YES", cls: "good", flag: "" },
+      ],
+    });
+    const metrics = medicationHeroMetrics(record);
+    expect(metrics.map((metric) => metric.label)).toEqual(["Max Dose", "Half-life", "GI Upset", "Post-MI Safe"]);
+    // Max Dose's flag:"hi" marks ceiling importance, not danger — keep red reserved.
+    expect(metrics.map((metric) => metric.tone)).toEqual(["clinical", "neutral", "warning", "success"]);
+  });
+
+  it("caps to four metrics and shortens over-long values", () => {
+    const record = buildRecord({
+      stats: [
+        { label: "Max Dose", value: "2 mg: max 20 pieces/day; 4 mg: max 10 pieces/day", cls: "hi", flag: "hi" },
+        { label: "Half-life", value: "13-28.4 h", cls: "", flag: "" },
+        { label: "Renal Adj.", value: "DOSE RED.", cls: "warn", flag: "warn" },
+        { label: "Efficacy", value: "MODERATE", cls: "warn", flag: "" },
+        { label: "Extra", value: "SHOULD NOT APPEAR", cls: "", flag: "" },
+      ],
+    });
+    const metrics = medicationHeroMetrics(record);
+    expect(metrics).toHaveLength(4);
+    expect(metrics.some((metric) => metric.label === "Extra")).toBe(false);
+    expect(metrics[0].value.endsWith("…")).toBe(true);
+    expect(metrics[0].value.length).toBeLessThanOrEqual(25);
+  });
+
+  it("marks genuine hi-risk stats as danger even though Max Dose is not", () => {
+    const record = buildRecord({
+      stats: [
+        { label: "Max Dose", value: "3 g/day", cls: "hi", flag: "hi" },
+        { label: "Lactic Acidosis", value: "CRITICAL", cls: "hi", flag: "warn" },
+      ],
+    });
+    const metrics = medicationHeroMetrics(record);
+    expect(metrics.find((metric) => metric.label === "Max Dose")?.tone).toBe("clinical");
+    expect(metrics.find((metric) => metric.label === "Lactic Acidosis")?.tone).toBe("danger");
+  });
+
+  it("backfills from crisp derived tokens when a record has fewer than four stats", () => {
+    const record = buildRecord({
+      schedule: "S8",
+      category: "Opioid",
+      stats: [{ label: "Max Dose", value: "Titrated", cls: "hi", flag: "hi" }],
+      quick: [{ label: "Usual dose", value: "PO 5 mg BD" }],
+    });
+    const metrics = medicationHeroMetrics(record);
+    expect(metrics.map((metric) => metric.label)).toEqual(["Max Dose", "Usual dose", "Schedule", "Category"]);
+    expect(metrics.find((metric) => metric.label === "Schedule")?.value).toBe("S8");
+    expect(metrics.find((metric) => metric.label === "Usual dose")?.value).toBe("PO 5 mg BD");
+  });
+
+  it("promotes Max Dose first even when the curated array lists it later, from the snapshot", () => {
+    const record = getMedicationRecord("metformin");
+    expect(record).toBeTruthy();
+    const metrics = medicationHeroMetrics(record as MedicationRecord);
+    expect(metrics.length).toBeLessThanOrEqual(4);
+    expect(metrics[0].label).toMatch(/max dose/i);
+    expect(metrics[0].tone).toBe("clinical");
+  });
+});
+
+describe("medicationIndication", () => {
+  it("crisps the indication to a single clinical clause", () => {
+    const record = buildRecord({
+      sections: [
+        {
+          title: "Indication",
+          type: "ind",
+          rows: [{ key: "Primary", val: "Alcohol dependence maintenance. Adjunct to psychosocial support." }],
+        },
+      ],
+    });
+    expect(medicationIndication(record)).toBe("Alcohol dependence maintenance");
+  });
+
+  it("falls back to taxonomy when no indication content exists", () => {
+    expect(medicationIndication(buildRecord({ subclass: "SSRI" }))).toBe("SSRI");
+  });
+});

From 6601ccf26749eb44ecd0213a752e0a9942b88674 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 13:47:48 +0000
Subject: [PATCH 2/6] Preserve genus abbreviations in firstClinicalSentence
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Routing medicationIndication through firstClinicalSentence made it treat a
single-letter clinical abbreviation followed by a period as a sentence end, so
Clarithromycin's "Atypical CAP, H. pylori eradication, severe MAC complex in
HIV." rendered as "Atypical CAP, H" — dropping the H. pylori indication — in the
detail header and search/cross-mode subtitles. Skip the split when the token
before the period is a lone letter (genus abbreviations like H. pylori, E. coli,
C. difficile). Adds unit coverage plus a Clarithromycin snapshot regression test.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 src/lib/medications.ts    |  4 ++++
 tests/medications.test.ts | 16 ++++++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/src/lib/medications.ts b/src/lib/medications.ts
index 17a029620..0912f91e0 100644
--- a/src/lib/medications.ts
+++ b/src/lib/medications.ts
@@ -179,6 +179,10 @@ export function firstClinicalSentence(value: string): string {
     if (next !== undefined && !/\s/.test(next)) continue;
     const before = text.slice(0, match.index);
     if (/(?:^|[\s(])(?:e\.g|i\.e|etc|vs|approx)$/i.test(before)) continue;
+    // Single-letter clinical abbreviations — genus abbreviations such as
+    // "H. pylori", "E. coli", "C. difficile" — are not sentence ends; splitting
+    // here would drop the rest of the clause (e.g. an indication list).
+    if (/(?:^|[\s(])[A-Za-z]$/.test(before)) continue;
     return before.trim();
   }
   return text;
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index fc8750777..4202f6e56 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -211,6 +211,14 @@ describe("medication action tone", () => {
     expect(firstClinicalSentence("Rash, nausea, etc. may occur. Stop if severe.")).toBe("Rash, nausea, etc. may occur");
     expect(firstClinicalSentence("No trailing period")).toBe("No trailing period");
   });
+
+  it("does not cut at single-letter genus abbreviations", () => {
+    expect(firstClinicalSentence("Atypical CAP, H. pylori eradication, severe MAC complex in HIV.")).toBe(
+      "Atypical CAP, H. pylori eradication, severe MAC complex in HIV",
+    );
+    expect(firstClinicalSentence("E. coli UTI. Second line.")).toBe("E. coli UTI");
+    expect(firstClinicalSentence("C. difficile colitis (severe). Oral only.")).toBe("C. difficile colitis (severe)");
+  });
 });
 
 describe("shortValue", () => {
@@ -317,4 +325,12 @@ describe("medicationIndication", () => {
   it("falls back to taxonomy when no indication content exists", () => {
     expect(medicationIndication(buildRecord({ subclass: "SSRI" }))).toBe("SSRI");
   });
+
+  it("keeps genus abbreviations intact from the snapshot (does not truncate H. pylori)", () => {
+    const record = getMedicationRecord("clarithromycin");
+    expect(record).toBeTruthy();
+    const indication = medicationIndication(record as MedicationRecord);
+    expect(indication).toContain("H. pylori");
+    expect(indication).not.toBe("Atypical CAP, H");
+  });
 });

From 91edfdd49d01175912ed2cc4b9eeef5f140b17d7 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 13:58:37 +0000
Subject: [PATCH 3/6] Clamp mobile dose cell and dedupe max-dose label check

Address CodeRabbit review on #663:
- The mobile prescribing-result dose value was unclamped while indication and
  action were bounded, so a long dose could still expand the card. Add
  line-clamp-2 to match the desktop dose cell and the fixed-line content-fit goal.
- Extract the duplicated /^(?:max\s+)?dose$/i test into a shared isMaxDoseLabel
  helper reused by heroMetricTone (tone) and medicationHeroMetrics (ordering) so
  the two stay aligned.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 .../medication-prescribing-workspace.tsx      |  2 +-
 src/lib/medications.ts                        | 40 ++++++++++++-------
 tests/medications.test.ts                     | 16 +++++++-
 3 files changed, 40 insertions(+), 18 deletions(-)

diff --git a/src/components/clinical-dashboard/medication-prescribing-workspace.tsx b/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
index 3c818f3aa..85a5aa68b 100644
--- a/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
+++ b/src/components/clinical-dashboard/medication-prescribing-workspace.tsx
@@ -629,7 +629,7 @@ function MedicationResults({
                   </div>
                 ) : null}
                 <div className="flex max-w-full flex-wrap items-center gap-1.5 text-sm-minus font-semibold text-[color:var(--text-heading)]">
-                  <span className="nums break-words">{result.dose}</span>
+                  <span className="nums line-clamp-2 break-words">{result.dose}</span>
                   <DoseCeiling value={result.ceiling} />
                 </div>
                 <p className="line-clamp-2 break-words text-pretty text-xs leading-[1.45] text-[color:var(--text-muted)]">
diff --git a/src/lib/medications.ts b/src/lib/medications.ts
index 0912f91e0..9eea1c15f 100644
--- a/src/lib/medications.ts
+++ b/src/lib/medications.ts
@@ -152,16 +152,18 @@ export function medicationIndication(record: MedicationRecord) {
   return firstClinicalSentence(raw.replace(/\*\*/g, "")) || raw;
 }
 
-// Shared "short & sharp" clamp for constrained surfaces: take the first clinical
-// clause (protecting decimals/abbreviations, see firstClinicalSentence), strip
-// markdown bold, then cap on a word boundary with an ellipsis. Generalises the
-// slice(0, N)… idiom already used by formulationShortLabel (medication-badges.ts)
-// so every glanceable cell/tile trims content the same way. Full text is always
-// preserved in the deep-content sections/reference — this only shapes previews.
+// Shared "short & sharp" length cap for constrained surfaces (hero-metric tiles,
+// search cells): strip markdown bold, then cap on a word boundary with an
+// ellipsis. Generalises the slice(0, N)… idiom already used by
+// formulationShortLabel (medication-badges.ts). It intentionally does NOT split
+// on sentences — stat values are curated tokens that can carry dotted
+// abbreviations ("L.O.T. DRUG", "b.d.") — so callers that want the first clause
+// of prose compose firstClinicalSentence themselves (e.g. medicationIndication,
+// medicationUsualDose). Full text always remains in the sections/reference.
 export function shortValue(text: string, cap = 24): string {
-  const clause = firstClinicalSentence((text ?? "").replace(/\*\*/g, "").trim());
-  if (clause.length <= cap) return clause;
-  const slice = clause.slice(0, cap);
+  const cleaned = (text ?? "").replace(/\*\*/g, "").trim();
+  if (cleaned.length <= cap) return cleaned;
+  const slice = cleaned.slice(0, cap);
   const boundary = slice.lastIndexOf(" ");
   const head = (boundary > cap * 0.6 ? slice.slice(0, boundary) : slice).replace(/[\s,;:]+$/, "");
   return `${head}…`;
@@ -326,11 +328,17 @@ export type MedicationHeroMetric = {
 };
 
 // "Max Dose" (and a bare "Dose") carry flag:"hi" to mark the prescribing
-// ceiling's importance, not a safety stop — render it as the primary/clinical
-// metric so red stays reserved for genuine risk signals (contraindications,
-// toxicity, teratogenicity), matching the #659 colour contract.
+// ceiling's importance, not a safety stop, so both tone and ordering treat it
+// specially — share one label test so the two stay aligned.
+function isMaxDoseLabel(label: string): boolean {
+  return /^(?:max\s+)?dose$/i.test(label.trim());
+}
+
+// Render Max Dose as the primary/clinical metric so red stays reserved for
+// genuine risk signals (contraindications, toxicity, teratogenicity), matching
+// the #659 colour contract.
 function heroMetricTone(stat: MedicationStat): SemanticTone {
-  if (/^(?:max\s+)?dose$/i.test(stat.label.trim())) return "clinical";
+  if (isMaxDoseLabel(stat.label)) return "clinical";
   return medicationStatTone(stat);
 }
 
@@ -343,8 +351,10 @@ function heroMetricTone(stat: MedicationStat): SemanticTone {
 // sentences (indication/dosing/contraindications) are not tiled here — they live
 // in the header subtitle and the detail sections instead.
 export function medicationHeroMetrics(record: MedicationRecord): MedicationHeroMetric[] {
-  const isMaxDose = (stat: MedicationStat) => /^(?:max\s+)?dose$/i.test(stat.label.trim());
-  const ordered = [...record.stats.filter(isMaxDose), ...record.stats.filter((stat) => !isMaxDose(stat))];
+  const ordered = [
+    ...record.stats.filter((stat) => isMaxDoseLabel(stat.label)),
+    ...record.stats.filter((stat) => !isMaxDoseLabel(stat.label)),
+  ];
   const metrics: MedicationHeroMetric[] = ordered.slice(0, 4).map((stat) => ({
     label: stat.label,
     value: shortValue(stat.value),
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index 4202f6e56..ba81d13a6 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -235,8 +235,11 @@ describe("shortValue", () => {
     expect(out).not.toMatch(/[\s,;:]…$/); // no space/punctuation immediately before the ellipsis
   });
 
-  it("keeps only the first clinical clause, protecting decimals and abbreviations", () => {
-    expect(shortValue("Start 1.5 mg NOCTE. Titrate weekly.", 40)).toBe("Start 1.5 mg NOCTE");
+  it("caps by length only — no sentence-splitting — so dotted abbreviations survive", () => {
+    // Stat values are curated tokens ("L.O.T. DRUG", "b.d."), not prose; splitting
+    // at an internal period would drop clinical label content.
+    expect(shortValue("L.O.T. DRUG")).toBe("L.O.T. DRUG");
+    expect(shortValue("Start 1.5 mg NOCTE. Titrate weekly.", 40)).toBe("Start 1.5 mg NOCTE. Titrate weekly.");
   });
 });
 
@@ -306,6 +309,15 @@ describe("medicationHeroMetrics", () => {
     expect(metrics[0].label).toMatch(/max dose/i);
     expect(metrics[0].tone).toBe("clinical");
   });
+
+  it("preserves dotted-abbreviation stat values from the snapshot (L.O.T. DRUG)", () => {
+    const record = getMedicationRecord("lorazepam");
+    expect(record).toBeTruthy();
+    const hepaticSafe = medicationHeroMetrics(record as MedicationRecord).find(
+      (metric) => metric.label === "Hepatic Safe",
+    );
+    expect(hepaticSafe?.value).toBe("L.O.T. DRUG");
+  });
 });
 
 describe("medicationIndication", () => {

From efbda6f18f50daa6c6da7ff63cd2d291bc086359 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 14:05:26 +0000
Subject: [PATCH 4/6] Use unique hero-metric tile keys for repeated stat labels
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Some records repeat a stat label (adrenaline-epinephrine has two "Route"
stats), so keying the hero tiles by label alone produced duplicate React keys
— risking tile reuse on client updates and a duplicate-key warning. Key by
`${label}-${index}` instead. Adds a snapshot test asserting the two Route
metrics both survive.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 .../clinical-dashboard/medication-record-page.tsx         | 6 ++++--
 tests/medications.test.ts                                 | 8 ++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/components/clinical-dashboard/medication-record-page.tsx b/src/components/clinical-dashboard/medication-record-page.tsx
index ee6653ff6..1b4f02e85 100644
--- a/src/components/clinical-dashboard/medication-record-page.tsx
+++ b/src/components/clinical-dashboard/medication-record-page.tsx
@@ -423,8 +423,10 @@ function MedicationRecordDetail({
           </section>
 
           <section className="grid grid-cols-2 gap-2.5 xl:grid-cols-4">
-            {metrics.map((metric) => (
-              <DetailTile key={metric.label} metric={metric} />
+            {metrics.map((metric, index) => (
+              // Some records repeat a stat label (e.g. adrenaline has two "Route"
+              // stats), so the label alone is not a unique key — include the index.
+              <DetailTile key={`${metric.label}-${index}`} metric={metric} />
             ))}
           </section>
 
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index ba81d13a6..41da224a7 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -318,6 +318,14 @@ describe("medicationHeroMetrics", () => {
     );
     expect(hepaticSafe?.value).toBe("L.O.T. DRUG");
   });
+
+  it("preserves repeated stat labels from the snapshot (adrenaline has two Route metrics)", () => {
+    // Duplicate labels mean the render must key tiles by index, not by label.
+    const record = getMedicationRecord("adrenaline-epinephrine");
+    expect(record).toBeTruthy();
+    const labels = medicationHeroMetrics(record as MedicationRecord).map((metric) => metric.label);
+    expect(labels.filter((label) => label === "Route")).toHaveLength(2);
+  });
 });
 
 describe("medicationIndication", () => {

From a1ec8b35be58b2a19b9e3c10bac8abdc0c410860 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 14:21:13 +0000
Subject: [PATCH 5/6] Narrow the genus-abbreviation guard in
 firstClinicalSentence
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The earlier genus fix skipped the sentence split after ANY lone letter before a
period, which over-merged legitimate single-letter sentence ends across ~62
snapshot rows (e.g. "Pregnancy Category D. ...", "Vitamin C. ...", "Penicillin
G. ..."). Restrict the skip to genus-species shape — a lone letter followed by a
lowercase species (H. pylori, E. coli, C. difficile) — by also checking the next
word starts lowercase. Adds counter-case tests that must still split.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 src/lib/medications.ts    | 11 +++++++----
 tests/medications.test.ts | 12 +++++++++++-
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/src/lib/medications.ts b/src/lib/medications.ts
index 9eea1c15f..70650dba9 100644
--- a/src/lib/medications.ts
+++ b/src/lib/medications.ts
@@ -181,10 +181,13 @@ export function firstClinicalSentence(value: string): string {
     if (next !== undefined && !/\s/.test(next)) continue;
     const before = text.slice(0, match.index);
     if (/(?:^|[\s(])(?:e\.g|i\.e|etc|vs|approx)$/i.test(before)) continue;
-    // Single-letter clinical abbreviations — genus abbreviations such as
-    // "H. pylori", "E. coli", "C. difficile" — are not sentence ends; splitting
-    // here would drop the rest of the clause (e.g. an indication list).
-    if (/(?:^|[\s(])[A-Za-z]$/.test(before)) continue;
+    // Genus abbreviations ("H. pylori", "E. coli", "C. difficile") are a lone
+    // letter followed by a lowercase species — not a sentence end; splitting
+    // there would drop the rest of the clause. Restrict the skip to that shape
+    // so genuine single-letter sentence ends ("Pregnancy Category D. ...",
+    // "Vitamin C. ...", "Penicillin G. ...") still split.
+    const nextWord = text.slice(match.index + 1).match(/^\s+(\S)/);
+    if (/(?:^|[\s(])[A-Za-z]$/.test(before) && nextWord && /[a-z]/.test(nextWord[1])) continue;
     return before.trim();
   }
   return text;
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index 41da224a7..7bab8bc37 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -212,13 +212,23 @@ describe("medication action tone", () => {
     expect(firstClinicalSentence("No trailing period")).toBe("No trailing period");
   });
 
-  it("does not cut at single-letter genus abbreviations", () => {
+  it("does not cut at single-letter genus abbreviations (lone letter + lowercase species)", () => {
     expect(firstClinicalSentence("Atypical CAP, H. pylori eradication, severe MAC complex in HIV.")).toBe(
       "Atypical CAP, H. pylori eradication, severe MAC complex in HIV",
     );
     expect(firstClinicalSentence("E. coli UTI. Second line.")).toBe("E. coli UTI");
     expect(firstClinicalSentence("C. difficile colitis (severe). Oral only.")).toBe("C. difficile colitis (severe)");
   });
+
+  it("still splits genuine single-letter sentence ends (lone letter + uppercase next word)", () => {
+    expect(firstClinicalSentence("ABSOLUTE — Pregnancy Category D. Associated with malformations.")).toBe(
+      "ABSOLUTE — Pregnancy Category D",
+    );
+    expect(firstClinicalSentence("Vitamin C. Essential antioxidant.")).toBe("Vitamin C");
+    expect(firstClinicalSentence("Depot formulation of Penicillin G. Slowly leaches.")).toBe(
+      "Depot formulation of Penicillin G",
+    );
+  });
 });
 
 describe("shortValue", () => {

From a0ca895015df8ebe6ae57fa8a811a1fbb240e623 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 14 Jul 2026 14:29:12 +0000
Subject: [PATCH 6/6] Keep multi-variant max-dose ceilings complete in hero
 tiles
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Two review fixes on the hero metrics:

- P1: `shortValue` truncation could drop clinically relevant dose variants from
  the Max Dose tile (e.g. buprenorphine-sl-depot "Buvidal 160 mg/month;
  Sublocade 300 mg/month" → "Buvidal 160 mg/month…", nicotine-gum's 4 mg
  variant). With the Key-stats sidebar removed, the full ceiling lived nowhere
  else. Never truncate the ceiling — only the two long values in the snapshot are
  Max Dose, and the short non-ceiling stats keep shortValue as a safety net.
- Minor: the Schedule backfill now also passes through shortValue, matching the
  Usual dose / Category backfills.

Adds snapshot + synthetic tests locking the ceiling-stays-complete behavior.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBemWnYhqdGXBzwJUTQAqe
---
 src/lib/medications.ts    |  8 ++++++--
 tests/medications.test.ts | 25 ++++++++++++++++++++-----
 2 files changed, 26 insertions(+), 7 deletions(-)

diff --git a/src/lib/medications.ts b/src/lib/medications.ts
index 70650dba9..7afd5c557 100644
--- a/src/lib/medications.ts
+++ b/src/lib/medications.ts
@@ -360,7 +360,11 @@ export function medicationHeroMetrics(record: MedicationRecord): MedicationHeroM
   ];
   const metrics: MedicationHeroMetric[] = ordered.slice(0, 4).map((stat) => ({
     label: stat.label,
-    value: shortValue(stat.value),
+    // Never truncate the prescribing ceiling: multi-variant max doses (e.g.
+    // "Buvidal 160 mg/month; Sublocade 300 mg/month") must stay complete, and the
+    // full value lives nowhere else now that the Key-stats sidebar is gone. Other
+    // stats are short curated tokens, so shortValue is only a safety-net cap.
+    value: isMaxDoseLabel(stat.label) ? stat.value.replace(/\*\*/g, "").trim() : shortValue(stat.value),
     tone: heroMetricTone(stat),
   }));
   if (metrics.length >= 4) return metrics;
@@ -368,7 +372,7 @@ export function medicationHeroMetrics(record: MedicationRecord): MedicationHeroM
   const usualDose = medicationUsualDose(record);
   const backfills: MedicationHeroMetric[] = [
     { label: "Usual dose", value: shortValue(usualDose), tone: "clinical" },
-    record.schedule ? { label: "Schedule", value: record.schedule, tone: "neutral" } : null,
+    record.schedule ? { label: "Schedule", value: shortValue(record.schedule), tone: "neutral" } : null,
     record.category ? { label: "Category", value: shortValue(record.category), tone: "neutral" } : null,
   ].filter((metric): metric is MedicationHeroMetric => metric !== null);
 
diff --git a/tests/medications.test.ts b/tests/medications.test.ts
index 7bab8bc37..22f075e1d 100644
--- a/tests/medications.test.ts
+++ b/tests/medications.test.ts
@@ -269,11 +269,11 @@ describe("medicationHeroMetrics", () => {
     expect(metrics.map((metric) => metric.tone)).toEqual(["clinical", "neutral", "warning", "success"]);
   });
 
-  it("caps to four metrics and shortens over-long values", () => {
+  it("caps to four metrics; keeps the ceiling full but shortens other over-long values", () => {
     const record = buildRecord({
       stats: [
-        { label: "Max Dose", value: "2 mg: max 20 pieces/day; 4 mg: max 10 pieces/day", cls: "hi", flag: "hi" },
-        { label: "Half-life", value: "13-28.4 h", cls: "", flag: "" },
+        { label: "Max Dose", value: "Buvidal 160 mg/month; Sublocade 300 mg/month", cls: "hi", flag: "hi" },
+        { label: "Note", value: "An unusually long caution token that should be capped", cls: "", flag: "" },
         { label: "Renal Adj.", value: "DOSE RED.", cls: "warn", flag: "warn" },
         { label: "Efficacy", value: "MODERATE", cls: "warn", flag: "" },
         { label: "Extra", value: "SHOULD NOT APPEAR", cls: "", flag: "" },
@@ -282,8 +282,14 @@ describe("medicationHeroMetrics", () => {
     const metrics = medicationHeroMetrics(record);
     expect(metrics).toHaveLength(4);
     expect(metrics.some((metric) => metric.label === "Extra")).toBe(false);
-    expect(metrics[0].value.endsWith("…")).toBe(true);
-    expect(metrics[0].value.length).toBeLessThanOrEqual(25);
+    // The prescribing ceiling is never truncated — the full multi-variant value stays.
+    expect(metrics.find((metric) => metric.label === "Max Dose")?.value).toBe(
+      "Buvidal 160 mg/month; Sublocade 300 mg/month",
+    );
+    // ...but other over-long stat values are still capped.
+    const note = metrics.find((metric) => metric.label === "Note");
+    expect(note?.value.endsWith("…")).toBe(true);
+    expect(note?.value.length ?? 0).toBeLessThanOrEqual(25);
   });
 
   it("marks genuine hi-risk stats as danger even though Max Dose is not", () => {
@@ -336,6 +342,15 @@ describe("medicationHeroMetrics", () => {
     const labels = medicationHeroMetrics(record as MedicationRecord).map((metric) => metric.label);
     expect(labels.filter((label) => label === "Route")).toHaveLength(2);
   });
+
+  it("keeps a multi-variant max-dose ceiling complete from the snapshot (not truncated)", () => {
+    // The ceiling's full value is not repeated elsewhere on the page, so it must
+    // not be clamped away — both depot brand ceilings must survive.
+    const record = getMedicationRecord("buprenorphine-sl-depot");
+    expect(record).toBeTruthy();
+    const maxDose = medicationHeroMetrics(record as MedicationRecord).find((metric) => /max dose/i.test(metric.label));
+    expect(maxDose?.value).toBe("Buvidal 160 mg/month; Sublocade 300 mg/month");
+  });
 });
 
 describe("medicationIndication", () => {