code-review-checklist

Resultado esperado:

Problema de Seguridad Encontrado:

**Problema:** Vulnerabilidad de Inyección SQL

El código usa interpolación de cadenas para construir una consulta SQL, permitiendo a atacantes inyectar código SQL malicioso a través del parámetro email.

**Riesgo:** Los atacantes podrían extraer, modificar o eliminar contenidos de la base de datos.

**Recomendación:** Usa consultas parametrizadas:
```javascript
const query = 'SELECT * FROM users WHERE email = $1';
db.query(query, [email]);
```

Resultado esperado:

Problema de Calidad de Código Encontrado:

**Problema:** Nomenclatura poco clara

- Los nombres de variables 'a', 'b', 'c' no son descriptivos
- El nombre de función 'calc' no explica qué se está calculando
- Los números mágicos deberían reemplazarse con constantes nombradas

**Recomendación:** Usa nombres descriptivos:
```javascript
function calculateTotalPrice(quantity, unitPrice, tax) {
return quantity * unitPrice + tax;
}
```

Resultado esperado:

Lista de Pre-Revisión:

- [ ] Leer la descripción del PR y los issues vinculados
- [ ] Entender qué problema se está resolviendo
- [ ] Verificar que los tests pasen en CI/CD
- [ ] Obtener la rama y ejecutarla localmente
- [ ] Revisar cualquier cambio de documentación relacionado