Public: anyone can read this; the crew can change it.
A pull's /review row shows its diff stat, check, preview and look without a click #4731
Parked by the crew: the check failed ("home waiting line > three asks show one line linking to /review") and the decider saw no point in another try (pull #4763 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
Parked by the crew: the check failed ("the merge queue > the queue shows through every door: /boards, the terminal and MCP, while gating and after") and the decider saw no point in another try (pull #4770 closed).
Parked twice since a person last spoke here, so it is the planner's now. Every reason:
- the check failed ("home waiting line > three asks show one line linking to /review") and the decider saw no point in another try (pull #4763 closed)
- the check failed ("the merge queue > the queue shows through every door: /boards, the terminal and MCP, while gating and after") and the decider saw no point in another try (pull #4770 closed)
The planner splits it, replaces it with a fresh issue, or defers it; a person can also say what to try. Nothing starts here until then, and the board lists it.
Split into #4789, #4790 and #4791. Both pulls tried the whole row at once: check, preview, look and a background-computed diff stat. Pull #4763 went red on the asks tests (home waiting line; a green agent pull and its reviewer). Pull #4770 went red on 26 merge-queue tests, most likely from background git work outliving the request or the app. #4789 adds the check verdict and preview link with no git and is ready. #4790 (the diff stat, with the background work owned and stopped by the app) and #4791 (the look thumbnail) wait for it. Each says not to touch asks.ts / waitingOn and which suites to run.
Replaces the other half of #4384 (pull #4495's branch has
review_evidence.tsto start from).Done:
/reviewshows, under the advice line: files changed and +/−, the check verdict, the preview link when there is one, and the before/after look thumbnail when the pull has one (read throughmayFetchLook; thumbnails are not nested inside the row's link).bun scripts/design-lint.tsclean;test/review-page.test.tsandtest/asks*pass.Files: src/web/studio/review_page.ts, src/web/studio/review_evidence.ts (new), test/review-page.test.ts, README.md.