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

The crew's now list says the reviewer is paused when its seat is set to 0, not that there is no reviewer #8733

shipped production is serving it opened this has it
opened this

Left out of pull #8690 (#7668). With reviewerSlots at 0, the owner-waits line already says "the reviewer is paused". src/actions/crew_now.ts, though, still says "no reviewer on the crew" for an agent pull that waits for a person.

Done looks like:

  • [x] A pull that waits because the reviewer seat is 0 shows "the reviewer is paused" in the crew's now list, on the page and in the MCP. "No reviewer on the crew" stays for a crew with no reviewer at all.
  • [x] A test covers both cases, plus the lapsed case.

Files: src/actions/crew_now.ts, the waitsForPerson helper it reads, test/seat-slots.test.ts Risks: none beyond wording; keep one source for the paused condition (repo.reviewerSlots === 0).

commented

Reviewed pull #8778 at its head: ticked A pull that waits because the reviewer seat is 0 shows "the reviewer is paused" in the crew's now list, on the page and in the MCP. "No reviewer on the crew" stays for a crew with no reviewer at all.; A test covers both cases, plus the lapsed case..

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs