Get the FREE Ultimate OpenClaw Setup Guide →

requesting-code-review

Scanned
npx machina-cli add skill parthalon025/autonomous-coding-toolkit/requesting-code-review --openclaw
Files (1)
SKILL.md
1.5 KB

Requesting 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

  1. Step 1: Determine BASE_SHA (e.g., git rev-parse HEAD~1 or origin/main) and HEAD_SHA (git rev-parse HEAD)
  2. Step 2: Dispatch code-reviewer subagent with the Task tool, filling code-reviewer.md using the placeholders
  3. 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

Frequently Asked Questions

Add this skill to your agents
Sponsor this space

Reach thousands of developers