style: cerrar deuda de estilo ruff en src/ y tests/

Limpia los 80 hallazgos de `ruff check` acumulados en el repo (tipos
Optional/Dict/List modernizados a sintaxis X|None/dict/list, isinstance
duplicados fusionados, condicionales anidados aplanados, ClassVar en
dicts de clase de solo lectura, y manejo explícito de check=False en
subprocess.run). Incluye dos mejoras acotadas: mermaid_renderer.py
cierra el archivo temporal antes de invocar el subproceso externo
(requerido en Windows), y flowchart_agent.py documenta explícitamente
por qué su except Exception amplio es intencional.

Revisado por python-reviewer y security-reviewer sin hallazgos
CRITICAL/HIGH. Suite completa (207 tests) en verde.

Author: ljyudico

src/multiagent_core/code_auditor_agent.py

@@ -13,7 +13,7 @@
 import subprocess
 import sys
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any
 
 SKILL_METADATA = {
     "name": "code_auditor_agent",
@@ -31,7 +31,7 @@ class CodeAuditorAgent:
     def __init__(self):
         pass
 
-    def audit_style(self, code: str) -> List[str]:
+    def audit_style(self, code: str) -> list[str]:
         """Audita el estilo básico del código (PEP 8 simplificado)."""
         issues = []
         lines = code.split("\n")
@@ -50,7 +50,7 @@ def audit_style(self, code: str) -> List[str]:
 
         return issues
 
-    def audit_security(self, code: str) -> List[str]:
+    def audit_security(self, code: str) -> list[str]:
         """
         Escaneo estático de seguridad buscando riesgos asociados a código generado por IA
         y OWASP (ej. llaves expuestas, entradas no validadas, funciones peligrosas).
@@ -66,13 +66,16 @@ def audit_security(self, code: str) -> List[str]:
 
         # 1. Buscar uso de funciones peligrosas (eval, exec)
         for node in ast.walk(tree):
-            if isinstance(node, ast.Call) and isinstance(node.func, ast.Name):
-                if node.func.id in ["eval", "exec"]:
-                    issues.append(
-                        f"Riesgo de Seguridad: Uso de `{node.func.id}()`. "
-                        "Ejecutar cadenas de texto arbitrarias expone al sistema a inyecciones de código "
-                        "y ejecución remota de comandos no autorizados (Riesgo OWASP LLM-02)."
-                    )
+            if (
+                isinstance(node, ast.Call)
+                and isinstance(node.func, ast.Name)
+                and node.func.id in ["eval", "exec"]
+            ):
+                issues.append(
+                    f"Riesgo de Seguridad: Uso de `{node.func.id}()`. "
+                    "Ejecutar cadenas de texto arbitrarias expone al sistema a inyecciones de código "
+                    "y ejecución remota de comandos no autorizados (Riesgo OWASP LLM-02)."
+                )
 
         # 2. Buscar API Keys expuestas mediante heurística de variables
         api_key_pattern = re.compile(
@@ -89,7 +92,7 @@ def audit_security(self, code: str) -> List[str]:
 
         return issues
 
-    def run_pytest(self, test_file_path: Path) -> Dict[str, Any]:
+    def run_pytest(self, test_file_path: Path) -> dict[str, Any]:
         """Ejecuta pytest sobre el archivo de pruebas especificado y retorna el resultado."""
         try:
             result = subprocess.run(
@@ -117,7 +120,7 @@ def run_pytest(self, test_file_path: Path) -> Dict[str, Any]:
             }
 
     def generate_report(
-        self, student_code: str, test_file_path: Optional[Path] = None
+        self, student_code: str, test_file_path: Path | None = None
     ) -> str:
         """Genera un reporte pedagógico completo en Markdown."""
         style_issues = self.audit_style(student_code)

src/multiagent_core/content_auditor_agent.py

@@ -11,9 +11,9 @@
 
 import ast
 import re
-from functools import lru_cache
+from functools import cache
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any
 
 from .code_auditor_agent import CodeAuditorAgent
 from .mermaid_renderer import MermaidRenderer
@@ -38,8 +38,8 @@
 _ARGUMENTOS_SIN_TIPO_ESPERADO = {"self", "cls"}
 
 
-@lru_cache(maxsize=None)
-def _parse_programa_oficial(programa_path: Path) -> Dict[int, dict]:
+@cache
+def _parse_programa_oficial(programa_path: Path) -> dict[int, dict]:
     """Parsea el programa de asignatura oficial en subtemas y semanas por unidad.
 
     El resultado se cachea por ruta (`lru_cache`) porque `audit_all_units`
@@ -59,7 +59,7 @@ def _parse_programa_oficial(programa_path: Path) -> Dict[int, dict]:
     content = programa_path.read_text(encoding="utf-8")
     lines = content.split("\n")
 
-    mapeo: Dict[int, dict] = {}
+    mapeo: dict[int, dict] = {}
     unidad_actual = None
 
     for line in lines:
@@ -95,8 +95,8 @@ class ContentAuditorAgent:
 
     def __init__(
         self,
-        programa_path: Optional[Path] = None,
-        mermaid_renderer: Optional[MermaidRenderer] = None,
+        programa_path: Path | None = None,
+        mermaid_renderer: MermaidRenderer | None = None,
     ) -> None:
         self.code_auditor = CodeAuditorAgent()
         self.programa_path = (
@@ -106,7 +106,7 @@ def __init__(
             output_dir=Path.cwd() / "notebooks" / "assets" / "diagramas"
         )
 
-    def _audit_latex(self, content: str) -> List[str]:
+    def _audit_latex(self, content: str) -> list[str]:
         """Detecta delimitadores LaTeX desbalanceados y comandos mal formados.
 
         Args:
@@ -115,7 +115,7 @@ def _audit_latex(self, content: str) -> List[str]:
         Returns:
             Lista de descripciones de hallazgos; vacía si no hay problemas.
         """
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
 
         sin_fences = re.sub(r"```.*?```", "", content, flags=re.DOTALL)
         tokens = re.findall(r"\$\$|\$", sin_fences)
@@ -142,7 +142,7 @@ def _audit_latex(self, content: str) -> List[str]:
 
         return hallazgos
 
-    def _audit_codigo(self, python_blocks: List[str]) -> List[str]:
+    def _audit_codigo(self, python_blocks: list[str]) -> list[str]:
         """Audita bloques de código Python de ejemplo (docstrings, type hints, estilo).
 
         Ignora la longitud de línea (PEP8 79 cols) porque el código de
@@ -160,7 +160,7 @@ def _audit_codigo(self, python_blocks: List[str]) -> List[str]:
         Returns:
             Lista de descripciones de hallazgos; vacía si no hay problemas.
         """
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
 
         for code in python_blocks:
             if _CELDA_MAGICA_IPYTHON.search(code):
@@ -200,7 +200,7 @@ def _audit_codigo(self, python_blocks: List[str]) -> List[str]:
 
         return hallazgos
 
-    def _audit_pedagogico(self, bloques: List[tuple], content: str) -> List[str]:
+    def _audit_pedagogico(self, bloques: list[tuple], content: str) -> list[str]:
         """Verifica coherencia pedagógica: Hilo de Oro, analogías, y sintaxis Mermaid.
 
         Args:
@@ -210,7 +210,7 @@ def _audit_pedagogico(self, bloques: List[tuple], content: str) -> List[str]:
         Returns:
             Lista de descripciones de hallazgos; vacía si no hay problemas.
         """
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
 
         idiomas_presentes = {lang for _, lang, _ in bloques}
         hilo_de_oro_esperado = {"pseudocodigo", "mermaid", "python", "pytest"}
@@ -244,7 +244,7 @@ def _audit_pedagogico(self, bloques: List[tuple], content: str) -> List[str]:
 
         return hallazgos
 
-    def _audit_curricular(self, md_path: Path, content: str) -> List[str]:
+    def _audit_curricular(self, md_path: Path, content: str) -> list[str]:
         """Verifica cobertura temática y duración contra el programa oficial.
 
         Args:
@@ -264,7 +264,7 @@ def _audit_curricular(self, md_path: Path, content: str) -> List[str]:
         if numero_unidad not in mapeo:
             return []
 
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
         info = mapeo[numero_unidad]
         content_lower = content.lower()
 
@@ -297,8 +297,8 @@ def _audit_curricular(self, md_path: Path, content: str) -> List[str]:
     _UNIT_NUMBER_PATTERN = re.compile(r"unit_number\s*=\s*(\d+)")
 
     def _verifica_unit_number(
-        self, python_blocks: List[str], md_path: Path
-    ) -> List[str]:
+        self, python_blocks: list[str], md_path: Path
+    ) -> list[str]:
         """Verifica que unit_number en la celda de auto-evaluación coincida
         con el número de unidad indicado por el nombre del archivo.
 
@@ -316,7 +316,7 @@ def _verifica_unit_number(
             return []
         numero_esperado = int(unidad_match.group(1))
 
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
         for code in python_blocks:
             match = self._UNIT_NUMBER_PATTERN.search(code)
             if match and int(match.group(1)) != numero_esperado:
@@ -328,7 +328,7 @@ def _verifica_unit_number(
 
     _WRITEFILE_LINEA = re.compile(r"^\s*%%writefile\s+(\S+)", re.MULTILINE)
 
-    def _extrae_writefiles_reales(self, python_blocks: List[str]) -> List[tuple]:
+    def _extrae_writefiles_reales(self, python_blocks: list[str]) -> list[tuple]:
         """Extrae líneas reales de %%writefile (primera línea no en blanco
         de una celda), ignorando menciones en prosa o dentro de f-strings.
 
@@ -345,7 +345,7 @@ def _extrae_writefiles_reales(self, python_blocks: List[str]) -> List[tuple]:
         return resultado
 
     def _tiene_definicion_previa(
-        self, python_blocks: List[str], indice_bloque: int
+        self, python_blocks: list[str], indice_bloque: int
     ) -> bool:
         """Verifica si algún bloque anterior a indice_bloque define una
         función o clase (ast.parse + ast.walk), tolerando bloques con
@@ -372,8 +372,8 @@ def _tiene_definicion_previa(
         return False
 
     def _verifica_writefiles_con_definicion(
-        self, python_blocks: List[str]
-    ) -> List[str]:
+        self, python_blocks: list[str]
+    ) -> list[str]:
         """Verifica que todo %%writefile real tenga una definición de
         función/clase en algún bloque anterior del mismo MD.
 
@@ -384,7 +384,7 @@ def _verifica_writefiles_con_definicion(
             Lista con un hallazgo por cada %%writefile sin definición
             previa; vacía si todos tienen definición o no hay ninguno.
         """
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
         for nombre_archivo, indice_bloque in self._extrae_writefiles_reales(
             python_blocks
         ):
@@ -396,7 +396,7 @@ def _verifica_writefiles_con_definicion(
                 )
         return hallazgos
 
-    def _verifica_fences_balanceados(self, content: str) -> List[str]:
+    def _verifica_fences_balanceados(self, content: str) -> list[str]:
         """Detecta fences de apertura que extract_fenced_blocks() no pudo
         cerrar correctamente — señal indirecta: el último bloque extraído
         consume texto hasta el final del documento, aunque el .md real
@@ -435,14 +435,16 @@ def _verifica_fences_balanceados(self, content: str) -> List[str]:
         )
         if not cierre_encontrado:
             return [
-                f"Posible fence de código sin cerrar: un bloque abierto con "
-                f"'{ultimo_fence}' no encuentra su línea de cierre exacta en "
-                "el resto del documento — extract_fenced_blocks() pudo haber "
-                "consumido contenido no destinado a ser código."
+                (
+                    f"Posible fence de código sin cerrar: un bloque abierto con "
+                    f"'{ultimo_fence}' no encuentra su línea de cierre exacta en "
+                    "el resto del documento — extract_fenced_blocks() pudo haber "
+                    "consumido contenido no destinado a ser código."
+                )
             ]
         return []
 
-    def _verifica_celda_setup(self, content: str) -> List[str]:
+    def _verifica_celda_setup(self, content: str) -> list[str]:
         """Verifica que la celda estándar de setup (git clone + os.chdir)
         esté presente en el MD.
 
@@ -454,8 +456,10 @@ def _verifica_celda_setup(self, content: str) -> List[str]:
         """
         if "git clone" not in content or "os.chdir" not in content:
             return [
-                "No se encontró la celda de setup estándar (git clone + "
-                "os.chdir) en esta unidad."
+                (
+                    "No se encontró la celda de setup estándar (git clone + "
+                    "os.chdir) en esta unidad."
+                )
             ]
         return []
 
@@ -464,7 +468,7 @@ def _verifica_celda_setup(self, content: str) -> List[str]:
         r"^- La secuencia de la unidad anterior"
     )
 
-    def _verifica_seccion_prerequisitos(self, content: str) -> List[str]:
+    def _verifica_seccion_prerequisitos(self, content: str) -> list[str]:
         """Verifica que la sección de prerequisitos, si existe, tenga al
         menos una línea con el formato parseable esperado por
         CurriculumMapAgent.render_dag() (relación estructurada o la línea
@@ -499,16 +503,18 @@ def _verifica_seccion_prerequisitos(self, content: str) -> List[str]:
                 return []
 
         return [
-            "La sección '## 📚 Prerequisitos de esta unidad' existe pero "
-            "ninguna de sus líneas sigue el formato esperado "
-            "('- **término** (Unidad N)' o la línea alternativa de "
-            "'sin conceptos adicionales') — CurriculumMapAgent.render_dag() "
-            "no podrá extraer ninguna relación de esta unidad."
+            (
+                "La sección '## 📚 Prerequisitos de esta unidad' existe pero "
+                "ninguna de sus líneas sigue el formato esperado "
+                "('- **término** (Unidad N)' o la línea alternativa de "
+                "'sin conceptos adicionales') — CurriculumMapAgent.render_dag() "
+                "no podrá extraer ninguna relación de esta unidad."
+            )
         ]
 
     def _audit_invariantes_estructurales(
-        self, python_blocks: List[str], content: str, md_path: Path
-    ) -> List[str]:
+        self, python_blocks: list[str], content: str, md_path: Path
+    ) -> list[str]:
         """Audita invariantes estructurales: consistencia de la celda de
         auto-evaluación (donde ya existe), consistencia de la sección de
         prerequisitos (donde ya existe), e invariantes generales del
@@ -522,15 +528,15 @@ def _audit_invariantes_estructurales(
         Returns:
             Lista concatenada de hallazgos de las 5 verificaciones.
         """
-        hallazgos: List[str] = []
+        hallazgos: list[str] = []
         hallazgos.extend(self._verifica_unit_number(python_blocks, md_path))
         hallazgos.extend(self._verifica_writefiles_con_definicion(python_blocks))
         hallazgos.extend(self._verifica_fences_balanceados(content))
         hallazgos.extend(self._verifica_celda_setup(content))
         hallazgos.extend(self._verifica_seccion_prerequisitos(content))
         return hallazgos
 
-    def audit_unit(self, md_path: Path) -> Dict[str, Any]:
+    def audit_unit(self, md_path: Path) -> dict[str, Any]:
         """Audita una unidad del curso contra las 5 dimensiones de calidad.
 
         Args:

src/multiagent_core/curriculum_map_agent.py

@@ -10,7 +10,7 @@
 
 import re
 from pathlib import Path
-from typing import Dict, List
+from typing import ClassVar
 
 from .notebook_compiler_agent import extract_fenced_blocks
 
@@ -66,7 +66,7 @@
 class CurriculumMapAgent:
     """Agente que sugiere y renderiza el mapa de dependencias entre unidades."""
 
-    _NOMBRES_UNIDAD = {
+    _NOMBRES_UNIDAD: ClassVar[dict[int, str]] = {
         0: "Entorno de Trabajo",
         1: "Pensamiento Computacional",
         2: "Metodología y Pruebas",
@@ -83,7 +83,7 @@ def _numero_de_unidad(self, md_path: Path) -> int:
         match = _UNIT_NUMBER_FROM_FILENAME.match(md_path.name)
         return int(match.group(1)) if match else -1
 
-    def _leer_unidades_ordenadas(self, course_dir: Path) -> List[tuple]:
+    def _leer_unidades_ordenadas(self, course_dir: Path) -> list[tuple]:
         """Lee todas las UNIDAD_*.md y las retorna ordenadas por número.
 
         Returns:
@@ -107,7 +107,7 @@ def _buscar_evidencia_prosa(self, termino: str, content: str) -> str:
                 return linea.strip()
         return ""
 
-    def suggest_prerequisites(self, course_dir: Path) -> Dict[int, List[dict]]:
+    def suggest_prerequisites(self, course_dir: Path) -> dict[int, list[dict]]:
         """Sugiere candidatos de relación de concepto entre unidades.
 
         Para cada unidad, busca en unidades ANTERIORES apariciones de los
@@ -123,10 +123,10 @@ def suggest_prerequisites(self, course_dir: Path) -> Dict[int, List[dict]]:
             es {"unidad_origen": int, "termino": str, "evidencia": str}.
         """
         unidades = self._leer_unidades_ordenadas(course_dir)
-        resultado: Dict[int, List[dict]] = {}
+        resultado: dict[int, list[dict]] = {}
 
         for i, (numero_actual, content_actual, blocks_actual) in enumerate(unidades):
-            candidatos: List[dict] = []
+            candidatos: list[dict] = []
             unidades_anteriores = unidades[:i]
 
             for termino in TERMINOS_TECNICOS_CURADOS:
@@ -162,7 +162,7 @@ def suggest_prerequisites(self, course_dir: Path) -> Dict[int, List[dict]]:
 
         return resultado
 
-    def _extrae_relaciones_de_seccion(self, content: str) -> List[dict]:
+    def _extrae_relaciones_de_seccion(self, content: str) -> list[dict]:
         """Extrae relaciones (término, unidad_origen) de la sección de
         prerequisitos ya escrita, ignorando líneas que no matcheen el
         formato exacto.