Cuándo usar este skill

• El usuario pide “revisá este PR” o “mirá este diff”.
• Se pegó un diff o git diff en la conversación.
• El usuario pide feedback de calidad sobre código ya escrito.

Proceso

1. Leer el diff completo (no sólo líneas cambiadas — mirar contexto ±5 líneas).
2. Aplicar el checklist FIT en este orden:
   1. Correctness: el cambio hace lo que dice.
   2. Seguridad: inyecciones, secrets, permisos.
   3. Performance: N+1, queries sin índice, bucles O(n²) sobre datos grandes.
   4. Legibilidad: nombres, funciones chicas, sin comentarios ruidosos.
   5. Tests: cambios no triviales requieren test asociado.
3. Emitir feedback en 3 bloques: Debe cambiar / Sugerencias / Nits.

Anti-patrones FACTORIT

• Agregar any sin comentario justificando.
• console.log en código productivo.
• Deps nuevas sin justificar en el PR description.
• Comments que repiten el código (“increment counter”).

Ejemplo

Input: diff con un fetchUsers() que itera y hace query por cada user.

Output esperado:

### Debe cambiar
- src/users.ts:12 — N+1 detectado. Hacer un solo query con IN (ids).

### Sugerencias
- El nombre `u` es poco descriptivo; preferir `user` aunque sea 1 línea.