# Requesting Code Review

> Úsalo al completar tareas, implementar features mayores, o antes de mergear, para verificar que el trabajo cumple los requisitos.

Fuente: https://skillsagentes.com/skills/obra/superpowers/requesting-code-review
Markdown: https://skillsagentes.com/skills/obra/superpowers/requesting-code-review.md

Repositorio: https://github.com/obra/superpowers
Autor: obra
Licencia: MIT
Actualizado: el mes pasado
Coste de contexto: 27 tok instalada, 739 tok al activarse, 2.2k tok con todos los archivos del bundle
Bundle: 2 archivos, 8 KB
Permisos que pide: ninguno declarado

## Instalación

Un skill son archivos markdown: los mismos archivos valen para cualquier agente y lo único que cambia es el directorio de destino, es decir la bandera `--agent`. Añade `-g` para instalarlo en todos los proyectos de la máquina.

```bash
# Claude Code
npx -y skills add obra/superpowers --skill requesting-code-review --agent claude-code
# Cursor
npx -y skills add obra/superpowers --skill requesting-code-review --agent cursor
# Codex
npx -y skills add obra/superpowers --skill requesting-code-review --agent codex
# Gemini CLI
npx -y skills add obra/superpowers --skill requesting-code-review --agent gemini
# Windsurf
npx -y skills add obra/superpowers --skill requesting-code-review --agent windsurf
# Cline
npx -y skills add obra/superpowers --skill requesting-code-review --agent cline
```

## Qué hace

- Dispatcha un subagente code reviewer con contexto crafteado (no el historial de la sesión) para revisar cambios entre dos SHAs de git
- Da un flujo para actuar sobre el feedback: arreglar Critical de inmediato, Important antes de seguir, anotar Minor para después
- Provee una plantilla (code-reviewer.md) con placeholders DESCRIPTION, PLAN_OR_REQUIREMENTS, BASE_SHA, HEAD_SHA

## Cuándo usarla

- Después de cada tarea en desarrollo dirigido por subagentes
- Al completar una feature mayor
- Antes de mergear a main
- Cuando estás atascado, antes de refactorizar, o después de arreglar un bug complejo

## Qué la activa

- "Terminé la Tarea 2, pide una revisión de código antes de seguir"
- "Antes de mergear a main, dispatch un code reviewer subagent"
- "Revisa este cambio complejo con un reviewer fresco antes de refactorizar"

## Antes de instalar

- Requiere un repositorio git con SHAs accesibles (BASE_SHA y HEAD_SHA) y la plantilla code-reviewer.md.
- Necesita en el PATH: awk, git

## Archivos

- SKILL.md — 3 KB
- code-reviewer.md — 6 KB

## SKILL.md

Reproducido tal cual desde obra/superpowers bajo MIT. Esta sección es el documento original y está en inglés.

# 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:**
```bash
BASE_SHA=$(git rev-parse HEAD~1)  # or: git merge-base origin/main HEAD
HEAD_SHA=$(git rev-parse HEAD)
```

**2. Dispatch code reviewer subagent:**

Dispatch a `general-purpose` subagent, filling the template at [code-reviewer.md](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

| Excuse | Reality |
|--------|---------|
| "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](code-reviewer.md)

## Dónde encaja

- Categoría: [Testing y QA](https://skillsagentes.com/categorias/testing-qa.md) — Flujos de testing unitario, de integración y end-to-end.
- Creador: [obra](https://skillsagentes.com/creators/obra.md) — 14 skills en el directorio
- [Todas las skills](https://skillsagentes.com/skills.md)
- [Ranking de instalaciones](https://skillsagentes.com/ranking.md)

## Otras skills del mismo repositorio

- [Subagent Driven Development](https://skillsagentes.com/skills/obra/superpowers/subagent-driven-development.md): Úsalo al ejecutar planes de implementación con tareas independientes dentro de la sesión actual.
- [Brainstorming](https://skillsagentes.com/skills/obra/superpowers/brainstorming.md): Debes usarlo antes de cualquier trabajo creativo: crear features, construir componentes, añadir funcionalidad o modificar comportamiento. Explora la intención, requisitos y diseño antes de implementar.
- [Using Superpowers](https://skillsagentes.com/skills/obra/superpowers/using-superpowers.md): Úsala al iniciar cualquier conversación: establece cómo encontrar y usar skills, exigiendo su invocación antes de CUALQUIER respuesta, incluso preguntas aclaratorias.
- [Writing Skills](https://skillsagentes.com/skills/obra/superpowers/writing-skills.md): Úsala al crear nuevas skills, editar skills existentes o verificar que funcionan antes de desplegarlas.
- [Finishing A Development Branch](https://skillsagentes.com/skills/obra/superpowers/finishing-a-development-branch.md): Úsalo cuando la implementación esté completa, todos los tests pasen, y necesites decidir cómo integrar el trabajo.

## Skills relacionadas

- [Receiving Code Review](https://skillsagentes.com/skills/obra/superpowers/receiving-code-review.md): Úsalo al recibir feedback de code review, antes de implementar sugerencias, sobre todo si el feedback parece poco claro o técnicamente cuestionable: exige rigor técnico y verificación, no acuerdo performativo ni implementación ciega.
- [Systematic Debugging](https://skillsagentes.com/skills/obra/superpowers/systematic-debugging.md): Úsalo ante cualquier bug, fallo de test o comportamiento inesperado, antes de proponer arreglos.
- [Test Driven Development](https://skillsagentes.com/skills/obra/superpowers/test-driven-development.md): Úsalo al implementar cualquier feature o corrección de bug, antes de escribir código de implementación.
- [Verification Before Completion](https://skillsagentes.com/skills/obra/superpowers/verification-before-completion.md): Úsalo antes de afirmar que un trabajo está completo, corregido o pasando, antes de hacer commit o crear PRs: exige ejecutar comandos de verificación y confirmar la salida antes de cualquier afirmación de éxito.
- [Brainstorming](https://skillsagentes.com/skills/obra/superpowers/brainstorming.md): Debes usarlo antes de cualquier trabajo creativo: crear features, construir componentes, añadir funcionalidad o modificar comportamiento. Explora la intención, requisitos y diseño antes de implementar.

---

Skills Agentes · [Índice de páginas en markdown](https://skillsagentes.com/sitemap.md) · [Inicio](https://skillsagentes.com/index.md)