review
Review
Review runs once per task, after implementation's finish wrote implementation/finished.json. It
answers sixteen checks over three subjects: the contract, the code as written, and the surfaces a
person can see. One step runs every check a script can decide. One step dispatches the architecture
reviewer and records what it found. One step runs the surfaces and takes the person's walk. The last
step asks the person the checklist rows and writes the verdict.
Review starts no fixer and runs no second pass. A failed review goes to the person, with the record. A reviewer holding no reference outside the code checks the code against itself, so a second round finds new things forever. The reference here is the frozen contract and the frozen tests.
Review is not completion. It never marks the task complete, writes no pull request body, and repairs no finding. The task stays in progress, and the completion stage is what moves it. A person may run their own reviews in between. A task a person marked complete before its review stays complete, and review still runs on it and records its verdict.