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.
Converted and distributed by TomeVault — claim your Tome and manage your conversions.