refactor(no_conformidades,correo): consolidate managers, remove legacy correo_tareas/correos + add brass shim + cleanup tests

Author: DysTelefonica-legacy

.env.example

@@ -128,4 +128,3 @@ RIESGOS_CALIDAD_MENSUAL_FRECUENCIA_DIAS=30
 BRASS_FRECUENCIA_DIAS=1
 EXPEDIENTES_FRECUENCIA_DIAS=1
 AGEDYS_FRECUENCIA_DIAS=1
-CORREO_TAREAS_FRECUENCIA_DIAS=1

README.md

@@ -397,7 +397,6 @@ scripts-python/
 │   │   ├── __init__.py
 │   │   ├── brass_manager.py     # Gestor principal BRASS
 │   │   ├── brass_task.py        # Tareas BRASS
-│   │   └── run_brass.py         # Script BRASS interno
 │   ├── common/                  # Utilidades compartidas
 │   │   ├── __init__.py
 │   │   ├── access_connection_pool.py # Pool de conexiones Access
@@ -499,7 +498,6 @@ scripts-python/
 
 ## � Módulo No Conformidades – Funcionamiento, Arquitectura y Flujo
 
-> Esta sección ha sido revisada tras la refactorización: unificación de generación HTML, método genérico para recuperación de ARs por técnico, tipado fuerte y manejo específico de errores.
 
 ### 1. Propósito
 Automatizar el seguimiento y la comunicación sobre:
@@ -513,9 +511,8 @@ Automatizar el seguimiento y la comunicación sobre:
 |------------|---------|-----------------------|
 | Runner CLI | `scripts/run_no_conformidades.py` | Orquestación, flags de fuerza / dry-run, logging y registro de tareas. |
 | Manager | `src/no_conformidades/no_conformidades_manager.py` | Lógica de negocio, consultas SQL, flujo de generación interna. |
-| Registrador | `src/no_conformidades/report_registrar.py` | Construcción y registro final de emails (calidad / técnicos). |
 | Generador HTML | `src/common/html_report_generator.py` | Header / footer modernos y tablas unificadas. |
-| Tipos | `src/no_conformidades/types.py` | TypedDict para estructuras de datos AR técnicas y calidad. |
+| Tipos | Integrados en `no_conformidades_manager.py` | TypedDict para AR técnicas y calidad. |
 | Persistencia Avisos | Tabla `TbNCARAvisos` | Evita correos repetidos por AR y rango (0 / 7 / 15 días). |
 | Registro Email | Tabla `TbCorreosEnviados` (BD Tareas) | Trazabilidad de envíos y cuerpo HTML. |
 
@@ -1210,7 +1207,7 @@ python scripts/run_riesgos.py                   # Ejecución normal
 python scripts/run_riesgos.py --force           # Fuerza ejecución
 python scripts/run_riesgos.py --dry-run         # Modo simulación
 
-# Email Services - Servicio unificado de correo (remplaza correos y correo_tareas)
+# Email Services - Servicio unificado de correo (reemplaza módulos de correo previos)
 
 ### Manejo de errores transitorios SMTP (Refactor 2025)
 Los errores de conexión SMTP (p.ej. desconexión inesperada, `SMTPConnectError`, `ConnectionRefusedError`) ahora se consideran **transitorios** y no marcan el correo como fallido en la base de datos. El registro permanece pendiente para reintentos en futuros ciclos. Sólo errores definitivos (credenciales inválidas, destinatario rechazado, formato de mensaje inválido) marcan el correo como fallido. Esto incrementa la resiliencia ante caídas puntuales del servidor de correo.

infra/analysis/Dockerfile

@@ -0,0 +1,38 @@
+# Usar una imagen base de Python
+FROM python:3.9-slim
+
+# Variables de entorno para las versiones de las herramientas
+ENV SONAR_SCANNER_VERSION=4.7.0.2747
+ENV DEPENDENCY_CHECK_VERSION=7.1.1
+
+# Instalar dependencias del sistema
+RUN apt-get update && apt-get install -y \
+    unzip \
+    default-jre \
+    && rm -rf /var/lib/apt/lists/*
+
+# Instalar SonarScanner
+RUN apt-get update && apt-get install -y wget && \
+    wget https://binaries.sonarsource.com/Distribution/sonar-scanner-cli/sonar-scanner-cli-${SONAR_SCANNER_VERSION}-linux.zip && \
+    unzip sonar-scanner-cli-${SONAR_SCANNER_VERSION}-linux.zip && \
+    mv sonar-scanner-${SONAR_SCANNER_VERSION}-linux /opt/sonar-scanner && \
+    rm sonar-scanner-cli-${SONAR_SCANNER_VERSION}-linux.zip
+ENV PATH="/opt/sonar-scanner/bin:${PATH}"
+
+# Instalar OWASP Dependency-Check
+RUN wget https://github.com/jeremylong/DependencyCheck/releases/download/v${DEPENDENCY_CHECK_VERSION}/dependency-check-${DEPENDENCY_CHECK_VERSION}-release.zip && \
+    unzip dependency-check-${DEPENDENCY_CHECK_VERSION}-release.zip && \
+    mv dependency-check /opt/dependency-check && \
+    rm dependency-check-${DEPENDENCY_CHECK_VERSION}-release.zip
+ENV PATH="/opt/dependency-check/bin:${PATH}"
+
+# Establecer el directorio de trabajo
+WORKDIR /usr/src
+
+# --- CORRECCIÓN AÑADIDA AQUÍ ---
+# Copiar el script de análisis desde su ubicación correcta (analysis/run-analysis.sh)
+COPY analysis/run-analysis.sh .
+RUN chmod +x run-analysis.sh
+
+# Comando por defecto que se ejecutará al iniciar el contenedor
+CMD ["./run-analysis.sh"]

infra/analysis/run-analysis.sh

@@ -0,0 +1,19 @@
+#!/bin/bash
+
+echo "Starting SonarQube analysis..."
+sonar-scanner \
+  -Dsonar.projectKey=scripts-python \
+  -Dsonar.sources=/usr/src/src \
+  -Dsonar.host.url=http://sonarqube:9000 \
+  -Dsonar.login=sqp_... # Placeholder for SonarQube token, user needs to replace this
+
+echo "Starting OWASP Dependency-Check analysis..."
+/opt/dependency-check/bin/dependency-check.sh \
+  --project "scripts-python" \
+  --scan /usr/src \
+  --format HTML \
+  --out /usr/src/owasp-report \
+  --enableExperimental \
+  --enableRetired
+
+echo "Analysis complete."

infra/grafana/provisioning/datasources/loki-datasource.yml

@@ -0,0 +1,9 @@
+apiVersion: 1
+
+datasources:
+  - name: Loki
+    type: loki
+    access: proxy
+    url: http://loki:3100
+    isDefault: true
+    editable: true

infra/loki/loki-config.yml

@@ -0,0 +1,49 @@
+auth_enabled: false
+
+server:
+  http_listen_port: 3100
+  grpc_listen_port: 9096
+
+common:
+  path_prefix: /loki
+  storage:
+    filesystem:
+      chunks_directory: /loki/chunks
+      rules_directory: /loki/rules
+  replication_factor: 1
+  ring:
+    instance_addr: 127.0.0.1
+    kvstore:
+      store: inmemory
+
+query_range:
+  results_cache:
+    cache:
+      embedded_cache:
+        enabled: true
+        max_size_mb: 100
+
+schema_config:
+  configs:
+    - from: 2020-10-24
+      store: boltdb-shipper
+      object_store: filesystem
+      schema: v11
+      index:
+        prefix: index_
+        period: 24h
+
+ruler:
+  alertmanager_url: http://localhost:9093
+
+# By default, Loki will send anonymous, but uniquely-identifiable usage and configuration
+# analytics to Grafana Labs. These statistics are sent to https://stats.grafana.org/
+#
+# Statistics help us better understand how Loki is used, and they show us performance
+# levels for most users. This helps us prioritize features and documentation.
+# For more information on what's sent: https://github.com/grafana/loki/blob/main/docs/sources/configuration/telemetry.md
+# Refer to the buildReport method to see what goes into a report.
+#
+# If you would like to disable reporting, uncomment the following lines:
+#analytics:
+#  reporting_enabled: false

pytest.ini

@@ -3,4 +3,10 @@
 python_files = test_*.py
 python_classes = Test*
 python_functions = test_*
+markers =
     slow: tests lentos
+    integration: tests de integración
+filterwarnings =
+    ignore::DeprecationWarning
+    ignore::UserWarning
+    ignore::pytest.PytestUnknownMarkWarning

scripts/README.md

@@ -40,7 +40,7 @@ El **Master Runner** es el corazón del sistema de monitoreo continuo. Ejecuta d
 ## 📧 Scripts de Producción
 
 ### run_email_services.py
-- **Función**: Gestión unificada de servicios de correo (reemplaza correos y correo_tareas)
+- **Función**: Gestión unificada de servicios de correo (reemplaza módulos históricos de envío)
 - **Tipo**: Tarea continua (ejecutada en cada ciclo)
 - **Descripción**: Procesa colas de correo de múltiples orígenes (correos, tareas) y envía notificaciones