Public: anyone can read this; the crew can change it.
A crew reviewer's changes-requested queues the builder's follow-up within one tick, with the review as its brief #1175
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
Part of #1161 (P7 Command): tonight Sentinel's changes-requested on Mason's pulls (speed-to-lead, listing-launch, deal-clock) started nothing until a person commented, though
src/crew/dispatch.ts(pull.reviewed, ~line 390) means to queue a feedback run for a reviewer's verdict.Done looks like.
emailVerifiedfor an agent actor,skipsFeedback/nothingNewinsrc/crew/guard.tsnot counting an agent's review as new,agentForon a pull, or the run cap — and fix that step only.autonomousandassisted, a crew reviewer'schanges_requestedon a crew pull queues the builder'sfeedbackrun with no person speaking; the run's brief carries the review body and its line comments.off, nothing is queued.test/crew-loop.test.ts: review → follow-up queued with no comment (fails on today's main);offwaits.nowlists the run with its reasonfeedbackand the review it answers).Files.
src/crew/dispatch.ts,src/crew/guard.ts,src/crew/brief.ts,test/crew-loop.test.ts.Risks. Waking the builder on every verdict could loop with the reviewer; the three-round bound is the next issue, so keep the run cap and breaker in force here.
Fits. Mason (builder brief).
Before pushing.
bun x tsc --noEmitand the test files named here, quoted in the pull. One pull, green on its first push.One pillar: this serves exactly the one it is linked under. Done means reachable on the page, through MCP, in a test and by eye (#334).