requesting-code-review
Scannednpx machina-cli add skill parthalon025/autonomous-coding-toolkit/requesting-code-review --openclawRequesting Code Review
Dispatch superpowers:code-reviewer subagent to catch issues before they cascade.
Core principle: Review early, review often.
When to Request Review
Mandatory:
- After each task in subagent-driven development
- After completing major feature
- Before merge to main
Optional but valuable:
- When stuck (fresh perspective)
- Before refactoring (baseline check)
- After fixing complex bug
How to Request
1. Get git SHAs:
BASE_SHA=$(git rev-parse HEAD~1) # or origin/main
HEAD_SHA=$(git rev-parse HEAD)
2. Dispatch code-reviewer subagent:
Use Task tool with superpowers:code-reviewer type, fill template at code-reviewer.md
Placeholders:
{WHAT_WAS_IMPLEMENTED}- What you just built{PLAN_OR_REQUIREMENTS}- What it should do{BASE_SHA}- Starting commit{HEAD_SHA}- Ending commit{DESCRIPTION}- Brief summary
3. Act on feedback:
- Fix Critical issues immediately
- Fix Important issues before proceeding
- Note Minor issues for later
- Push back if reviewer is wrong (with reasoning)
Integration with Workflows
Subagent-Driven Development:
- Review after EACH task
- Catch issues before they compound
Executing Plans:
- Review after each batch (3 tasks)
See template at: requesting-code-review/code-reviewer.md
Source
git clone https://github.com/parthalon025/autonomous-coding-toolkit/blob/main/skills/requesting-code-review/SKILL.mdView on GitHub Overview
This skill streamlines requesting a code review at key points in development. It helps catch issues early and ensure work aligns with requirements before merging or releasing.
How This Skill Works
You define a baseline and end point with git SHAs, then dispatch the code-reviewer subagent using a filled template. The reviewer returns structured feedback, which you triage by priority (Critical, Important, Minor) and address accordingly to keep work compliant with requirements.
When to Use It
- After each task in subagent-driven development
- After completing a major feature
- Before merging to main
- When stuck and you need a fresh perspective
- Before refactoring to establish a baseline
Quick Start
- Step 1: Determine BASE_SHA (e.g., git rev-parse HEAD~1 or origin/main) and HEAD_SHA (git rev-parse HEAD)
- Step 2: Dispatch code-reviewer subagent with the Task tool, filling code-reviewer.md using the placeholders
- Step 3: Act on feedback: fix Critical immediately, fix Important before proceeding, note Minor for later, and push updates (re-run review if needed)
Best Practices
- Capture BASE_SHA and HEAD_SHA for every request
- Fill the code-reviewer.md template precisely with WHAT_WAS_IMPLEMENTED, PLAN_OR_REQUIREMENTS, DESCRIPTION, BASE_SHA, and HEAD_SHA
- Provide a concise DESCRIPTION and clear PLAN_OR_REQUIREMENTS so reviewers can verify scope
- Prioritize feedback: fix Critical first, then Important, and note Minor issues for later
- Push back with reasoning if you disagree with reviewer feedback and document the rationale
Example Use Cases
- After implementing a new authentication feature, request a code review before merging to main
- Before merging a bug fix that touches security-critical code paths
- During a refactor to establish a safe baseline and catch regressions early
- After completing a batch of changes in a multi-task plan to ensure consistency
- When you’re stuck and need a fresh perspective on implementation details