Fix: Agrega logs para las operaciones en la base de datos #4

Merged
A8065384 merged 8 commits from fix/logs into main 2026-02-24 22:37:15 +00:00
Member

PR para #2

Resumen

Este PR refuerza la estabilidad y observabilidad del agente: endurece validaciones, amplía el manejo de errores y actualiza todo el stack de logging sin interrumpir la ejecución del servicio.

Cambios principales

Nuevo logger centralizado con soporte para log_structured_entry en formato JSON.
🛡️ Validaciones adicionales para la conexión a la base de datos y manejo robusto de excepciones en puntos críticos.
📝 Refactor de importaciones y adición de logs de arranque para depurar flujos con mayor precisión.
💾 Persistencia de errores sin cerrar la app, permitiendo investigar fallos post‑ejecución.
Ajustes finales para que ruff y ty pasen al 100 %.

Características destacadas

  • Logging estructurado listo para integrarse con el stack de observabilidad actual.
  • Evita cierres inesperados: las excepciones se capturan, se registran y se conservan para análisis.
  • Validaciones “fail fast” al abrir conexiones, con mensajes accionables.
  • Instrumentación adicional que deja trazabilidad completa del ciclo de vida del agente.
  • Estilo y tipado alineados con las herramientas (uv run ruff check, uv run ty check).
> PR para #2 ## Resumen Este PR refuerza la estabilidad y observabilidad del agente: endurece validaciones, amplía el manejo de errores y actualiza todo el stack de logging sin interrumpir la ejecución del servicio. ### Cambios principales ✨ Nuevo logger centralizado con soporte para `log_structured_entry` en formato JSON. 🛡️ Validaciones adicionales para la conexión a la base de datos y manejo robusto de excepciones en puntos críticos. 📝 Refactor de importaciones y adición de logs de arranque para depurar flujos con mayor precisión. 💾 Persistencia de errores sin cerrar la app, permitiendo investigar fallos post‑ejecución. ✅ Ajustes finales para que `ruff` y `ty` pasen al 100 %. ### Características destacadas - Logging estructurado listo para integrarse con el stack de observabilidad actual. - Evita cierres inesperados: las excepciones se capturan, se registran y se conservan para análisis. - Validaciones “fail fast” al abrir conexiones, con mensajes accionables. - Instrumentación adicional que deja trazabilidad completa del ciclo de vida del agente. - Estilo y tipado alineados con las herramientas (`uv run ruff check`, `uv run ty check`).
A8080816 added 7 commits 2026-02-24 18:35:31 +00:00
A8080816 requested review from A8065384 2026-02-24 18:35:31 +00:00
A8080816 added this to the Vaia v1.0 project 2026-02-24 18:35:31 +00:00
A8080816 changed title from Fix: Agrega logs par alas operaciones en la Base de Datos to Fix: Agrega logs para las operaciones en la base de datos 2026-02-24 19:42:38 +00:00
A8065384 requested changes 2026-02-24 21:56:39 +00:00
@@ -0,0 +14,4 @@
def _setup_logger() -> logging.Logger:
"""Create or return the singleton evaluation logger."""
log_name = "va_agent-evaluation-logs"
Owner

Convertir en parametro dentro de Settings

Convertir en parametro dentro de `Settings`
A8080816 marked this conversation as resolved
@@ -0,0 +16,4 @@
"""Create or return the singleton evaluation logger."""
log_name = "va_agent-evaluation-logs"
logger = logging.getLogger(log_name)
cfg = Settings.model_validate({})
Owner

Se instancia Settings dos veces, podemos centralizarlo en config

Se instancia `Settings` dos veces, podemos centralizarlo en `config`
A8080816 marked this conversation as resolved
@@ -0,0 +24,4 @@
client = google.cloud.logging.Client(project=cfg.project_id)
handler = CloudLoggingHandler(client, name=log_name) # async transport
logger.addHandler(handler)
logger.setLevel(logging.INFO)
Owner

Convertir LOG LEVEL en parámetro de ambiente también

Convertir LOG LEVEL en parámetro de ambiente también
A8080816 marked this conversation as resolved
A8080816 added 1 commit 2026-02-24 22:14:36 +00:00
A8065384 merged commit e9b4c93a20 into main 2026-02-24 22:37:15 +00:00
A8065384 deleted branch fix/logs 2026-02-24 22:37:15 +00:00
A8080816 removed this from the Vaia v1.0 project 2026-02-24 22:45:41 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: va/knowledge-search-mcp#4