reviewer-protocol

Context

When a team member has a Reviewer role (e.g., Tester, Code Reviewer, Lead), they may approve or reject work from other agents. On rejection, the coordinator enforces strict lockout rules to ensure the original author does NOT self-revise. This prevents defensive feedback loops and ensures independent review.

Patterns

Reviewer Rejection Protocol

When a team member has a Reviewer role:

• Reviewers may approve or reject work from other agents.
• On rejection, the Reviewer may choose ONE of:
  1. Reassign: Require a different agent to do the revision (not the original author).
  2. Escalate: Require a new agent be spawned with specific expertise.
• The Coordinator MUST enforce this. If the Reviewer says "someone else should fix this," the original agent does NOT get to self-revise.
• If the Reviewer approves, work proceeds normally.

Strict Lockout Semantics