Skip to content

docs: Update reviewers for multi-phase [doc-updater] #686

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jwbron merged 2 commits into from
Feb 15, 2026
+16 −9
Merged

docs: Update reviewers for multi-phase [doc-updater] #686

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8b627dd
docs: Update reviewer section for multi-phase execution
jwbron Feb 14, 2026
1d10ee9
docs: Clarify refine phase wording to single-worker
egg-reviewer[bot] Feb 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 16 additions & 9 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,18 +127,25 @@ During the **implement** phase, work is divided across these specialized agents:

**Execution model**: Wave-based with dependencies. The coder runs first, then tester and documenter run in parallel (both depend on coder's output). The integrator runs after the coder and tester complete (it does not wait for the documenter).

### Reviewer Roles (Implement Phase)
### Reviewer Roles

After the integrator completes, reviewers validate the implementation output:
Reviewers run as part of multi-agent wave execution in refine, plan, and implement phases:

| Role | Responsibility |
|------|----------------|
| **Unified Reviewer** | Comprehensive review across all criteria |
| **Code Reviewer** | Security, correctness, code quality |
| **Contract Reviewer** | Verify acceptance criteria met, task completion status |
| **Agent Design Reviewer** | Check for agent-mode anti-patterns, autonomous operation capability |
**Refine Phase:**
- **Unified Reviewer**: Analysis quality and completeness
- **Agent Design Reviewer**: Agent-mode alignment and anti-patterns

**Plan Phase:**
- **Unified Reviewer**: Plan quality, task structure, acceptance criteria
- **Agent Design Reviewer**: Agent-mode alignment and anti-patterns

**Implement Phase:**
- **Unified Reviewer**: Comprehensive review across all criteria
- **Code Reviewer**: Security, correctness, code quality
- **Contract Reviewer**: Verify acceptance criteria met, task completion status
- **Agent Design Reviewer**: Check for agent-mode anti-patterns, autonomous operation capability

**Execution model**: All reviewers depend on the integrator and run in parallel after it completes, producing structured verdicts (approved/needs_revision). Reviewers currently only operate in the implement phase.
**Execution model**: Reviewers run in parallel as the final wave of each phase's dependency graph. In implement phase, reviewers depend on the integrator. In plan phase, reviewers depend on the task planner and risk analyst. In refine phase (single-worker), reviewers run after the coder completes.

### Plan Phase Roles

Expand Down