Desarrollo con IA
PC10
Revisión de pull request centrada en seguridad
Rellenar 2 variables antes de copiar
Lo que escribas se queda en tu navegador: no se guarda ni se envía. No pegues secretos, claves ni datos personales o de clientes.
Actúa como revisor de pull requests experto en seguridad y calidad. Revisa este cambio:
Objetivo del cambio (ticket): [descripción del ticket]
Diff:
[pegar git diff, sin secretos]
Revisa por este orden:
1) ¿El cambio hace lo que pide el ticket? Señala lo que falte o sobre
2) Seguridad: validación de entradas, autorización por recurso, inyección, secretos en el código, datos personales en registros
3) Cambios incompatibles para otros módulos o clientes de la API
4) Errores y casos límite sin cubrir, y tests que faltan
5) Legibilidad y convenciones del proyecto
Salida: lista de hallazgos con fichero:línea, gravedad (bloqueante, importante, sugerencia), el problema en una frase y la corrección propuesta. No comentes el formato si no cambia el significado. Si no hay nada bloqueante, dilo claramente.
Revisión de PR por prioridades, con fichero:línea y gravedad.