Prompts de revisión de código: 16 casos listos

Línea 1 · crítico · errorcart=[] se crea una sola vez, al importar el módulo, y no en cada llamada: el segundo cliente sigue llenando el carrito del primero, y así hasta que el proceso se reinicia.Corrección: cart: list | None = None y, como primera línea del cuerpo, if cart is None: cart = [].

Línea 3 · grave · errorif not balance atrapa por igual None, 0 y 0.0: quien ha gastado hasta el último céntimo ve "sin datos" en lugar de un cero honesto.Corrección: if balance is None — la ausencia se comprueba de forma explícita y el cero pasa al formateador como cualquier otro número.

Línea 2 · crítico · errordatetime.now() es hora local naive y expires_at llega de la base en UTC: en un servidor de Madrid los tokens viven dos horas de más y, frente a un valor aware, la comparación revienta directamente con TypeError.Corrección: datetime.now(timezone.utc) y una regla para todo el proyecto — lo que se guarda es aware y en UTC.

Líneas 15-16 · crítico · errorLa comprobación eliminada era el único límite de percent: con 150 el total del pedido se vuelve negativo y la caja lo convierte en una devolución.Corrección: devolver la comprobación o moverla a la validación de la petición, y añadir el caso percent=150 a los tests.

Línea 2 · crítico · seguridademail se pega al texto SQL: un valor con una comilla cierra la condición y todo lo que va detrás lo ejecuta la base como consulta tuya, DROP TABLE incluido.Corrección: un parámetro en vez de la concatenación — conn.execute("SELECT id, email FROM users WHERE email = %s", [email]); de paso desaparece el SELECT * que hoy se trae también el hash de la contraseña.

Líneas 2-3 · crítico · seguridadUn token de producción está escrito en un archivo versionado en git: lo tiene en su historial todo el que haya clonado el repositorio, y echo lo copia además al log de despliegue, que lee mucha más gente.Corrección: leerlo del entorno ($API_TOKEN sin valor por defecto), quitar el echo y dar la clave por filtrada — hay que rotarla.

Línea 3 · crítico · seguridados.path.join sale sin problemas de /var/app/uploads cuando el nombre es ../../etc/passwd, y una ruta absoluta descarta directamente el primer argumento: se descarga cualquier archivo que el proceso pueda leer.Corrección: os.path.basename(name), después comparar el os.path.realpath del resultado con la carpeta de subidas y servir el archivo solo si está dentro.

Líneas 3-5 · grave · rendimientoDos consultas por usuario: mil usuarios son dos mil idas y vueltas, y el tiempo se va en la red, no en los cálculos.Corrección: una consulta con JOIN y GROUP BY users.id, o dos consultas con IN y la unión hecha en memoria.

Líneas 3-4 · grave · rendimientodate_trunc sobre created_at deja el índice inservible y un LIKE con porcentaje inicial no puede usarlo en absoluto: queda un seq scan de toda la tabla, y SELECT * arrastra además columnas que nadie mira.Corrección: comparar created_at contra un rango (desde la medianoche del 1 de septiembre y menor que la medianoche del día 2), indexar el dominio aparte o guardarlo en su propia columna, y seleccionar solo los campos que se usan.

Línea 2 · grave · rendimientoread() sube el archivo completo a memoria, split lo duplica y la lista dentro de len() guarda una tercera copia: con un log de ocho gigas llega antes el OOM killer que el resultado. Además el archivo no se cierra nunca.Corrección: with open(path) as f y sum(1 for line in f if "ERROR" in line) — línea a línea y con memoria constante.

Líneas 2-5 · menor · legibilidadCuatro if anidados son una sola regla convertida en escalera: para saber a quién le llega el correo hay que sostener las cuatro condiciones a la vez, y PEP 8 pide la forma plana.Corrección: salidas tempranas — if user is None: return False y así sucesivamente — y el cuerpo se queda en un único nivel de sangría.

Líneas 1-4 · menor · estándaresNaming/MethodName: en Ruby los nombres de métodos van en snake_case y el CamelCase se lee aquí como una constante. Los nombres o, t e i no dicen nada de su contenido, y acumular a mano es justo el trabajo de sum.Corrección: def calc_total(items) e items.sum do |item| item.price * item.qty end — cuatro líneas se quedan en una.

Líneas 2 y 5 · grave · arquitecturaUna regla de negocio escrita dos veces ya se ha separado: la factura aplica el 20 por ciento y el recibo el 19, así que el cliente ve dos totales distintos para un mismo pedido.Corrección: una constante VAT_RATE y una función que usen las dos; así el tipo se cambia en un solo sitio.

Línea 1 · grave · testsHay un caso cubierto: un descuento normal sobre un pedido sin pagar. Nada de cero, ni cien por cien, ni valores negativos, ni un pedido ya pagado, ni el redondeo sobre 33,33 — cualquiera de esas ramas puede romperse sin que nadie se entere.Corrección: parametrize con los valores límite y un test aparte con pytest.raises para percent=150 y para el pedido ya pagado.

Líneas 5-6 · crítico · errorexcept Exception: pass se traga todo y devuelve "ok" aunque la escritura no haya ocurrido: el usuario ve éxito, los datos no están y el log está vacío. Con dos operaciones bajo un mismo try, un fallo del reindexado tampoco se distingue de un fallo de la base.Corrección: capturar las excepciones concretas, registrar con logger.exception y devolver un estado honesto; sacar el reindexado aparte para que su fallo no anule el guardado.

Líneas 2-4 · crítico · errorFalta set -euo pipefail: si pg_dump falla el archivo se crea igual, vacío, gzip lo comprime tan tranquilo y find borra las copias buenas de más de una semana; a los siete días no queda ni una íntegra. Un $DATABASE_URL sin valor pasa igual de desapercibido.Corrección: set -euo pipefail en la primera línea, comprobar el tamaño del dump después de pg_dump y borrar las copias antiguas solo cuando la nueva haya salido bien.