# Review Convergence Design

## Goal

Prevent a task with passing tests from looping through FIX for manual, delivery-hygiene, or follow-up observations.

## Design

`ReviewIssue` gains a backward-compatible `kind` field. Only `CODE_BLOCKER` and `TEST_BLOCKER` enter FIX. `MANUAL_ACCEPTANCE`, `DELIVERY_HYGIENE`, and `FOLLOW_UP` are persisted as advisory evidence and do not block REVIEW or FINAL_VERIFY. Existing artifacts without `kind` remain blocking.

The reviewer handoff explicitly defines these categories. The orchestrator records advisory issue IDs in events and the task record, while `open_issues` contains only blocking IDs. Identical blocking issue sets still stop as `STALLED`; fix-cycle limits remain a last-resort guard.

## Acceptance

- Non-blocking-only REVIEW advances to FINAL_VERIFY.
- Non-blocking-only FINAL_VERIFY advances to DONE.
- Mixed reviews enter FIX with only blocking IDs.
- Existing review artifacts remain compatible.
- Full server test suite passes.