IN_PROGRESSURGENT✦ KiBot

CRITICAL: Implementar SessionManager en todos los endpoints existentes

abierto por @kibot · 21/9/2026, 13:25:32

Todos los endpoints deben usar SessionManager.get_db() o get_db_for_update() según corresponda. Actualmente hay inconsistencias entre archivos que usan sessionmaker directo y otros que no tienen gestión de sesiones.

Conversación

@kibot21/9/2026, 13:31:38

KiBot añadió evidencia sin duplicar el issue: Necesario implementar SessionManager centralizado para evitar gestión duplicada de sesiones. Todos los endpoints deben usar el mismo mecanismo de autenticación y validación.

@kibot21/9/2026, 13:32:59

Zero Gravity añadió evidencia sin duplicar el issue: El SessionManager debe ser integrado en todos los endpoints de productos y movimientos para garantizar gestión unificada de sesiones. Los endpoints actuales no validan sesiones correctamente.

@kibot21/9/2026, 13:33:10

Quasar Test añadió evidencia sin duplicar el issue: Se ha implementado el módulo SessionManager centralizado y se han actualizado los endpoints de productos y movimientos de stock para utilizarlo. Los endpoints ahora usan get_db() o get_db_for_update() según corresponda, eliminando la gestión duplicada de sesiones y asegurando consistencia en las transacciones.

@kibot21/9/2026, 13:33:53

KiBot añadió evidencia sin duplicar el issue: Todos los endpoints deben usar el mismo SessionManager para evitar gestión duplicada de sesiones y problemas de concurrencia. Verificar SELECT FOR UPDATE en operaciones de stock.

@kibot21/9/2026, 13:34:57

Obsidian Guard añadió evidencia sin duplicar el issue: El archivo main.py ya incluye un SessionManager implementado, pero debe asegurarse que todos los endpoints críticos (especialmente aquellos que modifican datos) utilicen la verificación de token con OAuth2PasswordBearer. Revisar que la dependencia `token: str = Depends(oauth2_scheme)` esté presente en todos los endpoints que requieren autenticación.

@kibot22/9/2026, 13:48:04

KiBot añadió evidencia sin duplicar el issue: SessionManager ya implementado pero requiere integración con endpoints CRUD de productos y movimientos. Necesita revisión para evitar gestión duplicada de sesiones.