Fix workflow: pass_audit→pending_approval, fail_audit→todo

Tinker

Home Epics Fix workflow: pass_audit→pending_approval, fail_audit→todoEdit

Cancel

Title *

Project *

Description

Workflow fixes needed for the review/audit process.

**Issue 1: fail_audit goes to wrong state**
- Currently: `fail_audit` → `in_progress` (incorrect)
- Should be: `fail_audit` → `todo`
- Worker should not continue working without addressing reviewer feedback

**Issue 2: pass_audit should go to pending_approval, not "review"**
- Currently: `pass_audit` → `review` (vague/confusing)
- Should be: `pass_audit` → `pending_approval`
- Clear semantic: awaiting final product owner sign-off

**Corrected Workflow:**
```
pending_audit → (pass_audit) → pending_approval  [Reviewer approves, awaiting PO]
pending_audit → (fail_audit) → todo             [Rejected with feedback, re-queue]
```

**Files Likely to Need Changes:**
- State machine configuration (AASM) in Ticket model
- Workflow/transitions logic
- Any controller handling ticket state transitions

**Acceptance Criteria:**
- fail_audit transition moves tickets to `todo` (not `in_progress`)
- pass_audit transition moves tickets to `pending_approval` (not `review`)
- Remove/alias "review" status to "pending_approval"
- Failing audit requires a comment explaining why
- Reviewer feedback/comments are preserved when returning to todo
- Tickets can be claimed from todo after failed audit