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

A person can ask the crew to review their pull (route, button and @mention); today person-authored pulls are never reviewed #4568

open deferred: behind the Later gate opened this
Held by the Later gate, set by trevato . The crew picks nothing here until it is lifted.
opened this

Probe, 2026-10-21 17:40Z. Pulls authored by a person never get a crew review. sweepReviews and sendToReview skip any pull where !pull.authorIsAgent (src/crew/review.ts:303, :376). There is no route to ask for one (no /pulls/:n/reviewers or review-request route), and the author can't approve their own pull (self_review). So arm #491 and #502, steward-carried fixes with a green check, sit until the owner reviews them by hand.

Done when:

  • A person can ask the crew to review: POST /api/v1/repos/:o/:n/pulls/:number/review-requests {reviewer?: "<agent>"}, plus a "Ask the crew to review" button on the pull page. Without a named reviewer it goes to the least-busy reviewer (#4486).
  • The review runs like an agent pull's: the same rounds and verdicts, and an approval lets landIfGreen land it.
  • An @mention of a crew reviewer in a pull comment does the same.
  • Test: a person-authored green pull, a requested review, the reviewer approves, and the pull lands.
commented

Parked by the crew: the check failed ("a variant run opens a pull that carries its group > two variants open two pulls in the returned group, on different branches") and the decider saw no point in another try (pull #4569 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.

commented

Parked by the crew: the check failed ("(unnamed)") and the decider saw no point in another try (pull #4574 closed).

Parked twice since a person last spoke here, so it is the planner's now. Every reason:

  1. the check failed ("a variant run opens a pull that carries its group > two variants open two pulls in the returned group, on different branches") and the decider saw no point in another try (pull #4569 closed)
  2. the check failed ("(unnamed)") and the decider saw no point in another try (pull #4574 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.

commented

Live triage (automatic, not written by a person): no pillar named, so it waits behind the Later gate; name a pillar to move it.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs