Habilidad

Revisar un pull request

Revisa un diff buscando fallos reales, no preferencias de estilo.

v1.0.0 Desarrollo de software OpenClawClaudeChatGPT #código #revisión #calidad
revisar-un-pull-request

Objetivo

Encontrar en un cambio de código lo que va a romperse en producción, antes de que se fusione.

Instrucciones

Lees el diff completo y su contexto. Priorizas corrección sobre estilo: un comentario sobre comillas simples no vale lo mismo que una condición invertida.

Workflow

  1. Entiende qué intenta lograr el cambio.
  2. Busca errores de lógica: condiciones invertidas, casos límite, valores nulos.
  3. Busca problemas de seguridad: entrada sin validar, consultas concatenadas, secretos en el código.
  4. Busca fugas de recursos y consultas dentro de bucles.
  5. Comprueba si hay pruebas para el camino que se modificó.
  6. Ordena los hallazgos de mayor a menor gravedad.

Reglas

  • No señales estilo si el proyecto tiene formateador automático.
  • Cada hallazgo debe incluir un escenario concreto de fallo, no una sospecha.
  • Si el cambio es correcto, dilo en una línea y no inventes objeciones.
  • No propongas reescribir el archivo entero.

Inputs

  • diff: cambio a revisar
  • contexto: archivos relacionados, si están disponibles

Outputs

  • hallazgos: lista ordenada con archivo, línea, problema y escenario de fallo
  • veredicto: aprobar, aprobar con cambios menores, o solicitar cambios

Ejemplos

Un cambio que añade un filtro por usuario pero olvida el índice recibe un hallazgo de rendimiento con la consulta concreta que se degrada.

También te puede servir

0 Tu paquete 0 agentes · 0 habilidades