fix: defensive cohort fallbacks (normalized forge approval, undefined-safe verdict rank + actorCount)

Author: jddunn

src/dashboard/src/components/layout/VerdictModal.tsx

@@ -34,7 +34,14 @@ function CohortVerdictDetails({ v }: { v: Record<string, unknown> }) {
   const keyDivergence = String(v.keyDivergence || '');
   const rankings = (Array.isArray(v.rankings) ? v.rankings : []) as CohortRankingEntry[];
   const reasoning = String(v.reasoning || '');
-  const actorCount = Array.isArray(v.actors) ? (v.actors as unknown[]).length : rankings.length;
+  // Schema requires `rankings` to carry ≥ 2 entries, but a malformed
+  // verdict payload could still arrive empty; the floor of 1 keeps the
+  // kicker from rendering "Cohort of 0" which would never match the
+  // banner's actor count and looks like a bug to the user.
+  const actorCount = Math.max(
+    1,
+    Array.isArray(v.actors) ? (v.actors as unknown[]).length : rankings.length,
+  );
 
   return (
     <div className={styles.cohortDetails}>
@@ -57,11 +64,11 @@ function CohortVerdictDetails({ v }: { v: Record<string, unknown> }) {
             const color = getActorColorVar(r.actorIndex);
             return (
               <li
-                key={`${r.rank}-${r.actorName}`}
+                key={`${r.rank ?? 'unranked'}-${r.actorName}-${r.actorIndex}`}
                 className={styles.cohortRankingEntry}
                 style={{ '--actor-color': color } as CSSProperties}
               >
-                <div className={styles.cohortRankBadge}>#{r.rank}</div>
+                <div className={styles.cohortRankBadge}>#{r.rank ?? '?'}</div>
                 <div className={styles.cohortRankBody}>
                   <div className={styles.cohortRankName}>{r.actorName}</div>
                   <div className={styles.cohortRankScores}>

src/dashboard/src/components/viz/CohortSwarmGrid.tsx

@@ -85,15 +85,20 @@ function buildForgeFeedFor(events: ProcessedEvent[]): { attempts: ForgeAttempt[]
   for (const evt of events) {
     if (evt.type === 'forge_attempt') {
       const d = evt.data || {};
+      // Anthropic SSE payloads sometimes serialize the boolean as the
+      // string 'true' instead of `true`; the dashboard accepts both.
+      // Compute the normalized value once and reuse it for both the
+      // attempts entry and the firstByName lookup so they can't drift.
+      const isApproved = d.approved === true || d.approved === 'true';
       attempts.push({
         turn: Number(d.turn ?? 0),
         eventIndex: Number(d.eventIndex ?? 0),
         department: String(d.department || ''),
         name: String(d.name || ''),
-        approved: d.approved === true || d.approved === 'true',
+        approved: isApproved,
         confidence: typeof d.confidence === 'number' ? d.confidence : undefined,
       });
-      if (d.name && d.department && d.approved === true && !firstByName.has(String(d.name))) {
+      if (d.name && d.department && isApproved && !firstByName.has(String(d.name))) {
         firstByName.set(String(d.name), String(d.department));
       }
       continue;