Code Review Playbook
Repo anchors (autocodex)
- •INTERNAL_PATH:
internal/ - •DOCS_PATH:
docs/
When to use
- •Reviewing PRs or local diffs before merge.
Preconditions
- •You can see the diff and tests run.
Inputs to confirm
- •Intended behavior change
- •Test evidence
Required artifacts
- •Findings list ordered by severity
- •Open questions
- •Short summary
Quick path
- •Scan for correctness, then safety, then tests.
Steps
- •Identify behavior changes and edge cases.
- •Verify observability and error handling.
- •Check tests and docs.
Failure modes and responses
- •No tests: flag as blocking.
- •Hidden behavior changes: request clarification.
Definition of done
- •Findings are clear, actionable, and prioritized.
Example (minimal)
- •Finding: Missing error handling in plugin handshake.