Auditoria funcionando

backend/app/api/schemas/audit.py

@@ -1,7 +1,7 @@
-"""
+"""
 Audit Schemas - ServiceManagerWeb
 
-Schemas Pydantic para endpoints de auditoría
+Schemas Pydantic para endpoints de auditoría
 """
 
 from pydantic import BaseModel, Field, UUID4
@@ -11,7 +11,7 @@ from datetime import datetime
 
 class AuditLogBase(BaseModel):
     """Schema base para audit logs."""
-    action: str = Field(..., description="Acci├│n realizada (ej: ticket.create)")
+    action: str = Field(..., description="Acción realizada (ej: ticket.create)")
     resource_type: str = Field(..., description="Tipo de recurso (ticket, user, etc.)")
     resource_id: Optional[UUID4] = Field(None, description="ID del recurso afectado")
     extra_metadata: Optional[Dict[str, Any]] = Field(None, description="Metadata adicional", alias="metadata")
@@ -21,18 +21,18 @@ class AuditLogResponse(AuditLogBase):
     """
     Schema de respuesta para audit logs.
     
-    Incluye toda la informaci├│n del log con datos del usuario.
+    Incluye toda la información del log con datos del usuario.
     """
     id: UUID4
     tenant_id: UUID4
     user_id: Optional[UUID4]
     
-    # Informaci├│n del usuario (si existe)
+    # Información del usuario (si existe)
     user_email: Optional[str] = None
     user_name: Optional[str] = None
     user_role: Optional[str] = None
     
-    # Contexto de la acci├│n
+    # Contexto de la acción
     ip_address: Optional[str]
     user_agent: Optional[str]
     correlation_id: Optional[UUID4]
@@ -45,7 +45,7 @@ class AuditLogResponse(AuditLogBase):
     created_at: datetime
     
     # Display friendly
-    action_display: str = Field(description="Acci├│n en formato amigable")
+    action_display: str = Field(description="Acción en formato amigable")
     
     class Config:
         from_attributes = True
@@ -55,15 +55,15 @@ class AuditLogFilters(BaseModel):
     """
     Filtros para consulta de audit logs.
     
-    Permite filtrar por m├║ltiples criterios.
+    Permite filtrar por múltiples criterios.
     """
-    # Paginaci├│n
-    page: int = Field(default=1, ge=1, description="Número de página")
-    per_page: int = Field(default=50, ge=1, le=100, description="Elementos por página")
+    # Paginación
+    page: int = Field(default=1, ge=1, description="Número de página")
+    per_page: int = Field(default=50, ge=1, le=100, description="Elementos por página")
     
     # Filtros
     user_id: Optional[UUID4] = Field(None, description="Filtrar por usuario")
-    action: Optional[str] = Field(None, description="Filtrar por acción específica")
+    action: Optional[str] = Field(None, description="Filtrar por acción específica")
     resource_type: Optional[str] = Field(None, description="Filtrar por tipo de recurso")
     resource_id: Optional[UUID4] = Field(None, description="Filtrar por ID de recurso")
     
@@ -71,26 +71,25 @@ class AuditLogFilters(BaseModel):
     date_from: Optional[datetime] = Field(None, description="Fecha inicio (ISO 8601)")
     date_to: Optional[datetime] = Field(None, description="Fecha fin (ISO 8601)")
     
-    # B├║squeda
-    search: Optional[str] = Field(None, description="B├║squeda en acciones o recursos")
+    # Búsqueda
+    search: Optional[str] = Field(None, description="Búsqueda en acciones o recursos")
 
 
 class AuditLogStats(BaseModel):
     """
-    Estadísticas de auditoría.
+    Estadísticas de auditoría.
     
     Resumen de actividad del sistema.
     """
     total_actions: int = Field(description="Total de acciones registradas")
-    actions_today: int = Field(description="Acciones en las ├║ltimas 24 horas")
-    actions_this_week: int = Field(description="Acciones en los últimos 7 días")
-    critical_actions_today: int = Field(description="Acciones críticas hoy (delete, cambios sensibles)")
+    actions_today: int = Field(description="Acciones en las últimas 24 horas")
+    actions_this_week: int = Field(description="Acciones en los últimos 7 días")
     
     # Top acciones
-    top_actions: Dict[str, int] = Field(description="Acciones más frecuentes")
+    top_actions: Dict[str, int] = Field(description="Acciones más frecuentes")
     
     # Top usuarios
-    top_users: Dict[str, int] = Field(description="Usuarios más activos")
+    top_users: Dict[str, int] = Field(description="Usuarios más activos")
     
     # Actividad por tipo de recurso
     by_resource_type: Dict[str, int] = Field(description="Acciones por tipo de recurso")
@@ -102,9 +101,9 @@ class AuditLogListResponse(BaseModel):
     """
     logs: list[AuditLogResponse]
     total: int = Field(description="Total de registros")
-    page: int = Field(description="Página actual")
-    per_page: int = Field(description="Registros por página")
-    total_pages: int = Field(description="Total de páginas")
+    page: int = Field(description="Página actual")
+    per_page: int = Field(description="Registros por página")
+    total_pages: int = Field(description="Total de páginas")
     
     class Config:
         from_attributes = True

backend/app/api/v1/endpoints/audit.py

@@ -1,7 +1,7 @@
-"""
+"""
 Audit Endpoints - ServiceManagerWeb
 
-Endpoints para consulta de logs de auditoría.
+Endpoints para consulta de logs de auditoría.
 Solo accesible por roles: ADMIN, SUPPORT_MANAGER, AUDITOR
 """
 
@@ -34,14 +34,14 @@ def require_auditor_role(current_user: User = Depends(get_current_user)) -> User
     """
     Dependency que verifica que el usuario tenga rol de auditor.
     
-    Solo ADMIN, SUPPORT_MANAGER y AUDITOR pueden ver logs de auditoría.
+    Solo ADMIN, SUPPORT_MANAGER y AUDITOR pueden ver logs de auditoría.
     """
     allowed_roles = [UserRole.ADMIN, UserRole.SUPPORT_MANAGER, UserRole.AUDITOR]
     
     if current_user.role not in allowed_roles:
         raise HTTPException(
             status_code=status.HTTP_403_FORBIDDEN,
-            detail="Solo usuarios con rol ADMIN, SUPPORT_MANAGER o AUDITOR pueden acceder a logs de auditoría"
+            detail="Solo usuarios con rol ADMIN, SUPPORT_MANAGER o AUDITOR pueden acceder a logs de auditoría"
         )
     
     return current_user
@@ -49,38 +49,36 @@ def require_auditor_role(current_user: User = Depends(get_current_user)) -> User
 
 @router.get("/", response_model=AuditLogListResponse)
 async def get_audit_logs(
-    # Paginaci├│n
-    page: int = Query(default=1, ge=1, description="Número de página"),
-    per_page: int = Query(default=50, ge=1, le=100, description="Registros por página"),
+    # Paginación
+    page: int = Query(default=1, ge=1, description="Número de página"),
+    per_page: int = Query(default=50, ge=1, le=100, description="Registros por página"),
     
     # Filtros
     user_id: Optional[uuid.UUID] = Query(None, description="Filtrar por usuario"),
-    action: Optional[str] = Query(None, description="Filtrar por acci├│n"),
+    action: Optional[str] = Query(None, description="Filtrar por acción"),
     resource_type: Optional[str] = Query(None, description="Filtrar por tipo de recurso"),
     resource_id: Optional[uuid.UUID] = Query(None, description="Filtrar por ID de recurso"),
     date_from: Optional[datetime] = Query(None, description="Fecha desde"),
     date_to: Optional[datetime] = Query(None, description="Fecha hasta"),
-    search: Optional[str] = Query(None, description="B├║squeda en acci├│n o email"),
-        # Multi-tenant filters (solo ADMIN/SUPPORT_MANAGER)
-    tenant_id: Optional[uuid.UUID] = Query(None, description="Ver logs de un tenant específico"),
-    all_tenants: bool = Query(False, description="Ver logs de todos los tenants"),
-        # Dependencies
+    search: Optional[str] = Query(None, description="Búsqueda en acción o email"),
+    
+    # Dependencies
     current_user: User = Depends(require_auditor_role),
     current_tenant: Tenant = Depends(get_current_tenant),
     db: AsyncSession = Depends(get_db)
 ):
     """
-    Obtener logs de auditoría con filtros y paginación.
+    Obtener logs de auditoría con filtros y paginación.
     
     **Permisos**: ADMIN, SUPPORT_MANAGER, AUDITOR
     
     **Filtros disponibles**:
-    - `user_id`: Acciones de un usuario específico
-    - `action`: Tipo de acci├│n (ej: "ticket.create")
+    - `user_id`: Acciones de un usuario específico
+    - `action`: Tipo de acción (ej: "ticket.create")
     - `resource_type`: Tipo de recurso (ej: "ticket")
-    - `resource_id`: ID de recurso específico
+    - `resource_id`: ID de recurso específico
     - `date_from`, `date_to`: Rango de fechas
-    - `search`: B├║squeda en acciones
+    - `search`: Búsqueda en acciones
     
     **Retorna**: Lista paginada de audit logs
     """
@@ -92,28 +90,17 @@ async def get_audit_logs(
             "user_id": str(user_id) if user_id else None,
             "action": action,
             "resource_type": resource_type,
-            "page": page,
-            "tenant_filter": str(tenant_id) if tenant_id else None,
-            "all_tenants": all_tenants
+            "page": page
         }
     )
     
-    # Determinar el filtro de tenant
-    # Solo ADMIN y SUPPORT_MANAGER pueden ver otros tenants o todos los tenants
-    can_see_all_tenants = current_user.role in [UserRole.ADMIN, UserRole.SUPPORT_MANAGER]
-    
-    # Query base con filtro de tenant dinámico
-    query = select(AuditLog).options(selectinload(AuditLog.user))
-    
-    if all_tenants and can_see_all_tenants:
-        # Ver todos los tenants (no agregar filtro de tenant)
-        pass
-    elif tenant_id and can_see_all_tenants:
-        # Ver un tenant específico
-        query = query.where(AuditLog.tenant_id == tenant_id)
-    else:
-        # Ver solo el tenant actual (comportamiento default)
-        query = query.where(AuditLog.tenant_id == current_tenant.id)
+    # Query base - solo logs del tenant actual
+    # Usar selectinload para cargar la relación user (eager loading para async)
+    query = (
+        select(AuditLog)
+        .where(AuditLog.tenant_id == current_tenant.id)
+        .options(selectinload(AuditLog.user))
+    )
     
     # Aplicar filtros
     if user_id:
@@ -132,15 +119,16 @@ async def get_audit_logs(
         query = query.where(AuditLog.created_at >= date_from)
     
     if date_to:
-        # El frontend ya envía el timestamp correcto
-        query = query.where(AuditLog.created_at < date_to)
+        # Agregar 1 día para incluir todo el día
+        date_to_end = date_to + timedelta(days=1)
+        query = query.where(AuditLog.created_at < date_to_end)
     
     if search:
-        # B├║squeda en action
+        # Búsqueda en action
         search_filter = AuditLog.action.ilike(f"%{search}%")
         query = query.where(search_filter)
     
-    # Ordenar por fecha descendente (más recientes primero)
+    # Ordenar por fecha descendente (más recientes primero)
     query = query.order_by(desc(AuditLog.created_at))
     
     # Contar total antes de paginar
@@ -148,7 +136,7 @@ async def get_audit_logs(
     total_result = await db.execute(count_query)
     total = total_result.scalar() or 0
     
-    # Aplicar paginaci├│n
+    # Aplicar paginación
     offset = (page - 1) * per_page
     query = query.offset(offset).limit(per_page)
     
@@ -156,7 +144,7 @@ async def get_audit_logs(
     result = await db.execute(query)
     logs = result.scalars().all()
     
-    # Calcular total de páginas
+    # Calcular total de páginas
     total_pages = (total + per_page - 1) // per_page
     
     # Convertir a response schema (agregar info del usuario)
@@ -200,68 +188,61 @@ async def get_audit_logs(
 
 @router.get("/stats", response_model=AuditLogStats)
 async def get_audit_stats(
-    all_tenants: bool = Query(False, description="Ver stats de todos los tenants"),
     current_user: User = Depends(require_auditor_role),
     current_tenant: Tenant = Depends(get_current_tenant),
     db: AsyncSession = Depends(get_db)
 ):
     """
-    Obtener estadísticas de auditoría del tenant (o todos los tenants si es ADMIN).
+    Obtener estadísticas de auditoría del tenant.
     
     **Permisos**: ADMIN, SUPPORT_MANAGER, AUDITOR
     
     **Retorna**: Estadísticas de actividad
     """
-    can_see_all_tenants = current_user.role in [UserRole.ADMIN, UserRole.SUPPORT_MANAGER]
-    
     logger.info(
         "Fetching audit stats",
         user_id=str(current_user.id),
-        tenant_id=str(current_tenant.id),
-        all_tenants=all_tenants,
-        can_see_all=can_see_all_tenants
+        tenant_id=str(current_tenant.id)
     )
     
     now = datetime.utcnow()
     
-    # Determinar si aplicar filtro de tenant
-    apply_tenant_filter = not (all_tenants and can_see_all_tenants)
-    
     # Total de acciones
-    total_query = select(func.count()).select_from(AuditLog)
-    if apply_tenant_filter:
-        total_query = total_query.where(AuditLog.tenant_id == current_tenant.id)
+    total_query = select(func.count()).select_from(AuditLog).where(
+        AuditLog.tenant_id == current_tenant.id
+    )
     total_result = await db.execute(total_query)
     total_actions = total_result.scalar() or 0
     
-    # Acciones hoy (├║ltimas 24 horas)
+    # Acciones hoy (últimas 24 horas)
     today_start = now - timedelta(days=1)
     today_query = select(func.count()).select_from(AuditLog).where(
-        AuditLog.created_at >= today_start
+        and_(
+            AuditLog.tenant_id == current_tenant.id,
+            AuditLog.created_at >= today_start
+        )
     )
-    if apply_tenant_filter:
-        today_query = today_query.where(AuditLog.tenant_id == current_tenant.id)
     today_result = await db.execute(today_query)
     actions_today = today_result.scalar() or 0
     
-    # Acciones esta semana (últimos 7 días)
+    # Acciones esta semana (últimos 7 días)
     week_start = now - timedelta(days=7)
     week_query = select(func.count()).select_from(AuditLog).where(
-        AuditLog.created_at >= week_start
+        and_(
+            AuditLog.tenant_id == current_tenant.id,
+            AuditLog.created_at >= week_start
+        )
     )
-    if apply_tenant_filter:
-        week_query = week_query.where(AuditLog.tenant_id == current_tenant.id)
     week_result = await db.execute(week_query)
     actions_this_week = week_result.scalar() or 0
     
-    # Top 5 acciones más frecuentes
+    # Top 5 acciones más frecuentes
     top_actions_query = select(
         AuditLog.action,
         func.count(AuditLog.id).label('count')
-    )
-    if apply_tenant_filter:
-        top_actions_query = top_actions_query.where(AuditLog.tenant_id == current_tenant.id)
-    top_actions_query = top_actions_query.group_by(
+    ).where(
+        AuditLog.tenant_id == current_tenant.id
+    ).group_by(
         AuditLog.action
     ).order_by(
         desc('count')
@@ -274,10 +255,9 @@ async def get_audit_stats(
     by_resource_query = select(
         AuditLog.resource_type,
         func.count(AuditLog.id).label('count')
-    )
-    if apply_tenant_filter:
-        by_resource_query = by_resource_query.where(AuditLog.tenant_id == current_tenant.id)
-    by_resource_query = by_resource_query.group_by(
+    ).where(
+        AuditLog.tenant_id == current_tenant.id
+    ).group_by(
         AuditLog.resource_type
     ).order_by(
         desc('count')
@@ -286,48 +266,14 @@ async def get_audit_stats(
     by_resource_result = await db.execute(by_resource_query)
     by_resource_type = {row.resource_type: row.count for row in by_resource_result}
     
-    # Top usuarios (con join a users para obtener nombres)
-    top_users_query = select(
-        User.email,
-        func.count(AuditLog.id).label('count')
-    ).join(
-        User, AuditLog.user_id == User.id
-    )
-    if apply_tenant_filter:
-        top_users_query = top_users_query.where(AuditLog.tenant_id == current_tenant.id)
-    top_users_query = top_users_query.group_by(
-        User.email
-    ).order_by(
-        desc('count')
-    ).limit(5)
-    
-    top_users_result = await db.execute(top_users_query)
-    top_users = {row.email: row.count for row in top_users_result}
-    
-    # Acciones críticas hoy (delete, update sensibles, etc.)
-    critical_conditions = [
-        AuditLog.created_at >= today_start,
-        or_(
-            AuditLog.action.like('%.delete'),
-            AuditLog.action.like('user.update'),
-            AuditLog.action.like('%.assign'),
-            AuditLog.action.in_(['user.login_failed', 'user.logout'])
-        )
-    ]
-    if apply_tenant_filter:
-        critical_conditions.append(AuditLog.tenant_id == current_tenant.id)
-    
-    critical_actions_query = select(func.count()).select_from(AuditLog).where(
-        and_(*critical_conditions)
-    )
-    critical_result = await db.execute(critical_actions_query)
-    critical_actions_today = critical_result.scalar() or 0
+    # Top usuarios (necesitamos hacer join - simplificado por ahora)
+    # En producción podrías hacer un join con users para obtener nombres
+    top_users = {}  # Placeholder - implementar con join si es necesario
     
     return AuditLogStats(
         total_actions=total_actions,
         actions_today=actions_today,
         actions_this_week=actions_this_week,
-        critical_actions_today=critical_actions_today,
         top_actions=top_actions,
         top_users=top_users,
         by_resource_type=by_resource_type
@@ -342,7 +288,7 @@ async def get_audit_log_detail(
     db: AsyncSession = Depends(get_db)
 ):
     """
-    Obtener detalle de un audit log específico.
+    Obtener detalle de un audit log específico.
     
     **Permisos**: ADMIN, SUPPORT_MANAGER, AUDITOR
     

backend/app/api/v1/endpoints/auth.py

@@ -4,13 +4,14 @@ Authentication Endpoints - ServiceManagerWeb
 Endpoints para autenticación y autorización
 """
 
-from fastapi import APIRouter, HTTPException, status, Depends
+from fastapi import APIRouter, HTTPException, status, Depends, Request
 from fastapi.security import OAuth2PasswordBearer, OAuth2PasswordRequestForm
 from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy import select
 from sqlalchemy.orm import selectinload
 from pydantic import BaseModel, EmailStr
 from typing import Optional
+from datetime import datetime  # Para actualizar last_login
 import structlog
 
 from app.core.database import get_db
@@ -18,7 +19,8 @@ from app.core.security import security
 from app.core.config import get_settings
 from app.models.user import User
 from app.models.tenant import Tenant
-from app.services.audit_service import AuditService
+from app.services.audit_service import AuditService  # Servicio de auditoría
+from app.services.token_service import TokenService  # Servicio de tokens
 
 router = APIRouter()
 logger = structlog.get_logger(__name__)
@@ -68,13 +70,17 @@ class TokenResponse(BaseModel):
 @router.post("/login", response_model=LoginResponse)
 async def login(
     login_data: LoginRequest,
+    request: Request,  # Agregar Request para capturar IP y user agent
     db: AsyncSession = Depends(get_db)
 ):
     """
     Authenticate user and return access/refresh tokens.
     
+    **Auditoría**: Registra login exitoso y fallido en audit_logs
+    
     Args:
         login_data: Login credentials
+        request: FastAPI Request (para auditoría)
         db: Database session
         
     Returns:
@@ -101,21 +107,22 @@ async def login(
             email=login_data.email
         )
         
-        # Registrar intento fallido en auditoría (si el usuario existe)
-        if user:
-            try:
-                await AuditService.log(
-                    db=db,
-                    tenant_id=user.tenant_id,
-                    user_id=None,  # Login fallido = sin user_id
-                    action="user.login_failed",
-                    resource_type="user",
-                    resource_id=user.id,
-                    metadata={"email": login_data.email, "reason": "invalid_password"}
-                )
-                await db.commit()
-            except Exception as e:
-                logger.warning("Failed to log audit entry", error=str(e))
+        # 🔍 AUDITORÍA: Registrar intento fallido
+        if user:  # Solo si el usuario existe (password incorrecto)
+            await AuditService.log(
+                db=db,
+                tenant_id=user.tenant_id,
+                user_id=None,  # NULL porque aún no autenticado
+                action="user.login_failed",
+                resource_type="user",
+                resource_id=user.id,
+                metadata={
+                    "email": login_data.email,
+                    "reason": "invalid_password"
+                },
+                request=request
+            )
+            await db.commit()  # Commit del audit log
         
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
@@ -128,11 +135,31 @@ async def login(
             "Login failed - user inactive",
             email=login_data.email
         )
+        
+        # 🔍 AUDITORÍA: Registrar intento con usuario inactivo
+        await AuditService.log(
+            db=db,
+            tenant_id=user.tenant_id,
+            user_id=None,
+            action="user.login_failed",
+            resource_type="user",
+            resource_id=user.id,
+            metadata={
+                "email": login_data.email,
+                "reason": "user_inactive"
+            },
+            request=request
+        )
+        await db.commit()
+        
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
             detail="Usuario inactivo"
         )
     
+    # Actualizar último login
+    user.last_login = datetime.utcnow()
+    
     # Create tokens
     token_data = {
         "sub": str(user.id),
@@ -144,20 +171,61 @@ async def login(
     access_token = security.create_access_token(token_data)
     refresh_token = security.create_refresh_token(token_data)
     
-    # Registrar login exitoso en auditoría
-    try:
-        await AuditService.log(
-            db=db,
-            tenant_id=user.tenant_id,
-            user_id=user.id,
-            action="user.login",
-            resource_type="user",
-            resource_id=user.id,
-            metadata={"email": user.email, "success": True}
-        )
-        await db.commit()
-    except Exception as e:
-        logger.warning("Failed to log audit entry", error=str(e))
+    # Extraer información del dispositivo para rastreo
+    user_agent = request.headers.get("user-agent")
+    client_ip = request.client.host if request.client else None
+    
+    # device_id: El frontend puede enviarlo en el futuro como header
+    device_id = request.headers.get("x-device-id")  # Opcional
+    
+    # device_name: Simplificado del user-agent (ej: "Chrome en Windows")
+    device_name = None
+    if user_agent:
+        # Parseo básico para obtener un nombre legible
+        if "Chrome" in user_agent:
+            device_name = "Chrome"
+        elif "Firefox" in user_agent:
+            device_name = "Firefox"
+        elif "Safari" in user_agent:
+            device_name = "Safari"
+        elif "Edge" in user_agent:
+            device_name = "Edge"
+        else:
+            device_name = "Unknown Browser"
+            
+        # Agregar OS
+        if "Windows" in user_agent:
+            device_name += " en Windows"
+        elif "Macintosh" in user_agent or "Mac OS" in user_agent:
+            device_name += " en macOS"
+        elif "Linux" in user_agent:
+            device_name += " en Linux"
+        elif "Android" in user_agent:
+            device_name += " en Android"
+        elif "iPhone" in user_agent or "iPad" in user_agent:
+            device_name += " en iOS"
+    
+    # Persistir refresh token en BD con tracking completo
+    await TokenService.create_refresh_token(
+        db=db,
+        user=user,
+        refresh_token=refresh_token,
+        device_id=device_id,
+        device_name=device_name,
+        user_agent=user_agent,
+        ip_address=client_ip
+    )
+    
+    # 🔍 AUDITORÍA: Registrar login exitoso
+    await AuditService.log_login(
+        db=db,
+        user=user,
+        request=request,
+        success=True
+    )
+    
+    # Commit de todos los cambios (last_login + refresh token + audit log)
+    await db.commit()
     
     logger.info(
         "Login successful",
@@ -192,6 +260,9 @@ async def refresh_token(
     """
     Refresh access token using refresh token.
     
+    Verifica el JWT, valida en BD que no esté revocado/expirado,
+    actualiza estadísticas de uso y genera nuevo access token.
+    
     Args:
         refresh_data: Refresh token data
         db: Database session
@@ -204,18 +275,26 @@ async def refresh_token(
     """
     logger.info("Token refresh attempt")
     
-    # Verify refresh token
+    # 1. Verify refresh token JWT signature
     payload = security.verify_token(refresh_data.refresh_token)
     if not payload or payload.get("type") != "refresh":
-        logger.warning("Token refresh failed - invalid token")
+        logger.warning("Token refresh failed - invalid JWT")
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
             detail="Invalid refresh token"
         )
     
-    # TODO: Check if refresh token exists in database and is not revoked
+    # 2. Verificar que el token exista en BD y no esté revocado/expirado
+    #    También actualiza last_used_at y usage_count automáticamente
+    db_token = await TokenService.verify_refresh_token(db, refresh_data.refresh_token)
+    if not db_token:
+        logger.warning("Token refresh failed - token not found, revoked or expired")
+        raise HTTPException(
+            status_code=status.HTTP_401_UNAUTHORIZED,
+            detail="Invalid or revoked refresh token"
+        )
     
-    # Create new access token
+    # 3. Create new access token
     token_data = {
         "sub": payload["sub"],
         "email": payload["email"],
@@ -225,7 +304,15 @@ async def refresh_token(
     
     access_token = security.create_access_token(token_data)
     
-    logger.info("Token refresh successful", user_id=payload["sub"])
+    # 4. Commit actualización de estadísticas de uso
+    await db.commit()
+    
+    logger.info(
+        "Token refresh successful",
+        user_id=payload["sub"],
+        token_id=str(db_token.id),
+        usage_count=db_token.usage_count
+    )
     
     return TokenResponse(
         access_token=access_token,
@@ -239,14 +326,17 @@ async def logout(
     db: AsyncSession = Depends(get_db)
 ):
     """
-    Logout user and revoke refresh token.
+    Logout user and revoke all refresh tokens.
+    
+    Revoca todos los tokens del usuario (logout en todos los dispositivos)
+    y registra quién ejecutó la revocación para auditoría.
     
     Args:
         token: Access token
         db: Database session
         
     Returns:
-        Success message
+        Success message with count of revoked tokens
     """
     logger.info("Logout attempt")
     
@@ -258,30 +348,31 @@ async def logout(
             detail="Invalid token"
         )
     
-    # TODO: Revoke refresh token in database
-    
-    # Registrar logout en auditoría
-    try:
-        import uuid
-        user_id = uuid.UUID(payload["sub"])
-        tenant_id = uuid.UUID(payload["tenant_id"])
-        
-        await AuditService.log(
-            db=db,
-            tenant_id=tenant_id,
-            user_id=user_id,
-            action="user.logout",
-            resource_type="user",
-            resource_id=user_id,
-            metadata={"email": payload.get("email")}
+    # Extraer user_id del token
+    user_id_str = payload.get("sub")
+    if not user_id_str:
+        raise HTTPException(
+            status_code=status.HTTP_401_UNAUTHORIZED,
+            detail="Invalid token payload"
         )
-        await db.commit()
-    except Exception as e:
-        logger.warning("Failed to log audit entry", error=str(e))
     
-    logger.info("Logout successful", user_id=payload["sub"])
+    user_id = uuid.UUID(user_id_str)
     
-    return {"message": "Successfully logged out"}
+    # Revocar todos los tokens del usuario con auditoría de quién lo revocó
+    count = await TokenService.revoke_all_user_tokens(
+        db=db,
+        user_id=user_id,
+        revoked_by_user_id=user_id  # El usuario se revoca a sí mismo
+    )
+    
+    await db.commit()
+    
+    logger.info("Logout successful", user_id=str(user_id), tokens_revoked=count)
+    
+    return {
+        "message": "Successfully logged out from all devices",
+        "tokens_revoked": count
+    }
 
 
 @router.get("/me")

backend/app/api/v1/endpoints/tickets.py

@@ -21,7 +21,6 @@ from app.models.comment import TicketComment
 from app.models.attachment import TicketAttachment
 from app.api.schemas.attachment import AttachmentResponse
 from app.core.file_handler import file_handler
-from app.services.audit_service import AuditService
 import uuid
 
 router = APIRouter()
@@ -144,27 +143,6 @@ async def create_ticket(
             await db.commit()
             await db.refresh(db_ticket)
             
-            # Registrar creación en auditoría
-            try:
-                await AuditService.log(
-                    db=db,
-                    tenant_id=current_user.tenant_id,
-                    user_id=current_user.id,
-                    action="ticket.create",
-                    resource_type="ticket",
-                    resource_id=db_ticket.id,
-                    new_values={
-                        "ticket_number": db_ticket.ticket_number,
-                        "subject": db_ticket.subject,
-                        "priority": db_ticket.priority.value,
-                        "status": db_ticket.status.value
-                    }
-                )
-                await db.commit()
-            except Exception as e:
-                # No fallar si falla el audit log
-                pass
-            
             # ✅ Éxito - retornar ticket creado
             return {
                 "id": str(db_ticket.id),
@@ -262,7 +240,7 @@ async def get_tickets(
             "status": t.status.value,
             "priority": t.priority.value,
             "category_id": str(t.category_id) if t.category_id else None,
-            "affected_system_id": str(t.affected_system_id) if t.affected_system_id else None,
+            "affected_system_id": str(t.affected_system_id) if t.affected_system_id else None,  # ✅ CORREGIDO
             "created_by": str(t.created_by),
             "assigned_to": str(t.assigned_to) if t.assigned_to else None,
             "created_at": t.created_at,
@@ -523,15 +501,6 @@ async def update_ticket(
             detail=f"Ticket {ticket_id} not found"
         )
     
-    # Guardar valores anteriores para audit
-    old_values = {
-        "subject": db_ticket.subject,
-        "description": db_ticket.description,
-        "status": db_ticket.status.value,
-        "priority": db_ticket.priority.value,
-        "assigned_to": str(db_ticket.assigned_to) if db_ticket.assigned_to else None
-    }
-    
     try:
         update_data = ticket_update.dict(exclude_unset=True)
         
@@ -550,34 +519,6 @@ async def update_ticket(
         await db.commit()
         await db.refresh(db_ticket)
         
-        # Registrar actualización en auditoría
-        try:
-            new_values = {
-                "subject": db_ticket.subject,
-                "description": db_ticket.description,
-                "status": db_ticket.status.value,
-                "priority": db_ticket.priority.value,
-                "assigned_to": str(db_ticket.assigned_to) if db_ticket.assigned_to else None
-            }
-            
-            # Si cambió assigned_to, registrar como acción de asignación
-            action = "ticket.assign" if old_values["assigned_to"] != new_values["assigned_to"] else "ticket.update"
-            
-            await AuditService.log(
-                db=db,
-                tenant_id=current_user.tenant_id,
-                user_id=current_user.id,
-                action=action,
-                resource_type="ticket",
-                resource_id=db_ticket.id,
-                old_values=old_values,
-                new_values=new_values
-            )
-            await db.commit()
-        except Exception as e:
-            # No fallar si falla el audit log
-            pass
-        
         # ✅ CORREGIDO: Usar affected_system_id
         return {
             "id": str(db_ticket.id),
@@ -846,33 +787,9 @@ async def delete_ticket(
             detail=f"Ticket {ticket_id} not found"
         )
     
-    # Guardar datos del ticket antes de eliminar para audit
-    old_values = {
-        "ticket_number": db_ticket.ticket_number,
-        "subject": db_ticket.subject,
-        "status": db_ticket.status.value,
-        "priority": db_ticket.priority.value
-    }
-    
     await db.delete(db_ticket)
     await db.commit()
     
-    # Registrar eliminación en auditoría
-    try:
-        await AuditService.log(
-            db=db,
-            tenant_id=current_user.tenant_id,
-            user_id=current_user.id,
-            action="ticket.delete",
-            resource_type="ticket",
-            resource_id=ticket_uuid,
-            old_values=old_values
-        )
-        await db.commit()
-    except Exception as e:
-        # No fallar si falla el audit log
-        pass
-    
     return {"message": "Ticket deleted successfully"}
 
 # ===================================

backend/app/api/v1/endpoints/users.py

@@ -9,7 +9,6 @@ import uuid
 from app.core.database import get_db
 from app.core.security import security
 from app.models.user import User, UserRole
-from app.services.audit_service import AuditService
 from app.api import deps 
 
 router = APIRouter()
@@ -148,28 +147,6 @@ async def create_user(
     db.add(db_user)
     await db.commit()
     await db.refresh(db_user)
-    
-    # Registrar creación en auditoría
-    try:
-        await AuditService.log(
-            db=db,
-            tenant_id=current_user.tenant_id,
-            user_id=current_user.id,
-            action="user.create",
-            resource_type="user",
-            resource_id=db_user.id,
-            new_values=AuditService.sanitize_values({
-                "email": db_user.email,
-                "first_name": db_user.first_name,
-                "last_name": db_user.last_name,
-                "role": db_user.role.value
-            })
-        )
-        await db.commit()
-    except Exception as e:
-        # No fallar si falla el audit log
-        pass
-    
     return db_user
 
 
@@ -237,15 +214,6 @@ async def update_user(
             detail="User not found"
         )
     
-    # Guardar valores anteriores para audit
-    old_values = {
-        "email": db_user.email,
-        "first_name": db_user.first_name,
-        "last_name": db_user.last_name,
-        "role": db_user.role.value,
-        "is_active": db_user.is_active
-    }
-    
     # Verificar email único si se está cambiando
     update_data = user_update.model_dump(exclude_unset=True)
     if "email" in update_data and update_data["email"] != db_user.email:
@@ -271,32 +239,6 @@ async def update_user(
     
     await db.commit()
     await db.refresh(db_user)
-    
-    # Registrar actualización en auditoría
-    try:
-        new_values = {
-            "email": db_user.email,
-            "first_name": db_user.first_name,
-            "last_name": db_user.last_name,
-            "role": db_user.role.value,
-            "is_active": db_user.is_active
-        }
-        
-        await AuditService.log(
-            db=db,
-            tenant_id=current_user.tenant_id,
-            user_id=current_user.id,
-            action="user.update",
-            resource_type="user",
-            resource_id=db_user.id,
-            old_values=AuditService.sanitize_values(old_values),
-            new_values=AuditService.sanitize_values(new_values)
-        )
-        await db.commit()
-    except Exception as e:
-        # No fallar si falla el audit log
-        pass
-    
     return db_user
 
 
@@ -364,28 +306,6 @@ async def delete_user(
     # Soft delete
     db_user.is_active = False
     await db.commit()
-    
-    # Registrar eliminación en auditoría
-    try:
-        await AuditService.log(
-            db=db,
-            tenant_id=current_user.tenant_id,
-            user_id=current_user.id,
-            action="user.delete",
-            resource_type="user",
-            resource_id=db_user.id,
-            old_values={
-                "email": db_user.email,
-                "role": db_user.role.value,
-                "was_active": True
-            },
-            metadata={"action_type": "soft_delete"}
-        )
-        await db.commit()
-    except Exception as e:
-        # No fallar si falla el audit log
-        pass
-    
     return None
 
 

backend/app/api/v1/router.py

@@ -23,6 +23,13 @@ api_router.include_router(
     tags=["authentication"]
 )
 
+# Audit routes (antes de otros para logging)
+api_router.include_router(
+    audit.router,
+    prefix="/audit",
+    tags=["audit"]
+)
+
 api_router.include_router(
     tenants.router,
     prefix="/tenants",
@@ -59,11 +66,4 @@ api_router.include_router(
     client_profile.router,
     prefix="/client-profile",
     tags=["client-profile"]
-)
-
-# Audit routes
-api_router.include_router(
-    audit.router,
-    prefix="/audit",
-    tags=["audit"]
 )

backend/app/main.py

@@ -23,8 +23,8 @@ from app.models.user import User
 from app.models.ticket import Ticket
 from app.models.comment import TicketComment
 from app.models.attachment import TicketAttachment
-from app.models.audit import AuditLog
-from app.models.refresh_token import RefreshToken
+from app.models.audit import AuditLog  # Nuevo modelo para auditoría
+from app.models.refresh_token import RefreshToken  # Modelo para refresh tokens
 
 from app.core.logging import setup_logging
 from app.api.v1.router import api_router

backend/app/models/audit.py

@@ -1,7 +1,7 @@
-"""
+"""
 Audit Log Model - ServiceManagerWeb
 
-Modelo para bitácora de auditoría y compliance.
+Modelo para bitácora de auditoría y compliance.
 Registra todas las acciones importantes del sistema.
 """
 
@@ -21,15 +21,15 @@ if TYPE_CHECKING:
 
 class AuditLog(Base):
     """
-    Bitácora de auditoría para tracking completo de acciones.
+    Bitácora de auditoría para tracking completo de acciones.
     
     Registra:
-    - Qui├®n hizo la acci├│n (user_id)
-    - Qu├® hizo (action)
-    - Sobre qu├® recurso (resource_type + resource_id)
-    - Cuándo lo hizo (created_at)
-    - Desde d├│nde (ip_address, user_agent)
-    - Qu├® cambi├│ (old_values, new_values)
+    - Quién hizo la acción (user_id)
+    - Qué hizo (action)
+    - Sobre qué recurso (resource_type + resource_id)
+    - Cuándo lo hizo (created_at)
+    - Desde dónde (ip_address, user_agent)
+    - Qué cambió (old_values, new_values)
     """
     
     __tablename__ = "audit_logs"
@@ -42,7 +42,7 @@ class AuditLog(Base):
         index=True
     )
     
-    # Usuario que ejecut├│ la acci├│n (NULL = acci├│n del sistema)
+    # Usuario que ejecutó la acción (NULL = acción del sistema)
     user_id: Mapped[Optional[uuid.UUID]] = mapped_column(
         UUID(as_uuid=True),
         ForeignKey("users.id", ondelete="SET NULL"),
@@ -50,7 +50,7 @@ class AuditLog(Base):
         index=True
     )
     
-    # Acci├│n realizada (ej: "user.login", "ticket.create", "ticket.assign")
+    # Acción realizada (ej: "user.login", "ticket.create", "ticket.assign")
     action: Mapped[str] = mapped_column(
         String(100),
         nullable=False,
@@ -87,14 +87,14 @@ class AuditLog(Base):
         nullable=True
     )
     
-    # Valores despu├®s del cambio (JSON)
+    # Valores después del cambio (JSON)
     new_values: Mapped[Optional[Dict[str, Any]]] = mapped_column(
         JSONB,
         nullable=True
     )
     
     # Metadata adicional (cualquier info relevante)
-    # Nota: 'metadata' está reservado en SQLAlchemy, usamos 'extra_metadata'
+    # Nota: 'metadata' está reservado en SQLAlchemy, usamos 'extra_metadata'
     extra_metadata: Mapped[Optional[Dict[str, Any]]] = mapped_column(
         'metadata',  # Nombre real de la columna en BD
         JSONB,
@@ -113,14 +113,14 @@ class AuditLog(Base):
     tenant: Mapped["Tenant"] = relationship("Tenant", foreign_keys=[tenant_id])
     user: Mapped[Optional["User"]] = relationship("User", foreign_keys=[user_id])
     
-    # Índices compuestos para queries comunes
+    # Índices compuestos para queries comunes
     __table_args__ = (
         Index('idx_audit_logs_tenant_action', 'tenant_id', 'action'),
         Index('idx_audit_logs_resource', 'resource_type', 'resource_id'),
         Index('idx_audit_logs_user_created', 'user_id', 'created_at'),
     )
     
-    # Configuraci├│n del mapper: excluir updated_at porque audit logs son inmutables
+    # Configuración del mapper: excluir updated_at porque audit logs son inmutables
     __mapper_args__ = {
         "exclude_properties": ["updated_at"]
     }
@@ -130,19 +130,19 @@ class AuditLog(Base):
     
     @property
     def action_display(self) -> str:
-        """Formato amigable de la acci├│n."""
+        """Formato amigable de la acción."""
         parts = self.action.split('.')
         if len(parts) == 2:
             resource, verb = parts
             verb_map = {
-                'create': 'cre├│',
-                'update': 'actualiz├│',
-                'delete': 'elimin├│',
-                'login': 'inici├│ sesi├│n',
-                'logout': 'cerr├│ sesi├│n',
-                'assign': 'asign├│',
-                'close': 'cerr├│',
-                'reopen': 'reabri├│'
+                'create': 'creó',
+                'update': 'actualizó',
+                'delete': 'eliminó',
+                'login': 'inició sesión',
+                'logout': 'cerró sesión',
+                'assign': 'asignó',
+                'close': 'cerró',
+                'reopen': 'reabrió'
             }
             return f"{verb_map.get(verb, verb)} {resource}"
         return self.action

backend/app/models/refresh_token.py

@@ -1,7 +1,7 @@
-"""
+"""
 Refresh Token Model - ServiceManagerWeb
 
-Modelo para persistencia de refresh tokens con revocaci├│n y tracking.
+Modelo para persistencia de refresh tokens con revocación y tracking.
 """
 
 from sqlalchemy import String, Boolean, DateTime, ForeignKey, Index, Integer
@@ -19,16 +19,16 @@ if TYPE_CHECKING:
 
 class RefreshToken(Base):
     """
-    Refresh Token persistente para gesti├│n de sesiones.
+    Refresh Token persistente para gestión de sesiones.
     
-    Almacena refresh tokens con informaci├│n de dispositivo y permite
-    revocaci├│n para mejorar la seguridad.
+    Almacena refresh tokens con información de dispositivo y permite
+    revocación para mejorar la seguridad.
     
-    Características:
+    Características:
     - Token hasheado (no se guarda en texto plano)
     - Device fingerprinting
-    - Revocaci├│n individual con tracking
-    - Auto-expiraci├│n
+    - Revocación individual con tracking
+    - Auto-expiración
     - Tracking de IP y uso
     """
     
@@ -72,14 +72,14 @@ class RefreshToken(Base):
         nullable=True
     )
     
-    # Expiraci├│n del token
+    # Expiración del token
     expires_at: Mapped[datetime] = mapped_column(
         DateTime(timezone=True),
         nullable=False,
         index=True
     )
     
-    # Estado de revocaci├│n
+    # Estado de revocación
     revoked: Mapped[bool] = mapped_column(
         Boolean,
         default=False,
@@ -125,11 +125,11 @@ class RefreshToken(Base):
         server_default="NOW()"
     )
     
-    # Relaci├│n con usuario
+    # Relación con usuario
     user: Mapped["User"] = relationship("User", foreign_keys=[user_id], back_populates="refresh_tokens")
     revoker: Mapped[Optional["User"]] = relationship("User", foreign_keys=[revoked_by])
     
-    # Índices compuestos
+    # Índices compuestos
     __table_args__ = (
         Index('idx_refresh_tokens_user_expires', 'user_id', 'expires_at'),
     )
@@ -140,10 +140,10 @@ class RefreshToken(Base):
     @property
     def is_valid(self) -> bool:
         """
-        Verificar si el token es válido.
+        Verificar si el token es válido.
         
-        Un token es válido si:
-        - No está revocado
+        Un token es válido si:
+        - No está revocado
         - No ha expirado
         """
         return not self.revoked and self.expires_at > datetime.utcnow()
@@ -158,7 +158,7 @@ class RefreshToken(Base):
         Marcar el token como revocado.
         
         Args:
-            revoked_by: ID del usuario que revoc├│ el token
+            revoked_by: ID del usuario que revocó el token
         """
         self.revoked = True
         self.revoked_at = datetime.utcnow()

backend/app/models/user.py

@@ -78,11 +78,12 @@ class User(Base):
         back_populates="assigned_to_user", 
         foreign_keys="Ticket.assigned_to"
     )
+    # Refresh tokens: especificar user_id como FK (hay 2 FKs: user_id y revoked_by)
     refresh_tokens: Mapped[List["RefreshToken"]] = relationship(
         "RefreshToken",
         back_populates="user",
-        foreign_keys="RefreshToken.user_id",
-        cascade="all, delete-orphan"
+        cascade="all, delete-orphan",
+        foreign_keys="[RefreshToken.user_id]"
     )
     
     # Unique constraint por tenant

backend/app/services/audit_service.py

@@ -1,8 +1,8 @@
-"""
+"""
 Audit Service - ServiceManagerWeb
 
-Funciones helper para facilitar el registro de auditoría.
-Simplifica el proceso de logging en toda la aplicaci├│n.
+Funciones helper para facilitar el registro de auditoría.
+Simplifica el proceso de logging en toda la aplicación.
 """
 
 from typing import Optional, Dict, Any
@@ -19,9 +19,9 @@ logger = structlog.get_logger(__name__)
 
 class AuditService:
     """
-    Servicio centralizado para registro de auditoría.
+    Servicio centralizado para registro de auditoría.
     
-    Uso básico:
+    Uso básico:
         await AuditService.log(
             db=db,
             tenant_id=tenant.id,
@@ -47,25 +47,25 @@ class AuditService:
         request: Optional[Request] = None
     ) -> AuditLog:
         """
-        Registra una acción en la bitácora de auditoría.
+        Registra una acción en la bitácora de auditoría.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             tenant_id: ID del tenant
-            action: Acci├│n realizada (formato: "recurso.verbo")
+            action: Acción realizada (formato: "recurso.verbo")
                    Ejemplos: "user.login", "ticket.create", "ticket.assign"
             resource_type: Tipo de recurso ("user", "ticket", "comment", etc.)
             resource_id: ID del recurso afectado (opcional)
-            user_id: ID del usuario que ejecut├│ la acci├│n (opcional = sistema)
+            user_id: ID del usuario que ejecutó la acción (opcional = sistema)
             old_values: Valores antes del cambio (opcional)
-            new_values: Valores despu├®s del cambio (opcional)
-            metadata: Informaci├│n adicional (opcional)
+            new_values: Valores después del cambio (opcional)
+            metadata: Información adicional (opcional)
             request: Request de FastAPI para extraer IP y user agent (opcional)
             
         Returns:
             AuditLog creado
         """
-        # Extraer información del request si está disponible
+        # Extraer información del request si está disponible
         ip_address = None
         user_agent = None
         correlation_id = None
@@ -81,7 +81,7 @@ class AuditService:
             # Correlation ID (si existe en el request state)
             correlation_id = getattr(request.state, "correlation_id", None)
         
-        # Crear registro de auditoría
+        # Crear registro de auditoría
         audit_log = AuditLog(
             tenant_id=tenant_id,
             user_id=user_id,
@@ -97,7 +97,7 @@ class AuditService:
         )
         
         db.add(audit_log)
-        await db.flush()  # No commit, se hará con la transacción principal
+        await db.flush()  # No commit, se hará con la transacción principal
         
         # Log estructurado para debugging
         logger.info(
@@ -122,8 +122,8 @@ class AuditService:
         Registra un intento de login.
         
         Args:
-            db: Sesi├│n de base de datos
-            user: Usuario que intent├│ loguearse
+            db: Sesión de base de datos
+            user: Usuario que intentó loguearse
             request: Request de FastAPI
             success: Si el login fue exitoso
             
@@ -154,8 +154,8 @@ class AuditService:
         Registra un logout.
         
         Args:
-            db: Sesi├│n de base de datos
-            user: Usuario que cerr├│ sesi├│n
+            db: Sesión de base de datos
+            user: Usuario que cerró sesión
             request: Request de FastAPI
             
         Returns:
@@ -182,12 +182,12 @@ class AuditService:
         request: Optional[Request] = None
     ) -> AuditLog:
         """
-        Registra la creaci├│n de un recurso.
+        Registra la creación de un recurso.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             tenant_id: ID del tenant
-            user_id: ID del usuario que cre├│ el recurso
+            user_id: ID del usuario que creó el recurso
             resource_type: Tipo de recurso ("ticket", "user", etc.)
             resource_id: ID del recurso creado
             new_values: Valores del nuevo recurso
@@ -219,12 +219,12 @@ class AuditService:
         request: Optional[Request] = None
     ) -> AuditLog:
         """
-        Registra la actualizaci├│n de un recurso.
+        Registra la actualización de un recurso.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             tenant_id: ID del tenant
-            user_id: ID del usuario que actualiz├│
+            user_id: ID del usuario que actualizó
             resource_type: Tipo de recurso
             resource_id: ID del recurso
             old_values: Valores anteriores
@@ -257,12 +257,12 @@ class AuditService:
         request: Optional[Request] = None
     ) -> AuditLog:
         """
-        Registra la eliminaci├│n de un recurso.
+        Registra la eliminación de un recurso.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             tenant_id: ID del tenant
-            user_id: ID del usuario que elimin├│
+            user_id: ID del usuario que eliminó
             resource_type: Tipo de recurso
             resource_id: ID del recurso eliminado
             old_values: Valores del recurso antes de eliminar

backend/app/services/token_service.py

@@ -1,7 +1,7 @@
-"""
+"""
 Token Service - ServiceManagerWeb
 
-Servicio para gesti├│n de refresh tokens persistentes.
+Servicio para gestión de refresh tokens persistentes.
 """
 
 from sqlalchemy.ext.asyncio import AsyncSession
@@ -21,9 +21,9 @@ settings = get_settings()
 
 class TokenService:
     """
-    Servicio para gesti├│n de refresh tokens.
+    Servicio para gestión de refresh tokens.
     
-    Proporciona m├®todos para crear, validar, revocar y limpiar
+    Proporciona métodos para crear, validar, revocar y limpiar
     refresh tokens persistentes.
     
     NOTA: Los tokens se almacenan directamente en BD (no hash)
@@ -44,10 +44,10 @@ class TokenService:
         Crear y persistir un refresh token.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             user: Usuario propietario del token
             refresh_token: Token JWT generado (se almacena directamente)
-            device_id: ID ├║nico del dispositivo (UUID generado por cliente)
+            device_id: ID único del dispositivo (UUID generado por cliente)
             device_name: Nombre del dispositivo (ej: "Chrome en Windows")
             user_agent: User agent completo del navegador
             ip_address: IP del cliente
@@ -55,7 +55,7 @@ class TokenService:
         Returns:
             RefreshToken creado
         """
-        # Calcular expiraci├│n
+        # Calcular expiración
         expires_at = datetime.utcnow() + timedelta(
             days=settings.REFRESH_TOKEN_EXPIRE_DAYS
         )
@@ -92,16 +92,16 @@ class TokenService:
         refresh_token: str
     ) -> Optional[RefreshToken]:
         """
-        Verificar que el refresh token exista y sea válido.
+        Verificar que el refresh token exista y sea válido.
         
         Busca el JWT directamente en la BD y verifica su estado.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             refresh_token: Token JWT a verificar
             
         Returns:
-            RefreshToken si es válido, None si no existe o está revocado/expirado
+            RefreshToken si es válido, None si no existe o está revocado/expirado
         """
         # Buscar token directamente en BD (sin hash)
         query = select(RefreshToken).where(
@@ -114,7 +114,7 @@ class TokenService:
             logger.warning("Refresh token not found in database")
             return None
         
-        # Verificar si es válido (usa property is_valid del modelo)
+        # Verificar si es válido (usa property is_valid del modelo)
         if not db_token.is_valid:
             logger.warning(
                 "Invalid refresh token",
@@ -124,7 +124,7 @@ class TokenService:
             )
             return None
         
-        # Actualizar estadísticas de uso
+        # Actualizar estadísticas de uso
         db_token.track_usage()
         await db.flush()
         
@@ -142,15 +142,15 @@ class TokenService:
         revoked_by_user_id: Optional[uuid.UUID] = None
     ) -> bool:
         """
-        Revocar un refresh token específico.
+        Revocar un refresh token específico.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             refresh_token: Token JWT a revocar
-            revoked_by_user_id: ID del usuario que revoca (para auditoría)
+            revoked_by_user_id: ID del usuario que revoca (para auditoría)
             
         Returns:
-            True si se revoc├│, False si no se encontr├│
+            True si se revocó, False si no se encontró
         """
         # Buscar token directamente (sin hash)
         query = select(RefreshToken).where(
@@ -163,7 +163,7 @@ class TokenService:
             logger.warning("Refresh token not found for revocation")
             return False
         
-        # Revocar usando m├®todo del modelo
+        # Revocar usando método del modelo
         db_token.revoke(revoked_by=revoked_by_user_id)
         await db.flush()
         
@@ -183,15 +183,15 @@ class TokenService:
         """
         Revocar todos los tokens activos de un usuario.
         
-        Útil para logout en todos los dispositivos.
+        Útil para logout en todos los dispositivos.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             user_id: ID del usuario
-            revoked_by_user_id: ID del usuario que ejecuta la revocación (para auditoría)
+            revoked_by_user_id: ID del usuario que ejecuta la revocación (para auditoría)
             
         Returns:
-            N├║mero de tokens revocados
+            Número de tokens revocados
         """
         # Buscar todos los tokens activos del usuario
         query = select(RefreshToken).where(
@@ -226,12 +226,12 @@ class TokenService:
         Tarea de mantenimiento para limpiar tokens antiguos.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             
         Returns:
-            N├║mero de tokens eliminados
+            Número de tokens eliminados
         """
-        # Eliminar tokens expirados hace más de 7 días
+        # Eliminar tokens expirados hace más de 7 días
         cutoff_date = datetime.utcnow() - timedelta(days=7)
         
         query = delete(RefreshToken).where(
@@ -254,7 +254,7 @@ class TokenService:
         Obtener todos los tokens activos de un usuario.
         
         Args:
-            db: Sesi├│n de base de datos
+            db: Sesión de base de datos
             user_id: ID del usuario
             
         Returns:

backend/create_test_user.py

@@ -1,67 +0,0 @@
-"""
-Script para crear/actualizar usuario de prueba con contraseña conocida
-"""
-import asyncio
-from sqlalchemy import select, update
-from app.core.database import AsyncSessionLocal
-from app.core.security import security
-from app.models.user import User, UserRole
-from app.models.tenant import Tenant
-import uuid
-
-async def create_test_user():
-    async with AsyncSessionLocal() as db:
-        # Buscar tenant
-        tenant_query = select(Tenant).where(Tenant.slug.like('%aduanasoft%')).limit(1)
-        result = await db.execute(tenant_query)
-        tenant = result.scalar_one_or_none()
-        
-        if not tenant:
-            print("❌ No se encontró tenant")
-            return
-        
-        print(f"✅ Tenant encontrado: {tenant.name} ({tenant.slug})")
-        
-        # Buscar o crear usuario admin
-        user_query = select(User).where(
-            User.email == "admin@aduanasoft.com",
-            User.tenant_id == tenant.id
-        )
-        result = await db.execute(user_query)
-        user = result.scalar_one_or_none()
-        
-        # Hash de la contraseña "admin123"
-        password_hash = security.hash_password("admin123")
-        
-        if user:
-            # Actualizar contraseña
-            user.password_hash = password_hash
-            user.is_active = True
-            user.email_verified = True
-            await db.commit()
-            print(f"✅ Usuario actualizado: {user.email}")
-        else:
-            # Crear usuario nuevo
-            user = User(
-                id=uuid.uuid4(),
-                tenant_id=tenant.id,
-                email="admin@aduanasoft.com",
-                first_name="Admin",
-                last_name="Sistema",
-                password_hash=password_hash,
-                role=UserRole.ADMIN,
-                is_active=True,
-                email_verified=True
-            )
-            db.add(user)
-            await db.commit()
-            print(f"✅ Usuario creado: {user.email}")
-        
-        print(f"\n📋 Credenciales de prueba:")
-        print(f"   Email: admin@aduanasoft.com")
-        print(f"   Password: admin123")
-        print(f"   Tenant: {tenant.slug}")
-        print(f"   Role: ADMIN")
-
-if __name__ == "__main__":
-    asyncio.run(create_test_user())

backend/migrations/versions/a1b2c3d4e5f6_add_audit_logs_table.py

@@ -1,11 +1,11 @@
-"""add_audit_logs_table
+"""add_audit_logs_table
 
 Revision ID: a1b2c3d4e5f6
 Revises: 13362e8c493a
 Create Date: 2026-02-12 10:00:00.000000
 
 Registra el modelo AuditLog en Alembic.
-La tabla audit_logs ya existe en schema.sql, esta migraci├│n solo
+La tabla audit_logs ya existe en schema.sql, esta migración solo
 la registra en el control de versiones de Alembic.
 """
 from alembic import op
@@ -23,8 +23,8 @@ def upgrade():
     """
     Verificar que audit_logs existe y registrarla en Alembic.
     
-    La tabla fue creada por schema.sql, esta migraci├│n solo verifica
-    que exista y está disponible para usar.
+    La tabla fue creada por schema.sql, esta migración solo verifica
+    que exista y está disponible para usar.
     """
     from sqlalchemy import inspect
     
@@ -33,10 +33,10 @@ def upgrade():
     tables = inspector.get_table_names()
     
     if 'audit_logs' in tables:
-        print(" Tabla audit_logs encontrada (creada por schema.sql)")
-        print(" Modelo AuditLog registrado en Alembic")
+        print("✅ Tabla audit_logs encontrada (creada por schema.sql)")
+        print("✅ Modelo AuditLog registrado en Alembic")
         
-        # Verificar que tenga los índices necesarios
+        # Verificar que tenga los índices necesarios
         existing_indexes = [idx['name'] for idx in inspector.get_indexes('audit_logs')]
         missing_indexes = []
         
@@ -53,17 +53,17 @@ def upgrade():
                 missing_indexes.append(idx)
         
         if missing_indexes:
-            print(f"ÔÜá´©Å  ├ìndices faltantes: {', '.join(missing_indexes)}")
+            print(f"⚠️  Índices faltantes: {', '.join(missing_indexes)}")
             print("   (Esto es normal si usaste schema.sql completo)")
         else:
-            print("Ô£à Todos los ├¡ndices necesarios est├ín presentes")
+            print("✅ Todos los índices necesarios están presentes")
         
     else:
-        print("ÔÜá´©Å  La tabla audit_logs NO existe")
+        print("⚠️  La tabla audit_logs NO existe")
         print("   Ejecuta: docker-compose exec -T postgres psql -U postgres -d servicemanager < db/schema.sql")
         print("   O crea la tabla manualmente desde schema.sql")
         
-        # No crear la tabla aquí - debe venir de schema.sql para mantener consistencia
+        # No crear la tabla aquí - debe venir de schema.sql para mantener consistencia
         raise Exception(
             "La tabla audit_logs no existe. "
             "Por favor ejecuta el schema.sql completo primero."
@@ -76,6 +76,6 @@ def downgrade():
     
     Solo des-registrar de Alembic.
     """
-    print("Ôä╣´©Å  Tabla audit_logs NO ser├í eliminada (creada por schema.sql)")
-    print(" Modelo AuditLog des-registrado de Alembic")
+    print("ℹ️  Tabla audit_logs NO será eliminada (creada por schema.sql)")
+    print("✅ Modelo AuditLog des-registrado de Alembic")
 

backend/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "servicemanager-backend"
-version = "1.6.0"
+version = "1.5.1"
 description = "ServiceManagerWeb Backend - Mesa de Ayuda B2B"
 authors = [
     {name = "Aduanasoft", email = "dev@aduanasoft.com"}

db/schema.sql

@@ -369,14 +369,10 @@ CREATE TABLE audit_logs (
 CREATE INDEX idx_audit_logs_tenant_id ON audit_logs(tenant_id);
 CREATE INDEX idx_audit_logs_user_id ON audit_logs(user_id);
 CREATE INDEX idx_audit_logs_action ON audit_logs(action);
+CREATE INDEX idx_audit_logs_resource ON audit_logs(resource_type, resource_id);
 CREATE INDEX idx_audit_logs_correlation_id ON audit_logs(correlation_id);
 CREATE INDEX idx_audit_logs_created_at ON audit_logs(created_at);
 
--- Índices compuestos para queries comunes de auditoría
-CREATE INDEX idx_audit_logs_tenant_action ON audit_logs(tenant_id, action);
-CREATE INDEX idx_audit_logs_resource ON audit_logs(resource_type, resource_id);
-CREATE INDEX idx_audit_logs_user_created ON audit_logs(user_id, created_at);
-
 -- ===================================
 -- FUNCIONES Y TRIGGERS
 -- ===================================

frontend-client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@servicemanager/client-frontend",
-  "version": "1.6.0",
+  "version": "1.5.1",
   "private": true,
   "type": "module",
   "scripts": {

frontend-client/src/lib/stores/app.ts

@@ -3,6 +3,9 @@ import { auth } from './auth.js';
 import { get } from 'svelte/store';
 import type { Writable } from 'svelte/store';
 
+// API Configuration
+const API_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
+
 // Types
 export interface Category {
   id: string;
@@ -30,7 +33,7 @@ const initialState: AppState = {
 async function apiCall(endpoint: string, options: RequestInit = {}) {
   const authState = get(auth);
   
-  const response = await fetch(`/api/v1${endpoint}`, {
+  const response = await fetch(`${API_URL}/v1${endpoint}`, {
     ...options,
     headers: {
       'Content-Type': 'application/json',

frontend-client/src/lib/stores/auth.ts

@@ -1,6 +1,9 @@
 import { writable } from 'svelte/store';
 import type { Writable } from 'svelte/store';
 
+// API Configuration
+const API_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
+
 // Types
 export interface User {
   id: string;
@@ -79,7 +82,7 @@ function createAuthStore() {
       update(state => ({ ...state, isLoading: true }));
       
       try {
-        const response = await fetch('/api/v1/auth/login', {
+        const response = await fetch(`${API_URL}/v1/auth/login`, {
           method: 'POST',
           headers: {
             'Content-Type': 'application/json',

frontend-client/src/lib/stores/tickets.ts

@@ -2,6 +2,9 @@ import type { Writable } from 'svelte/store';
 import { get, writable } from 'svelte/store';
 import { auth } from './auth';
 
+// API Configuration
+const API_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
+
 // Types
 interface FastAPIValidationError {
   loc: (string | number)[];
@@ -84,7 +87,7 @@ async function apiCall(endpoint: string, options: RequestInit = {}) {
     throw new Error('Not authenticated');
   }
 
-  const response = await fetch(`/api/v1${endpoint}`, {
+  const response = await fetch(`${API_URL}/v1${endpoint}`, {
     ...options,
     headers: {
       'Content-Type': 'application/json',
@@ -259,7 +262,7 @@ function createTicketsStore() {
           throw new Error('Not authenticated');
         }
 
-        const response = await fetch(`/api/v1/tickets/${ticketId}/attachments`, {
+        const response = await fetch(`${API_URL}/v1/tickets/${ticketId}/attachments`, {
           method: 'POST',
           headers: {
             'Authorization': `Bearer ${authState.token}`,
@@ -338,7 +341,7 @@ function createTicketsStore() {
         throw new Error('Not authenticated');
       }
 
-      const response = await fetch(`/api/v1/tickets/${ticketId}/attachments/${attachmentId}/download`, {
+      const response = await fetch(`${API_URL}/v1/tickets/${ticketId}/attachments/${attachmentId}/download`, {
         method: 'GET',
         headers: {
           'Authorization': `Bearer ${authState.token}`,

frontend-client/src/routes/profile/+page.svelte

@@ -4,6 +4,9 @@
   import { toast } from '$lib/stores/toast.js';
   import { onMount } from 'svelte';
 
+  // API Configuration
+  const API_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
+
   let currentPassword = '';
   let newPassword = '';
   let confirmPassword = '';
@@ -82,7 +85,7 @@
         return;
       }
 
-      const response = await fetch('/api/v1/client-profile/', {
+      const response = await fetch(`${API_URL}/v1/client-profile/`, {
         headers: {
           Authorization: `Bearer ${$auth.token}`,
           'X-Tenant-ID': $auth.user.tenant_id
@@ -185,7 +188,7 @@
     isUpdatingProfile = true;
 
     try {
-      const response = await fetch('/api/v1/auth/profile', {
+      const response = await fetch(`${API_URL}/v1/auth/profile`, {
         method: 'PATCH',
         headers: {
           'Content-Type': 'application/json',
@@ -218,7 +221,7 @@
     isChangingPassword = true;
 
     try {
-      const response = await fetch('/api/v1/auth/change-password', {
+      const response = await fetch(`${API_URL}/v1/auth/change-password`, {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',
@@ -269,7 +272,7 @@
         profileData.credit_limit = parseFloat(profileData.credit_limit);
       }
 
-      const response = await fetch('/api/v1/client-profile/', {
+      const response = await fetch(`${API_URL}/v1/client-profile/`, {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',

frontend-internal/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@servicemanager/internal-frontend",
-  "version": "1.6.0",
+  "version": "1.5.1",
   "private": true,
   "type": "module",
   "scripts": {

frontend-internal/src/lib/components/Modal.svelte

@@ -1,9 +1,7 @@
 <script>
-  import { createEventDispatcher } from 'svelte';
+  import { createEventDispatcher, onMount, onDestroy } from 'svelte';
   
-  export let open = false;
   export let title = '';
-  export let size = 'lg'; // sm, md, lg, xl, 2xl
   
   const dispatch = createEventDispatcher();
   
@@ -12,72 +10,37 @@
   }
 
   function handleKeydown(e) {
-    if (e.key === 'Escape' && open) {
+    if (e.key === 'Escape') {
       close();
     }
   }
-  
-  function handleBackdropClick(e) {
-    if (e.target === e.currentTarget) {
-      close();
-    }
-  }
-  
-  const sizeClasses = {
-    sm: 'sm:max-w-sm',
-    md: 'sm:max-w-md',
-    lg: 'sm:max-w-lg',
-    xl: 'sm:max-w-xl',
-    '2xl': 'sm:max-w-2xl'
-  };
 </script>
 
 <svelte:window on:keydown={handleKeydown}/>
 
-{#if open}
-  <div class="fixed inset-0 z-50 overflow-y-auto" aria-labelledby="modal-title" role="dialog" aria-modal="true">
+<div class="fixed inset-0 z-50 overflow-y-auto" aria-labelledby="modal-title" role="dialog" aria-modal="true">
     <div class="flex items-end justify-center min-h-screen px-4 pt-4 pb-20 text-center sm:block sm:p-0">
       
-      <!-- Backdrop -->
-      <div 
-        class="fixed inset-0 transition-opacity bg-gray-500 bg-opacity-75" 
-        aria-hidden="true" 
-        on:click={handleBackdropClick}
-      ></div>
+      <div class="fixed inset-0 transition-opacity bg-gray-500 bg-opacity-75" aria-hidden="true" on:click={close}></div>
 
-      <!-- Center trick -->
       <span class="hidden sm:inline-block sm:align-middle sm:h-screen" aria-hidden="true">&#8203;</span>
 
-      <!-- Modal panel -->
-      <div class="inline-block w-full align-bottom bg-white rounded-lg shadow-xl transform transition-all sm:my-8 sm:align-middle {sizeClasses[size]} sm:w-full">
-        <!-- Header -->
-        <div class="px-6 py-4 border-b border-gray-200 flex items-center justify-between">
-          <h3 class="text-lg font-semibold text-gray-900" id="modal-title">
-            {title}
-          </h3>
-          <button 
-            type="button"
-            on:click={close}
-            class="text-gray-400 hover:text-gray-500 focus:outline-none focus:ring-2 focus:ring-primary-500 rounded-lg p-1"
-          >
-            <svg class="w-5 h-5" fill="none" stroke="currentColor" viewBox="0 0 24 24">
-              <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M6 18L18 6M6 6l12 12" />
-            </svg>
-          </button>
+      <div class="inline-block px-4 pt-5 pb-4 overflow-hidden text-left align-bottom transition-all transform bg-white rounded-lg shadow-xl sm:my-8 sm:align-middle sm:max-w-lg sm:w-full sm:p-6">
+        <div class="sm:flex sm:items-start">
+          <div class="mt-3 text-center sm:mt-0 sm:ml-4 sm:text-left w-full">
+            <h3 class="text-lg leading-6 font-medium text-gray-900" id="modal-title">
+              {title}
+            </h3>
+            <div class="mt-2 text-sm text-gray-500">
+              <slot />
+            </div>
+          </div>
         </div>
-        
-        <!-- Body -->
-        <div class="px-6 py-4 max-h-[70vh] overflow-y-auto">
-          <slot />
-        </div>
-        
-        <!-- Footer (optional) -->
         {#if $$slots.footer}
-          <div class="px-6 py-4 bg-gray-50 border-t border-gray-200 rounded-b-lg">
+          <div class="mt-5 sm:mt-6 sm:flex sm:flex-row-reverse">
             <slot name="footer" />
           </div>
         {/if}
       </div>
     </div>
   </div>
-{/if}

frontend-internal/src/lib/stores/auth.ts

@@ -1,6 +1,9 @@
 import { writable } from 'svelte/store';
 import type { Writable } from 'svelte/store';
 
+// API Configuration
+const API_URL = import.meta.env.VITE_API_URL || 'http://localhost:8000';
+
 // Types
 export interface InternalUser {
   id: string;
@@ -95,7 +98,7 @@ function createAuthStore() {
       update(state => ({ ...state, isLoading: true }));
       
       try {
-        const response = await fetch('/api/v1/auth/login', {
+        const response = await fetch(`${API_URL}/v1/auth/login`, {
           method: 'POST',
           headers: {
             'Content-Type': 'application/json',
@@ -147,7 +150,7 @@ function createAuthStore() {
         update (state => ({ ...state, isLoading: true }));
 
         try {
-            const response = await fetch('/api/v1/auth/refresh', {
+            const response = await fetch(`${API_URL}/v1/auth/refresh`, {
                 method: 'POST',
                 headers: {
                     'Content-Type': 'application/json',

frontend-internal/src/routes/categories/+page.svelte

@@ -145,7 +145,8 @@
   </div>
 </div>
 
-<Modal open={showModal} title={editingCategory ? 'Editar Categoría' : 'Nueva Categoría'} on:close={() => showModal = false}>
+{#if showModal}
+<Modal title={editingCategory ? 'Editar Categoría' : 'Nueva Categoría'} on:close={() => showModal = false}>
   <form on:submit|preventDefault={handleSubmit} class="space-y-4">
     <div>
       <label for="name" class="block text-sm font-medium text-gray-700">Nombre</label>
@@ -182,3 +183,4 @@
     </div>
   </form>
 </Modal>
+{/if}

frontend-internal/src/routes/systems/+page.svelte

@@ -118,7 +118,8 @@
   </div>
 </div>
 
-<Modal open={showModal} title={editingSystem ? 'Editar Sistema' : 'Nuevo Sistema'} on:close={() => showModal = false}>
+{#if showModal}
+<Modal title={editingSystem ? 'Editar Sistema' : 'Nuevo Sistema'} on:close={() => showModal = false}>
   <form on:submit|preventDefault={handleSubmit} class="space-y-4">
     <div>
       <label for="name" class="block text-sm font-medium text-gray-700">Nombre</label>
@@ -145,3 +146,4 @@
     </div>
   </form>
 </Modal>
+{/if}

frontend-internal/src/routes/tenants/+page.svelte

@@ -126,7 +126,8 @@
   </div>
 </div>
 
-<Modal open={showModal} title={editingTenant ? 'Editar Cliente' : 'Nuevo Cliente'} on:close={() => showModal = false}>
+{#if showModal}
+<Modal title={editingTenant ? 'Editar Cliente' : 'Nuevo Cliente'} on:close={() => showModal = false}>
   <form on:submit|preventDefault={handleSubmit} class="space-y-4">
     <div>
       <label for="name" class="block text-sm font-medium text-gray-700">Nombre</label>
@@ -164,3 +165,4 @@
     </div>
   </form>
 </Modal>
+{/if}

frontend-internal/src/routes/tickets/+page.svelte

@@ -522,7 +522,8 @@
 </div>
 
 <!-- Modal Crear Ticket -->
-<Modal open={showModal} title="Nuevo Ticket" on:close={() => showModal = false}>
+{#if showModal}
+<Modal title="Nuevo Ticket" on:close={() => showModal = false}>
   <form on:submit|preventDefault={handleSubmit} class="space-y-4">
     <div>
       <label for="subject" class="block text-sm font-medium text-gray-700">Asunto *</label>
@@ -608,9 +609,11 @@
     </div>
   </form>
 </Modal>
+{/if}
 
 <!-- Modal Editar Ticket -->
-<Modal open={showEditModal} title="Editar Ticket #{selectedTicket?.ticket_number || ''}" on:close={() => showEditModal = false}>
+{#if showEditModal}
+<Modal title="Editar Ticket #{selectedTicket?.ticket_number || ''}" on:close={() => showEditModal = false}>
   <form on:submit|preventDefault={handleUpdate} class="space-y-4">
     <div>
       <label for="editStatus" class="block text-sm font-medium text-gray-700">Estado</label>
@@ -674,9 +677,11 @@
     </div>
   </form>
 </Modal>
+{/if}
 
 <!-- Modal Eliminar Ticket -->
-<Modal open={showDeleteModal} title="Eliminar Ticket" on:close={() => showDeleteModal = false}>
+{#if showDeleteModal}
+<Modal title="Eliminar Ticket" on:close={() => showDeleteModal = false}>
   <div class="space-y-4">
     <div class="bg-red-50 border border-red-200 rounded-md p-4">
       <div class="flex">
@@ -718,4 +723,5 @@
       </button>
     </div>
   </div>
-</Modal>
+</Modal>
+{/if}

frontend-internal/src/routes/users/+page.svelte

@@ -172,7 +172,8 @@
   </div>
 </div>
 
-<Modal open={showModal} title={editingUser ? 'Editar Usuario' : 'Nuevo Usuario'} on:close={() => showModal = false}>
+{#if showModal}
+<Modal title={editingUser ? 'Editar Usuario' : 'Nuevo Usuario'} on:close={() => showModal = false}>
   <form on:submit|preventDefault={handleSubmit} class="space-y-4">
     <div class="grid grid-cols-2 gap-4">
       <div>
@@ -229,3 +230,4 @@
     </div>
   </form>
 </Modal>
+{/if}