/vt-d-4-review¶
Phase 4 - Comprehensive code review using the implementation-orchestrator. Runs a diff-scoped set of parallel reviewers (constitutional floor ∪ diff-triggers ∪ deliverable-type reviewers) for security, quality, performance, and patterns.
Appears in: knowledge-work / Phases, unified-product-development / Implementation phases
Implemented by: plugins/vt-product-dev/skills/workflow-4-review/SKILL.md
Runs in: context: fork
Output contract: multi-step
Agents¶
Routes to implementation-orchestrator (declared as agent: in its frontmatter, which is what gives a forked phase a routable identity — a context: fork skill without it falls back to the type name general-purpose, which is not an address; BUG-046).
Declared agent requirements (SPEC-189):
| Agent | Role |
|---|---|
| spec-compliance-reviewer | gate-critical |
| security-sentinel | gate-critical |
Reviewer roster, as the selector computes it¶
Derived by running plugins/vt-product-dev/skills/workflow-4-review/scripts/persona-select.sh, which the skill itself calls the single source of truth for which reviewers a review dispatches. The roster is diff-scoped, so it differs per deliverable type.
--types code — 13 reviewers
- accessibility-reviewer
- code-simplicity-reviewer
- data-integrity-guardian
- design-implementation-reviewer
- julik-frontend-races-reviewer
- kieran-python-reviewer
- kieran-typescript-reviewer
- pattern-recognition-specialist
- performance-oracle
- php-reviewer
- security-sentinel
- spec-compliance-reviewer
- tdd-compliance-reviewer
--types document — 3 reviewers
- document-quality-reviewer
- security-sentinel
- spec-compliance-reviewer
Hooks¶
Hooks are triggered by tools, not by phases — there is no phase binding to read. See the hooks reference for which hook intercepts which tool; any step that writes a file passes the Write/Edit hooks, and any step that runs a command passes the Bash hooks.