Code Review desde Market Data

[GBRCenter]

@claude
Por favor lee toda la documentación en la carpeta planning
El backend de Market Data fue implementado con pruebas. Por favor lleva a cabo una revisión de código exhaustiva, corre todas las pruebas, y escribe tus conclusiones en un archivo MARKET_DATA_REVIEW.md en la carpeta planning.

[claude]

Claude finished @GBRCenter's task in 4m 4s —— View job • claude/issue-5-20260902-1240

---

Tareas

• Leer documentación en planning/
• Explorar el código del backend de Market Data
• Ejecutar la suite de pruebas — bloqueado por permisos (ver nota)
• Realizar revisión de código exhaustiva
• Escribir conclusiones en planning/MARKET_DATA_REVIEW.md
• Hacer commit y push, y proveer enlace de PR

Resumen

Leí toda la documentación en planning/ (incluyendo PLAN.md, MARKET_DATA_SUMMARY.md, MARKET_DATA_DESIGN.md y la revisión archivada previa) y revisé exhaustivamente los 8 módulos de backend/app/market/ (730 líneas) y sus 7 archivos de tests (96 tests, 1,161 líneas), línea por línea, contrastando cada test contra el código que ejercita.

⚠️ No pude ejecutar la suite de pruebas. En este entorno, uv sync, uv run pytest e incluso python3 -c ... requieren aprobación manual y no hay ningún humano disponible para aprobarlos en una ejecución disparada por un issue. En su lugar hice una revisión estática exhaustiva, verificando que cada test efectivamente ejercita el comportamiento real del código (no solo que "parece" correcto). Si quieres un resultado real de pass/fail y cobertura, vuelve a lanzar la tarea añadiendo Bash(uv sync:*) y Bash(uv run:*) a --allowedTools.

Hallazgos principales (todos documentados con detalle en planning/MARKET_DATA_REVIEW.md):

• El módulo está muy bien construido: patrón strategy limpio, PriceCache como única fuente de verdad con locking correcto, matemática GBM correcta, y los 3 TODOs que PLAN.md marcaba como pendientes (keepalive SSE, historial rodante, endpoint /history) están implementados y probados.
• De los 7 issues de la revisión archivada (planning/archive/MARKET_DATA_REVIEW.md, 2026-02-10), 5 están corregidos (config de build de hatchling, fragilidad de tests de Massive, tipo de retorno de _generate_events, acceso a atributo privado en get_tickers, imports sin usar).
• Issue nuevo/reabierto (severidad Media): create_stream_router/create_history_router en stream.py reutilizan un APIRouter a nivel de módulo en vez de crear uno nuevo por llamada. Antes era un "riesgo latente"; ahora los propios tests de test_stream.py lo demuestran — llaman a create_history_router() seis veces y acumulan seis rutas duplicadas en el mismo router, funcionando solo porque toman routes[-1]. Esto puede causar rutas duplicadas silenciosas cuando se integre con FastAPI (próximo trabajo según PLAN.md §13) si la app se reconstruye más de una vez por proceso, algo común en fixtures de pytest. Recomiendo corregirlo antes de esa integración.
• Dos issues menores (severidad Baja/Trivial): PriceCache.update() trata un timestamp=0.0 explícito como "no dado" (ts = timestamp or time.time()), y PriceCache.version lee fuera del lock (inofensivo bajo CPython/GIL).
• Una observación de diseño: si MassiveDataSource recibe un AuthError después de que el poller ya inició, la tarea de fondo muere silenciosamente sin que nada lo reporte — vale la pena que el futuro endpoint GET /api/health verifique la salud de esa tarea.

El documento completo incluye una tabla módulo por módulo de cobertura de tests, comparación contra la revisión anterior, y recomendaciones priorizadas.

Crear un PR