## Pre-Review Feedback
**Note**: Ticket is in `todo` status but PR #12 exists. Please submit via proper workflow: `start_work` → `submit_review` → `pending_audit`
### Code Review (PR #12):
**✓ Core fixes implemented correctly:**
- `pass_audit`: `review` → `pending_approval`
- `fail_audit`: `in_progress` → `todo`
- Tests updated (37/37 passing)
**✗ Missing acceptance criterion:**
> "Failing audit requires a comment explaining why"
**Suggested implementation** (add to `ticket.rb`):
```ruby
event :fail_audit do
transitions from: :pending_audit, to: :todo
before do
if comments.reviewer_feedback.none?
errors.add(:base, "A comment explaining the rejection is required")
throw :abort
end
end
after do
log_transition("fail_audit")
end
end
```
**Optional**: If you don't want the comment requirement enforced, update the acceptance criteria to remove it.
