88 lines
4.6 KiB
Markdown
88 lines
4.6 KiB
Markdown
# OBSERVATIONS — Sprint 5
|
||
|
||
> Hallazgos descubiertos durante el sprint actual.
|
||
> Este archivo se vacía al cerrar el sprint: los hallazgos se promocionan a TECH_DEBT.md o BACKLOG.md, o quedan resueltos.
|
||
>
|
||
> **Sprint 4 cerrado el 2026-06-18.** Observaciones archivadas — ítems migrados a BACKLOG.md y TECH_DEBT.md.
|
||
|
||
---
|
||
|
||
## Convención de etiquetas
|
||
|
||
- `[CURRENT-SPRINT]` — Se resuelve en este sprint
|
||
- `[DEFERRED-TECH-DEBT]` — Se va a TECH_DEBT.md al cerrar el sprint
|
||
- `[DEFERRED-BACKLOG]` — Se va a BACKLOG.md al cerrar el sprint
|
||
- `[RESOLVED]` — Resuelto durante el sprint, archivar al cierre
|
||
|
||
---
|
||
|
||
## Hallazgos activos
|
||
|
||
### [RESOLVED] Etapa 1 — Workspace loading fix + carry-over skeleton
|
||
|
||
**Estado:** OK formal del director — 2026-06-19.
|
||
**Fix principal:** `process-store.ts` y `useReportData.ts` usaban `getSession()` sin timeout; reemplazados por `getSessionSafe()`. Error Boundary + estado `error` inline con retry button agregados.
|
||
**Carry-over a Etapa 2:** `WorkspaceSkeleton.tsx` fue creado pero el spinner del router-level guard (TanStack Router) intercepta antes de que el componente monte. El skeleton no aparece en la práctica. Corregir en Etapa 2 como Fix 0 (bajo esfuerzo).
|
||
**SupabaseAuthService.ts:** su `getSession()` directo documentado como excepción intencional (necesita el objeto sesión real, no es barrera descartable). Sin callers activos. Migrar a `getSessionSafe()` cuando tenga callers — pendiente registrar en TECH_DEBT.
|
||
|
||
---
|
||
|
||
## Hallazgos resueltos durante el sprint
|
||
|
||
### [RESOLVED] Etapas 3 + 3B — Heatmap gradiente + toggle + persistencia
|
||
|
||
**Estado:** OK formal del director — 2026-06-19.
|
||
**Entregado:** gradiente 4 stops (#10b981→#f59e0b→#f97316→#ef4444), modos percentile/minmax, `useHeatmapMode` hook con localStorage, HeatmapModeToggle, leyenda PDF 4 stops, zoom controls movidos a bottom-left.
|
||
**Lección operativa:** Dokploy con Autodeploy desactivado requiere `git push origin main` explícito antes de validar en producción. Los commits `a65bec4` y `e8a4a1b` estaban locales sin pushear — Dokploy desplegaba Etapa 2 mientras se validaba Etapa 3. La validación fallida fue contra código desactualizado, no contra bugs reales.
|
||
|
||
### [RESOLVED] Etapa 2 — Zoom BPMN fix + controles + skeleton
|
||
|
||
**Estado:** OK formal del director — 2026-06-19.
|
||
**Fixes:** fit-viewport `'auto'` agregado (una línea), zoom-in/zoom-out en esquina inferior derecha (Lucide icons, 1.2x factor, límites 0.1x–4x), skeleton wired correctamente en WorkspacePage (guard `isLoading` simple sin `!currentProcess`).
|
||
**Carry-over a Etapa 3:** controles de zoom solapan el logo bpmn.io (esquina inferior derecha). Fix: mover a esquina inferior izquierda (`left: 12` en lugar de `right: 12`). Una línea CSS.
|
||
|
||
---
|
||
|
||
### [RESOLVED] Etapa 3 — Heatmap gradiente 4 stops + persistencia de modo + zoom position
|
||
|
||
**Estado:** pendiente OK formal del director.
|
||
**Fixes Etapa 3:**
|
||
- `colors.ts`: gradiente continuo verde→amarillo→naranja→rojo (interpolación genérica N-stops), modos `'percentile'`/`'minmax'` reemplazan `'relative'`/`'absolute'`
|
||
- `useHeatmapMode.ts` (hook nuevo): persiste preferencia en `localStorage` con lazy initializer en `useState`
|
||
- `ReportPage.tsx`: reemplaza `useState<HeatmapMode>('relative')` por `useHeatmapMode()`
|
||
- `BpmnCanvas.tsx`: zoom-in/zoom-out movidos de `right:12` a `left:12` (evita solapar logo bpmn.io)
|
||
- PDF: leyenda de 4 stops con interpolación. Tests: 528 Vitest verdes. E2E recalibrados con distancia euclidiana.
|
||
|
||
**Fixes Etapa 3B verificados (incluidos en Etapa 3):**
|
||
- Fix 1 (repintado de canvas al cambiar toggle): `mode` ya estaba como dependencia del `useEffect` en `HeatmapCanvas.tsx` desde el MVP. El bug fue observado durante implementación intermedia; resuelto con los renombres de Etapa 3.
|
||
- Fix 2 (persistencia localStorage): `useHeatmapMode()` con lazy initializer correcto; `ReportPage` usa el hook.
|
||
- Fix 3 (zoom position): `left: 12` en `BpmnCanvas.tsx`.
|
||
|
||
---
|
||
|
||
## Hallazgos diferidos (promocionar al cerrar sprint)
|
||
|
||
*(vacío)*
|
||
|
||
---
|
||
|
||
## Notas operativas
|
||
|
||
- Los hallazgos resueltos quedan acá hasta el cierre del sprint. Al cerrar, se archiva o purga.
|
||
- Los hallazgos sin resolver se promocionan a `TECH_DEBT.md` o `BACKLOG.md` según corresponda.
|
||
- Este archivo NO contiene roadmap futuro ni deuda histórica — solo hallazgos del sprint en curso.
|
||
|
||
---
|
||
|
||
## Cierre del sprint (checklist)
|
||
|
||
Cuando el sprint cierre, ejecutar:
|
||
|
||
```
|
||
[ ] Verificar que todos los items [CURRENT-SPRINT] están RESOLVED
|
||
[ ] Mover items [DEFERRED-TECH-DEBT] a TECH_DEBT.md (si no están ya)
|
||
[ ] Mover items [DEFERRED-BACKLOG] a BACKLOG.md (si no están ya)
|
||
[ ] Archivar o purgar este archivo
|
||
[ ] Crear OBSERVATIONS.md vacío para el sprint siguiente
|
||
```
|