Revisión de código con criterio senior

Cuándo usarlo

Antes de mergear código generado por IA o cuando heredas un archivo que nadie quiere tocar.

Actúa como ingeniero senior revisando un pull request. Tu prioridad es correctitud, seguridad y mantenibilidad, en ese orden.

Contexto del proyecto: {{contexto}}
Lenguaje y stack: {{stack}}

Código:
```
{{codigo}}
```

Entrega:

1. BLOQUEANTES: bugs, condiciones de carrera, fugas de datos o fallos de seguridad. Cada uno con línea, explicación y corrección concreta.
2. RIESGOS: lo que funciona hoy pero va a doler (acoplamiento, falta de tests, manejo de errores, rendimiento bajo carga).
3. MEJORAS: solo si aportan valor medible. Sin preferencias de estilo.
4. TESTS que faltan, con el caso concreto que rompería el código actual.
5. VEREDICTO: aprobar, aprobar con cambios, o rechazar. Justifica en dos frases.

Reglas: cita el fragmento exacto que comentas. Si no estás seguro de un bug, dilo y explica cómo comprobarlo. No reescribas el archivo entero.

Variables que debes rellenar

Consejos

• ·Pega el diff completo, no solo el archivo: sin contexto inventa dependencias.
• ·Pide los bloqueantes primero y en sesión aparte las mejoras; mezclados se diluyen.
• ·Toda corrección que te dé, pídela como parche con el código exacto antes y después.