Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions docs/docs/guides/getting-started/ai-development.md
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,21 @@ re-verify their own findings against the pushed code (nothing is closed on the f
alone) β†’ after at most 4 fix rounds, the PR either converges (CI green, all review threads
resolved) or is handed to a human with the open disagreements listed.

**How deep-review findings reach the fixer.** The deep review posts on two channels, and which
one a finding takes decides whether the loop acts on it. Blocking findings (πŸ”΄ Critical, 🟠
Major, 🟑 Minor) are filed as **inline review comments** on the lines they concern; those
threads are the fix agent's work items, and it answers each with `Fixed in <sha>` and
deliberately does not resolve it β€” the reviewer re-checks that claim in its `verify` pass and
resolves only what is genuinely fixed. πŸ”΅ Advisory findings never go inline: they appear only as
Comment thread
allxsmith marked this conversation as resolved.
Resolved
rows in the summary review (the one carrying the `<!-- claude-deep-review -->` marker), because
an inline thread is an actionable work item and advisories must not spin the loop.

One consequence is worth knowing before reading a run's job list: **a deep review that reports
`0 blocking` creates no threads, so the `verify` pass has nothing to check and does not run.**
That is the design working rather than a stalled loop β€” most reviews do land at zero blocking.
`verify` missing from a run means the review found nothing that blocks, not that verification
was skipped.
Comment thread
allxsmith marked this conversation as resolved.
Resolved

**Screenshots at handoff.** When the loop flips a PR to `needs-human-review` it also
dispatches a screenshot pass (`story-screenshots.yml`): Playwright captures the Storybook
stories affected by the PR's changed files β€” once light, once dark β€” and posts them to the PR
Expand Down
Loading