Realiza una revisión de código exhaustiva como un senior developer, detectando bugs, problemas de seguridad, performance y deuda técnica.
Cuándo usarlo: Antes de hacer merge en main, en pull requests o cuando revisas código de un junior.
Herramienta recomendada: Claude
Actúa como un senior software engineer con 10+ años de experiencia en code reviews. Revisa el siguiente código: ```[LENGUAJE] [PEGA EL CÓDIGO AQUÍ] ``` Contexto: - Propósito del código: [QUÉ HACE] - Stack tecnológico: [STACK] - Entorno: [PRODUCCIÓN / DESARROLLO / MICROSERVICIO / etc.] - Tráfico esperado: [BAJO / MEDIO / ALTO] Realiza una code review completa evaluando: ## 🔴 CRÍTICO (bloquea el merge) - Bugs que causarán fallos en producción - Vulnerabilidades de seguridad (SQLi, XSS, IDOR, secrets expuestos) - Race conditions o problemas de concurrencia ## 🟡 IMPORTANTE (debe corregirse pronto) - Problemas de performance (N+1, llamadas innecesarias, sin índices) - Manejo de errores incompleto - Código no testeable ## 🟢 SUGERENCIAS (mejora la calidad) - Naming y legibilidad - Principios SOLID violados - Duplicación de lógica (DRY) - Complejidad ciclomática alta ## ✅ LO QUE ESTÁ BIEN (Reconoce 2-3 cosas bien hechas) ## RESUMEN - Puntuación general: X/10 - ¿Apruebas el merge? SÍ / NO / CON CAMBIOS - Top 3 cambios prioritarios