The orchestrator treats exit 0 from pi-review.sh as approval and gets the HEAD commit hash. But REQUEST_CHANGES also exits 0 (it just adds feedback without committing). The orchestrator should either: 1. Check if there's actually a new commit (compare before/after HEAD) 2. Or pi-review.sh should exit with a distinct code for REQUEST_CHANGES (e.g. exit 2)
Current behavior: orchestrator marks task done even when reviewer requested changes.