Skip to content
cccskills
FreePublished on GitHub

requesting-code-review

Request a focused code review after implementation or before merging. Give a separate reviewer agent the requirements, change summary, and commit range, then address findings according to their severity before continuing.

OriginalShow the original description

Use when completing tasks, implementing major features, or before merging to verify work meets requirements

See how to install

Good for

  • Reviewing a completed major feature
  • Checking changes before merging into main
  • Reviewing a complex bug fix
  • Checking existing code before refactoring

Files included (2)

  • SKILL.md2.9 KB
  • code-reviewer.md5.5 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

Requesting Code Review

Dispatch a code reviewer subagent to catch issues before they cascade. The reviewer gets precisely crafted context for evaluation — never your session's history.

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:

Dispatch a general-purpose subagent, filling the template at code-reviewer.md

Placeholders:

  • {DESCRIPTION} - Brief summary of what you built
  • {PLAN_OR_REQUIREMENTS} - What it should do
  • {BASE_SHA} - Starting commit
  • {HEAD_SHA} - Ending commit

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)

Example

[Just completed Task 2: Add verification function]

You: Let me request code review before proceeding.

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[Dispatch code reviewer subagent]
  DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types
  PLAN_OR_REQUIREMENTS: Task 2 from docs/superpowers/plans/deployment-plan.md
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661

[Subagent returns]:
  Strengths: Clean architecture, real tests
  Issues:
    Important: Missing progress indicators
    Minor: Magic number (100) for reporting interval
  Assessment: Ready to proceed

You: [Fix progress indicators]
[Continue to Task 3]

Common Rationalizations

ExcuseReality
"I'll just review the diff myself instead of dispatching a reviewer"You're the coordinator — reviewing the diff inline burns the context window you need to keep driving the work. Dispatch a reviewer subagent: the diff and the evaluation live in its context, and only the findings come back to you.
"The reviewer needs my whole session history to understand the change"Hand it precisely crafted context, never your session's history. That keeps the reviewer on the work product, not your thought process.

Red Flags

Never:

  • Skip review because "it's simple"
  • Ignore Critical issues
  • Proceed with unfixed Important issues
  • Argue with valid technical feedback

If reviewer wrong:

  • Push back with technical reasoning
  • Show code/tests that prove it works
  • Request clarification

See template at: code-reviewer.md

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

brainstorming

FreeSummary

Turn development ideas into clear designs through guided discussion. The skill clarifies goals, requirements, and constraints, then adapts the process to a feasibility investigation, a small existing-code change, or a larger architectural design.

  • Clarifying requirements before adding a feature
  • Exploring whether an idea is feasible
  • Comparing designs for a new project
spacering-net/codeg4KUpdated Oct 20, 2026

Find academic papers, retrieve publication details, and build BibTeX reference files for scholarly writing. Check identifiers, missing information, duplicate entries, and mismatches between manuscript citations and the bibliography, with tools for formatting and validation.

  • Finding papers for a manuscript or thesis
  • Converting paper identifiers into BibTeX entries
  • Cleaning duplicate or incomplete references
spacering-net/codeg4KUpdated Oct 20, 2026

Assign independent bugs or test failures to separate agents that work at the same time. Define each agent’s scope and expected report, then check for conflicting changes and run the full test suite before integrating fixes.

  • Investigating test failures with unrelated causes
  • Fixing independent problems in separate subsystems
  • Defining focused tasks for multiple agents
spacering-net/codeg4KUpdated Oct 20, 2026

executing-plans

FreeSummary

Carry out a written implementation plan in a separate session, reviewing it before starting and tracking each task through verification. The workflow pauses for blockers and hands completed work to a development branch finishing process.

  • Executing a written implementation plan
  • Tracking task progress and verification
  • Reviewing gaps before implementation
spacering-net/codeg4KUpdated Oct 20, 2026

Plan experiments before collecting data by choosing how to assign samples, combine conditions, and order runs. Generate reproducible allocation schedules and experimental layouts while checking for design flaws that could make results difficult to interpret.

  • Assigning subjects or samples to comparison groups
  • Screening many factors to identify promising ones
  • Planning around batch, day, or plate-position differences
spacering-net/codeg4KUpdated Oct 20, 2026

Explore scientific data files to understand their structure, statistical patterns, and quality. References cover over 200 formats, with workflows for producing Markdown reports that summarize findings and recommend preprocessing, visualizations, and further analysis.

  • Checking missing values and outliers in experimental tables
  • Examining sequence lengths and quality scores in FASTQ files
  • Inspecting microscopy image dimensions and metadata
spacering-net/codeg4KUpdated Oct 20, 2026

See all skills by spacering-net

Report a problem with this skill