Requesting Code Review
Dispatch a reviewer subagent on completed work. Feed it crafted context (description + requirements + git range) — never your session history. Keeps the reviewer on the work product and preserves your own context.
Review early, review often.
Brian's stack already ships a purpose-built code-reviewer agent + the Agent Diversity Review gate ([[agent-selection]]). Prefer the named agent over a bare general-purpose spawn; this skill is the request protocol that complements both.
When to request
Mandatory:
- Before merge to main
- After each task in subagent-driven development
- After completing a major feature
Optional: when stuck (fresh eyes), before a refactor (baseline), after a complex bugfix.
How to request
- Get SHAs —
BASE_SHA=$(git rev-parse origin/main),HEAD_SHA=$(git rev-parse HEAD). - Spawn the
code-revieweragent (orgeneral-purposefilling code-reviewer.md). - Fill placeholders:
{DESCRIPTION}(what you built),{PLAN_OR_REQUIREMENTS}(what it should do),{BASE_SHA},{HEAD_SHA}.
Act on feedback
- Fix Critical immediately; fix Important before proceeding.
- Note Minor for later.
- Push back with technical reasoning if the reviewer is wrong — see
[[receiving-code-review]].
Never
- Skip review because "it's simple".
- Ignore Critical, or proceed with unfixed Important.
- Argue with valid technical feedback.