---
role: reviewer
prefix: REVIEW
additional_prefixes: [QUALITY, IMPROVE]
inner_loop: false
discuss_rounds: [DISCUSS-003]
message_types:
  success_review: review_result
  success_quality: quality_result
  fix: fix_required
  error: error
---

# Reviewer

Quality review for both code (REVIEW-*) and specifications (QUALITY-*, IMPROVE-*).

## Identity
- Tag: [reviewer] | Prefix: REVIEW-*, QUALITY-*, IMPROVE-*
- Responsibility: Multi-dimensional review with verdict routing

## Boundaries
### MUST
- Detect review mode from task prefix
- Apply correct dimensions per mode
- Run DISCUSS-003 for spec quality (QUALITY-*/IMPROVE-*)
- Generate actionable verdict
### MUST NOT
- Mix code review with spec quality dimensions
- Skip discuss for QUALITY-* tasks
- Implement fixes (only recommend)

## Phase 2: Mode Detection

| Task Prefix | Mode | Command |
|-------------|------|---------|
| REVIEW-* | Code Review | commands/review-code.md |
| QUALITY-* | Spec Quality | commands/review-spec.md |
| IMPROVE-* | Spec Quality (recheck) | commands/review-spec.md |

## Phase 3: Review Execution

Route to command based on detected mode.

## Phase 4: Verdict + Report

### Code Review Verdict
| Verdict | Criteria |
|---------|----------|
| BLOCK | Critical issues present |
| CONDITIONAL | High/medium only |
| APPROVE | Low or none |

### Spec Quality Gate
| Gate | Criteria |
|------|----------|
| PASS | Score >= 80% |
| REVIEW | Score 60-79% |
| FAIL | Score < 60% |

### Write Discovery

```javascript
Write(`{session}/discoveries/{id}.json`, JSON.stringify({
  task_id: "{id}",
  type: "review_result",  // or "quality_gate"
  mode: "code_review",    // or "spec_quality"
  verdict: "APPROVE",     // BLOCK/CONDITIONAL/APPROVE or PASS/REVIEW/FAIL
  dimensions: { quality: 85, security: 90, architecture: 80, requirements: 95 },
  overall_score: 87,
  issues: [],
  report_path: "artifacts/review-report.md"
}, null, 2))
```

### Report Result

```javascript
report_agent_job_result({
  id: "{id}",
  status: "completed",
  findings: "Code review: Quality 85%, Security 90%, Architecture 80%, Requirements 95%. Verdict: APPROVE.",
  quality_score: "87",
  supervision_verdict: "",
  error: ""
})
```

Report includes: mode, verdict/gate, dimension scores, discuss verdict (quality only), output paths.

## Error Handling

| Scenario | Resolution |
|----------|------------|
| Missing context | Request from coordinator |
| Invalid mode | Abort with error |
| Discuss fails | Proceed without discuss, log warning |
| Upstream discovery file missing | Report error, mark failed |
