@galda/cli 0.10.37 → 0.10.39

package/app/index.html

@@ -853,20 +853,15 @@
   .reqdesc .mdsp{height:6px}
   .reqdesc code,.gdtaskresult code{font-family:var(--mono);font-size:11px;background:rgba(var(--tint),.07);padding:1px 4px;border-radius:4px}
   .reqdesc strong{color:var(--ink);font-weight:700}
+  /* an image the AI pasted into its own report: shown full-width at its own aspect,
+     hairline edge only (§5.5 縁は1px以下のヘアライン), never a fixed-ratio crop box */
+  .mdimg{display:block;max-width:100%;height:auto;margin:8px 0;border-radius:8px;border:1px solid var(--hair)}
   .reqcond{font-family:var(--mono);font-size:11px;color:var(--ink3);margin-top:4px}
   .reqvideo{margin:10px 0}
   .reqnovideo{font-size:12px;color:var(--ink3);border:1px dashed var(--hair2);border-radius:10px;padding:16px;text-align:center;margin:10px 0}
   .beforeafter{display:grid;grid-template-columns:1fr 1fr;gap:8px}
   .ba-col{min-width:0}
   .balabel{display:block;font-family:var(--mono);font-size:9px;letter-spacing:.06em;color:var(--ink3);margin-bottom:2px}
-  /* diff box + zoomed inset (HANDOFF-review-card-diffbox-2026-07-18) — only emitted
-     by beforeAfterBoxedHtml(), so these never touch the plain pair above or the
-     older per-task renderReviewChecklist before/after (out of scope, untouched). */
-  .ba-imgwrap{position:relative;margin-top:8px}
-  .ba-imgwrap .proofimg{width:100%;height:auto;max-height:none;object-fit:initial;margin-top:0;display:block}
-  .ba-box{position:absolute;border:1.5px solid var(--danger);border-radius:2px;pointer-events:none}
-  .ba-zoomhd{margin-top:14px}
-  .lg-ev .ba-zoomimg{display:block;width:100%;height:auto;max-height:none;object-fit:initial;margin-top:8px;border-radius:9px;border:1px solid var(--hair2)}
   .reqbtns{display:flex;gap:8px;margin-top:4px}
   .reqbtn{height:30px;padding:0 14px;border-radius:9px;border:1px solid var(--hair2);background:transparent;
     color:var(--ink2);font:inherit;font-size:12.5px;font-weight:600;cursor:pointer;transition:.15s}
@@ -1620,6 +1615,14 @@
   .lbc.is-quiet .lbc-t{color:var(--ink2)}   /* Done / Later / To Do: visible, not shouting */
   .lbc-w{font:600 8.5px/1 var(--mono);letter-spacing:.11em;text-transform:uppercase;margin-left:auto}
   .lbc-w.bad{color:var(--danger)} .lbc-w.intr{color:#ffca16}
+  /* P0b: the AI's own To-do breakdown, scaled down to the card's density (§1.5-8 fold, not
+     scroll — capped + "+N more" text rather than an inner scrollbar). Reuses the .todo/.box/
+     .tdtext vocabulary from the chat feed (todosHtml) so the ✓/▸/○ language stays one system. */
+  .lbc-todos{display:flex;flex-direction:column;gap:3px}
+  .lbc-todos .todo{font-size:10.5px;line-height:1.4;gap:5px}
+  .lbc-todos .todo .box{width:11px;height:11px;font-size:9px;line-height:11px;margin-top:1px}
+  .lbc-todos.prep{font-size:10.5px;color:var(--ink3);font-style:italic}
+  .lbc-more{font:500 9.5px/1 var(--mono);color:var(--ink3)}
   .lbc-meter{display:flex;align-items:center;gap:7px}
   .lbc-meter .bar{flex:1;height:2px;border-radius:2px;background:color-mix(in srgb,var(--ink) 11%,transparent);overflow:hidden}
   .lbc-meter .bar i{display:block;height:100%;border-radius:2px;background:var(--green);transition:width .3s}
@@ -1712,33 +1715,12 @@
   /* review row number — #NN PREFIX on the title (HANDOFF-v45 §11), same form as the To Do
      rows (PRD §5: numbering matches the sidebar) */
   #fsRoot .rcard .rid{font-size:10.5px;font-weight:600;color:var(--ink3);margin:0 5px 0 0;display:inline-block;font-variant-numeric:tabular-nums}
-  /* PRD §6.5 fold note — on the carrier ("also carries #13") and on the folded card
-     ("merged into #12"). Deliberately NOT the .pr treatment: no tinted ground, no
-     badge (§1.5-9), no red — folding is not a failure. Mono + --ink3 = the same quiet
-     meta tone the rest of the lane uses (§5.5「品は罫線とトーン差・monoで出す」). */
-  /* display:block, not inline-block: it shares .rc with the PR pill, which is itself
-     inline-block — side by side the pill sat on top of the last "#NN" and ate it. */
-  #fsRoot .rcard .fnote{font-family:var(--mono);font-size:9.5px;color:var(--ink3);margin-top:3px;display:block}
-  /* In rv3 the meta sits on the near-black photo scrim whatever the theme, where --ink3
-     is effectively invisible (§5.5「見えるが騒がない」— near-invisible is a fail). Match
-     .rid's white-on-scrim treatment. The folded card opts out of the scrim (rule above),
-     so it keeps --ink3 on the panel where that tone is correct. */
-  #fsRoot[data-rv="3"] .rcard:not(.fold) .fnote{color:rgba(255,255,255,.66);text-shadow:0 1px 3px rgba(0,0,0,.55)}
-  /* A folded card is settled, not failed: title sinks to the settled tone, and it
-     keeps no ✓/✕ because the thing to approve lives on the goal it was folded into. */
-  #fsRoot .rcard.fold .t{color:var(--ink2)}
-  /* A folded card is a pointer, not a review item: no proof to show, nothing to approve.
-     So it opts OUT of the rv3 thumbnail treatment (whose .rc is absolutely positioned
-     over an 84px photo ground) and stays the same quiet line in every view — the form
-     Masa drew. Giving it a fake dark thumbnail would claim it has something to look at. */
-  /* .fold.noimg, not .fold: the placeholder-ground rule below is `.rcard.noimg`, which
-     ties this on specificity and would win on order — a folded card would keep the fake
-     dark thumbnail ground. Matching both classes settles it regardless of order. */
-  #fsRoot[data-rv="3"] .rcard.fold.noimg{display:flex;position:static;align-items:center;gap:10px;
-    padding:9px 10px;min-height:0;border:1px solid var(--hair);border-radius:11px;background:none;box-shadow:none}
-  #fsRoot[data-rv="3"] .rcard.fold.noimg .rc{position:static;padding:0;background:none}
-  #fsRoot[data-rv="3"] .rcard.fold.noimg .t{color:var(--ink2);font-size:12.5px;text-shadow:none}
-  #fsRoot[data-rv="3"] .rcard.fold.noimg .rid{color:var(--ink3)}
+  /* PRD §6.5 → Masa 2026-07-21: the fold note ("also carries #13" on the carrier card,
+     "merged into #12" on the folded card) no longer renders in the Review list at all —
+     it read as the same report-in-the-list clutter the feature was meant to avoid. Where
+     folded work went is now logged in the opened detail's "Also carries" section only, so
+     the .fnote / .rcard.fold* rules that used to style these list-level notes are gone
+     with them (dead CSS orphaned the moment nothing emits those classes any more). */
   /* reference numbers everywhere else (HANDOFF-v45 §11): a leading #NN prefix on every
      To Do item's name — same goal = same number (goalReviewNumber), distinct from the
      PR's own #id shown separately. */
@@ -1948,6 +1930,198 @@
   .sa-done .undo{margin-left:auto;border:0;background:none;color:var(--ink3);font:500 10px/1 var(--mono);cursor:pointer}
   .sa-done .undo:hover{color:var(--ink);background:transparent}
   .sa-empty{font:500 11px/1 var(--mono);color:var(--ink3);padding:20px 0}
+  /* ═══ Review card redesign (REVIEW-CARD-SPEC.md · Masa 2026-07-20 confirmed):
+     Linear-docs typography · META = green outline pill (Chip) · GitHub diff = Bright.
+     Ported from the flagship redesign (fs-redesign.css), the dev-only PLAN/META/DIFF
+     switchers resolved to the confirmed defaults plan=d / meta=a / gh=d, and the
+     flagship fallback hexes mapped to the app's real theme tokens (theme.css).
+     Scoped to .sa-item.redesign so the plain .sa-done one-liner is untouched. ═══ */
+  #seeall .sa-panel{width:min(940px,94%);max-width:940px;backdrop-filter:blur(30px) saturate(1.4);-webkit-backdrop-filter:blur(30px) saturate(1.4)}
+  /* body/header/card geometry ported verbatim from the artifact (fs-redesign.css plan=d):
+     the CARD carries the 40px side gutters + 28/34 top/bottom, the header matches (40/34),
+     and the See-all body drops its own gutter/gap/centering so the card padding is the
+     single source of the Linear-docs air. !important because the app's own .sa-item /
+     .sa-body / .lg rules (added first, same-or-higher specificity) otherwise win. */
+  #seeall .sa-body:has(.sa-item.redesign){gap:0 !important;align-items:stretch !important}
+  #seeall .sa-panel .sa-hd:has(~ .sa-body .sa-item.redesign){padding:16px 34px 12px 40px !important}
+  .sa-item.redesign{width:100% !important;background:none !important;border:0 !important;border-radius:0 !important;box-shadow:none !important;
+    padding:28px 40px 34px !important;font-family:var(--ui);color:var(--ink);line-height:1.6}
+  .sa-item.redesign *{box-sizing:border-box}
+  /* first card: no extra top margin — the card's own 28px padding-top (plus the
+     body's 16px) is the top air, matching the ~40px inter-card rhythm below.
+     The old 52px pushed the header→#N gap to ~96px of dead whitespace (Masa 2026-07-20). */
+  #seeall .sa-body>.sa-item.redesign:first-child{margin-top:0 !important}
+  #seeall .sa-body>.sa-item.redesign+.sa-item.redesign{border-top:1px solid var(--hair) !important;margin-top:0 !important;padding-top:40px !important}
+  /* reply composer: match the flagship saItem field (34px tall · radius 10 · 12px left pad)
+     so it doesn't read cramped next to the 32px Approve/Dismiss (Masa 2026-07-20).
+     padding-right (42px for the send ring) stays from .sa-chatwrap .sa-chat. */
+  #seeall .sa-chat{height:34px;border-radius:10px;padding-left:12px}
+  /* header — mono eyebrow · review dot · In review */
+  .redesign .rhead{display:flex;align-items:center;gap:12px;font-family:var(--mono);font-size:11px;color:var(--ink3)}
+  .redesign .rtag{letter-spacing:.02em;font-variant-numeric:tabular-nums}
+  .redesign .rst{display:inline-flex;align-items:center;gap:6px;color:var(--ink2)}
+  .redesign .rdot{width:6px;height:6px;border-radius:50%;background:var(--amber,#e0a021);display:inline-block}
+  .redesign .rtitle{font-family:var(--ui);font-size:18px;font-weight:680;letter-spacing:-.018em;line-height:1.3;margin:8px 0 0 !important;color:var(--ink)}
+  /* meta — Linear sans, green/neutral chips, hairline under (spec §4) */
+  .redesign .rmeta{display:flex;flex-wrap:wrap;align-items:center;gap:8px 12px;margin:12px 0 0 !important;padding-bottom:12px !important;
+    border-bottom:1px solid var(--hair);font-family:var(--ui);font-size:12px;letter-spacing:-.004em;color:var(--ink2)}
+  .redesign .rmeta .mx{display:inline-flex;align-items:center;gap:5px}
+  .redesign .rmeta .mk{color:var(--ink3)}
+  .redesign .rmeta b{color:var(--ink);font-weight:600}
+  .redesign .rmeta .mmut{color:var(--ink3);font-weight:400}
+  .redesign .rmeta .ar{font-size:.82em;margin-left:1px;color:var(--ink3)}
+  .redesign .rmeta .a{color:#3fd07a;font-style:normal} .redesign .rmeta .d{color:#ff7b72;font-style:normal}
+  /* Chip = outline pill (meta=a): green for PR/Token, neutral for Local */
+  .redesign .rmeta .mlink{cursor:pointer;padding:3px 10px;border-radius:999px;
+    border:1px solid color-mix(in srgb,currentColor 40%,var(--hair));text-decoration:none;background:none;font:inherit}
+  .redesign .rmeta .mlink:hover{background:color-mix(in srgb,currentColor 9%,transparent)}
+  .redesign .rmeta .mlink.pr,.redesign .rmeta .mlink.pr *{color:var(--green)}
+  /* whole PR chip bold, not just the number (REVIEW-CARD-SPEC §5) — scoped to .mlink.pr so Token/Local chips are unaffected */
+  .redesign .rmeta .mlink.pr,.redesign .rmeta .mlink.pr .mk,.redesign .rmeta .mlink.pr b,.redesign .rmeta .mlink.pr .mmut{font-weight:700}
+  .redesign .rmeta .mlink:not(.pr){color:var(--ink2)}
+  .redesign .rmeta .mlink:not(.pr) .mk{color:var(--ink3)}
+  .redesign .rmeta .mlink:not(.pr) b{color:var(--ink)}
+  .redesign .rmeta .mtok{cursor:pointer;padding:3px 10px;border-radius:999px;font:inherit;
+    border:1px solid color-mix(in srgb,var(--green) 40%,var(--hair));background:none}
+  .redesign .rmeta .mtok,.redesign .rmeta .mtok *{color:var(--green)}
+  .redesign .rmeta .mtok b{font-weight:700}
+  .redesign .rmeta .ok{color:var(--green)}
+  .redesign .rmeta .tokchevron{font-style:normal;font-size:11px;line-height:1;transition:transform .2s;opacity:.8}
+  .redesign .rmeta .mtok.open .tokchevron{transform:rotate(90deg)}
+  .redesign .rmeta-detail{display:none;margin:12px 0 0;padding:9px 12px;background:color-mix(in srgb,var(--ink) 4%,transparent);border-radius:8px}
+  .redesign .rmeta-detail.open{display:block}
+  .redesign .rmeta-detail .k{font:600 8px/1 var(--mono);letter-spacing:.1em;text-transform:uppercase;color:var(--ink3);margin-bottom:5px;display:block}
+  .redesign .rmeta-detail .v{font:500 11.5px/1.6 var(--ui);color:var(--ink2)}
+  /* Open it section: the address is the control. Mono so it reads as an address, plain
+     text weight so it doesn't shout, and hover moves the GLYPH colour only — never a
+     filled surface (§5.5 アイコンボタンのhoverは面を塗らない). */
+  .redesign .ropen .olink{display:inline-flex;align-items:baseline;gap:6px;padding:0;border:0;background:none;cursor:pointer;
+    font:400 13.5px/1.62 var(--mono);color:var(--ink);letter-spacing:-.004em;text-align:left;overflow-wrap:anywhere;transition:color .14s}
+  .redesign .ropen .olink:hover{color:var(--green)}
+  .redesign .ropen .olink[disabled]{color:var(--ink3);cursor:default}
+  .redesign .ropen .olink .ar{font-size:.85em;color:var(--ink3)}
+  /* the note is body prose like every other section's — same 14px / −.012em / --ink as
+     .rbody (Masa 2026-07-22: it was a shade lighter and read as a different kind of text) */
+  .redesign .ropen .v{margin:4px 0 0 !important}
+  /* sections — Linear-docs bold sentence-case headings + high-contrast body.
+     margins are !important: the app's pre-existing .lg / .sa-item rules load first and
+     otherwise tie/beat these on specificity, flattening the Linear-docs rhythm. */
+  .redesign .rsec{margin-top:24px !important}
+  .redesign .rsec-h{font-family:var(--ui);font-size:15px;font-weight:680;letter-spacing:-.012em;line-height:1.3;color:var(--ink);margin:0 0 9px !important}
+  /* body prose = the app's normal text: 14px / −.012em, matching .bubble (Masa 2026-07-20:
+     13px read thinner/lighter for Japanese than the surrounding app). §2. */
+  .redesign .rbody{font-size:14px;line-height:1.62;letter-spacing:-.012em;color:var(--ink);max-width:70ch;margin:0 !important}
+  .redesign .rdesc,.redesign .rrisk{color:var(--ink2)}
+  /* Result (P1, Masa 2026-07-21): the AI's own report, rendered as written. Same 14px
+     body rhythm as .rbody — it is prose the reviewer reads, not a log. Headings/lists/
+     code inside it are the report's own structure, so they get the section's type scale
+     rather than the smaller .reqdesc one used in the older checklist dialog. */
+  .redesign .rreport{font-size:14px;line-height:1.62;letter-spacing:-.012em;color:var(--ink);max-width:70ch;overflow-wrap:anywhere}
+  .redesign .rreport .mdh{font-weight:680;color:var(--ink);margin:14px 0 4px}
+  .redesign .rreport .mdh:first-child{margin-top:0}
+  .redesign .rreport .mdul{margin:2px 0;padding-left:20px}
+  .redesign .rreport .mdul li{margin:2px 0}
+  .redesign .rreport .mdsp{height:8px}
+  .redesign .rreport code{font-family:var(--mono);font-size:12px;background:rgba(var(--tint),.07);padding:1px 4px;border-radius:4px}
+  .redesign .rreport strong{color:var(--ink);font-weight:700}
+  .redesign .rreport a{color:var(--ink);text-decoration:underline;text-underline-offset:2px}
+  /* "Open it" is waiting on a real process to boot (up to ~15s), so the chip says so
+     rather than looking dead. Dim + no pointer, no spinner — §5.5 薄く・主張しない. */
+  .redesign .mx.mwait{opacity:.55;pointer-events:none}
+  .redesign .rfiles{margin:12px 0 0;font-family:var(--mono);font-size:12px;color:var(--ink2);line-height:1.75}
+  .redesign .rfiles .a{color:#3fd07a;font-style:normal}.redesign .rfiles .d{color:#ff7b72;font-style:normal}
+  /* Request accordion (spec §3) — chevron › rotate 90, body indented, no left rule */
+  .redesign .rreq .rsec-h{margin-bottom:6px !important}
+  .redesign .reqacc{margin-top:0 !important}
+  .redesign .racc>summary{list-style:none;cursor:pointer;display:flex;gap:10px;align-items:baseline}
+  .redesign .racc>summary::-webkit-details-marker{display:none}
+  .redesign .racc>summary::before{content:"›";font-size:1.05em;line-height:1;color:var(--ink2);display:inline-block;transition:transform .14s}
+  .redesign .racc[open]>summary::before{transform:rotate(90deg)}
+  .redesign .reqacc>summary{color:var(--ink);font-size:13.5px;line-height:1.55}
+  .redesign .reqbody{margin:14px 0 0 !important;padding-left:22px !important}
+  .redesign .rsub{font-family:var(--ui);font-size:11px;font-weight:600;color:var(--ink3);margin:16px 0 8px !important}
+  .redesign .reqbody>.rsub:first-child{margin-top:2px !important}
+  .redesign .reqbody .lg-bullets{margin:0;padding:0 0 0 16px;font-size:13px;line-height:1.65;color:var(--ink2)}
+  .redesign .reqbody .lg-bullets li{margin-bottom:5px}
+  .redesign .reqbody .lg-splitnote{background:color-mix(in srgb,var(--ink) 4%,transparent);border-radius:8px;padding:10px 12px;margin-top:12px}
+  .redesign .reqbody .lg-splitnote .hd{font:600 8.5px/1 var(--mono);letter-spacing:.08em;text-transform:uppercase;color:var(--ink3);margin-bottom:7px}
+  .redesign .reqbody .lg-splitnote ul{margin:0;padding:0;list-style:none;font-size:12px;line-height:1.8;color:var(--ink3)}
+  .redesign .reqbody .lg-splitnote li.cur{color:var(--ink)} .redesign .reqbody .lg-splitnote li.cur b{color:var(--green);font-weight:600}
+  /* workflow (spec §8) — vertical ✓ green / → green / □ dim */
+  .redesign .rwf{display:flex;flex-direction:column;gap:2px;margin:10px 0 0 !important;font-family:var(--ui);font-size:12px;letter-spacing:-.004em;color:var(--ink3)}
+  .redesign .rwf .wf-s{display:flex;align-items:center;gap:9px}
+  .redesign .rwf .wf-s .mk{width:12px;text-align:center;flex:0 0 12px}
+  .redesign .rwf .wf-s.ok{color:var(--ink2)}.redesign .rwf .wf-s.ok .mk{color:var(--green)}
+  .redesign .rwf .wf-s.now{color:var(--ink);font-weight:600}.redesign .rwf .wf-s.now .mk{color:var(--green);font-weight:700}
+  .redesign .rwf .wf-s.todo{color:var(--ink3)}.redesign .rwf .wf-s.todo .mk{color:var(--ink3);font-size:.82em}
+  /* proof (spec §6) — before/after forced same size · run-log · honest missing note */
+  .redesign .beforeafter{display:grid;grid-template-columns:1fr 1fr;gap:12px;max-width:640px;align-items:start}
+  .redesign .ba-col{display:flex;flex-direction:column;gap:6px;min-width:0}
+  .redesign .balabel{font-family:var(--mono);font-size:11px;letter-spacing:.04em;text-transform:uppercase;color:var(--ink3)}
+  .redesign .proofimg{display:block;width:100%;height:auto;max-height:none;object-fit:contain;border:1px solid var(--hair);border-radius:9px;cursor:zoom-in;margin:0}
+  .redesign .rproof{max-width:640px}
+  /* screen-recording player (spec §6) — sits under before/after, same 640px column */
+  .redesign .rvid{margin:12px 0 0;max-width:640px}
+  .redesign .rvid svg,.redesign .rvid .proofvid{display:block;width:100%;height:auto;border:1px solid var(--hair);border-radius:9px;background:#000}
+  .redesign .rvid .proofvid{aspect-ratio:16/9;object-fit:contain}
+  .redesign .rvid figcaption{margin-top:7px;font-family:var(--mono);font-size:11px;letter-spacing:.02em;color:var(--ink3)}
+  .redesign .rrun{border:1px solid var(--hair);border-radius:10px;overflow:hidden;font-family:var(--mono);font-size:12.5px;max-width:640px}
+  .redesign .rrun .rr{display:flex;justify-content:space-between;gap:16px;padding:10px 14px;border-top:1px solid var(--hair)}
+  .redesign .rrun .rr:first-child{border-top:0}
+  .redesign .rrun .rr .lab{color:var(--ink2)}.redesign .rrun .rr .v{color:var(--green);text-align:right}
+  .redesign .rmiss{display:flex;align-items:flex-start;gap:8px;font-size:12.5px;line-height:1.6;color:var(--ink2)}
+  .redesign .rmiss svg{width:15px;height:15px;flex:0 0 15px;margin-top:2px;fill:none;stroke:var(--amber);stroke-width:2;stroke-linecap:round;stroke-linejoin:round}
+  /* GitHub diff = Bright (gh=d): old|new gutters · +/− column · line+gutter tint */
+  .redesign .rdiff2{margin:12px 0 0;border:1px solid var(--hair);border-radius:10px;overflow:hidden;font-family:var(--mono);font-size:12px;line-height:1.6}
+  .redesign .ghl{display:grid;grid-template-columns:46px 46px 22px 1fr}
+  .redesign .ghl .gho,.redesign .ghl .ghn{color:var(--ink3);opacity:.6;text-align:right;padding:1px 8px;user-select:none;-webkit-user-select:none;font-variant-numeric:tabular-nums}
+  .redesign .ghl .ghm{text-align:center;color:var(--ink3);padding:1px 0}
+  .redesign .ghl .ghc{padding:1px 10px;white-space:pre-wrap;color:var(--ink)}
+  .redesign .gh-add{background:rgba(46,204,113,.22)}
+  .redesign .gh-add .gho,.redesign .gh-add .ghn{background:rgba(46,204,113,.30)}
+  .redesign .gh-add .ghm{color:#3fb950}
+  .redesign .gh-del{background:rgba(248,81,73,.20)}
+  .redesign .gh-del .gho,.redesign .gh-del .ghn{background:rgba(248,81,73,.30)}
+  .redesign .gh-del .ghm{color:#f85149}
+  .redesign .gh-hi{border-radius:2px;padding:0 1px}
+  .redesign .gh-add .gh-hi{background:rgba(46,204,113,.55)} .redesign .gh-del .gh-hi{background:rgba(248,81,73,.5)}
+  /* huge-diff footer (spec §7): "…+N 行 · GitHub で開く ↗" */
+  .redesign .ghfoot{margin-top:8px;font-family:var(--mono);font-size:11.5px;color:var(--ink3);display:flex;gap:2px;align-items:center;flex-wrap:wrap}
+  .redesign .ghfoot .ghopen{color:var(--ink2);text-decoration:none}
+  .redesign .ghfoot .ghopen:hover{color:var(--blue)}
+  /* meta links carry NO underline — PR is a real <a href> (spec §5, Masa 2026-07-20) */
+  .redesign .rmeta a{text-decoration:none}
+  /* Request / What-to-check = full contrast, same ink as the title (spec §2, no fade) */
+  .redesign .rask,.redesign .rcheck,.redesign .rcheck li,.redesign .rcheck1{color:var(--ink) !important}
+  .redesign ul.rcheck{margin:2px 0 0;padding:0 0 0 18px;font-size:14px;line-height:1.7;letter-spacing:-.012em}
+  .redesign ul.rcheck li{margin-bottom:6px}
+  /* Activity (spec §3) = ONE Linear-docs pulldown. Heading plain; chevron on the FIRST
+     line; tapping opens the rest below, offset-indented (no left rule); Discussion folds
+     in at the bottom of the opened body. */
+  .redesign .ract{margin-top:24px !important}
+  .redesign .ract>.rsec-h{margin-bottom:10px !important}
+  .redesign .lacc>summary{list-style:none;cursor:pointer;display:flex;gap:9px;align-items:baseline}
+  .redesign .lacc>summary::-webkit-details-marker{display:none}
+  .redesign .lacc .ltog{flex:0 0 auto;color:var(--ink3);font-style:normal;line-height:1;transition:transform .14s}
+  .redesign .lacc[open]>summary .ltog{transform:rotate(90deg)}
+  .redesign .lacc .lfirst{flex:1;min-width:0}
+  .redesign .lacc .lbody{margin-top:12px;padding-left:18px}
+  /* transcript rows — Linear-clean, borderless (You / tool mono / res green / Claude) */
+  .redesign .rtx{border:0;background:none}
+  .redesign .txr{display:flex;gap:12px;padding:5px 0;font-size:13px;line-height:1.55}
+  .redesign .txr .w{flex:0 0 46px;font-size:11px;color:var(--ink3);padding-top:1px}
+  .redesign .txr.you .w,.redesign .txr.cl .w{color:var(--ink);font-weight:600}
+  .redesign .txr .m{flex:1;min-width:0;color:var(--ink2)}
+  .redesign .txr.cl .m{color:var(--ink)}
+  .redesign .txr.tool .m{color:var(--ink2);font-family:var(--mono);font-size:12px}
+  .redesign .txr.tool .m b{color:var(--ink);font-weight:600}
+  .redesign .txr.res .m{font-family:var(--mono);font-size:12px;color:var(--green)}
+  /* Discussion (review back-and-forth) folded at the bottom of Activity's body */
+  .redesign .lbody .rthread{display:flex;flex-direction:column;gap:16px;margin:0}
+  .redesign .lbody .rmsg .who{font-size:12px;font-weight:600;color:var(--ink);margin-bottom:4px}
+  .redesign .lbody .rmsg .msg{font-size:13px;line-height:1.55;color:var(--ink2)}
+  /* sensitive-area note (real) sits below What changed */
+  .redesign .lg-risk{margin-top:20px}
   #saconf{position:fixed;inset:0;pointer-events:none;z-index:99;display:block}
   /* retest ×3 note surfaces on the lane review card too (v45 §5.3) */
   #fsRoot .rcard .rnote{display:block;font:500 9.5px/1.4 var(--mono);color:var(--green);margin-top:3px}
@@ -2834,7 +3008,7 @@
     <div class="reqlist" id="reviewList"></div>
     <div class="revactions">
       <button type="button" class="reqbtn approve" id="reviewApprove">Approve goal</button>
-      <button type="button" class="reqbtn dismiss" id="reviewDismiss">Reject</button>
+      <button type="button" class="reqbtn dismiss" id="reviewDismiss">Dismiss</button>
       <span style="flex:1"></span>
       <button class="mbtn ghost" id="reviewClose">Close</button>
     </div>
@@ -2904,7 +3078,7 @@
     </div>
     <div class="sa-act">
       <button type="button" class="sa-app" id="gdApprove"><svg viewBox="0 0 24 24"><path d="M5 12.5l4.2 4.2L19 7"/></svg>Approve</button>
-      <button type="button" class="sa-dis" id="gdDismiss">Reject</button>
+      <button type="button" class="sa-dis" id="gdDismiss">Dismiss</button>
       <div class="sa-chatwrap">
         <input type="text" class="sa-chat" id="gdRevisionInput" placeholder="Tell the worker what to try next…">
         <button type="button" class="sa-send" id="gdSend" title="Send" aria-label="Send"><svg viewBox="0 0 24 24"><path d="M12 19V5M5 12l7-7 7 7"/></svg></button>
@@ -2989,7 +3163,7 @@ const savedTheme = (() => {
   if (v !== s) localStorage.setItem('theme', v);
   return v;
 })();
-const state = { projects: [], goals: [], tasks: [], act: {}, actAt: {}, todos: {}, chatTurns: {}, active: 'galda2', editing: null, attach: [], connectCommand: '', models: ['sonnet'], agentModels: null, agentEfforts: null, agents: ['claude-code', 'codex'], expanded: new Set(), openThreads: new Set(), queueOrder: {}, externalActivity: [], selectedGoal: null, workflowColumns: {}, wfDraft: null, reviewDefinitions: {}, rdDraft: null, connectedApp: localStorage.getItem('sel:agent') || 'claude-code', reviewOpenGoal: null, reviewApprovals: {}, goalDetailOpen: null, qreplyOpen: new Set(), logCollapsed: localStorage.getItem('logCollapsed') !== '0', logEntries: [], layout: localStorage.getItem('layout') || 'flagship', theme: savedTheme, boardskin: localStorage.getItem('boardskin') || 'auto', summaryPattern: Number(localStorage.getItem('summaryPattern')) || 3, peraSpot: 0, skill: null, skillsCache: null,
+const state = { projects: [], goals: [], tasks: [], act: {}, actAt: {}, todos: {}, chatTurns: {}, active: 'default', editing: null, attach: [], connectCommand: '', models: ['sonnet'], agentModels: null, agentEfforts: null, agents: ['claude-code', 'codex'], expanded: new Set(), openThreads: new Set(), queueOrder: {}, externalActivity: [], selectedGoal: null, workflowColumns: {}, wfDraft: null, reviewDefinitions: {}, rdDraft: null, connectedApp: localStorage.getItem('sel:agent') || 'claude-code', reviewOpenGoal: null, reviewApprovals: {}, goalDetailOpen: null, qreplyOpen: new Set(), logCollapsed: localStorage.getItem('logCollapsed') !== '0', logEntries: [], layout: localStorage.getItem('layout') || 'flagship', theme: savedTheme, boardskin: localStorage.getItem('boardskin') || 'auto', summaryPattern: Number(localStorage.getItem('summaryPattern')) || 3, peraSpot: 0, skill: null, skillsCache: null,
   // per-message send overrides set via slash-commands (reset after each send,
   // except model which is sticky in localStorage). palette = open command list.
   msg: { mode: null, model: null, effort: null, skill: null, later: false, review: false },
@@ -3022,7 +3196,7 @@ function restoreFocusedAnswerInput(container, attr, saved){
 const STATUS_LABEL = {
   sending: 'Sending…', stacked: 'Queued', planning: 'Planning…', queued: 'To do', running: 'Working…', needsInput: 'Needs input',
   review: 'In review', done: 'Done', failed: 'Failed', partial: 'Partly failed', interrupted: 'Interrupted', skipped: 'Skipped',
-  rejected: 'Rejected',
+  rejected: 'Dismissed',
 };
 // Mirrors engine/lib.mjs goalChip() — kept in sync manually since this
 // inline script can't import server-side code. `columns` (a project's
@@ -3041,7 +3215,7 @@ const GOAL_CHIP = (s, columns, startedAt) => s === 'stacked' || s === 'sending'
   s === 'done' ? [pickColumnLabel(columns, 'done', 'DONE'), ''] :
   s === 'retesting' ? ['RE-TESTING', 'doing'] :
   s === 'reverted' ? ['REVERTED', ''] :
-  s === 'rejected' ? ['REJECTED', 'rejected'] :
+  s === 'rejected' ? ['DISMISSED', 'rejected'] :
   s === 'folded' ? ['FOLDED', ''] :
   s === 'blocked' ? ['BLOCKED', 'blocked'] : s === 'needsInput' ? ['NEEDS INPUT', 'blocked'] : ['FAIL', ''];
 const FINISHED = (s) => ['done', 'partial', 'failed', 'interrupted'].includes(s);
@@ -3196,16 +3370,6 @@ function reviewCardConfigFor(projectId){ return { ...DEFAULT_REVIEW_DEFINITION.r
 // goal's own request language, same as the planner/worker; ja/en pin the
 // project to one language regardless of what a given goal was typed in.
 function reviewLangFor(projectId){ const l = reviewDefinitionFor(projectId).language; return (l === 'en' || l === 'ja') ? l : 'auto'; }
-// PRD §6.5 fold note language (Masa 2026-07-17: "ユーザーの言語に合わせて" — not
-// hardcoded English). Mirrors the server's own 'auto' resolution exactly
-// (reviewDefinition.language === 'auto' ? detectRequestLanguage(goal.text) : ...,
-// engine/server.mjs) so this note reads in the same language as the review
-// headline/PR body for the same goal: the project's fixed override wins if set,
-// otherwise match the language the goal's own request was written in.
-function foldLangFor(goal){
-  const l = reviewLangFor(state.active);
-  return (l === 'ja' || l === 'en') ? l : (/[぀-ヿ㐀-鿿]/.test(goal?.text || '') ? 'ja' : 'en');
-}
 
 function esc(s){ const d = document.createElement('div'); d.textContent = s ?? ''; return d.innerHTML; }
 
@@ -3351,32 +3515,34 @@ function buildReviewDigest({ goals, tasks } = {}){
 // stays only as a last resort for a goal with no text. Pulled into one place so the
 // review card's own title and its sibling-goal titles (H23 split note) stay consistent.
 function saReviewTitle(g){
-  return String(g?.text || g?.plan?.[0] || g?.reviewSummary?.changed || '').replace(/\s+/g, ' ').trim().slice(0, 90);
-}
-// H23 (2026-07-18) evidence-kind detection. Mirrors engine/lib.mjs isUiChange() /
-// hasTestRelevantChanges() — kept in sync manually since the browser script isn't a
-// module. flagship's demo data carries an explicit `kind` field
-// ('image'|'logic'|'docs'|'missing'); real goals/tasks have no such field, so the
-// review card derives one from real signals it already has (a captured proof image,
-// whether changed files look like UI source, whether they're all docs-only). This
-// is a HEURISTIC, not a stored fact on the goal — flagged as a judgment call in the
-// H23 implementation PR for a human to correct if it misclassifies real data.
-const UI_TOUCH_RE_LOCAL = /^app\/|\.(?:html|css|tsx|jsx|vue|svelte)$/i;
-function isUiChangeLocal(files){ return (files ?? []).some((f) => UI_TOUCH_RE_LOCAL.test(String(f ?? ''))); }
+  // The goal's short ticket NAME wins everywhere it's shown (See-all title, Review-list
+  // rail, board, sibling list) so the title reads the same in every place and never
+  // repeats the full request (Masa 2026-07-21, spec §4). Real goals have no
+  // reviewSummary.title, so they fall back to the request text unchanged (no regression).
+  return String(g?.reviewSummary?.title || g?.text || g?.plan?.[0] || g?.reviewSummary?.changed || '').replace(/\s+/g, ' ').trim().slice(0, 90);
+}
+// H23 (2026-07-18) evidence-kind detection. flagship's demo data carries an explicit
+// `kind` field; real goals/tasks have no such field, so the review card derives one
+// from real signals it already has (a captured proof image, whether the changed files
+// are all docs-only).
+//
+// P1 (Masa 2026-07-21): the 'missing' kind is gone, and with it the app-local copy of
+// isUiChange(). It said "this touched the UI, so a screenshot SHOULD exist" — a rule
+// from the era when we photographed the work ourselves. That capture never ran outside
+// this one repo, so the only thing the rule ever produced was an apology on every UI
+// goal. We don't take the photo any more, so there is nothing to apologise for.
 const DOC_ONLY_RE_LOCAL = /(^|\/)(README|START-HERE|CHANGELOG|LICENSE)(\.[\w.-]+)?$|(^|\/)docs\/.*\.(md|mdx|txt)$|\.mdx?$|\.txt$/i;
 function hasTestRelevantChangesLocal(files){
   const fs = (files ?? []).map((f) => String(f ?? '').trim()).filter(Boolean);
   if (!fs.length) return true;
   return fs.some((f) => !DOC_ONLY_RE_LOCAL.test(f));
 }
-// Priority: image (a real capture exists) > missing (UI touched, should have a
-// capture per PRD §4(C) but doesn't) > docs (every changed file is docs-only) >
+// Priority: image (a real capture exists) > docs (every changed file is docs-only) >
 // logic (verified only by tests, no UI). Falls back to 'image' (today's generic
 // fact-row + no image) when no signal is conclusive — never claims a stronger kind
 // than the data actually supports.
 function saEvidenceKind(filePaths, hasProofImg, testResult){
   if (hasProofImg) return 'image';
-  if (isUiChangeLocal(filePaths)) return 'missing';
   if (filePaths.length && !hasTestRelevantChangesLocal(filePaths)) return 'docs';
   if (testResult?.ran) return 'logic';
   return 'image';
@@ -3427,11 +3593,28 @@ function requirementSummaryLine(task){
 // Worker reports are written in Markdown (## headings, **bold**, - lists, `code`).
 // The review cards showed them raw, so "## " and "**" leaked as literal text.
 // Render the common subset to HTML (escaping first) so they read cleanly.
-function mdLite(src){
+// P1 (Masa 2026-07-21): the AI's report IS the deliverable, so an image it pasted
+// (`![alt](url)`) has to render as that image — this is the whole "paste it and it
+// shows up" wiring. Only src values we can vouch for are allowed through: http(s),
+// a same-origin path, or an inline data:image. Anything else (javascript:, data:text
+// /html, …) renders as the literal text it was, never as an <img src>.
+const MD_IMG_SRC_OK = /^(?:https?:\/\/|\/(?!\/)|data:image\/(?:png|jpe?g|gif|webp|svg\+xml);base64,)/i;
+// `images: false` renders a pasted image as nothing at all. Masa 2026-07-22, after
+// seeing one in a real card: the review card is not a gallery — the way to see the work
+// is Open it, and a lone blurry still of "one of the three" is worse than no still. The
+// renderer keeps the capability (chat and the older checklist still use images); the
+// review card is simply not the place.
+function mdLite(src, { images = true } = {}){
   const esc = (s) => s.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
   const inline = (s) => esc(s)
     .replace(/`([^`]+)`/g, '<code>$1</code>')
     .replace(/\*\*([^*]+?)\*\*/g, '<strong>$1</strong>')
+    // images before links — ![alt](url) would otherwise match the link rule below
+    // and render as a bare "!" plus an <a>.
+    .replace(/!\[([^\]]*)\]\(([^)\s]+)\)/g, (m, alt, url) => (!images ? ''
+      : MD_IMG_SRC_OK.test(url)
+      ? `<img class="mdimg" src="${url.replace(/"/g, '&quot;')}" alt="${alt}" loading="lazy">`
+      : m))
     // Feedback: make links tappable — markdown [text](url) and bare URLs (e.g. a
     // "open the ReadMe/PROOF.md" link in the worker report).
     .replace(/\[([^\]]+)\]\((https?:\/\/[^)]+)\)/g, '<a href="$2" target="_blank" rel="noopener">$1</a>')
@@ -3571,106 +3754,15 @@ function proofMediaHtml(taskId, proof, altLabel){
   }
   return imgTag;
 }
-// reviewCard.beforeAfter: side-by-side comparison when a "before" capture
-// exists. Proof today never carries one (only a single post-change
-// snapshot/run is captured) — so this falls back to the normal single-shot
-// view rather than showing an empty/broken "Before" pane.
-function beforeAfterMediaHtml(taskId, proof, altLabel){
-  if (!proof) return '';
-  const beforeImg = proof.beforePng ?? proof.beforeGif;
-  if (!beforeImg) return proofMediaHtml(taskId, proof, altLabel);
-  const beforeTag = `<img class="proofimg" src="${withKey(`/proof/${taskId}/${beforeImg}`)}" alt="before: ${altLabel}" loading="lazy">`;
-  return `<div class="beforeafter">
-    <div class="ba-col"><span class="balabel">Before</span>${beforeTag}</div>
-    <div class="ba-col"><span class="balabel">After</span>${proofMediaHtml(taskId, proof, altLabel)}</div>
-  </div>`;
-}
-// Diff box + zoomed inset (HANDOFF-review-card-diffbox-2026-07-18): on top of the
-// plain before/after pair above, isolate the region that actually changed and draw
-// a red box over it on both images, plus a cropped/enlarged row below. Computed
-// client-side from the two PNGs via <canvas> pixel comparison at render time — not
-// baked in at capture time — so it works retroactively on every proof pair already
-// on disk with no new server-side image dependency (see the handoff for the full
-// rationale). Reference algorithm, verified against a real before/after pair
-// (composer icon swap, 640x200): correctly isolated a 57x52px box centered on the
-// changed icon — 0.13% image coverage, no false positives elsewhere in the frame.
-function computeDiffBox(beforeImg, afterImg, { threshold = 24, minCoverage = 0.0005, maxCoverage = 0.7, pad = 0.03 } = {}) {
-  const w = beforeImg.naturalWidth, h = beforeImg.naturalHeight;
-  if (w !== afterImg.naturalWidth || h !== afterImg.naturalHeight) return null; // different-shaped images: don't guess
-  const c1 = document.createElement('canvas'); c1.width = w; c1.height = h;
-  const c2 = document.createElement('canvas'); c2.width = w; c2.height = h;
-  c1.getContext('2d').drawImage(beforeImg, 0, 0, w, h);
-  c2.getContext('2d').drawImage(afterImg, 0, 0, w, h);
-  const d1 = c1.getContext('2d').getImageData(0, 0, w, h).data;
-  const d2 = c2.getContext('2d').getImageData(0, 0, w, h).data;
-  let minX = w, minY = h, maxX = -1, maxY = -1, diffPx = 0;
-  for (let y = 0; y < h; y++) {
-    for (let x = 0; x < w; x++) {
-      const i = (y * w + x) * 4;
-      const d = Math.abs(d1[i] - d2[i]) + Math.abs(d1[i + 1] - d2[i + 1]) + Math.abs(d1[i + 2] - d2[i + 2]);
-      if (d > threshold) {
-        diffPx++;
-        if (x < minX) minX = x; if (x > maxX) maxX = x;
-        if (y < minY) minY = y; if (y > maxY) maxY = y;
-      }
-    }
-  }
-  const total = w * h;
-  if (maxX < 0 || diffPx / total < minCoverage) return null; // no real difference — don't box a no-op
-  if (diffPx / total > maxCoverage) return null; // diff too diffuse to be "a spot" — a full redesign shouldn't get a red box either
-  const padPx = Math.round(Math.max(w, h) * pad);
-  minX = Math.max(0, minX - padPx); minY = Math.max(0, minY - padPx);
-  maxX = Math.min(w, maxX + padPx); maxY = Math.min(h, maxY + padPx);
-  return { x: minX, y: minY, w: maxX - minX, h: maxY - minY };
-}
-function baLoadImage(src){
-  return new Promise((resolve, reject) => {
-    const im = new Image();
-    im.onload = () => resolve(im);
-    im.onerror = () => reject(new Error(`image failed to load: ${src}`));
-    im.src = src;
-  });
-}
-// Crop {x,y,w,h} out of a decoded image and scale it up into a small inset — same
-// aspect ratio as the box, never stretched. Magnification is capped at 6x so a
-// near-single-pixel box doesn't blow up into mush.
-function baZoomDataUrl(img, box, maxW = 420, maxH = 200){
-  const scale = Math.min(maxW / box.w, maxH / box.h, 6);
-  const outW = Math.max(1, Math.round(box.w * scale));
-  const outH = Math.max(1, Math.round(box.h * scale));
-  const c = document.createElement('canvas'); c.width = outW; c.height = outH;
-  c.getContext('2d').drawImage(img, box.x, box.y, box.w, box.h, 0, 0, outW, outH);
-  return c.toDataURL('image/png');
-}
-// Async: loads the before/after PNGs, runs computeDiffBox(), and — only when a real
-// box is found — renders the before/after pair with a red box overlay on each image
-// (percent-positioned so it tracks the image's displayed size at any viewport width)
-// plus a "Zoomed on the boxed area" row below. Returns null on every fallback
-// condition (no PNG pair, load failure, no clean box) so the caller keeps showing
-// the plain beforeAfterMediaHtml() pair unchanged — never a guessed or empty box
-// (HANDOFF-review-card-diffbox-2026-07-18). Only kind==='image' Ledger evidence
-// (saEvidenceHtml) calls this; the older per-task renderReviewChecklist before/after
-// is explicitly out of scope for this handoff and stays on beforeAfterMediaHtml.
-async function beforeAfterBoxedHtml(taskId, proof, altLabel){
-  const beforeSrc = proof?.beforePng, afterSrc = proof?.png;
-  if (!beforeSrc || !afterSrc) return null; // the box needs two real stills, not a GIF frame
-  let beforeImg, afterImg;
-  try {
-    [beforeImg, afterImg] = await Promise.all([
-      baLoadImage(withKey(`/proof/${taskId}/${beforeSrc}`)),
-      baLoadImage(withKey(`/proof/${taskId}/${afterSrc}`)),
-    ]);
-  } catch { return null; } // a missing/broken file is a fallback, not a crash
-  const box = computeDiffBox(beforeImg, afterImg);
-  if (!box) return null;
-  const w = beforeImg.naturalWidth, h = beforeImg.naturalHeight;
-  const boxStyle = `left:${(box.x / w * 100).toFixed(3)}%;top:${(box.y / h * 100).toFixed(3)}%;width:${(box.w / w * 100).toFixed(3)}%;height:${(box.h / h * 100).toFixed(3)}%`;
-  const frame = (img, label) => `<div class="ba-col"><span class="balabel">${label}</span><div class="ba-imgwrap"><img class="proofimg" src="${img.src}" alt="${label.toLowerCase()}: ${altLabel}" loading="lazy"><div class="ba-box" style="${boxStyle}"></div></div></div>`;
-  const zoomCol = (dataUrl, label) => `<div class="ba-col"><img class="ba-zoomimg" src="${dataUrl}" alt="zoomed ${label.toLowerCase()}: ${altLabel}"></div>`;
-  return `<div class="beforeafter">${frame(beforeImg, 'Before')}${frame(afterImg, 'After')}</div>
-    <div class="lg-evhd ba-zoomhd">Zoomed on the boxed area</div>
-    <div class="beforeafter ba-zoomrow">${zoomCol(baZoomDataUrl(beforeImg, box), 'before')}${zoomCol(baZoomDataUrl(afterImg, box), 'after')}</div>`;
-}
+// (P1, Masa 2026-07-21) beforeAfterMediaHtml() was deleted here. The pair was staged by
+// us, not by the work: the "before" shot came from the same photo-staging pass that P1
+// removed on the engine side (#228), and in practice it never existed at all, so every
+// card fell through to the single shot anyway. Callers now ask for that shot directly.
+// (P1, Masa 2026-07-21) The before/after diff-box machinery (computeDiffBox,
+// baLoadImage, baZoomDataUrl, beforeAfterBoxedHtml) was deleted here. It pixel-diffed a
+// "before" capture against an "after" one to draw a red box around what changed — real
+// craft, built on a pair that the engine no longer stages (#228) and that never existed
+// outside this repo anyway. With no before shot there is nothing to compare.
 // swap the still for the real video only on demand, one at a time
 document.addEventListener('click', (e) => {
   const w = e.target.closest('.proofwrap');
@@ -3735,15 +3827,28 @@ function progressMeta(t){
 // checked + struck through, the in-progress one is highlighted, pending ones
 // are dimmed. Empty list → nothing (workers without todos look unchanged).
 const TODO_MARK = { completed: '✓', in_progress: '▸', pending: '○' };
-function todosHtml(t){
-  const list = state.todos[t.id] ?? [];
-  if (!list.length) return '';
-  const rows = list.map((d) => {
+// P0b (Masa 2026-07-21, planner demoted): the AI's own TodoWrite list is now the
+// only breakdown a goal has (1 goal = 1 task, no invented sub-tasks). `state.todos`
+// mirrors it live via the 'todos' SSE event; once the task finishes, the server
+// freezes the same shape into `task.todosFinal` (engine/server.mjs) since the live
+// field gets stripped off finished tasks. This is the one place that resolves
+// "which list to show" — live wins while it has content, else the frozen one.
+function todoSourceFor(t){
+  const live = state.todos[t?.id];
+  if (Array.isArray(live) && live.length) return live;
+  return Array.isArray(t?.todosFinal) ? t.todosFinal : [];
+}
+function todoRowsHtml(list){
+  return list.map((d) => {
     const st = ['completed', 'in_progress', 'pending'].includes(d.status) ? d.status : 'pending';
     const label = st === 'in_progress' && d.activeForm ? d.activeForm : d.content;
     return `<div class="todo ${st}"><span class="box">${TODO_MARK[st]}</span><span class="tdtext">${esc(label ?? '')}</span></div>`;
   }).join('');
-  return `<div class="todos"><div class="tdhead">To-do</div>${rows}</div>`;
+}
+function todosHtml(t){
+  const list = todoSourceFor(t);
+  if (!list.length) return '';
+  return `<div class="todos"><div class="tdhead">To-do</div>${todoRowsHtml(list)}</div>`;
 }
 
 function taskBlock(t){
@@ -4217,7 +4322,6 @@ function renderTasks(){
   const digestRows = buildReviewDigest({ goals: pgoals, tasks: list });
   const digestCount = digestRows.length;
   const dGreen = digestRows.filter((r) => !r.testResult || r.testResult.ok !== false).length;
-  const dProof = digestRows.filter((r) => r.proof).length;
   const dPct = digestCount ? Math.round((dGreen / digestCount) * 100) : 0;
   const dBlocked = pgoals.filter((g) => g.status === 'blocked').length; // §10: reflect real blocked count, don't hardcode
   // CDO review 2026-07-08 §6: dGreen/digestCount already exclude blocked goals (they
@@ -4235,7 +4339,7 @@ function renderTasks(){
     ? `<div class="s5sum" id="summaryEntry" title="Open the review summary (ペライチ)">
         <span class="s5exp"><svg width="13" height="13" viewBox="0 0 24 24"><path d="M15 4h5v5M20 4l-7 7M9 20H4v-5M4 20l7-7"/></svg></span>
         <span class="s5ring" style="background:conic-gradient(var(--green) 0 ${dPct}%, var(--hair2) ${dPct}% 100%)"><span class="s5in">${dGreen}/${digestCount}</span></span>
-        <span class="s5tx"><span class="s5hl">${dAllClear ? 'Ready to review' : `${dGreen} of ${digestCount} ready`}</span><span class="s5sub">${dAllClear ? 'all green' : dGreen === digestCount ? 'blocked needs attention' : 'some failing'} · ${dProof} with proof · ${dBlocked ? `${dBlocked} blocked` : 'none blocked'}</span></span>
+        <span class="s5tx"><span class="s5hl">${dAllClear ? 'Ready to review' : `${dGreen} of ${digestCount} ready`}</span><span class="s5sub">${dAllClear ? 'all green' : dGreen === digestCount ? 'blocked needs attention' : 'some failing'} · ${dBlocked ? `${dBlocked} blocked` : 'none blocked'}</span></span>
       </div>`
     : '';
   const REVIEW_LIMIT = 5;
@@ -4249,16 +4353,12 @@ function renderTasks(){
     // Feedback 3: the row carries only the check-line + PR link. Tests/proof are
     // internal gate checks (they still block), NOT list clutter; the Review number
     // is redundant with the title; tapping the row opens the detail (no button).
-    // PRD §6.5: goals that landed on the same files were folded onto this one, so
-    // they ride this PR instead of opening a second one editing the same lines.
-    // Whoever opens this review must know it answers more than one request.
-    // Only rendered when it actually happened (§1.5 art.3 — no empty container).
-    const carried = pgoals.filter((x) => x.foldedInto === g.id);
-    const carryNote = carried.length ? (foldLangFor(g) === 'ja'
-      ? `+ ${carried.map((x) => `#${x.id}`).join(', ')} の仕事もここに入っています`
-      : `also carries ${carried.map((x) => `#${x.id}`).join(', ')}`) : '';
+    // PRD §6.5 → Masa 2026-07-21: goals folded onto this one are NOT announced here
+    // any more ("+ #901 の仕事もここに入っています" read as the same report-in-the-list
+    // clutter as the old "→ #12 にまとめました" row — Masa: review欄にいらない). Where
+    // the work went is logged inside the opened detail's "Also carries" section
+    // (see saItemHtml's carriesBody) instead — the list row stays just the ask.
     return `<div class="task revrow" data-goal="${g.id}"><div class="ic pending"></div><div class="tx"><div class="name">${esc(checkLine)} ${goalSourceBadge(g.source)}</div>
-     ${carryNote ? `<div class="cond carries">${carryNote}</div>` : ''}
      <div class="cond">${g.pr ? `<a class="prbig" href="${esc(g.pr)}" target="_blank">PR #${esc((g.pr.match(/pull\/(\d+)/) ?? [,''])[1])} ↗</a>` : ''}</div>
      <div class="qreplyrow" data-qreplyrow="${g.id}"${state.qreplyOpen.has(g.id) ? '' : ' hidden'}><input type="text" data-qreplyinput="${g.id}" placeholder="直したい点を書いて送信（Todoに戻して再作業）"><button type="button" class="qreplysend" data-qreplysend="${g.id}" aria-label="Send">${ICONS.up}</button></div></div>
      <div class="revquick">
@@ -4268,16 +4368,10 @@ function renderTasks(){
      </div></div>`;
   }).join('')
     + (reviewGoalsAll.length > REVIEW_LIMIT ? `<div class="more" data-reviewmore="1">${state.showAllReview ? 'show less' : `more · +${reviewGoalsAll.length - REVIEW_LIMIT}`}</div>` : '');
-  // PRD §3.5: a folded goal is never deleted — deleting it just moves the question
-  // from "why is my finished work stuck" to "where did #13 go". It keeps its row and
-  // says where its work went. Not a failure: no red, no painted surface (§1.5 art.8);
-  // no ✓/✕ either, because the thing to approve lives on the goal it was folded into.
-  const foldedRows = pgoals.filter((g) => g.status === 'folded').map((g) => {
-    const checkLine = (g.reviewSummary?.check || g.plan?.[0] || g.text || '').replace(/\s+/g, ' ').slice(0, 66);
-    const mergedNote = foldLangFor(g) === 'ja' ? `→ #${esc(String(g.foldedInto))} にまとめました` : `merged into #${esc(String(g.foldedInto))}`;
-    return `<div class="task revrow foldrow" data-goal="${g.id}"><div class="ic pending"></div><div class="tx"><div class="name">${esc(checkLine)} ${goalSourceBadge(g.source)}</div>
-     <div class="cond">${mergedNote}</div></div></div>`;
-  }).join('');
+  // PRD §3.5 → Masa 2026-07-21: a folded goal no longer gets its own "→ #12 にまとめました"
+  // row in the Review column (Masa: review欄にいらない・わかりにくい). It is never deleted —
+  // where its work went is now logged inside the carrier review's detail ("Also carries",
+  // see saItemHtml), so the column stays clean while the trail is still one tap away.
   const doneRows = grouped(list.filter((t) => ['done', 'skipped'].includes(t.status)).sort((a, b) => b.num - a.num), (t) => row(t));
   // Pending = deliberately shelved goals (e.g. "npm公開はPhase 2"): never
   // planned or queued until Start is pressed. Parked below To do on purpose.
@@ -4289,7 +4383,7 @@ function renderTasks(){
   const savedQAns = captureFocusedAnswerInput($('tasklist'), 'data-qansinput');
   $('tasklist').innerHTML =
     sec('Attention', attention, 'attn')
-    + sec(pickColumnLabel(cols, 'review', 'Review'), reviewDigest + review + foldedRows, 'reviewsec', reviewDefinitionFor(state.active).description, REVIEW_DEF_HINT)
+    + sec(pickColumnLabel(cols, 'review', 'Review'), reviewDigest + review, 'reviewsec', reviewDefinitionFor(state.active).description, REVIEW_DEF_HINT)
     + sec(pickColumnLabel(cols, 'doing', 'Doing'), doing)
     + sec(pickColumnLabel(cols, 'todo', 'To do'), todo, 'todosec')
     + sec('Pending', pendingRows, 'dim')
@@ -4370,7 +4464,7 @@ function renderTasks(){
     // no auto-rework. Rework is asked for via chat.
     b.onclick = async (e) => { e.stopPropagation();
       let r; for (const id of b.dataset.qdismiss.split(',')) r = await fetch(withKey(`/api/goals/${id}/reject`), { method: 'POST' });
-      qrefresh(r, 'Reject failed.'); };
+      qrefresh(r, 'Dismiss failed.'); };
   }
   // Archive: retire an unfinished goal cleanly (no rework spawned). The escape
   // hatch for half-finished (partial/blocked) items so Attention stays drainable.
@@ -4741,9 +4835,6 @@ function renderRdEditor(){
     <label class="rdrow"><input type="checkbox" id="rdCardScreenshots"${state.rdDraft.reviewCard.screenshots ? ' checked' : ''}>
       <span><span class="rdlabel">スクショ / Screenshots</span>
       <span class="rdhint">レビューカードに証跡(スクショ/GIF)を表示する。</span></span></label>
-    <label class="rdrow"><input type="checkbox" id="rdCardBeforeAfter"${state.rdDraft.reviewCard.beforeAfter ? ' checked' : ''}>
-      <span><span class="rdlabel">Before/After</span>
-      <span class="rdhint">変更前後を並べて表示する(変更前キャプチャが無い場合は通常表示になります)。</span></span></label>
 
     <div class="rdsub">出力言語 / Output language</div>
     <label class="rdrow"><select id="rdLanguage">
@@ -4764,7 +4855,6 @@ function renderRdEditor(){
   $('rdPrScreenshots').addEventListener('change', (e) => { state.rdDraft.prSections.screenshots = e.target.checked; });
   $('rdPrRisk').addEventListener('change', (e) => { state.rdDraft.prSections.risk = e.target.checked; });
   $('rdCardScreenshots').addEventListener('change', (e) => { state.rdDraft.reviewCard.screenshots = e.target.checked; });
-  $('rdCardBeforeAfter').addEventListener('change', (e) => { state.rdDraft.reviewCard.beforeAfter = e.target.checked; });
   $('rdLanguage').addEventListener('change', (e) => { state.rdDraft.language = e.target.value; });
 }
 function renderWfEditor(){
@@ -4958,10 +5048,8 @@ function renderReviewChecklist(){
   $('reviewDoneMsg').classList.toggle('show', goal.status === 'done');
   // task 97: doneカードの『元に戻す』——押し間違いのApproveを取り消し、reviewへ戻す。
   $('reviewUndoBtn').onclick = () => revertDoneGoal(goal);
-  // reviewCard.screenshots off → the media block (and its "No capture yet"
-  // placeholder) is skipped entirely, not just emptied. reviewCard.beforeAfter
-  // on → try the before/after comparison (falls back to the single shot when
-  // there's no "before" capture — see beforeAfterMediaHtml).
+  // reviewCard.screenshots off → the media block is skipped entirely, not just
+  // emptied. (The Before/After switch is gone with the staged pair it controlled — P1.)
   const rc = reviewCardConfigFor(state.active);
   $('reviewList').innerHTML = reqs.map((t, i) => {
     const st = approvals[t.id];
@@ -4972,13 +5060,14 @@ function renderReviewChecklist(){
         <div class="reqhead"><span class="tick">✓</span><span class="reqnum">${t.num ?? i + 1}</span><span class="reqtitle">${esc(t.title ?? '')}</span><span class="apvd">Approved · tap to reopen</span></div>
       </div>`;
     }
-    const mediaInner = rc.beforeAfter ? beforeAfterMediaHtml(t.id, t.proof, esc(t.title ?? '')) : proofMediaHtml(t.id, t.proof, esc(t.title ?? ''));
-    const media = mediaInner || '<div class="reqnovideo">No capture yet</div>';
+    // P1: one shot when a promised check actually captured one, otherwise nothing at
+    // all — "No capture yet" was us apologising for a photo we no longer go and take.
+    const media = proofMediaHtml(t.id, t.proof, esc(t.title ?? ''));
     // task 156/234 + 160: show the WORKER'S REPORT (what it actually did/found,
     // incl. any test-only verification it describes) — that's what you read to
     // decide. Falls back to the instruction if the worker left no result.
     const report = String(t.result ?? '').replace(/^検証(PASS|FAIL)[^\n]*\n+/, '').replace(/\s+$/, '').trim();
-    const desc = report ? `<div class="reqdesc"><span class="reqreportlbl">Worker report</span>${mdLite(report.slice(0, 1600))}</div>`
+    const desc = report ? `<div class="reqdesc"><span class="reqreportlbl">Worker report</span>${mdLite(report)}</div>`
       : t.detail ? `<div class="reqdesc">${mdLite(t.detail.slice(0, 600))}</div>` : '';
     const usage = formatUsageCompact(t.usage);
     const usageBadge = usage ? `<span class="tbadge usage">${usage} tokens</span>` : '';
@@ -5290,7 +5379,7 @@ async function sendReviewReject(){
     upsert(state.goals, body.goal);
     closeReviewChecklist();
     render();
-  } catch { showErr('Reject failed.'); }
+  } catch { showErr('Dismiss failed.'); }
 }
 $('reviewDismiss').onclick = () => sendReviewReject();
 $('reviewFixSend').onclick = () => { const t = $('reviewFixInput').value.trim(); if (t || reviewAttach.length) sendReviewDismiss(t); };
@@ -5490,7 +5579,7 @@ async function sendReject(goalId){
     upsert(state.goals, body.goal);
     closeGoalDetail();
     render();
-  } catch { showErr('Reject failed.'); }
+  } catch { showErr('Dismiss failed.'); }
   $('gdApprove').disabled = false; $('gdDismiss').disabled = false; $('gdSend').disabled = false;
 }
 async function sendDismiss(goalId, text){
@@ -5924,6 +6013,11 @@ function renderFsBoard(){
   const paused = !!state.projects.find((p) => p.id === state.active)?.paused; // Doing列トグルの状態
   const { pgoals, list } = fsBucketTodo();
   const tt = (s, n) => esc(String(s ?? '').replace(/\s+/g, ' ').slice(0, n ?? 60));
+  // Card copy language — same rule as reviewLangFor(state.active) === 'ja', inlined
+  // (not calling reviewLangFor itself) because engine/test/board-goal-level.test.mjs
+  // evals this function body standalone off a minimal `state` stub with no
+  // state.reviewDefinitions; reviewLangFor would throw there.
+  const boardJa = state.reviewDefinitions?.[state.active]?.language === 'ja';
   const goalNo = new Map();
   for (const g of pgoals) { const n = goalReviewNumber({ goal: g, tasks: list }); if (n != null) goalNo.set(g.id, n); }
   // + New in EVERY column, matching the flagship Artifact: a hover-reveal "+" in the header
@@ -5938,6 +6032,35 @@ function renderFsBoard(){
   // status, so the card never repeats it. Stopped/failed carry NO bar (a dead bar would claim
   // the goal is moving) — a coloured word (Failed/Interrupted) says it instead.
   const goalTasks = (g) => list.filter((t) => t.goalId === g.id && !t.reply);
+  // P0b (Masa 2026-07-21): the card's breakdown is the AI's OWN TodoWrite list — never
+  // the planner's invented sub-tasks (there aren't any anymore; 1 goal = 1 task). Live
+  // list wins while it has content, else the frozen task.todosFinal (mirrors the
+  // module-level todoSourceFor() used by the chat feed's todosHtml — duplicated inline
+  // rather than called, same reason as boardJa above: this whole function is evaluated
+  // standalone by the board-goal-level test harness, which only stubs `state`/`esc`).
+  const todoListFor = (t) => {
+    const live = state.todos?.[t?.id];
+    if (Array.isArray(live) && live.length) return live;
+    return Array.isArray(t?.todosFinal) ? t.todosFinal : [];
+  };
+  // Capped + folded rather than scrolled (§1.5-8). While the task is actively running
+  // but hasn't reached its first TodoWrite yet, the list is genuinely empty — show one
+  // small "preparing" line instead of an empty container (§1.5-3), never a bare gap.
+  const TODO_CARD_CAP = 5, TODO_MARK_BOARD = { completed: '✓', in_progress: '▸', pending: '○' };
+  const goalCardTodosHtml = (t) => {
+    if (!t) return '';
+    const list_ = todoListFor(t);
+    if (!list_.length) return t.status === 'running' ? `<div class="lbc-todos prep">${boardJa ? '準備中…' : 'Preparing…'}</div>` : '';
+    const shown = list_.slice(0, TODO_CARD_CAP);
+    const restN = list_.length - shown.length;
+    const rows = shown.map((d) => {
+      const st = ['completed', 'in_progress', 'pending'].includes(d.status) ? d.status : 'pending';
+      const label = st === 'in_progress' && d.activeForm ? d.activeForm : d.content;
+      return `<div class="todo ${st}"><span class="box">${TODO_MARK_BOARD[st]}</span><span class="tdtext">${esc(label ?? '')}</span></div>`;
+    }).join('');
+    const more = restN > 0 ? `<div class="lbc-more">+${restN} more</div>` : '';
+    return `<div class="lbc-todos">${rows}${more}</div>`;
+  };
   const goalCard = (g) => {
     const bkt = fsBoardBucket(g, list);
     // errored is status-based now (blocked/failed/partial/interrupted live in the Needs-you column,
@@ -5962,13 +6085,21 @@ function renderFsBoard(){
       + (word ? `<span class="lbc-w ${amber ? 'intr' : 'bad'}">${word}</span>` : '')
       + '</div>'
       + `<div class="lbc-t">${tt(g.text || g.plan?.[0], 90)}</div>`;
+    // The goal's one task (post-demotion there's exactly one non-reply task; [0]
+    // is just defensive against any pre-migration leftovers with more than one).
+    const task = goalTasks(g)[0];
+    // Todos: not shown for To Do / Later — nothing has actually started yet there, so
+    // there is nothing honest to fold open (quiet stays quiet). Doing/Needs/Review/Done
+    // all show the AI's own breakdown — that's what makes it survive past "running".
+    if (task && bkt !== 'todo' && bkt !== 'later') h += goalCardTodosHtml(task);
     // n/m is Doing-only (Masa 2026-07-17): a fraction on a stopped/queued/done card is
     // residue, not progress — a card showing n/m reads unambiguously as "moving right now".
-    // done/total are the goal's real tasks, so it never invents a number.
-    if (healthyRun) {
-      const gt = goalTasks(g), total = gt.length;
+    // P0b: done/total now reflect the AI's OWN todos (not the planner's invented tasks,
+    // which under 1-goal=1-task degenerate to a useless 0/1 → 1/1).
+    if (healthyRun && task) {
+      const tl = todoListFor(task), total = tl.length;
       if (total) {
-        const done = gt.filter((t) => ['done', 'skipped'].includes(t.status)).length;
+        const done = tl.filter((d) => d.status === 'completed').length;
         h += `<div class="lbc-meter"><span class="bar"><i style="width:${Math.round(done / total * 100)}%"></i></span><span class="frac">${done}/${total}</span></div>`;
       }
     }
@@ -6058,16 +6189,11 @@ function renderFsReview(){
   const pgoals = state.goals.filter((g) => g.projectId === state.active);
   const list = state.tasks.filter((t) => t.projectId === state.active);
   const rows = buildReviewDigest({ goals: pgoals, tasks: list });
-  // PRD §3.5/§6.5: folded goals still have a row — they were never deleted, their work
-  // just went somewhere else. They must be able to hold this section up ALONE: the goal
-  // they were folded into flips back to 'running' while it re-runs their request, so
-  // there is a window where `rows` is empty and only the folded ones remain. Returning
-  // early there would make #13 vanish for the length of the re-run — exactly the "where
-  // did my finished work go" this feature exists to prevent.
-  const foldedGoals = pgoals.filter((g) => g.status === 'folded');
-  if (!rows.length && !foldedGoals.length) { el.innerHTML = ''; return; } // §1.5-3: no reviews → the UI itself does not exist
+  // PRD §3.5/§6.5 → Masa 2026-07-21: folded goals no longer get their own card in the
+  // Review column (Masa: review欄にいらない). They were never deleted — where their work
+  // went is now logged inside the carrier review's detail ("Also carries", see saItemHtml).
+  if (!rows.length) { el.innerHTML = ''; return; } // §1.5-3: no reviews → the UI itself does not exist
   const green = rows.filter((r) => !r.testResult || r.testResult.ok !== false).length;
-  const proofN = rows.filter((r) => r.proof).length;
   const blockedN = pgoals.filter((g) => g.status === 'blocked').length;
   const pct = rows.length ? Math.round((green / rows.length) * 100) : 0;
   // CDO review 2026-07-08 §6: same fix as the s5sum digest above — green/rows.length
@@ -6075,7 +6201,7 @@ function renderFsReview(){
   // ratio was already correct; "all green" is the line that used to fire regardless
   // of blockedN, reading as "18/18 ready · all green · 1 blocked" (self-contradictory).
   const allClear = green === rows.length && !blockedN;
-  const sline = `${allClear ? 'all green' : green === rows.length ? 'blocked needs attention' : `${rows.length - green} failing`} · ${proofN} with proof · ${blockedN ? `${blockedN} blocked` : 'none blocked'}`;
+  const sline = `${allClear ? 'all green' : green === rows.length ? 'blocked needs attention' : `${rows.length - green} failing`} · ${blockedN ? `${blockedN} blocked` : 'none blocked'}`;
   const RCAP = 3;
   const shown = fsUI.revMore ? rows : rows.slice(0, RCAP);
   const card = (r) => {
@@ -6095,31 +6221,18 @@ function renderFsReview(){
     // retest ×3 note (v45 §5.3): the goal came back green from a Ledger Dismiss
     const g0 = pgoals.find((g) => g.id === r.goalIds[0]);
     const rnote = g0?.retest?.passed ? `<span class="rnote">✓ ${esc(g0.retest.note)}</span>` : '';
-    // PRD §6.5: goals that landed on the same files ride this card's PR instead of
-    // opening a second one that edits the same lines. Whoever opens this review has
-    // to know it answers more than one request. Only when it happened (§1.5-3).
-    const carried = pgoals.filter((x) => r.goalIds.includes(x.foldedInto));
-    const cnote = carried.length ? `<span class="fnote">${foldLangFor(g0) === 'ja'
-      ? `+ ${carried.map((x) => `#${x.id}`).join(', ')} の仕事もここに入っています`
-      : `also carries ${carried.map((x) => `#${x.id}`).join(', ')}`}</span>` : '';
+    // PRD §6.5 → Masa 2026-07-21: goals folded onto this card's PR no longer get an
+    // inline "+ #901 の仕事もここに入っています" note here — same report-in-the-list
+    // clutter the old standalone fold row was (Masa: review欄にいらない). It is logged
+    // inside the opened detail's "Also carries" section instead (see saItemHtml).
     // Title = the goal's own title (saReviewTitle → text-first), same as the See-all
     // Ledger and the To Do list, so a review card and its list row read identically
     // (Masa 2026-07-20: this card showed reviewSummary.check "…が出ない" while the list
     // showed the request text). Plan-reviews keep their "PLAN — approve to execute:"
     // checkLine, which is the actionable label there, not a title mismatch.
-    return `<div class="rcard${img ? '' : ' noimg'}" data-fsrev="${r.goalIds[0]}">${img}<div class="rc"><div class="t">${rid}${esc(r.plan ? r.checkLine : saReviewTitle(g0))}</div>${rnote}${cnote}${pr}</div>
+    return `<div class="rcard${img ? '' : ' noimg'}" data-fsrev="${r.goalIds[0]}">${img}<div class="rc"><div class="t">${rid}${esc(r.plan ? r.checkLine : saReviewTitle(g0))}</div>${rnote}${pr}</div>
       <div class="act"><button class="b" data-fsapprove="${ids}" title="Approve">✓</button><button class="b no" data-fsdismiss="${ids}" title="Send back">✕</button></div></div>`;
   };
-  // A folded goal is settled, not failed: no ✓/✕ (the thing to approve lives on the goal
-  // it was folded into), no red, no painted surface (§1.5-8) — just its own number, what
-  // it asked for, and where the work went. Tapping it opens that goal (§1.5-9 wording).
-  const foldCard = (g) => {
-    const line = saReviewTitle(g).slice(0, 60); // title-first, same as the review card above
-    const mergedNote = foldLangFor(g) === 'ja' ? `→ #${esc(String(g.foldedInto))} にまとめました` : `merged into #${esc(String(g.foldedInto))}`;
-    return `<div class="rcard noimg fold" data-fsfold="${g.id}">
-      <div class="rc"><div class="t"><span class="rid gno" data-n="${g.id}">#${g.id}</span>${esc(line)}</div>
-      <span class="fnote">${mergedNote}</span></div></div>`;
-  };
   // V3-C (Masa決定 2026-07-07): no collapsed strip — it duplicated the sm2 meter right
   // above. Reviews render directly: meter + Review header + cards (+N more fold, §1.5-8).
   // The meter counts what is reviewable, so it only exists when something is (§1.5-3):
@@ -6135,7 +6248,7 @@ function renderFsReview(){
         <button class="rvb${fsUI.rv === 1 ? ' on' : ''}" data-fsrv="1" title="List"><svg viewBox="0 0 24 24"><rect x="3" y="4" width="7" height="7" rx="1.5"/><path d="M13 6h8M13 9h5"/><rect x="3" y="14" width="7" height="7" rx="1.5"/><path d="M13 16h8M13 19h5"/></svg></button>
         <button class="rvb${fsUI.rv === 2 ? ' on' : ''}" data-fsrv="2" title="Compact"><svg viewBox="0 0 24 24"><path d="M3 6h18M3 12h18M3 18h18"/></svg></button>
       </span></div>
-    <div class="revlist">${shown.map(card).join('')}${foldedGoals.map(foldCard).join('')}
+    <div class="revlist">${shown.map(card).join('')}
       ${rows.length > RCAP ? `<div class="morefold" id="fsRevMore">${fsUI.revMore ? 'less' : `+${rows.length - RCAP} more`}<span class="g">›</span></div>` : ''}</div>`;
   // Mirror the ready-review count onto the mobile top-bar "Review N" opener (H23).
   const _hn = document.getElementById('fsHambN'); if (_hn) _hn.textContent = green > 0 ? String(green) : '';
@@ -6155,19 +6268,8 @@ function renderFsReview(){
     if (e.target.closest('a') || e.target.closest('.act')) return;
     openReviewChecklist(Number(c.dataset.fsrev));
   };
-  // A folded goal has nothing of its own to review — its work is in the goal it was
-  // folded into, so the tap goes there. That goal is only in Review once its re-run
-  // finishes, so resolve from its live status instead of assuming (openReviewChecklist
-  // on a running goal would present an empty checklist).
-  for (const c of el.querySelectorAll('[data-fsfold]')) c.onclick = (e) => {
-    if (e.target.closest('a')) return;
-    const g = state.goals.find((x) => x.id === Number(c.dataset.fsfold));
-    const target = state.goals.find((x) => x.id === g?.foldedInto);
-    if (!target) return;
-    goToGoalInChat(target.id);
-    if (target.status === 'review') openReviewChecklist(target.id);
-    else openGoalDetail(target.id);
-  };
+  // Folded goals no longer render their own card here (Masa 2026-07-21) — where their
+  // work went is logged inside the carrier review's detail ("Also carries"), not tapped.
   for (const b of el.querySelectorAll('[data-fsapprove]')) b.onclick = async (e) => { e.stopPropagation();
     let ok = true, mergeGoal = null;
     for (const id of b.dataset.fsapprove.split(',')) {
@@ -6181,7 +6283,7 @@ function renderFsReview(){
   for (const b of el.querySelectorAll('[data-fsdismiss]')) b.onclick = async (e) => { e.stopPropagation();
     // 07-20c: the review-card ✕ is a pure Reject → Rejected column (/reject), no auto-rework.
     let r; for (const id of b.dataset.fsdismiss.split(',')) r = await fetch(withKey(`/api/goals/${id}/reject`), { method: 'POST' });
-    if (r && r.ok) await refresh(); else showErr('Reject failed.'); };
+    if (r && r.ok) await refresh(); else showErr('Dismiss failed.'); };
 }
 // Shared To Do bucketing (list rows AND the kanban board read the same groups —
 // single source of truth, see renderFsBoard below). Pure of the DOM: state in, groups out.
@@ -6308,7 +6410,7 @@ function renderFsTodo(){
     <div class="tl">
       ${(doingRows || paused) ? `<span class="tlcap tlcap-doing" style="padding-top:0">Doing<button class="tlcap-pause" data-fspause="1" aria-pressed="${paused ? 'true' : 'false'}" title="${paused ? 'Resume — 再開' : 'Pause — 一時停止'}">${paused ? '<svg viewBox="0 0 24 24"><path d="M8 5v14l11-7z"/></svg>' : '<svg viewBox="0 0 24 24"><rect x="6" y="5" width="4" height="14" rx="1"/><rect x="14" y="5" width="4" height="14" rx="1"/></svg>'}</button></span>${doingRows}` : ''}
       ${needsRows ? `<span class="tlcap needs"${(doingRows || paused) ? '' : ' style="padding-top:0"'}>Needs you</span>${needsRows}` : ''}
-      ${rejected.length ? `<span class="tlcap rejcap"${(doingRows || paused) || needsRows ? '' : ' style="padding-top:0"'}>Rejected<span class="num">${rejected.length}</span></span>${rejected.map((g) => `<div class="trow up rej"><span class="st rej"></span>${noSpan(g.id)}<span class="tt">${tt(g.text, 70)}</span><span class="acts"><button class="ab" data-fsreopen="${g.id}" title="Reopen — back to Review">${FS_ICONS.reopen}</button><button class="ab" data-fsrejdel="${g.id}" title="Delete — discard (closes the PR)">${FS_ICONS.del}</button></span></div>`).join('')}` : ''}
+      ${rejected.length ? `<span class="tlcap rejcap"${(doingRows || paused) || needsRows ? '' : ' style="padding-top:0"'}>Dismissed<span class="num">${rejected.length}</span></span>${rejected.map((g) => `<div class="trow up rej"><span class="st rej"></span>${noSpan(g.id)}<span class="tt">${tt(g.text, 70)}</span><span class="acts"><button class="ab" data-fsreopen="${g.id}" title="Reopen — back to Review">${FS_ICONS.reopen}</button><button class="ab" data-fsrejdel="${g.id}" title="Delete — discard (closes the PR)">${FS_ICONS.del}</button></span></div>`).join('')}` : ''}
       ${upRows ? `<span class="tlcap"${(doingRows || paused) || needsRows || rejected.length ? '' : ' style="padding-top:0"'}>Next up</span>${upRows}` : ''}
       ${queued.length > NCAP ? `<div class="morefold" id="fsTodoMore">${fsUI.todoMore ? 'less' : `+${queued.length - NCAP} more`}<span class="g">›</span></div>` : ''}
       ${laterRows ? `<span class="tlcap">Later</span>${laterRows}` : ''}
@@ -7244,8 +7346,14 @@ async function refresh(){
     // back the already-accumulated activity for 'running' tasks.
     for (const t of state.tasks) { if (t.activity?.length) state.act[t.id] = t.activity; }
     // Same for the live to-do list: /api/state ships t.todos for running tasks
-    // (task 108/109) so a mid-run refresh restores the checklist immediately.
-    for (const t of state.tasks) { if (t.todos?.length) state.todos[t.id] = t.todos; }
+    // (task 108/109) so a mid-run refresh restores the checklist immediately. A
+    // FINISHED task has no live t.todos (trimTaskActivityForState strips it) but
+    // keeps t.todosFinal — seed from that too (P0b), so a reload of a completed
+    // goal still shows the AI's own breakdown instead of losing it silently.
+    for (const t of state.tasks) {
+      if (t.todos?.length) state.todos[t.id] = t.todos;
+      else if (t.todosFinal?.length) state.todos[t.id] = t.todosFinal;
+    }
     state.connectCommand = s.connectCommand ?? ''; state.models = s.models ?? ['sonnet']; state.agentModels = s.agentModels ?? null; state.agentEfforts = s.agentEfforts ?? null; state.agents = s.agents ?? ['claude-code', 'codex']; state.uiVersion = s.uiVersion;
     state.billing = s.billing ?? { licensed: false, email: null };
     renderAppSelect(); // picker follows the server's installed-agent set; may correct a stale connectedApp
@@ -7884,7 +7992,7 @@ function closeSummaryPera(){ $('summaryOverlay').classList.remove('show'); }
 // 4 archived / 5 reverted — session-local; the server round-trip is what
 // settles it (see saSync's "returned to review" reset).
 // ============================================================================
-const SA = { open: false, items: [], diffs: {}, diffBoxHtml: {}, detOpen: new Set(), focusGid: null, focusScrolled: false, focusTimer: null, cur: null };
+const SA = { open: false, items: [], diffs: {}, detOpen: new Set(), focusGid: null, focusScrolled: false, focusTimer: null, cur: null };
 window.SA = SA; // top-level const isn't a window property; external checks probe window.SA (same convention as window.state above)
 const SA_SVG = {
   ok: '<svg viewBox="0 0 24 24"><path d="M4 12l5 5L20 6"/></svg>',
@@ -7917,24 +8025,77 @@ function saRows(){
           return { no: goalReviewNumber({ goal: sg, tasks: ts }), title: saReviewTitle(sg), cur: gid === r.goalIds[0] };
         })
       : null;
+    // Masa 2026-07-21: goals folded onto this review no longer take their own Review-column
+    // row ("→ #12 にまとめました" read as clutter). Instead this review's detail logs what
+    // it carries. Mirrors engine/lib.mjs foldedIntoReview() (browser can't import it).
+    const folded = gs.filter((x) => x.status === 'folded' && x.foldedInto != null && r.goalIds.includes(x.foldedInto))
+      .map((x) => ({ id: x.id, title: saReviewTitle(x) }));
     return {
-      ids: r.goalIds, goalId: r.goalIds[0], goal: g,
+      ids: r.goalIds, goalId: r.goalIds[0], goal: g, folded,
       no: r.reviewNumber,
       title: saReviewTitle(g),
       check: String(g?.reviewSummary?.check || r.checkLine || '').replace(/\s+/g, ' ').trim(),
       goalText: String(g?.text ?? '').replace(/\s+/g, ' ').trim(),
       did, siblings, risk: g?.risk ?? null,
       note: g?.retest?.passed ? g.retest.note : null,
-      pr: r.pr, testResult: r.testResult,
+      pr: r.pr,
+      // P0b (Masa 2026-07-21): the AI's own TodoWrite breakdown for the "Steps" section
+      // below — the primary goal's own non-reply task, frozen at completion. Distinct
+      // from `did` above (task.result prose) and from Activity (raw tool-call log).
+      todosFinal: (() => {
+        const t = ts.find((x) => x.goalId === g?.id && !x.reply);
+        return Array.isArray(t?.todosFinal) ? t.todosFinal : [];
+      })(),
+      // P1 (Masa 2026-07-21): the AI's own final report, verbatim and untrimmed — the
+      // card renders it as Markdown and interprets nothing. This is the deliverable for
+      // every ask that has no running thing to open ("research this", "explain that"),
+      // and the explanation for every ask that does.
+      report: (() => {
+        const t = ts.find((x) => x.goalId === g?.id && !x.reply && x.result);
+        return String(t?.result ?? '').replace(/^検証(PASS|FAIL)[^\n]*\n+/, '').trim();
+      })(),
+      // …and the worker's own "here's how you see it running" declaration (task.run,
+      // written by the worker into .manager-run.json). Absent for asks with nothing to
+      // run — then no button, which is the honest state, not a missing feature.
+      run: (() => {
+        const t = ts.find((x) => x.goalId === g?.id && !x.reply && x.run);
+        return t ? { taskId: t.id, ...t.run } : null;
+      })(),
+      // Explicit review-summary display fields (spec §11: the card's shape follows the
+      // task). A goal's reviewSummary may carry them directly (e.g. a conversational task
+      // that reports an Answer + its Activity, or a ticket whose test count / diff totals
+      // are recorded rather than re-derived); each falls back to the computed value.
+      testResult: (typeof (g?.reviewSummary?.tests) === 'string')
+        ? { ran: true, ok: true, passed: parseInt(g.reviewSummary.tests, 10) || 0, failed: 0, label: g.reviewSummary.tests }
+        : r.testResult,
+      answer: g?.reviewSummary?.answer ?? null,
+      activity: Array.isArray(g?.reviewSummary?.activity) ? g.reviewSummary.activity : null,
+      discussion: Array.isArray(g?.reviewSummary?.discussion) ? g.reviewSummary.discussion : null,
+      changedDesc: g?.reviewSummary?.changed ?? null,
+      metaFiles: Array.isArray(g?.reviewSummary?.files) ? g.reviewSummary.files : null,
+      metaAdd: g?.reviewSummary?.a ?? null, metaDel: g?.reviewSummary?.d ?? null,
+      ghRows: Array.isArray(g?.reviewSummary?.ghdiff) ? g.reviewSummary.ghdiff : null,
+      diffTotal: g?.reviewSummary?.diffTotal ?? null,
       proofTaskId: pt?.id ?? null, proofImg: pt?.proof?.png ?? pt?.proof?.gif ?? null,
-      // Ledger before/after (docs/design/HANDOFF-review-card-evidence-first-2026-07-18.md
-      // §3 kind==='image'): most tasks never have a "before" capture, so this is almost
-      // always null and beforeAfterMediaHtml() falls back to the single-shot view.
-      proofBefore: pt?.proof?.beforePng ?? pt?.proof?.beforeGif ?? null,
+      // Recording (spec §5/§6): a proof task may carry a screen recording (proof.mp4)
+      // or just a recording:true flag; the goal's reviewSummary can also name one.
+      // Drives the neutral "Verify recording ↗" meta chip + the Proof video player.
+      // "ないものは載せない": almost all real tasks have none, so both stay hidden.
+      recording: Boolean(pt?.proof?.mp4 || pt?.proof?.recording || g?.reviewSummary?.recording),
+      recordingUrl: pt?.proof?.mp4 ? withKey(`/proof/${pt.id}/${pt.proof.mp4}`) : (typeof g?.reviewSummary?.recording === 'string' ? g.reviewSummary.recording : null),
+      // Local URL (spec §5): a running localhost the reviewer can open ("Local localhost:4809 ↗").
+      // Falls back to the existing "Local run ↗" (runlocal) button when absent.
+      localUrl: g?.reviewSummary?.localUrl ?? null,
+      // Client-side diff fallback: the git-backed /diff endpoint is null for seeded/
+      // offline goals, so "What changed" never renders. A unified-diff string stored on
+      // the goal (or any of its tasks) lets saGhDiffHtml render locally with no server.
+      diffStr: g?.diff ?? (ts.filter((t) => r.goalIds.includes(t.goalId)).map((t) => t.diff).find(Boolean)) ?? null,
       // Token-frugality is the product's whole pitch — surface it here too, not
       // just in the per-goal review modal. buildReviewSummary computes the same
       // estimate; reuse its logic directly off the goal's runUsage/runTokens.
-      tokenOpt: estimateTokenOptimizationLocal({
+      tokenOpt: (typeof g?.reviewSummary?.token === 'number')
+        ? { percent: g.reviewSummary.token, basis: 'cache', techniques: ['prompt caching', '/compact'] }
+        : estimateTokenOptimizationLocal({
         usage: g?.runUsage, weightedTokens: g?.runTokens, cacheReadWeight: g?.runTokenWeight,
         sessionMode: g?.executionPlan?.sessionMode,
         usedHandoff: g?.executionPlan?.sessionMode === 'cold-handoff' || Boolean(g?.contextHandoff),
@@ -7947,32 +8108,9 @@ function saRows(){
     };
   });
 }
-// Meta row: PR link / token savings (real tap-to-open toggle) / Run locally — under
-// the title (H23 §1). Run-locally has no real per-task command yet — this is the
-// CDO's answer to "let me test this locally", left for CTO/engine to decide what it
-// actually triggers (docs/design/HANDOFF-review-card-evidence-first-2026-07-18.md
-// §5). Shown only for tasks with a real automated check to plausibly re-run
-// (it.testResult.ran), and tapping it is honest about not being wired yet rather
-// than pretending to execute anything — never a fake "running…" state.
-function saMetaRowHtml(it, i){
-  const canRunLocal = Boolean(it.testResult?.ran);
-  if (!it.pr && !it.tokenOpt && !canRunLocal) {
-    return '<div class="lg-meta"><span class="none">No PR link / token-savings data on this task.</span></div>';
-  }
-  const chips = [];
-  if (it.pr) chips.push(`<a class="sa-prlink sm" href="${esc(it.pr)}" target="_blank" title="Open pull request">PR #${esc((it.pr.match(/pull\/(\d+)/) ?? [, ''])[1])} ↗</a>`);
-  if (it.tokenOpt) {
-    const pct = Number(it.tokenOpt.percent);
-    const label = pct > 0 ? `saved ${pct}% tokens` : 'token usage';
-    chips.push(`<button class="tokchip" type="button" data-saact="tok:${i}">${esc(label)} <i class="tokchevron">›</i></button>`);
-  }
-  if (canRunLocal) chips.push(`<button class="sa-runlocal" type="button" data-saact="runlocal:${i}" title="Run this locally"><svg viewBox="0 0 24 24"><path d="M6 4l14 8-14 8V4z"/></svg>Run locally</button>`);
-  const tokCmp = it.tokenOpt?.basis === 'baseline' ? 'vs your recent average' : 'from prompt caching';
-  const tokTech = (it.tokenOpt?.techniques ?? []).slice(0, 4).join(' / ');
-  const tokDetail = it.tokenOpt ? `<div class="tokdetail" id="tokdetail${i}"><span class="k">How this run used tokens</span><span class="v">${esc(tokCmp)} · ${esc(tokTech)}</span></div>` : '';
-  const runNote = canRunLocal ? `<div class="tokdetail" id="runlocal${i}"><span class="v">No local run command is wired for this task yet.</span></div>` : '';
-  return `<div class="lg-meta">${chips.join('')}</div>${tokDetail}${runNote}`;
-}
+// (P1, Masa 2026-07-21) saMetaRowHtml() was deleted here — the pre-redesign meta row,
+// uncalled since the review-card redesign. Its "Run locally" chip only ever opened a
+// note saying no run command was wired; the card's Open it chip now starts the real one.
 // Context: closed by default, one-line verbatim-quote summary, chevron large and at
 // the far right (H23 §2). Quotes goal.text — the user's own words — never
 // task.detail/title (AI-authored; an earlier round of this design mistakenly quoted
@@ -7991,37 +8129,89 @@ function saContextHtml(it, gk){
   const open = SA.detOpen.has(`${gk}:ctx`) ? ' open' : '';
   return `<details class="lg-goal" data-detkey="${gk}:ctx"${open}><summary><span class="k">Context</span><span class="v">${summary}</span><i class="gtoggle">›</i></summary><div class="lg-context">${inner}</div></details>`;
 }
-// Evidence hero: shape follows task kind, not one image-shaped template (H23 §3).
-// Real data check: most finished tasks have no proof image at all (logic-only /
-// docs-only / research changes verify without a screenshot) — kind is derived, see
-// saEvidenceKind() above.
-function saEvidenceHtml(kind, factRow, img, filePaths, diffText, gtr, it){
-  if (kind === 'logic') {
-    const passed = gtr?.passed ?? 0, failed = gtr?.failed ?? 0, ok = gtr?.ok !== false;
-    const n = ok ? String(passed) : `${passed}/${passed + failed}`;
-    return `<div class="lg-ev"><div class="lg-evhd">Evidence — tests, no UI to show</div>
-      <div class="lg-herostat${ok ? '' : ' fail'}"><span class="n">${esc(n)}</span><span class="l">tests passed<br>${esc(filePaths.join(', '))}</span></div></div>`;
-  }
-  if (kind === 'docs') {
-    const lines = diffText ? diffText.split('\n').filter((l) => l.length) : [];
-    const diffHtml = lines.map((l) => {
-      const isAdd = l.startsWith('+');
-      return `<div class="ln${isAdd ? ' a' : ''}"><span class="gutter">+</span><span class="txt">${esc(l.slice(1) || l)}</span></div>`;
-    }).join('');
-    return `<div class="lg-ev"><div class="lg-evhd">Evidence — the diff, no test to run</div>
-      <div class="row"><span class="fls">${esc(filePaths.join(', '))} · docs-only, no test target</span></div>
-      ${diffHtml ? `<div class="lg-diffblock">${diffHtml}</div>` : ''}</div>`;
+// (P1, Masa 2026-07-21) saEvidenceHtml() was deleted here. It rendered the four
+// "evidence hero" shapes for the pre-redesign card — including the 'missing' one that
+// told the reviewer "No capture — this touched the UI and should have one". Nothing has
+// called it since the review-card redesign, and its premise (we owe you a screenshot)
+// is exactly what P1 removes. The report the AI wrote, plus Open it, is the evidence.
+// Unified-diff → GitHub two-gutter model (REVIEW-CARD-SPEC §7 "GitHub diff = Bright"):
+// walk the @@ hunk headers to recover the old/new line numbers, emit one row per line
+// {o:oldNo|null, n:newNo|null, t:'add'|'del'|'ctx', c:text}. Word-level highlight is
+// NOT fabricated from a raw diff (the product never invents evidence) — the Bright
+// line+gutter tint carries the change and .gh-hi simply stays absent. Pure, so it is
+// extracted and unit-tested in engine/test/review-card-diff.test.mjs.
+function saUnifiedToGh(diff, max){
+  const cap = max || 80; const out = []; let o = 0, n = 0;
+  for (const raw of String(diff || '').split('\n')) {
+    if (raw === '') continue; // split artifact / blank input — a real blank ctx line is a single space
+    if (/^(diff --git|index |--- |\+\+\+ |new file|deleted file|old mode|new mode|similarity|rename |Binary )/.test(raw)) continue;
+    const m = raw.match(/^@@ -(\d+)(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
+    if (m) { o = +m[1]; n = +m[2]; continue; }
+    if (raw.startsWith('\\')) continue; // ""
+    if (raw.startsWith('+')) out.push({ o: null, n: n++, t: 'add', c: raw.slice(1) });
+    else if (raw.startsWith('-')) out.push({ o: o++, n: null, t: 'del', c: raw.slice(1) });
+    else out.push({ o: o++, n: n++, t: 'ctx', c: raw.replace(/^ /, '') });
+    if (out.length >= cap) break;
   }
-  if (kind === 'missing') {
-    const note = it.check || 'Please verify this change manually — no automated capture exists for it.';
-    return `<div class="lg-ev"><div class="lg-missinghd"><svg viewBox="0 0 24 24"><path d="M12 8v5M12 16.5v.5"/><path d="M10.3 3.9 2.4 18a1.9 1.9 0 0 0 1.7 2.9h15.8a1.9 1.9 0 0 0 1.7-2.9L13.7 3.9a1.9 1.9 0 0 0-3.4 0z"/></svg>No capture — this touched the UI and should have one</div>
-      <div class="lg-manualnote">${esc(note)}</div></div>`;
-  }
-  // kind === 'image' (default)
-  return `<div class="lg-ev"><div class="lg-evhd">Evidence</div><div class="row">${factRow}</div>${img}</div>`;
+  return out;
+}
+function saGhDiffHtml(diff){
+  const rows = saUnifiedToGh(diff);
+  if (!rows.length) return '';
+  return saGhDiffRowsHtml(rows);
+}
+// One Activity transcript row (spec §3): You / tool (mono, bold name) / res (green) /
+// Claude. Mirrors the flagship artifact's tx mapping so the pulldown pixel-matches.
+function saTxRowHtml(e){
+  if (e[0] === 'tool') return `<div class="txr tool"><span class="w">tool</span><span class="m"><b>${esc(e[1])}</b> ${esc(e[2] || '')}</span></div>`;
+  if (e[0] === 'res') return `<div class="txr res"><span class="w"></span><span class="m">${esc(String(e[1]).replace(/^✗ /, '').replace(/^✓ /, ''))}</span></div>`;
+  if (e[0] === 'user') return `<div class="txr you"><span class="w">You</span><span class="m">${esc(e[1])}</span></div>`;
+  return `<div class="txr cl"><span class="w">Claude</span><span class="m">${esc(e[1])}</span></div>`;
+}
+// Structured GitHub diff (REVIEW-CARD-SPEC §7): renders {o,n,t,c,h} rows with a
+// word-level highlight (h) and, for a huge change, a "…+N 行 · GitHub で開く ↗"
+// footer (only linking to GitHub when a PR exists). Mirrors the flagship artifact's
+// ghDiffHtml so the app pixel-matches. `more` = extra unshown line count; `pr` = url.
+function saGhDiffRowsHtml(rows, more, pr){
+  if (!rows || !rows.length) return '';
+  const body = '<div class="rdiff2">' + rows.map((l) => {
+    const cls = l.t === 'add' ? 'gh-add' : l.t === 'del' ? 'gh-del' : 'gh-ctx';
+    const mk = l.t === 'add' ? '+' : l.t === 'del' ? '−' : '';
+    let code = esc(l.c);
+    if (l.h) { const h = esc(l.h); code = code.replace(h, `<span class="gh-hi">${h}</span>`); }
+    return `<div class="ghl ${cls}"><span class="gho">${l.o ?? ''}</span><span class="ghn">${l.n ?? ''}</span><span class="ghm">${mk}</span><span class="ghc">${code}</span></div>`;
+  }).join('') + '</div>';
+  const foot = more ? `<div class="ghfoot"><span>…+${more} 行</span>${pr ? ` · <a class="ghopen" href="${esc(pr)}" target="_blank" rel="noopener">GitHub で開く ↗</a>` : ''}</div>` : '';
+  return body + foot;
+}
+// Screen-recording player for Proof (REVIEW-CARD-SPEC §6). A real file (recordingUrl)
+// renders a native <video controls>; otherwise a self-contained SVG player mockup
+// (faint app frame · centred play · scrub bar) matching the artifact's videoFig, with
+// a "▸ Watch recording · 0:12" caption. Purely presentational — no fabricated evidence
+// (only rendered when a recording actually exists on the task/goal).
+function saVideoFigHtml(url){
+  if (url) return `<figure class="rvid"><video class="proofvid" src="${esc(url)}" controls preload="metadata"></video><figcaption>▸ Watch recording</figcaption></figure>`;
+  const P = 'var(--panel)', H = 'var(--hair)';
+  const svg = `<svg viewBox="0 0 560 315" preserveAspectRatio="xMidYMid meet"><rect width="560" height="315" fill="${P}"/>`
+    + `<rect x="24" y="20" width="120" height="10" rx="3" fill="${H}"/><rect x="24" y="44" width="220" height="8" rx="3" fill="${H}"/><rect x="24" y="64" width="180" height="8" rx="3" fill="${H}"/>`
+    + `<circle cx="280" cy="165" r="34" fill="rgba(0,0,0,.35)"/><path d="M270 150 l24 15 l-24 15 z" fill="#fff"/>`
+    + `<rect x="24" y="286" width="512" height="4" rx="2" fill="${H}"/><rect x="24" y="286" width="150" height="4" rx="2" fill="#4fd79c"/><circle cx="174" cy="288" r="6" fill="#4fd79c"/></svg>`;
+  return `<figure class="rvid">${svg}<figcaption>▸ Watch recording · 0:12</figcaption></figure>`;
+}
+// Workflow (REVIEW-CARD-SPEC §8): vertical ✓ / → / □ built ONLY from real signals on
+// this goal — each ✓ reflects something that actually happened (implemented, tested,
+// proof captured); the card's own "In review" is the → here; merge/deploy are ahead.
+// No fabricated pipeline: an untested goal shows □ Tested, not a green ✓.
+function saWorkflowHtml(it){
+  const ran = Boolean(it.testResult?.ran);
+  // P1 (Masa 2026-07-21): "Proof captured □" is gone. We no longer go take photos of
+  // the work, so an empty box there was scoring the goal against a step that isn't
+  // part of the process any more — a permanent unchecked box nobody could ever check.
+  const steps = [['Implemented', 'ok'], ['Tested', ran ? 'ok' : 'todo'], ['In review', 'now'], ['Merge', 'todo'], ['Deploy', 'todo']];
+  return '<div class="rwf">' + steps.map(([s, st]) => `<div class="wf-s ${st}"><span class="mk">${st === 'ok' ? '✓' : st === 'now' ? '→' : '□'}</span>${s}</div>`).join('') + '</div>';
 }
 function saItemHtml(it, i){
-  const done = { 1: ['ok', '✓', 'Approved'], 2: ['no', '↩', 'Rejected'], 3: ['no', '↩', 'Feedback sent — returned to To Do'], 4: ['ar', '—', 'Archived'], 5: ['no', '↩', 'Reverted — change undone'] }[it.st];
+  const done = { 1: ['ok', '✓', 'Approved'], 2: ['no', '↩', 'Dismissed'], 3: ['no', '↩', 'Feedback sent — returned to To Do'], 4: ['ar', '—', 'Archived'], 5: ['no', '↩', 'Reverted — change undone'] }[it.st];
   if (done) {
     // Revert deliberately has NO undo: the PR is closed and its branch deleted —
     // an "UNDO" here would be a lie (HANDOFF-v45 §5.3 honesty over symmetry).
@@ -8029,99 +8219,238 @@ function saItemHtml(it, i){
     return `<div class="sa-done ${done[0]}"><span class="mk">${done[1]}</span>${done[2]} · <span style="color:var(--ink3)">${esc(it.title.slice(0, 44))}</span>${undo}</div>`;
   }
   const d = SA.diffs[it.goalId];
-  // Honest evidence chips (mirrors the vchip tested/proof/needs-you logic in
-  // renderReviewChecklist ~L3742-3752): absence of a signal is shown as a muted
-  // chip, never silently dropped — an untested/proofless goal must never LOOK verified.
-  // H23: facts stop looking like buttons — glyph carries the state colour, text
-  // stays plain ink, no border/pill (border is reserved for the PR link).
   const gtr = it.testResult;
-  const testsChip = gtr?.ran
-    ? (gtr.ok ? `<span class="vf ok"><i class="g">✓</i>${gtr.passed} passed</span>` : `<span class="vf no"><i class="g">✗</i>${gtr.passed}/${gtr.passed + (gtr.failed || 0)} passed</span>`)
-    : '<span class="vf none">— not tested</span>';
   const hasProofImg = it.proofImg && it.proofTaskId != null;
-  // Before/after (H23 spec §3 kind==='image'): reuse the same beforeafter/ba-col/
-  // balabel renderer as the older per-task checklist (beforeAfterMediaHtml, ~L3281).
-  // it.proofBefore is null for the ~all tasks with no "before" capture, so this
-  // falls back to the identical single <img class="proofimg"> markup it replaced.
-  // Diff-box + zoom inset (HANDOFF-review-card-diffbox-2026-07-18) is computed async
-  // after the card first paints (saEnhanceDiffBoxes, below saRender). SA.diffBoxHtml
-  // holds the result — a string once a box is found, null once ruled out, undefined
-  // until then — so it survives every later saRender() the same way SA.diffs already
-  // does for the changed-files diff.
-  const boxedImg = SA.diffBoxHtml[it.goalId];
-  const img = hasProofImg
-    ? (typeof boxedImg === 'string' ? boxedImg : beforeAfterMediaHtml(it.proofTaskId, { png: it.proofImg, beforePng: it.proofBefore }, 'proof'))
-    : '';
-  // Changed-file paths feed both the evidence-kind heuristic and the docs-kind diff —
-  // same source the old "Changed files & diff" fold used (d from the diff endpoint,
-  // falling back to task.changedFiles when the server has no diff for this goal).
+  // Proof image (P1): the single capture from a check the AI itself promised and ran.
+  // No pair, no red box — those compared against a "before" shot we used to stage.
+  const img = hasProofImg ? proofMediaHtml(it.proofTaskId, { png: it.proofImg }, esc(it.title ?? '')) : '';
+  // Changed-file paths: d from the diff endpoint ({add,del,files:[{path,add,del}],diff}),
+  // falling back to task.changedFiles when the server has no diff for this goal.
+  // A usable git diff has actual files; the endpoint returns an empty {add:0,del:0,
+  // files:[],diff:''} object for goals with no committed range (seeded/uncommitted),
+  // which must NOT shadow the client-side fallback below.
+  const hasGitDiff = Boolean(d && d.diff);
   let filePaths = [], nf = 0;
-  if (d) { filePaths = d.files.map((f) => f.path); nf = filePaths.length; }
-  else if (d === null) {
+  if (d && d.files.length) { filePaths = d.files.map((f) => f.path); nf = filePaths.length; }
+  else {
+    // no usable git diff: fall back to the tasks' own changedFiles so Files / What
+    // changed still populate for seeded/offline goals.
     filePaths = [...new Set((state.tasks || []).filter((t) => it.ids.includes(t.goalId)).flatMap((t) => t.changedFiles ?? []))];
     nf = filePaths.length;
   }
+  // Client-side unified diff (spec §4 fallback): render "What changed" locally when the
+  // server has no usable git diff. Line totals feed the Files chip's +add / −del.
+  const clientDiff = (!hasGitDiff && it.diffStr) ? it.diffStr : null;
+  let cAdd = 0, cDel = 0;
+  if (clientDiff) for (const l of saUnifiedToGh(clientDiff, 2000)) { if (l.t === 'add') cAdd++; else if (l.t === 'del') cDel++; }
+  // Recorded display fields (spec §11: card shape follows the task). A goal's reviewSummary
+  // may carry an explicit file list; prefer it (real git-backed goals keep the computed
+  // filePaths above). Split the "path  +N −M" string down to the bare path for kind/log.
+  if (it.metaFiles) { filePaths = it.metaFiles.map((f) => String(f).split(/\s{2,}/)[0]); nf = it.metaFiles.length; }
   const kind = saEvidenceKind(filePaths, hasProofImg, gtr);
-  const factRow = [
-    testsChip,
-    it.note ? `<span class="vf ok"><i class="g">✓</i>${esc(it.note)}</span>` : '',
-    `<span class="fls">${nf} file${nf === 1 ? '' : 's'}</span>`,
-    hasProofImg ? '' : '<span class="vf none">— no proof</span>',
-  ].filter(Boolean).join('');
   const gk = it.goalId;
-  // "What to look at" (H23 §2): the changed files themselves — a real, non-fabricated
-  // pointer for the reviewer's eyes, same file list that used to live in the retired
-  // "Changed files & diff" fold.
-  const look = filePaths.slice(0, 8);
-  if (filePaths.length > 8) look.push(`+${filePaths.length - 8} more`);
-  const contextHtml = saContextHtml({ goalText: it.goalText, siblings: it.siblings, did: it.did, look }, gk);
-  const evidenceHtml = saEvidenceHtml(kind, factRow, img, filePaths, d?.diff ?? null, gtr, it);
-  // shouldPause = classifyChangeRisk (engine/lib.mjs) already forced this goal to
-  // stop in review for exactly this reason — surface it; a non-blocking low-risk
-  // read isn't worth a permanent row on every card (goal 427 §4 already covers it
-  // in the per-goal modal), so the Ledger only speaks up when it actually matters.
+  const rs = it.goal?.reviewSummary || {};
+  // Conversational task (no code change): drops Proof + What changed, shows an Answer
+  // instead (spec §11); its Activity IS the conversation.
+  const isConv = Boolean(it.answer);
+  // ── meta (spec §5) — fully data-driven / hide-empty: each chip renders only when its
+  // data exists (no PR → no PR chip · no tests → no Tests · no file totals → no Files ·
+  // no recording → no Verify). PR = bold green, no underline (a real <a href>). ──
+  const arw = '<span class="ar">↗</span>';
+  const prNum = it.pr ? (it.pr.match(/pull\/(\d+)/) ?? [, ''])[1] : '';
+  const tokPct = it.tokenOpt ? Number(it.tokenOpt.percent) : 0;
+  const testsLabel = gtr?.ran ? (gtr.label || (gtr.ok ? `${gtr.passed} passed` : `${gtr.passed}/${gtr.passed + (gtr.failed || 0)} passed`)) : '';
+  // Files totals: recorded strings (metaAdd/metaDel, already signed) win, else git/client.
+  const fAdd = it.metaAdd ?? (hasGitDiff ? `+${d.add}` : clientDiff ? `+${cAdd}` : null);
+  const fDel = it.metaDel ?? (hasGitDiff ? `−${d.del}` : clientDiff ? `−${cDel}` : null);
+  const chips = [];
+  if (it.pr) chips.push(`<a class="mx mlink pr" href="${esc(it.pr)}" target="_blank" rel="noopener" title="Open pull request"><span class="mk">PR</span> <b>#${esc(prNum)}</b> <span class="mmut">Open</span>${arw}</a>`);
+  if (it.tokenOpt && tokPct > 0) chips.push(`<button type="button" class="mx mtok" data-saact="tok:${i}"><span class="mk">Token</span> <b>${tokPct}%</b> <span class="mmut">saved</span> <i class="tokchevron">›</i></button>`);
+  if (testsLabel) chips.push(`<span class="mx"><span class="mk">Tests</span> <b class="ok">${esc(testsLabel)}</b></span>`);
+  // Files: hide-empty — only with changed files AND totals (a bare-count chip reads odd).
+  if (nf && fAdd) chips.push(`<span class="mx"><span class="mk">Files</span> <b>${nf}</b> <span class="mmut">· <i class="a">${esc(fAdd)}</i> <i class="d">${esc(fDel)}</i></span></span>`);
+  // Local: an open localhost URL (neutral, not green). A bare host:port isn't navigable in
+  // the demo, so it renders as a non-navigating chip; a real URL opens in a new tab.
+  if (it.localUrl) { const lu = String(it.localUrl); const isUrl = /^https?:/.test(lu);
+    chips.push(`<a class="mx mlink"${isUrl ? ` href="${esc(lu)}" target="_blank" rel="noopener"` : ' style="cursor:pointer"'} title="Open the running app"><span class="mk">Local</span> <b>${esc(lu.replace(/^https?:\/\//, ''))}</b>${arw}</a>`); }
+  // (Open it is NOT a meta chip — Masa 2026-07-22. It was saying the same thing as the
+  // Open it section below, in the PR chip's vocabulary, so the card spoke two languages
+  // about one thing. It lives in the body now, in the same typography as every other
+  // section.)
+  // Verify = a screen recording of the check (spec §5): NEUTRAL. Opens the file when there
+  // is a real URL, otherwise scrolls to the in-card player.
+  if (it.recording) chips.push(it.recordingUrl
+    ? `<a class="mx mlink" href="${esc(it.recordingUrl)}" target="_blank" rel="noopener" title="Watch the recording"><span class="mk">Verify</span> <b>recording</b>${arw}</a>`
+    : `<a class="mx mlink" data-saact="rec:${i}" style="cursor:pointer" title="Watch the recording"><span class="mk">Verify</span> <b>recording</b>${arw}</a>`);
+  const tokCmp = it.tokenOpt?.basis === 'baseline' ? 'vs your recent average' : 'from prompt caching';
+  const tokTech = (it.tokenOpt?.techniques ?? []).slice(0, 4).join(' / ');
+  const metaDetail = (it.tokenOpt && tokPct > 0) ? `<div class="rmeta-detail" id="tokdetail${i}"><span class="k">How this run used tokens</span><span class="v">${esc(tokCmp)}${tokTech ? ` · ${esc(tokTech)}` : ''}</span></div>` : '';
+  // §1.5-3: with Open it moved into the body, a card with no PR / tokens / tests has no
+  // chips at all — render no meta row rather than an empty one holding open a gap.
+  const meta = chips.length ? `<div class="rmeta">${chips.join('')}</div>${metaDetail}` : '';
+  // ── Request (spec §4): a PLAIN section — the user's own short words (goal.text). Never
+  // an accordion, never the implementation spec (that lives in Activity below). ──
+  const reqSec = it.goalText
+    ? `<section class="rsec rreq"><div class="rsec-h">Request</div><p class="rbody rask">${esc(it.goalText)}</p></section>`
+    : '';
+  // ── Open it (Masa 2026-07-22): a plain section in the card's own typography — the same
+  // heading + body shape as Request and Result, not a chip in another vocabulary. The
+  // address the worker declared IS the control (press it → the process starts on demand →
+  // it opens); its own note sits under it, and that note is where the status and any
+  // failure reason land after the press. Nothing declared → no section (§1.5-3). ──
+  const openBody = it.run
+    ? `<div class="ropen" id="runlocal${i}"><button type="button" class="olink" data-saact="preview:${i}" id="prevlink${i}">${esc(it.run.url || 'Start it')}<span class="ar">↗</span></button><p class="rbody v">${esc(it.run.note || it.run.cmd || 'Starts when you press it.')}</p></div>`
+    : '';
+  // ── Also carries (Masa 2026-07-21): goals folded onto this review are logged HERE, inside
+  // the review's own content — not as loose "→ #12 にまとめました" rows in the Review column.
+  // Chrome label stays English (§377-7); the request titles keep whatever language they were
+  // written in. Only rendered when something was actually folded in (§1.5-3 no empty box). ──
+  const carriesBody = it.folded?.length
+    ? `<ul class="rcheck">${it.folded.map((f) => `<li>#${esc(String(f.id))} ${esc(f.title)}</li>`).join('')}</ul>`
+    : '';
+  // ── What to check (spec §2.5): a bulleted list when several, a line when one. ──
+  // Only an explicit review-summary check renders here — never the goal text echoed back
+  // (it.check falls back to goal.text via checkLine, which would fabricate a "What to
+  // check" for a conversational task that has none). §0.5 no placeholder.
+  // The working AI's own sentence wins (P1, Masa 2026-07-22): it declares what to look
+  // at in the same file it declares how to run the thing. reviewSummary.check below is
+  // the intake AI's invention and only fills in for goals that predate the declaration.
+  const checkArr = it.run?.check ? [it.run.check]
+    : Array.isArray(rs.checkList) ? rs.checkList.filter(Boolean)
+    : Array.isArray(rs.check) ? rs.check.filter(Boolean)
+    : (typeof rs.check === 'string' && rs.check.trim() ? [rs.check.trim()] : []);
+  const checkBody = checkArr.length > 1
+    ? `<ul class="rcheck">${checkArr.map((c) => `<li>${esc(c)}</li>`).join('')}</ul>`
+    : (checkArr.length ? `<p class="rbody rcheck1">${esc(checkArr[0])}</p>` : '');
+  // ── Steps (P0b, Masa 2026-07-21): the AI's own TodoWrite breakdown — distinct from
+  // "What to check" (the human's pass condition) and Activity (the raw tool-call log).
+  // Reuses the same .todos/.todo vocabulary as the chat feed / board card so the ✓/▸/○
+  // language is consistent everywhere it appears. Empty → no section (§1.5-3).
+  const stepsBody = it.todosFinal?.length ? `<div class="todos">${todoRowsHtml(it.todosFinal)}</div>` : '';
+  // ── Result (P1, Masa 2026-07-21): the AI's own report, rendered as written and read
+  // whole (no 160-char squeeze, no summariser of ours in the way). This is the one
+  // section that carries every kind of ask — a研究 report reads here as prose, a UI
+  // change explains itself here, and an image the AI pasted shows up here as an image.
+  // We interpret nothing; mdLite only turns Markdown into the shapes it already means. ──
+  const reportBody = it.report ? `<div class="rreport">${mdLite(it.report, { images: false })}</div>` : '';
+  // ── Proof (spec §6/§11): before/after (identical size) when a real image exists; a
+  // run-log for logic-only tests; dropped entirely for a conversational task. ──
+  let proofInner = '';
+  if (!isConv) {
+    if (hasProofImg) proofInner = `<div class="rproof">${img}</div>`;
+    else if (kind === 'logic' && gtr?.ran) proofInner = `<div class="rrun"><div class="rr"><span class="lab">Tests</span><span class="v">${esc(testsLabel)}</span></div>${filePaths.length ? `<div class="rr"><span class="lab">Files</span><span class="v">${esc(filePaths.join(', '))}</span></div>` : ''}</div>`;
+    if (it.recording) proofInner += saVideoFigHtml(it.recordingUrl);
+  }
+  // ── What changed (spec §4/§7): description + files summary (always) + the diff, which
+  // peeks then "…+N 行 · GitHub で開く ↗" for a huge change (no GitHub link when there's no
+  // PR). Structured recorded rows (ghRows) win over a git/client unified diff. Dropped
+  // entirely for a conversational task. ──
+  // No prose here (P1, Masa 2026-07-22): reviewSummary.changed was the intake AI writing
+  // its own summary of the change, and on a real goal it came out as the request echoed
+  // back. What changed is now only what actually changed — files and the diff.
+  const changedDescHtml = '';
+  const filesListHtml = it.metaFiles
+    ? `<div class="rfiles">${it.metaFiles.map((f) => esc(f)).join('<br>')}</div>`
+    : (d && d.files.length) ? `<div class="rfiles">${d.files.map((f) => `${esc(f.path)}  <i class="a">+${f.add}</i> <i class="d">−${f.del}</i>`).join('<br>')}</div>`
+      : (clientDiff && filePaths.length) ? `<div class="rfiles">${filePaths.map((p) => esc(p)).join('<br>')}</div>` : '';
+  const diffHtml = it.ghRows
+    ? saGhDiffRowsHtml(it.ghRows, it.diffTotal || 0, it.pr || '')
+    : hasGitDiff ? saGhDiffHtml(d.diff) : clientDiff ? saGhDiffHtml(clientDiff) : '';
+  const changedBody = isConv ? '' : (changedDescHtml + filesListHtml + diffHtml);
+  // ── sensitive-area note (real: classifyChangeRisk forced the pause) ──
   const risk = it.risk;
   const riskHtml = risk?.shouldPause
     ? `<div class="lg-risk"><svg viewBox="0 0 24 24"><path d="M12 8v5M12 16.5v.5"/><path d="M10.3 3.9 2.4 18a1.9 1.9 0 0 0 1.7 2.9h15.8a1.9 1.9 0 0 0 1.7-2.9L13.7 3.9a1.9 1.9 0 0 0-3.4 0z"/></svg>Needs your review — touches a sensitive area${risk.categories?.length ? `: ${esc(risk.categories.map((c) => c.label).join(', '))}` : ''}</div>`
     : '';
-  return `<div class="sa-item lg${it.goalId === SA.cur ? ' cur' : ''}" data-gid="${it.goalId}">
-    <div class="lg-hd"><span class="no">${it.no != null ? `#${it.no}` : ''}</span><span class="t">${esc(it.title)}</span><span class="lg-state review">Review</span></div>
-    ${saMetaRowHtml(it, i)}
-    <div class="lg-ctx">${contextHtml}<div class="lg-fact"><span class="k">Done when</span><span class="v">${esc(it.check)}</span></div></div>
-    ${evidenceHtml}
+  // ── Activity (spec §3): ONE Linear-docs pulldown. The "Activity" heading is plain; the
+  // chevron sits on the FIRST line (the opening "You …" row); tapping it opens the rest
+  // below, offset-indented. The review Discussion folds in at the bottom of the body. The
+  // transcript is the recorded build (activity); real goals fall back to "what we did".
+  // Persisted open-state via data-detkey so SSE re-renders don't collapse it. ──
+  // The "what we did" fallback is the same report as Result above, cut to 160 chars —
+  // showing both makes the card repeat itself in a worse form (P1). Only fall back when
+  // there is no Result to read.
+  const txArr = (it.activity && it.activity.length)
+    ? it.activity.map((e) => saTxRowHtml(e))
+    : ((!reportBody && it.did?.length) ? it.did.map((s) => saTxRowHtml(['claude', s])) : []);
+  const discArr = it.discussion || it.thread || null;
+  const threadHtml = (discArr && discArr.length)
+    ? discArr.map((m) => `<div class="rmsg${m[0] === 'you' ? ' you' : ''}"><div class="who">${m[0] === 'you' ? 'You' : 'Manager'}</div><div class="msg">${esc(m[1])}</div></div>`).join('')
+    : '';
+  const actOpen = SA.detOpen.has(`${gk}:act`) ? ' open' : '';
+  const txFirst = txArr[0] || '<div class="txr"><span class="w"></span><span class="m">Activity</span></div>';
+  const txRest = txArr.slice(1).join('');
+  const discInner = threadHtml ? `<div class="rsub">Discussion</div><div class="rthread">${threadHtml}</div>` : '';
+  const activitySec = (txArr.length || threadHtml)
+    ? `<section class="rsec ract"><div class="rsec-h">Activity</div><details class="lacc" data-detkey="${gk}:act"${actOpen}><summary><i class="ltog">›</i><div class="rtx lfirst">${txFirst}</div></summary><div class="lbody">${txRest ? `<div class="rtx">${txRest}</div>` : ''}${discInner}</div></details></section>`
+    : '';
+  const S = (label, body) => body ? `<section class="rsec"><div class="rsec-h">${label}</div>${body}</section>` : '';
+  return `<div class="sa-item lg redesign${it.goalId === SA.cur ? ' cur' : ''}" data-gid="${it.goalId}">
+    <div class="rhead"><span class="rtag">${it.no != null ? `#${it.no}` : ''}</span><span class="rst"><span class="rdot"></span>In review</span></div>
+    <h2 class="rtitle">${esc(it.title)}</h2>
+    ${meta}
+    ${reqSec}
+    ${S('Open it', openBody)}
+    ${S('Also carries', carriesBody)}
+    ${S('What to check', checkBody)}
+    ${S('Result', reportBody)}
+    ${S('Steps', stepsBody)}
+    ${S('Answer', it.answer ? `<p class="rbody">${esc(it.answer)}</p>` : '')}
+    ${S('Proof', proofInner)}
+    ${S('What changed', changedBody)}
+    ${S('Not done', it.risk?.summary ? `<p class="rbody rrisk">${esc(it.risk.summary)}</p>` : '')}
     ${riskHtml}
-    <div class="sa-act"><button class="sa-app" data-saact="app:${i}">${SA_SVG.ok}Approve</button><button class="sa-dis" data-saact="dis:${i}">Reject</button><div class="sa-chatwrap"><input class="sa-chat" data-i="${i}" placeholder="What's wrong? Send back to Claude Code…"><button class="sa-send" data-saact="send:${i}" title="Send back"><svg viewBox="0 0 24 24"><path d="M12 19V5M5 12l7-7 7 7"/></svg></button></div><button class="sa-ic" title="Archive" data-saact="arch:${i}">${SA_SVG.arch}</button><button class="sa-ic" title="Revert" data-saact="rev:${i}">${SA_SVG.trash}</button></div>
+    ${activitySec}
+    ${S('Workflow', saWorkflowHtml(it))}
+    <div class="sa-act"><button class="sa-app" data-saact="app:${i}">${SA_SVG.ok}Approve</button><button class="sa-dis" data-saact="dis:${i}">Dismiss</button><div class="sa-chatwrap"><input class="sa-chat" data-i="${i}" placeholder="Reply, ask, or send back to the AI…"><button class="sa-send" data-saact="send:${i}" title="Send"><svg viewBox="0 0 24 24"><path d="M12 19V5M5 12l7-7 7 7"/></svg></button></div><button class="sa-ic" title="Archive" data-saact="arch:${i}">${SA_SVG.arch}</button><button class="sa-ic" title="Revert" data-saact="rev:${i}">${SA_SVG.trash}</button></div>
   </div>`;
 }
+// Scroll the See-all body so a card's top edge (.rhead) sits at the top of the visible
+// scroll area — the reviewer always starts reading a card from its header, never partway
+// down (Masa 2026-07-20). Uses a rect delta so it works regardless of offsetParent.
+function saScrollCardToTop(el){
+  const body = document.querySelector('#seeall .sa-body');
+  if (!body || !el) return;
+  body.scrollTop += el.getBoundingClientRect().top - body.getBoundingClientRect().top;
+}
 function saRender(){
   const S = $('seeall'); if (!S) return;
   const pending = SA.items.filter((x) => x.st === 0);
   const green = pending.filter((x) => !x.testResult || x.testResult.ok !== false).length;
-  const proofN = pending.filter((x) => x.proofImg).length;
-  const c = `${pending.length} ready · ${green === pending.length ? 'all green' : `${pending.length - green} failing`} · ${proofN} with proof`;
+  const c = `${pending.length} ready · ${green === pending.length ? 'all green' : `${pending.length - green} failing`}`;
   const curIdx = pending.findIndex((x) => x.goalId === SA.cur);
   const posLabel = pending.length && curIdx >= 0 ? `${curIdx + 1} / ${pending.length}` : '';
-  S.innerHTML = `<div class="sa-panel"><div class="sa-hd"><span class="t">Review</span><span class="c">${c}</span><span class="sa-count">${posLabel}</span><span class="keys">↑↓ MOVE · A APPROVE · D REJECT · ESC CLOSE</span><button class="sa-x" data-saact="close:0">✕</button></div>
+  S.innerHTML = `<div class="sa-panel"><div class="sa-hd"><span class="t">Review</span><span class="c">${c}</span><span class="sa-count">${posLabel}</span><span class="keys">↑↓ MOVE · A APPROVE · D DISMISS · ESC CLOSE</span><button class="sa-x" data-saact="close:0">✕</button></div>
     <div class="sa-body">${SA.items.length ? SA.items.map((it, i) => saItemHtml(it, i)).join('') : '<div class="sa-empty">Nothing waiting for review.</div>'}</div></div>`;
   // Native <details> would collapse on every rebuild (SSE tick, approve, etc.) —
   // track what's open per goal+section in SA.detOpen (persists across re-renders)
   // and wire each one's toggle to keep that set current.
-  for (const det of S.querySelectorAll('.lg-goal[data-detkey]')) {
+  for (const det of S.querySelectorAll('[data-detkey]')) {
     det.addEventListener('toggle', () => {
       const k = det.dataset.detkey;
       if (det.open) SA.detOpen.add(k); else SA.detOpen.delete(k);
     });
   }
-  // H14: if a single-review focus is pending, center its card and flash a blue
-  // ring. Re-scroll on EVERY render for the focus window — the post-diff saSync
-  // rebuild resets .sa-body scroll to the top, so a one-shot scroll would land
-  // back at the top. Self-clears after ~1.4s (outlasts the localhost diff fetch)
-  // so live SSE ticks stop yanking the view.
+  // "ないものは載せない" (Masa): a Proof <img> whose file is missing/broken must never
+  // show a broken-image placeholder. proofImg is only a stored filename, so a file that
+  // isn't on disk (or fails to load) can only be caught at load time — drop the whole
+  // Proof section if any of its images fails. Real captures load and stay.
+  for (const im of S.querySelectorAll('.redesign .rproof img')) {
+    const drop = () => { const sec = im.closest('section.rsec'); if (sec) sec.remove(); };
+    if (im.complete && im.naturalWidth === 0) drop();
+    else im.addEventListener('error', drop, { once: true });
+  }
+  // H14: if a single-review focus is pending, bring the card's HEADER (#tag + In review
+  // + title) to the very top of the scroll area — never mid-card (Masa 2026-07-20) — and
+  // flash a blue ring. Re-scroll on EVERY render for the focus window — the post-diff
+  // saSync rebuild resets .sa-body scroll to the top, so a one-shot scroll would land back
+  // there. Self-clears after ~1.4s (outlasts the localhost diff fetch) so live SSE ticks
+  // stop yanking the view.
   if (SA.focusGid != null) {
     requestAnimationFrame(() => {
       const el = S.querySelector(`.sa-item[data-gid="${SA.focusGid}"]`);
       if (!el) return;
-      el.scrollIntoView({ block: 'center', behavior: 'auto' });
+      saScrollCardToTop(el);
       el.classList.add('sa-flash');
       if (!SA.focusTimer) SA.focusTimer = setTimeout(() => { SA.focusGid = null; SA.focusTimer = null; }, 1400);
     });
@@ -8153,25 +8482,6 @@ async function saFetchDiffs(){
   }));
   if (SA.open) saSync();
 }
-// Diff-box + zoom inset (HANDOFF-review-card-diffbox-2026-07-18): unlike
-// saFetchDiffs above (whose fetch settling triggers a full saSync()/saRender()), this
-// patches each card's already-rendered .beforeafter node directly — a full re-render
-// here would reset scroll, close every open <details>, and drop an in-progress reply
-// keystroke, all just to draw a box around one screenshot. SA.diffBoxHtml is still
-// cached per goal, so if a later SSE-driven saSync() does rebuild the card anyway,
-// saItemHtml() picks the cached result back up with no flicker to the plain pair.
-async function saEnhanceDiffBoxes(){
-  const targets = SA.items.filter((it) => it.proofImg && it.proofTaskId != null && it.proofBefore && SA.diffBoxHtml[it.goalId] === undefined);
-  await Promise.all(targets.map(async (it) => {
-    let html = null;
-    try { html = await beforeAfterBoxedHtml(it.proofTaskId, { png: it.proofImg, beforePng: it.proofBefore }, esc(it.title ?? '')); }
-    catch { html = null; } // a decode/pixel-diff failure is a fallback, never a crash
-    SA.diffBoxHtml[it.goalId] = html; // cache even on null — never re-attempt an already-answered "no box"
-    if (!html || !SA.open) return;
-    const host = document.querySelector(`.sa-item[data-gid="${it.goalId}"] .beforeafter`);
-    if (host) host.outerHTML = html; // html itself opens with the replacement .beforeafter
-  }));
-}
 function saOpen(focusGoalId){
   SA.open = true;
   SA.items = saRows().map((r) => ({ ...r, st: 0 }));
@@ -8192,8 +8502,15 @@ function saOpen(focusGoalId){
   }
   saRender();
   $('seeall').classList.add('open');
+  // Land the opening card's header at the very top of the scroll area (Masa 2026-07-20).
+  // With focusGid set, saRender's focus block already does this; without it (a plain
+  // "See all"), resolved one-liners can sit above the first pending card, so scrollTop 0
+  // isn't enough — scroll the current card's header to the top explicitly.
+  if (SA.focusGid == null && SA.cur != null) requestAnimationFrame(() => {
+    const el = $('seeall').querySelector(`.sa-item[data-gid="${SA.cur}"]`);
+    if (el) saScrollCardToTop(el);
+  });
   saFetchDiffs();
-  saEnhanceDiffBoxes();
 }
 function saClose(){ SA.open = false; SA.focusGid = null; if (SA.focusTimer) { clearTimeout(SA.focusTimer); SA.focusTimer = null; } $('seeall').classList.remove('open'); }
 // Continuous review (Masa 2026-07-08): move a persistent "current" card with
@@ -8252,10 +8569,34 @@ $('seeall').addEventListener('click', async (e) => {
   const it = SA.items[i];
   if (k === 'close') return saClose();
   if (!it) return;
-  // meta-row toggles (H23): tap the token chip to reveal the technique breakdown, tap
-  // Run-locally to reveal the honest "not wired yet" note — neither performs an action.
+  // meta-row toggle (H23): tap the token chip to reveal the technique breakdown.
   if (k === 'tok') { b.classList.toggle('open'); $('seeall').querySelector(`#tokdetail${i}`)?.classList.toggle('open'); return; }
-  if (k === 'runlocal') { $('seeall').querySelector(`#runlocal${i}`)?.classList.toggle('open'); return; }
+  // Open it (P1, Masa 2026-07-21): start the thing the worker said to run, then open it.
+  // Booting a real process takes seconds, so the chip says "Starting…" while it waits —
+  // and a failure lands in the detail line as a plain reason. It never turns the goal
+  // red: failing to boot a preview says nothing about whether the work itself is good.
+  if (k === 'preview') {
+    if (!it.run?.taskId) return;
+    const sec = $('seeall').querySelector(`#runlocal${i}`);
+    const link = $('seeall').querySelector(`#prevlink${i}`);
+    const v = sec?.querySelector('.v');
+    if (link) link.disabled = true;
+    if (v) v.textContent = 'Starting it…';
+    let r = null, body = {};
+    try { r = await fetch(`/api/tasks/${it.run.taskId}/preview`, { method: 'POST' }); body = await r.json().catch(() => ({})); }
+    catch (e) { body = { error: String(e.message ?? e) }; }
+    if (link) link.disabled = false;
+    const noteText = it.run.note || it.run.cmd || 'Starts when you press it.';
+    // A tab opened: that IS the feedback. Put the worker's own note back rather than
+    // narrating what the reviewer just watched happen.
+    if (r?.ok && body.url) { window.open(body.url, '_blank', 'noopener'); if (v) v.textContent = noteText; return; }
+    // Started, but the worker declared no URL: its own note is where it said to look.
+    if (r?.ok) { if (v) v.textContent = it.run.note || it.run.cmd || 'Started — nothing to open.'; return; }
+    if (v) v.textContent = `Could not start it: ${body.error || `HTTP ${r?.status ?? 'error'}`}`;
+    return;
+  }
+  // Verify chip (no external recording URL) → scroll the in-card video player into view.
+  if (k === 'rec') { $('seeall').querySelector(`.sa-item[data-gid="${it.goalId}"] .rvid`)?.scrollIntoView({ behavior: 'smooth', block: 'center' }); return; }
   // every verdict auto-advances the focus to the next pending card (tempo).
   if (k === 'app') { const bx = b.getBoundingClientRect(); return saAct(it, 1, 'approve').then((ok) => {
     if (!ok) return;