board: respond gates on the review card format, and a review-loop timeline in every gate header

apps/board/AGENTS.md

@@ -43,10 +43,12 @@ resolves run to MR in the background; the request path never waits on it).
 
 ## Seeing every face
 
-`bun run storybook` from the repo root, then `Gates/Board/Gallery`: one
-story per gate kind and variant, built from real gate shapes with invented
-data (`gate-gallery.fixtures.ts`). Add a story when you add a gate kind or
-variant, with invented names only (the repo is public).
+`bun run storybook` from the repo root, then `Gates/Board`: four stories
+(review first round, review re-review, respond plan, respond post), each
+rebuilt from a real gate's shape with invented data
+(`gate-stories.fixtures.ts`). Keep it to gates a real run asks; a variant
+belongs in a DOM test, not a new story. Invented names only (the repo is
+public).
 
 ## Respond rounds
 

apps/board/package.json

@@ -27,6 +27,7 @@
     "@mattstack/tui-kit": "workspace:*",
     "@pierre/diffs": "catalog:",
     "invadrs": "^1.3.1",
+    "lucide-react": "catalog:",
     "motion": "14.2.0",
     "pngjs": "^7.0.0",
     "react": "catalog:",

apps/board/src/__tests__/respond-rounds.test.ts

@@ -7,8 +7,10 @@ import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
 import {
   attachRespondRounds,
   deleteRespondRound,
+  readRespondOutcomes,
   readRespondRounds,
   recordRespondRound,
+  reviewerOutcome,
 } from '../respond-rounds.ts';
 import { openStateDb } from '../state/db.ts';
 
@@ -31,6 +33,21 @@ describe('respond rounds', () => {
     expect(recordRespondRound(URL_A, 'john', 2, db)).toBe(2);
   });
 
+  test('outcomes come back in round order, null where none was recorded', () => {
+    recordRespondRound(URL_A, 'john', 1, db);
+    recordRespondRound(URL_A, 'john', 2, db, 'comment');
+    recordRespondRound(URL_A, 'john', 3, db, 'approve');
+    const out = readRespondOutcomes(db);
+    expect(out.get(URL_A)).toEqual({ john: [null, 'comment', 'approve'] });
+    expect(
+      attachRespondRounds([{ webUrl: URL_A }], readRespondRounds(db), out)[0]
+    ).toEqual({
+      webUrl: URL_A,
+      respondRounds: { john: 3 },
+      respondOutcomes: { john: [null, 'comment', 'approve'] },
+    });
+  });
+
   test('rounds are per person', () => {
     recordRespondRound(URL_A, 'john', 1, db);
     recordRespondRound(URL_A, 'john', 2, db);
@@ -71,3 +88,38 @@ describe('respond rounds', () => {
     expect(out[2]).toBe(none);
   });
 });
+
+describe('the outcome a response round records', () => {
+  const reviews = (r: object) => ({
+    reviews: { approvedBy: [], reviewers: [], ...r },
+  });
+  test('an approver who was never an assigned reviewer approved', () => {
+    expect(
+      reviewerOutcome(
+        reviews({ approvedBy: [{ username: 'renee' }] }) as never,
+        'renee'
+      )
+    ).toBe('approve');
+  });
+  test('an assigned reviewer whose state reads approved approved', () => {
+    expect(
+      reviewerOutcome(
+        reviews({
+          reviewers: [{ username: 'renee', reviewState: 'APPROVED' }],
+        }) as never,
+        'renee'
+      )
+    ).toBe('approve');
+  });
+  test('anyone else commented', () => {
+    expect(
+      reviewerOutcome(
+        reviews({
+          reviewers: [{ username: 'renee', reviewState: 'REVIEWED' }],
+        }) as never,
+        'renee'
+      )
+    ).toBe('comment');
+    expect(reviewerOutcome(undefined, 'renee')).toBe('comment');
+  });
+});

apps/board/src/__tests__/state-db.test.ts

@@ -132,7 +132,7 @@ describe('state db', () => {
     expect(again.query('SELECT COUNT(*) AS n FROM agent_states').get()).toEqual(
       { n: 1 }
     );
-    expect(SCHEMA_VERSION).toBe(5);
+    expect(SCHEMA_VERSION).toBe(6);
   });
 
   test('v5 adds respond_rounds, and a v4 db upgrades in place with its rows intact', () => {

apps/board/src/client/board/Board.tsx

@@ -2161,6 +2161,7 @@ export function Board() {
               onAnswered={retireActiveGate}
               onContinue={retireActiveGate}
               onLostChange={lost => queue.hold(lost ? activeGateId : null)}
+              self={seat}
               readOnly={gateReadOnlyReason(
                 queue.active.gate,
                 queue.active.mr,