---
asset_id: MIN-EL-Jesus-20260629-v01
version: v02
tipo: MIN — Minuta del día
owner: Jesús
sherpa: Jess
ratificador: Jesús
fecha: 2026-06-29
intellibank: IB-EL-EmpowerLabs
subbank: EQ-EL-Equipo
proyecto: tribus-rrhh · mejora de performance
---

# Minuta del día — Jesús · 2026-06-29

## Qué se hizo

- Arrancó el room bajo WORX con `/arrancaroom` — contexto y perfil del operador cargados.

- **Análisis de auditoría de performance** (`Auditoria-home.md`) del proyecto `tribus-rrhh`:
  - Revisión de `app-routing.module.ts` (50 rutas, todas eager, cero NgModules por página).
  - Revisión de `angular.json` (budgets en 135 MB inútiles, faltan `vendorChunk` y `buildOptimizer`).
  - Revisión de `index.html` (sin preconnect a genniux.net, sin skeleton, Stripe síncrono, bug con dos `</head>`).
  - Revisión de `inicio.component.ts` (6 llamadas API en constructor, no en ngOnInit).
  - Revisión de `ngsw-config.json` (un solo grupo, timeout 30s, poco granular).
  - Identificado `congras.gif` (506 KB) activo en `registro-comprador.component.html`.
  - Creado `data/auditoria/Analisis-factibilidad-home.md` con evaluación completa por ítem.

- **Análisis comparativo con rebelocity-club** (`Factibilidad-home.md`):
  - Verificado qué de la Fase 1 y 2 ya existe en rebelocity-club.
  - Confirmado plan 100% replicable: mismo stack, mismo API, mismos problemas.
  - Actualizado `Analisis-factibilidad-home.md` con comparativa y plan de réplica.

- **Fase 1 implementada en tribus-rrhh** (7 cambios aplicados, ver detalle abajo):
- **Iteración servicio de banners:** resuelto bug que impedía la correcta visualización en dispositivos móviles.
- **Junta con Ángeles y Gustavo:** revisión de avances y nuevos ajustes sobre la plataforma de Rebelocity Club.
- **Análisis de notificaciones** (`src/app/custom/top-bar` + `src/app/services/notifications.service.ts`):
  - 12 issues documentados en `data/auditoria/Analisis-notificaciones.md`.
  - 4 issues de severidad alta: `connect()` sin guard (listeners acumulados), crash por `notifData.last` undefined, `BreakpointObserver` sin `takeUntil`, y `location.replace()` que rompe el modelo SPA.
  - Issues medios: `playAudio()` crea Audio nuevo por notificación, `validRoutesF()` lanza 4 HTTP calls en cada invocación, 15 subscriptions HTTP sin `takeUntil`.
  - Issues bajos/mínimos: `window.addEventListener` sin cleanup, `BroadcastChannel` no cerrado, `destroyed$` sin `.complete()`, código muerto del bloque `isBroadcast`, `console.error` en producción.
- **Fase 2 implementada en tribus-rrhh**:
  - `index.html`: preconnect + dns-prefetch a genniux.net, orden fonts corregido, skeleton screen con shimmer (`#342d78`), bug doble `</head>` eliminado, script Stripe restaurado (se había eliminado por error).
  - `angular.json`: `vendorChunk: true`, `buildOptimizer: true`, `optimization: true`, budgets reales (10 MB warning / 15 MB error).
  - `src/app/directives/lazy-image.directive.ts`: creada `LazyImageDirective` + declarada en `app.module.ts` — aplica `loading="lazy" decoding="async"` a todas las `<img>` sin atributo loading.
  - `congras.gif` convertido a WebM con ffmpeg (506 KB → 172 KB, −66%); `registro-comprador.component.html` usa `<video>` con WebM como fuente primaria y GIF como fallback.
  - `ngsw-config.json`: 3 grupos granulares (api-static/performance/1h · api-content/freshness/5m/3s · api-dynamic/freshness/5m/3s). Timeout bajó de 30s a 3s. Ajuste vs rebelocity: `/php/**` explícito en api-dynamic y `/domains/**` en api-static (tribus usa más endpoints PHP que rebelocity).

## Decisiones

- El plan de rebelocity-club es la referencia directa para tribus-rrhh — no se rediseña, se replica.
- **Fix #3b (Stripe defer) y Fix #4 (diferir API calls): en espera** — mismo criterio que rebelocity. Stripe requiere prueba en staging; API calls requieren mapear qué endpoints alimentan contenido above-the-fold.
- Lazy routing (Fase 3): pendiente, no iniciado en ninguno de los dos proyectos.
- Orden de ejecución: Fase 1 ✅ → Fase 2 (en curso) → Fase 3.

## NEXTs

- [x] ~~Ejecutar Fase 1 en tribus-rrhh~~ ✅ completada
- [x] ~~Ejecutar Fase 2 en tribus-rrhh~~ ✅ completada
  - [x] `ngsw-config.json` → 3 grupos granulares ✅
  - [ ] Diferir API calls del constructor de `inicio.component.ts` *(en espera — mapear primero above-the-fold)*
- [ ] **Fixes `ad-banner` component** (`src/app/pages/ads/ad-banner`):
  - [ ] 🔴 Agregar `takeUntil(this.destroy$)` al `subscribe` de `serveAd` en `loadNextBanner()` (línea 105)
  - [ ] 🟠 Agregar `probe.onerror` en `updateBannerMetrics` con fallback a área completa (línea 179)
  - [ ] 🟠 Throttle de `mousemove` con `requestAnimationFrame` en `onBannerMove` para evitar reflow por px (línea 212)
  - [ ] 🟡 Eliminar `transition: background-image` del CSS — no funciona en ningún browser (less:46)
  - [ ] ⚪ Eliminar getter redundante `activeBanners` y usar `banners` directo en template (línea 74)
- [ ] **Fixes `notifications.service.ts`** (`src/app/services/notifications.service.ts`):
  - [ ] 🔴 Agregar guard en `connect()`: si socket ya está conectado, return; si existe previo, `removeAllListeners()` + `disconnect()` (línea 43)
  - [ ] 🔴 Agregar `removeAllListeners()` en `closeConnection()` (línea 184)
  - [ ] 🟠 Convertir listeners anónimos de `checkPrimaryTab` a arrow properties (`onStorage`, `onBeforeUnload`) y removerlos en `destroy()` (líneas 108-119)
  - [ ] 🟡 Agregar `this.channel?.close()` en `destroy()` (línea 95)
- [ ] **Fixes `top-bar.component.ts`** (`src/app/custom/top-bar/top-bar.component.ts`):
  - [ ] 🔴 Inicializar `notifData.last: []` en la declaración de la propiedad — crash en runtime al recibir notificación socket (línea 757)
  - [ ] 🔴 Agregar `takeUntil(this.destroyed$)` al `breakpointObserver.observe()` (línea 104)
  - [ ] 🔴 Reemplazar `location.replace()` por `this.router.navigate()` en `showDetails()` y `showMoreDetails()` (líneas 841, 843, 861-875)
  - [ ] 🟠 Convertir `playAudio()` a instancia reutilizable (`private notifAudio`) con `currentTime = 0` (línea 1900)
  - [ ] 🟠 Agregar `takeUntil(this.destroyed$)` a las 15 subscriptions HTTP sin cleanup (líneas 149, 169, 190, 223, 377, 427, 618, 632, 651, 665, 762, 793, 889, 1598, 1626)
  - [ ] 🟡 Agregar `this.destroyed$.complete()` en `ngOnDestroy` (línea 129)
  - [ ] ⚪ Eliminar `console.error` de debug del tutorial en producción (líneas 1357, 1358, 1368, 1369, 1376, 1382)
- [ ] **Catálogo de playbooks para Katia:**
  - Crear lógica para obtener el catálogo de playbooks desde el API
  - Actualizar el MD que usa Katia por canal con los datos de cada playbook
  - Campos a guardar: título, autor, url, precio
- [ ] Fase 3 (evaluación futura):
  - [ ] Lazy loading de rutas (crear NgModule por página — 3-5 días)
  - [ ] Extraer Quill.js del bundle global
  - [ ] Fix #3b: Stripe con `defer` *(en espera — probar en staging)*
  - [ ] Angular Universal / SSR *(decisión de infraestructura)*