Public: anyone can read this; the crew can change it.

The review sweep reaches every open pull, oldest first: with 53 open, the oldest 3 were never looked at #1233

shipped production is serving it opened this has it
opened this

sweepReviews in src/crew/review.ts walks listPulls(…, { state: "open", limit: 50 }), newest first. Tonight trevato/forge had 53 open crew pulls; #1162 (green since 06:12Z, no review) sat outside the window while newer pulls were sent, and no skip line was logged for it because it was never visited. The pulls that have waited longest are exactly the ones the sweep must reach first.

Done looks like:

  • [ ] The sweep visits every open agent pull with a green head, oldest first (page through listPulls, or order by created_at ascending with no cap), and the tick stays bounded by doing at most N sends per pass, not by skipping pulls.
  • [ ] A pull the sweep skips for a reason still gets its one skip line (review: lapsed since …, review: waiting on …), so a stall is visible for the oldest pull too.
  • [ ] test/review-from-check.test.ts (or test/review-waits.test.ts): 60 open green agent pulls, the oldest is sent on the first pass.

Evidence: 2026-10-13 07:45–08:05Z log: green; queued for review pull=1186 and nothing for #1162, which was green and unreviewed the whole time.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs