**Problema 1: Símbolos no definidos de SDL_ttf** (CRÍTICO)
- Error: "Undefined symbols: _TTF_Init, _TTF_OpenFont, etc."
- Causa: LDFLAGS solo incluía -lSDL3 (faltaba -lSDL3_ttf)
- Solución: Añadido -lSDL3_ttf a LDFLAGS para Unix/macOS (línea 81)
- Afecta: Linux, macOS y otros sistemas Unix
**Problema 2: Mismatch class/struct SDL_Renderer** (WARNING)
- Warning: "class 'SDL_Renderer' was previously declared as a struct"
- Causa: ui_manager.h:7 declaraba "class SDL_Renderer"
- SDL3 lo declara como "struct SDL_Renderer" (SDL_render.h:119)
- Solución: Cambiado class → struct en ui_manager.h:7
- Evita warnings y potenciales errores de linker en MSVC
**Resultado:**
✅ make macos_release completa exitosamente
✅ DMG creado: vibe3_physics-2025-10-19-macos-apple-silicon.dmg (17.9 MB)
✅ Sin errores de enlazado, solo warnings menores de versión macOS
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**Cambios organizativos:**
- Archivos .md movidos de raíz a .claude/ (BOIDS_ROADMAP, CLAUDE, REFACTOR_*, ROADMAP, RULES)
- .claude/ ya está en .gitignore, archivos de sesión no versionados
**Nuevos recursos para release:**
- Añadido release/frameworks/SDL3_ttf.xcframework/ para macOS
- Añadidos release/SDL3.dll y release/SDL3_ttf.dll para Windows (forzado con -f)
**Configuración:**
- defines.h: APPLOGO_DISPLAY_INTERVAL 120→90 segundos (logo aparece más frecuente)
- defines.h: Ajustes de formato/indentación (sin cambios funcionales)
**Makefile windows_release:**
- Comandos Unix reemplazados por Windows CMD nativos:
- rm -rf → if exist + rmdir /S /Q
- mkdir -p → mkdir
- cp -f → copy /Y
- rm -f → if exist + del /Q
- Ahora funciona en Windows CMD sin necesitar Git Bash/MSYS2
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**Problemas corregidos:**
1. APP_SOURCES ahora usa wildcards automáticos (como CMakeLists.txt)
- Detecta automáticamente todos los .cpp en subdirectorios
- Faltaban 16 archivos: themes, state, input, scene, shapes_mgr, boids_mgr, text, ui, app_logo, logo_scaler
- Elimina mantenimiento manual de lista de archivos
2. macOS release ahora copia SDL3_ttf.xcframework correctamente
- Añadido a Contents/Frameworks en el .app
- Añadido a carpeta Frameworks temporal
3. Windows DLLs: línea mantenida para futuro uso
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**Problema:**
Cuando se iniciaba con `-m demo`, `-m demo-lite` o `-m logo`, se llamaba
a `setState()` directamente, que es un método de bajo nivel que solo
cambia el estado interno SIN ejecutar las acciones de configuración.
**Resultado del bug:**
- `-m demo`: NO aleatorizaba (tema default, primer escenario)
- `-m demo-lite`: NO aleatorizaba física
- `-m logo`: NO configuraba tema, PNG_SHAPE, ni pelotas pequeñas
**Arquitectura correcta:**
- `setState()` = Método primitivo bajo nivel (solo cambia estado)
- `toggleDemoMode()` = Método alto nivel (setState + randomize)
- `toggleDemoLiteMode()` = Método alto nivel (setState + randomize)
- `enterLogoMode()` = Método alto nivel (setState + configuración completa)
**Solución implementada:**
En lugar de llamar a setState() directamente, usar los métodos de
alto nivel que ejecutan las acciones de configuración:
```cpp
if (initial_mode == AppMode::DEMO) {
state_manager_->toggleDemoMode(...); // Entra a DEMO + randomiza
}
else if (initial_mode == AppMode::DEMO_LITE) {
state_manager_->toggleDemoLiteMode(...); // Entra a DEMO_LITE + randomiza
}
else if (initial_mode == AppMode::LOGO) {
state_manager_->enterLogoMode(...); // Entra a LOGO + configura todo
}
```
**Archivos modificados:**
- source/engine.cpp (líneas 249-263):
- Reemplazado setState() por toggleDemoMode/toggleDemoLiteMode/enterLogoMode
- Agregados comentarios explicativos
**Resultado esperado:**
- ✅ `-m demo` → Aleatoriza todo como si pulsaras D
- ✅ `-m demo-lite` → Aleatoriza física como si pulsaras Shift+D
- ✅ `-m logo` → Configura tema/PNG_SHAPE/pelotas como si pulsaras K
- ✅ Comportamiento consistente entre CLI y teclas
- ✅ Arquitectura correcta: alto nivel para acciones, bajo nivel para estado
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**1. Eliminado logging de debug del FADE_OUT:**
- Removido log de timer/delta_time/progress (FADE_OUT inicial)
- Removido log de alpha1/alpha2
- Removido log de animaciones (ZOOM, ELASTIC, SPIRAL, BOUNCE)
- Removido log de completado de FADE_OUT
- Consola limpia en modo producción
**2. Fix centro del logo en animación ZOOM_ONLY:**
**Problema:**
- Centro del logo se calculaba basándose en width/height escalados
- Cuando scale cambiaba (1.2 → 1.0), corner_x/corner_y se movían
- Resultado: logo se desplazaba lateralmente durante zoom
**Solución:**
- Calcular esquina BASE (sin escala): corner_x_base, corner_y_base
- Calcular centro FIJO basándose en base_width/base_height
- Calcular width/height escalados DESPUÉS (solo para vértices)
- Resultado: centro permanece fijo, zoom crece/decrece alrededor del centro
**Archivos modificados:**
- source/app_logo.cpp:
- Líneas 343-347: Eliminado log FADE_OUT inicial
- Línea 347: Eliminado log completado
- Líneas 365-366: Eliminado log alphas
- Líneas 381-383: Eliminado log ZOOM
- Líneas 396-398: Eliminado log ELASTIC
- Líneas 414-417: Eliminado log SPIRAL
- Líneas 444-446: Eliminado log BOUNCE
- Líneas 609-625: Reordenado cálculo de centro (FIJO) y tamaño (ESCALADO)
**Resultado esperado:**
- Sin spam en consola
- Animación ZOOM perfectamente centrada en esquina inferior derecha
- Logo crece/decrece sin desplazamiento lateral
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**Problema:**
- SDL_SetTextureAlphaMod() no funciona correctamente con SDL_RenderGeometry()
- El alpha de los vértices (hardcodeado a 1.0) overrideaba el alpha de textura
- Resultado: fade invisible o instantáneo a pesar de valores correctos
**Solución:**
- Eliminar SDL_SetTextureAlphaMod()
- Convertir alpha de 0-255 a 0.0-1.0 (alpha_normalized)
- Aplicar alpha_normalized directamente al canal alpha de los 4 vértices
- Ahora SDL_RenderGeometry respeta el fade correctamente
**Archivos modificados:**
- source/app_logo.cpp:
- Línea 630: Crear alpha_normalized en lugar de SetTextureAlphaMod
- Líneas 669, 680, 691, 702: Aplicar alpha_normalized a vértices
**Resultado esperado:**
- Fade visible y suave durante 2 segundos completos
- Logo 2 con retraso de 0.25s como esperado
- Sincronización perfecta entre animación y fade
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Log de timer_, delta_time, progress en cada frame
- Log de alpha1/alpha2 calculados
- Log de valores de animación por tipo (ZOOM/ELASTIC/SPIRAL/BOUNCE)
- Log de ease_t1 en ROTATE_SPIRAL para diagnosticar desincronización
- Log cuando FADE_OUT se completa
Propósito: Diagnosticar por qué el fade parece instantáneo
y desincronizado con la animación (serie en lugar de paralelo).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Restaura fade lineal para el alpha de los logos, eliminando el easing
que hacía la transición casi imperceptible. Renombra constante para
mayor claridad sobre lo que controla.
Problema identificado:
- El alpha usaba easeInOutQuad, acelerando la transición en el medio
- Con 2 segundos, el easing hacía que el logo pareciera aparecer/
desaparecer instantáneamente sin fade visible
- El usuario reportó "el logo termina y desaparece directamente"
Solución implementada:
1. **Fade lineal restaurado**:
- FADE_IN: alpha aumenta linealmente 0→255
- FADE_OUT: alpha disminuye linealmente 255→0
- Progreso visible y constante durante toda la duración
2. **Constante renombrada**:
- `APPLOGO_FADE_DURATION` → `APPLOGO_ANIMATION_DURATION`
- Nombre más claro: controla duración de toda la animación
- Actualizado valor a 2.0 segundos (configurable por usuario)
3. **Animaciones mantienen easing**:
- Zoom, rotación, squash, etc. siguen usando sus easings
- Solo el alpha es lineal para fade visible
Confirmaciones:
✅ Sistema time-based: usa delta_time correctamente
✅ Blend mode configurado: SDL_BLENDMODE_BLEND en todas las texturas
✅ Alpha se aplica: SDL_SetTextureAlphaMod en renderizado
Resultado con APPLOGO_ANIMATION_DURATION = 2.0s:
- t=0.0s → Alpha=0 (invisible)
- t=0.5s → Alpha=64 (25% visible)
- t=1.0s → Alpha=127 (50% visible)
- t=1.5s → Alpha=191 (75% visible)
- t=2.0s → Alpha=255 (100% visible)
Nota: El logo solo se muestra en modos DEMO/DEMO_LITE/LOGO,
no en SANDBOX. Para probar: ./vibe3_physics --mode demo
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Permite arrancar directamente en modo DEMO, DEMO_LITE, LOGO o SANDBOX
mediante argumentos de línea de comandos, eliminando necesidad de
cambiar manualmente el modo después del arranque.
Nuevos argumentos:
- `-m, --mode <mode>` - Establece modo inicial
- `sandbox` - Control manual (default)
- `demo` - Auto-play completo (figuras + temas + colisiones)
- `demo-lite` - Auto-play simple (solo física/figuras)
- `logo` - Modo logo (easter egg con convergencia)
Ejemplos de uso:
```bash
# Arrancar en modo DEMO
./vibe3_physics --mode demo
./vibe3_physics -m demo
# Arrancar en DEMO_LITE (solo física)
./vibe3_physics -m demo-lite
# Arrancar directo en LOGO
./vibe3_physics --mode logo
# Combinar con otros argumentos
./vibe3_physics -w 1920 -h 1080 --mode demo
./vibe3_physics -F -m demo-lite # Fullscreen + DEMO_LITE
```
Implementación:
1. main.cpp: Parsing de argumento --mode con validación
2. engine.h: Nuevo parámetro `initial_mode` en initialize()
3. engine.cpp: Aplicación del modo vía StateManager::setState()
Si no se especifica --mode, se usa SANDBOX (comportamiento actual).
El modo se aplica después de inicializar StateManager, garantizando
que todos los componentes estén listos antes del cambio de estado.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementa sistema de sincronización elegante entre logos con efecto de
"seguimiento" y fade suavizado para eliminar desincronización visual.
Cambios principales:
1. **Animación sincronizada**: Ambos logos usan la MISMA animación
- Eliminadas 4 variables independientes (logo1/logo2 × entry/exit)
- Una sola variable `current_animation_` compartida
- Misma animación para entrada y salida (simetría)
2. **Retraso de Logo 2**: 0.25 segundos detrás de Logo 1
- Logo 1 empieza en t=0.00s
- Logo 2 empieza en t=0.25s
- Efecto visual de "eco" o "seguimiento"
3. **Alpha independiente con retraso**:
- `logo1_alpha_` y `logo2_alpha_` separados
- Logo 2 aparece/desaparece más tarde visualmente
4. **Easing en alpha** (NUEVO):
- Aplicado `easeInOutQuad()` al fade de alpha
- Elimina problema de "logo deformado esperando a desvanecerse"
- Sincronización visual perfecta entre animación y fade
- Curva suave: lento al inicio, rápido en medio, lento al final
Comportamiento resultante:
FADE_IN:
- t=0.00s: Logo 1 empieza (alpha con easing)
- t=0.25s: Logo 2 empieza (alpha con easing + retraso)
- t=0.50s: Logo 1 completamente visible
- t=0.75s: Logo 2 completamente visible
FADE_OUT:
- t=0.00s: Logo 1 empieza a desaparecer (misma animación)
- t=0.25s: Logo 2 empieza a desaparecer
- t=0.50s: Logo 1 completamente invisible
- t=0.75s: Logo 2 completamente invisible
Archivos modificados:
- source/defines.h: +APPLOGO_LOGO2_DELAY
- source/app_logo.h: Reestructuración de variables de animación/alpha
- source/app_logo.cpp: Implementación de retraso + easing
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementa pre-escalado de alta calidad para eliminar artefactos de
escalado dinámico de SDL y mejorar la nitidez visual de los logos.
Características:
- 4 texturas pre-escaladas (2 logos × 2 resoluciones: base + nativa)
- Detección automática de resolución nativa del monitor
- Switching dinámico entre texturas al cambiar resolución (F4)
- Renderizado 1:1 sin escalado adicional (máxima calidad)
- Algoritmo Mitchell en espacio sRGB (balance calidad/velocidad)
- Todo en RAM, sin archivos temporales
Archivos nuevos:
- source/external/stb_image_resize2.h: Biblioteca de escalado stb
- source/logo_scaler.h/cpp: Clase helper para pre-escalado
Cambios en AppLogo:
- Reemplazadas shared_ptr<Texture> por SDL_Texture* raw pointers
- initialize(): Pre-escala logos a 2 resoluciones al inicio
- updateScreenSize(): Cambia entre texturas según resolución
- render(): Simplificado, siempre usa renderWithGeometry()
- ~AppLogo(): Libera 4 texturas SDL manualmente
El sistema detecta la resolución nativa al inicio y crea versiones
optimizadas. Al presionar F4, cambia automáticamente a la textura
nativa para calidad perfecta en fullscreen.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementación de sistema de 2 logos superpuestos con animaciones completamente independientes:
**Nuevas características:**
- Dos logos superpuestos (logo1.png + logo2.png) con animaciones independientes
- 4 tipos de animación: ZOOM_ONLY, ELASTIC_STICK, ROTATE_SPIRAL, BOUNCE_SQUASH
- Aleatorización independiente para entrada y salida de cada logo
- 256 combinaciones posibles (4×4 entrada × 4×4 salida)
**Ajuste de tamaño y posición:**
- Nueva constante APPLOGO_HEIGHT_PERCENT (40%) - altura del logo respecto a pantalla
- Nueva constante APPLOGO_PADDING_PERCENT (10%) - padding desde esquina inferior-derecha
- Logo anclado a esquina en lugar de centrado en cuadrante
- Valores fácilmente ajustables mediante constantes en defines.h
**Cambios técnicos:**
- Variables duplicadas para logo1 y logo2 (scale, squash, stretch, rotation)
- Variables compartidas para sincronización (state, timer, alpha)
- renderWithGeometry() acepta parámetro logo_index (1 o 2)
- Logo1 renderizado primero (fondo), Logo2 encima (overlay)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementación de deformación elástica con vértices para el logo:
FADE IN (0.5s):
- Scale: 120% → 100% con easing elástico (bounce)
- Squash Y: 0.6 → 1.0 con easing back (aplastamiento)
- Stretch X: compensación automática
- Efecto: logo se "pega" aplastándose y rebotando
FADE OUT (0.5s):
- Scale: 100% → 120% (aceleración cuadrática)
- Squash Y: 1.0 → 1.3 (estiramiento vertical)
- Stretch X: 1.0 → 0.8 (compresión horizontal)
- Rotación: 0° → ~5.7° (torsión sutil)
- Efecto: logo se "despega" estirándose y girando
Características técnicas:
- Enum AppLogoAnimationType (ZOOM_ONLY / ELASTIC_STICK)
- Renderizado con SDL_RenderGeometry para deformaciones
- Funciones de easing: easeOutElastic() y easeOutBack()
- Transformación de vértices con rotación y escala 2D
- Actualmente fijo en ELASTIC_STICK para testing
Limpieza adicional:
- Eliminado dbgtxt.h (no utilizado)
- Removidos SDL_Log de debug en HelpOverlay
- Comentada variable no usada en ShapeManager
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Nuevo sistema AppLogo que muestra el logo cada 20 segundos por 5 segundos
- Fade in/out suave de 0.5 segundos con alpha blending
- Máquina de estados: HIDDEN → FADE_IN → VISIBLE → FADE_OUT
- Logo posicionado en cuadrante inferior derecho (1/4 de pantalla)
- Añadido método setAlpha() a Texture para control de transparencia
- Habilitado SDL_BLENDMODE_BLEND en todas las texturas
- Filtrado LINEAR para suavizado del logo escalado
- Desactivado automáticamente en modo SANDBOX
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- Al cambiar de escenario (teclas 1-8) en modo BOIDS, la gravedad
se reseteaba a 720 en lugar de mantenerse en 0
- SceneManager::changeScenario() reinicializa bolas con gravedad default
- Esto rompía el invariante: "modo BOIDS = gravedad OFF siempre"
Solución:
- Añadido check en Engine::changeScenario() para forzar gravedad OFF
después del cambio de escenario si estamos en modo BOIDS
- Mantiene consistencia con el comportamiento de SHAPE mode
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**1. Bordes como obstáculos (no más wrapping):**
- Implementada fuerza de repulsión cuando boids se acercan a bordes
- Nueva regla: Boundary Avoidance (evitar bordes)
- Fuerza proporcional a cercanía (0% en margen, 100% en colisión)
- Constantes: BOID_BOUNDARY_MARGIN (50px), BOID_BOUNDARY_WEIGHT (7200 px/s²)
**2. Variables ajustables en runtime:**
- Añadidas 11 variables miembro en BoidManager (inicializadas con defines.h)
- Permite modificar comportamiento sin recompilar
- Variables: radios (separation/alignment/cohesion), weights, speeds, boundary
- Base para futuras herramientas de debug/tweaking visual
**3. Fix tecla G (BOIDS → PHYSICS):**
- Corregido: toggleBoidsMode() ahora acepta parámetro force_gravity_on
- handleGravityToggle() pasa explícitamente false para preservar inercia
- Transición BOIDS→PHYSICS ahora mantiene gravedad OFF correctamente
**Implementación:**
- defines.h: +2 constantes (BOUNDARY_MARGIN, BOUNDARY_WEIGHT)
- boid_manager.h: +11 variables miembro ajustables
- boid_manager.cpp:
- Constructor inicializa variables
- Todas las funciones usan variables en lugar de constantes
- applyBoundaries() completamente reescrito (repulsión vs wrapping)
- engine.h/cpp: toggleBoidsMode() con parámetro opcional
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Al entrar o salir del modo fullscreen real (F4), el área de juego cambia
de tamaño. Si estábamos en modo SHAPE, las bolas aparecían centradas pero
sin formar la figura.
PROBLEMA RAÍZ:
- changeScenario() recrea las bolas en nuevas posiciones
- NO se regeneraban los targets de la figura (puntos 3D)
- NO se reactivaba la atracción física hacia los targets
- Resultado: bolas centradas sin formar figura
SOLUCIÓN:
Después de changeScenario() en ambas transiciones de F4:
1. Llamar a generateShape() (Engine, no ShapeManager)
2. Reactivar enableShapeAttraction(true) en todas las bolas
Esto sigue el mismo patrón usado en Engine::changeScenario() (línea 515).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- En modo F3 (letterbox/integer scale), el debug HUD se pintaba
fuera del área de juego (en las barras negras laterales)
- SDL_GetRenderViewport() devuelve coordenadas LÓGICAS cuando hay
presentación lógica activa
- printAbsolute() trabaja en píxeles FÍSICOS
- Mismatch de coordenadas causaba alineación derecha incorrecta
Solución:
- Nuevo helper getPhysicalViewport() que:
1. Guarda estado de presentación lógica
2. Deshabilita presentación lógica temporalmente
3. Obtiene viewport en coordenadas físicas
4. Restaura presentación lógica
5. Retorna viewport físico
- UIManager::renderDebugHUD() ahora usa physical_viewport.w
para cálculo de alineación derecha (9 referencias actualizadas)
Resultado:
- Debug HUD alineado correctamente en F3 letterbox
- Debug HUD alineado correctamente en F4 integer scale
- Modo ventana sigue funcionando correctamente
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- En modo F3 (letterbox/integer scale), las notificaciones se pintaban
fuera del área de juego (en las barras negras)
- SDL_GetRenderViewport() devuelve coordenadas LÓGICAS cuando hay
presentación lógica activa
- printAbsolute() trabaja en píxeles FÍSICOS
- Mismatch de coordenadas causaba centrado incorrecto
Solución:
- Nuevo helper getPhysicalViewport() que:
1. Guarda estado de presentación lógica
2. Deshabilita presentación lógica temporalmente
3. Obtiene viewport en coordenadas físicas
4. Restaura presentación lógica
5. Retorna viewport físico
- Notifier::render() ahora usa physical_viewport.w para centrado
Resultado:
- Notificaciones centradas correctamente en F3 letterbox
- Notificaciones centradas correctamente en F4 integer scale
- Modo ventana sigue funcionando correctamente
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- Columna derecha del HUD (FPS, info de pelota) se alineaba usando dimensión física
- En modo letterbox (F3 INTEGER/LETTERBOX) aparecía en barras negras o fuera de pantalla
- Mismo issue que tenían Notifier y Help Overlay
Causa:
- ui_manager.cpp:renderDebugHUD() usaba `physical_window_width_` para alinear a la derecha
- En F3 letterbox: viewport visible < ventana física
- Ejemplo: ventana 1920px, viewport 1280px con offset 320px
- Cálculo: fps_x = 1920 - width - margin
- printAbsolute() aplicaba offset: 1920 - width + 320 = fuera de pantalla
- Resultado: texto del HUD invisible o en barras negras
Solución:
- Obtener viewport con SDL_GetRenderViewport() al inicio de renderDebugHUD()
- Reemplazar TODAS las referencias a `physical_window_width_` con `viewport.w`
- Coordenadas relativas al viewport, printAbsolute() aplica offset automáticamente
Código modificado:
- ui_manager.cpp:208-211 - Obtención de viewport
- ui_manager.cpp:315, 326, 333, 340, 347, 353, 360, 366, 375 - Alineación con viewport.w
Líneas afectadas (9 totales):
- FPS counter
- Posición X/Y primera pelota
- Velocidad X/Y
- Fuerza de gravedad
- Estado superficie
- Coeficiente de rebote (loss)
- Dirección de gravedad
- Convergencia (LOGO mode)
Resultado:
✅ HUD de debug alineado correctamente al borde derecho del viewport
✅ Columna derecha visible dentro del área de juego
✅ No aparece en barras negras en F3
✅ Funciona correctamente en ventana, F3 y F4
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- Notificaciones se centraban usando dimensión física de ventana
- En modo letterbox (F3 INTEGER/LETTERBOX) aparecían en barras negras
- Mismo issue que tenía Help Overlay
Causa:
- notifier.cpp:165 usaba `window_width_` para calcular centrado
- En F3 letterbox: viewport visible < ventana física
- Ejemplo: ventana 1920px, viewport 1280px con offset 320px
- Resultado: notificación descentrada fuera del área visible
Solución:
- Obtener viewport con SDL_GetRenderViewport() antes de calcular posición
- Usar `viewport.w` en lugar de `window_width_` para centrado
- Coordenadas relativas al viewport, printAbsolute() aplica offset automáticamente
Código modificado:
- notifier.cpp:162-170 - Centrado usando viewport dimensions
Resultado:
✅ Notificaciones centradas en área visible (viewport)
✅ No aparecen en barras negras en F3
✅ Funciona correctamente en ventana, F3 y F4
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
Problema:
- Solo columna 0 verificaba si cabía más texto antes de escribir
- Columna 1 (derecha) escribía fuera del overlay si no cabía
- En ventanas de 600px altura, columna 1 se desbordaba
Solución:
- Eliminada restricción `&& current_column == 0` del check de padding
- Ahora AMBAS columnas verifican si caben antes de escribir
- Si columna 1 está llena: omitir texto restante (continue)
- Si columna 0 está llena: cambiar a columna 1
Comportamiento preferido por el usuario:
"prefiero que 'falte texto' a que el texto se escriba por fuera del overlay"
Resultado:
✅ Columna 0 cambia a columna 1 cuando se llena
✅ Columna 1 omite texto que no cabe
✅ Overlay nunca muestra texto fuera de sus límites
✅ Funciona correctamente en ventanas pequeñas (600px)
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
Problemas resueltos:
- En modo F3 (letterbox), el overlay se centraba en pantalla física
en lugar de en el viewport visible, quedando desplazado
- Al salir de F3 a ventana, el overlay seguía roto
- Padding inferior no se respetaba correctamente
Cambios implementados:
1. render() ahora usa SDL_GetRenderViewport() para obtener área visible
2. Centrado calculado dentro del viewport (con offset de barras negras)
3. toggleFullscreen() restaura tamaño de ventana al salir de F3
4. Padding check movido ANTES de escribir línea (>= en lugar de >)
5. Debug logging añadido para diagnóstico de dimensiones
Resultado:
✅ Overlay centrado correctamente en F3 letterbox
✅ Overlay se regenera correctamente al salir de F3
✅ Padding inferior respetado en columna 0
Pendiente:
- Columna 2 (índice 1) todavía no respeta padding inferior
- Verificar que F4 (real fullscreen) siga funcionando correctamente
🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
Correcciones críticas para overlay en fullscreen y padding:
1. Fullscreen/resize roto CORREGIDO:
- Problema: orden incorrecto de actualizaciones causaba mezcla de
dimensiones antiguas (800x600) con font nuevo (24px)
- Solución: nuevo método updateAll() que actualiza font Y dimensiones
de forma atómica
- Flujo correcto: dimensiones físicas → font → recalcular box
- Antes: overlay gigante y descuadrado al cambiar fullscreen
- Ahora: overlay se reposiciona y escala correctamente
2. Padding inferior inexistente CORREGIDO:
- Problema: calculateTextDimensions() usaba num_lines/2 asumiendo
división perfecta entre columnas
- Problema 2: rebuildCachedTexture() no verificaba límite inferior
en columna 1
- Solución: contar líneas REALES en cada columna y usar el máximo
- Fórmula correcta: line_height*2 + max_column_lines*line_height + padding*2
- Ahora: padding inferior respetado siempre
3. Implementación técnica:
- HelpOverlay::updateAll(font, width, height) nuevo método unificado
- UIManager llama updateAll() en lugar de reinitializeFontSize() +
updatePhysicalWindowSize() separadamente
- Elimina race condition entre actualización de font y dimensiones
Resultado:
- F3/F4 (fullscreen) funciona correctamente
- Resize ventana (F1/F2) funciona correctamente
- Padding inferior respetado en ambas columnas
- Sin overlays gigantes o descuadrados
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Tres correcciones importantes para el Help Overlay:
1. Solapamiento de columnas corregido:
- Añadidos column1_width_ y column2_width_ para anchos reales
- calculateTextDimensions() ahora incluye encabezados en cálculo
- rebuildCachedTexture() usa anchos reales de columnas
- Columna 2 empieza en padding + column1_width_ + padding
- Elimina cálculo erróneo column_width = (box_width_ - padding*3)/2
2. Layout en alta resolución corregido:
- Eliminado ancho mínimo forzado del 90% de dimensión menor
- box_width_ ahora usa directamente text_width (justo lo necesario)
- Antes: 1920x1080 → min 972px aunque contenido necesite 600px
- Ahora: box ajustado al contenido sin espacio vacío extra
3. Fullscreen/resize corregido:
- reinitializeFontSize() ya NO llama a calculateBoxDimensions()
- Evita recalcular con physical_width_ y physical_height_ antiguos
- Confía en updatePhysicalWindowSize() que se llama después
- Antes: textura cacheada creada con dimensiones incorrectas
- Ahora: textura siempre creada con dimensiones correctas
Resultado:
- Columnas no se montan entre sí
- Box ajustado al contenido sin espacio vacío derecha
- Cambios fullscreen/ventana funcionan correctamente
- Overlay se recalcula apropiadamente en todos los casos
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Sistema de escalado dinámico de texto con 3 tamaños según área de ventana:
1. TextRenderer improvements:
- Añadido reinitialize(int new_font_size) para cambiar tamaño en runtime
- Almacena font_path_ para permitir recarga de fuente
- Cierra fuente anterior y abre nueva con diferente tamaño
- Verifica si tamaño es igual antes de reinicializar (optimización)
2. UIManager - Font size calculation:
- Añadido calculateFontSize() con stepped scaling por área:
* SMALL (< 800x600): 14px
* MEDIUM (800x600 a 1920x1080): 18px
* LARGE (> 1920x1080): 24px
- Tracking de current_font_size_ para detectar cambios
- Inicialización con tamaño dinámico en initialize()
- Reinitialización automática en updatePhysicalWindowSize()
3. UIManager - Propagation:
- Reinitializa 3 TextRenderer instances cuando cambia tamaño
- Propaga nuevo tamaño a HelpOverlay
- Detecta cambios solo cuando área cruza umbrales (eficiencia)
4. HelpOverlay integration:
- Acepta font_size como parámetro en initialize()
- Añadido reinitializeFontSize() para cambios dinámicos
- Recalcula dimensiones del box cuando cambia fuente
- Marca textura para rebuild completo tras cambio
Resultado:
- Ventanas pequeñas: texto 14px (más espacio para contenido)
- Ventanas medianas: texto 18px (tamaño original, óptimo)
- Ventanas grandes: texto 24px (mejor legibilidad)
- Cambios automáticos al redimensionar ventana (F1/F2/F3/F4)
- Sin impacto en performance (solo recalcula al cruzar umbrales)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Mejoras de rendimiento y usabilidad del Help Overlay:
1. Anchura dinámica basada en contenido:
- Ya no es siempre cuadrado (box_size_)
- Calcula ancho real según texto más largo por columna
- Mantiene mínimo del 90% dimensión menor como antes
- Nueva función calculateTextDimensions()
2. Render-to-texture caching para optimización:
- Renderiza overlay completo a textura una sola vez
- Detecta cambios de color con umbral (threshold 5/255)
- Soporta temas dinámicos con LERP sin rebuild constante
- Regenera solo cuando colores cambian o ventana redimensiona
3. Impacto en performance:
- Antes: 1200 FPS → 200 FPS con overlay activo
- Después: 1200 FPS → 1000-1200 FPS (casi sin impacto)
- Temas estáticos: 1 render total (~∞x más rápido)
- Temas dinámicos: regenera cada ~20-30 frames (~25x más rápido)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
El modo LOGO ahora incluye cambios automáticos de dirección de
gravedad como parte de sus variaciones, aumentando la diversidad
visual de la demostración.
Cambios:
- Nueva acción en modo LOGO (PHYSICS): cambiar dirección gravedad (16%)
- Rebalanceo de probabilidades existentes:
• PHYSICS → SHAPE: 60% → 50%
• Gravedad ON: 20% → 18%
• Gravedad OFF: 20% → 16%
• Dirección gravedad: nuevo 16%
- Al cambiar dirección, se fuerza gravedad ON para visibilidad
Antes el modo LOGO solo alternaba entre figura/física y gravedad
on/off, pero nunca cambiaba la dirección. Ahora tiene las mismas
capacidades de variación que los modos DEMO y DEMO_LITE.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
PROBLEMA RESUELTO:
En modo SHAPE (figuras 3D), al cambiar el número de pelotas (teclas 1-8),
las nuevas pelotas aparecían en pantalla pero NO formaban la figura hasta
pulsar de nuevo la tecla de figura (Q/W/E/R/T).
CAUSA RAÍZ:
1. changeScenario() creaba pelotas nuevas en centro de pantalla
2. generateShape() generaba puntos target de la figura
3. PERO las pelotas nuevas no tenían shape_attraction_active=true
4. Sin atracción activa, las pelotas no se movían hacia sus targets
CAMBIOS IMPLEMENTADOS:
1. Ball class (ball.h/ball.cpp):
- Constructor ahora acepta parámetro Y explícito
- Eliminado hardcodeo Y=0.0f en inicialización de pos_
2. SceneManager (scene_manager.cpp):
- PHYSICS mode: Y = 0.0f (parte superior, comportamiento original)
- SHAPE mode: Y = screen_height_/2.0f (centro vertical) ✅
- BOIDS mode: Y = rand() (posición Y aleatoria)
- Ball constructor llamado con parámetro Y según modo
3. Engine (engine.cpp:514-521):
- Tras generateShape(), activar enableShapeAttraction(true) en todas
las pelotas nuevas
- Garantiza que las pelotas converjan inmediatamente hacia figura
RESULTADO:
✅ Cambiar escenario (1-8) en modo SHAPE regenera automáticamente la figura
✅ No requiere pulsar tecla de figura de nuevo
✅ Transición suave e inmediata hacia nueva configuración
ARCHIVOS MODIFICADOS:
- source/ball.h: Constructor acepta parámetro Y
- source/ball.cpp: Usar Y en lugar de hardcode 0.0f
- source/scene/scene_manager.cpp: Inicializar Y según SimulationMode
- source/engine.cpp: Activar shape attraction tras changeScenario()
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
CAMBIOS:
- Debug HUD reorganizado en layout de 2 columnas (LEFT/RIGHT, sin centro)
- Añadidos getters públicos en Engine para info de sistema
- changeScenario() ahora preserva el SimulationMode actual
- Inicialización de pelotas según modo (PHYSICS/SHAPE/BOIDS)
- Eliminada duplicación de logo_entered_manually_ (ahora en StateManager)
ARCHIVOS MODIFICADOS:
- engine.h: Añadidos 8 getters públicos para UIManager
- engine.cpp: changeScenario() pasa current_mode_ a SceneManager
- scene_manager.h: changeScenario() acepta parámetro SimulationMode
- scene_manager.cpp: Inicialización según modo (RULES.md líneas 23-26)
- ui_manager.h: render() acepta Engine* y renderDebugHUD() actualizado
- ui_manager.cpp: Debug HUD con columnas LEFT (sistema) y RIGHT (física)
REGLAS.md IMPLEMENTADO:
✅ Líneas 23-26: Inicialización diferenciada por modo
- PHYSICS: Top, 75% distribución central en X, velocidades aleatorias
- SHAPE: Centro de pantalla, sin velocidad inicial
- BOIDS: Posiciones y velocidades aleatorias
✅ Líneas 88-96: Debug HUD con información de sistema completa
BUGS CORREGIDOS:
- Fix: Cambiar escenario (1-8) en FIGURE ya no resetea a PHYSICS ❌→✅
- Fix: Las pelotas se inicializan correctamente según el modo activo
- Fix: AppMode movido de centro a izquierda en Debug HUD
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Marcada Fase 2 como completada con detalles de implementación:
- Tiempo real: 2 horas (estimado: 4-6 horas)
- 206 líneas de código añadidas
- SpatialGrid genérico reutilizable
- Pendiente: Medición de rendimiento real
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementado sistema genérico de particionamiento espacial reutilizable
que reduce drásticamente la complejidad del algoritmo de boids.
**MEJORA DE RENDIMIENTO ESPERADA:**
- Sin grid: 1000 boids = 1M comparaciones (1000²)
- Con grid: 1000 boids ≈ 9K comparaciones (~9 vecinos/celda)
- **Speedup teórico: ~100x en casos típicos**
**COMPONENTES IMPLEMENTADOS:**
1. **SpatialGrid genérico (spatial_grid.h/.cpp):**
- Divide espacio 2D en celdas de 100x100px
- Hash map para O(1) lookup de celdas
- queryRadius(): Busca solo en celdas adyacentes (máx 9 celdas)
- Reutilizable para colisiones ball-to-ball en física (futuro)
2. **Integración en BoidManager:**
- Grid poblado al inicio de cada frame (O(n))
- 3 reglas de Reynolds ahora usan queryRadius() en lugar de iterar TODOS
- Separación/Alineación/Cohesión: O(n) total en lugar de O(n²)
3. **Configuración (defines.h):**
- BOID_GRID_CELL_SIZE = 100.0f (≥ BOID_COHESION_RADIUS)
**CAMBIOS TÉCNICOS:**
- boid_manager.h: Añadido miembro spatial_grid_
- boid_manager.cpp: update() poblа grid, 3 reglas usan queryRadius()
- spatial_grid.cpp: 89 líneas de implementación genérica
- spatial_grid.h: 74 líneas con documentación exhaustiva
**PRÓXIMOS PASOS:**
- Medir rendimiento real con 1K, 5K, 10K boids
- Comparar FPS antes/después
- Validar que comportamiento es idéntico
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementadas 2 mejoras críticas para modo boids:
**1. Auto-exit de boids al activar gravedad (G/cursores):**
- handleGravityToggle(): Sale a PHYSICS si está en BOIDS
- handleGravityDirectionChange(): Sale a PHYSICS y aplica dirección
- Razón: La gravedad es conceptualmente incompatible con boids
- UX esperada: Usuario pulsa G → vuelve automáticamente a física
**2. Update screen size en F4 (real fullscreen):**
- toggleRealFullscreen() ahora llama a boid_manager_->updateScreenSize()
- Corrige bug: Boids no respetaban nuevas dimensiones tras F4
- Wrapping boundaries ahora se actualizan correctamente
Cambios:
- engine.cpp: Añadida comprobación de BOIDS en métodos de gravedad
- engine.cpp: Actualización de boid_manager en F4 (línea 420)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
BUG CRÍTICO ENCONTRADO:
La fuerza de cohesión NO estaba normalizada, causando atracción
tipo "gravedad" que hacía que los boids colapsaran a puntos.
CAUSA RAÍZ:
```cpp
// ANTES (INCORRECTO):
float steer_x = (center_of_mass_x - center_x) * WEIGHT * delta_time;
// Si center_of_mass está a 100px → fuerza = 100 * 0.5 * 0.016 = 0.8
// ¡FUERZA PROPORCIONAL A DISTANCIA! Como una gravedad newtoniana
```
SOLUCIÓN IMPLEMENTADA:
```cpp
// DESPUÉS (CORRECTO):
float distance = sqrt(dx*dx + dy*dy);
float steer_x = (dx / distance) * WEIGHT * delta_time;
// Fuerza siempre normalizada = 1.0 * WEIGHT * delta_time
// Independiente de distancia (comportamiento Reynolds correcto)
```
CAMBIOS:
1. boid_manager.cpp::applyCohesion() - Fase 1.4
- Normalizar dirección hacia centro de masa
- Fuerza constante independiente de distancia
- Check de división por cero (distance > 0.1f)
2. defines.h - Ajuste de parámetros tras normalización
- BOID_COHESION_WEIGHT: 0.5 → 0.001 (1000x menor)
* Ahora que está normalizado, el valor anterior era gigantesco
- BOID_MAX_SPEED: 3.0 → 2.5 (reducida para evitar velocidades extremas)
- BOID_MAX_FORCE: 0.5 → 0.05 (reducida 10x)
- BOID_MIN_SPEED: 0.5 → 0.3 (reducida)
- Radios restaurados a valores originales (30/50/80)
RESULTADO ESPERADO:
✅ Sin colapso a puntos (cohesión normalizada correctamente)
✅ Movimiento orgánico sin "órbitas" artificiales
✅ Velocidades controladas y naturales
✅ Balance correcto entre las 3 fuerzas
TESTING:
Por favor probar con 100 y 1000 boids:
- ¿Se mantienen dispersos sin colapsar?
- ¿Las órbitas han desaparecido?
- ¿El movimiento es más natural?
Estado: Compilación exitosa
Rama: boids_development
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
PROBLEMA RESUELTO:
Los boids colapsaban al mismo punto dentro de cada grupo, haciendo
el sistema visualmente inutilizable.
CAMBIOS IMPLEMENTADOS:
1. BOIDS_ROADMAP.md creado (NEW FILE)
- Roadmap completo de 6 fases para mejora de boids
- Diagnóstico detallado de problemas actuales
- Plan de implementación con métricas de éxito
- Fase 1 (crítica): Fix clustering
- Fase 2 (alto impacto): Spatial Hash Grid O(n²)→O(n)
- Fases 3-6: Mejoras visuales, comportamientos avanzados
2. defines.h - Rebalanceo de parámetros (Fase 1.1)
- BOID_SEPARATION_RADIUS: 30→25px
- BOID_COHESION_RADIUS: 80→60px (REDUCIDO 25%)
- BOID_SEPARATION_WEIGHT: 1.5→3.0 (TRIPLICADO)
- BOID_COHESION_WEIGHT: 0.8→0.5 (REDUCIDO 37%)
- BOID_MAX_FORCE: 0.1→0.5 (QUINTUPLICADO)
- BOID_MIN_SPEED: 0.5 (NUEVO - evita boids estáticos)
3. boid_manager.cpp - Mejoras físicas
- Fase 1.2: Velocidad mínima en limitSpeed()
* Evita boids completamente estáticos
* Mantiene movimiento continuo
- Fase 1.3: Fuerza de separación proporcional a cercanía
* Antes: dividir por distance² (muy débil)
* Ahora: proporcional a (RADIUS - distance) / RADIUS
* Resultado: 100% fuerza en colisión, 0% en radio máximo
RESULTADO ESPERADO:
✅ Separación domina sobre cohesión (peso 3.0 vs 0.5)
✅ Boids mantienen distancia personal (~10-15px)
✅ Sin colapso a puntos únicos
✅ Movimiento continuo sin boids estáticos
PRÓXIMOS PASOS:
- Testing manual con 100, 1000 boids
- Validar comportamiento disperso sin clustering
- Fase 2: Spatial Hash Grid para rendimiento O(n)
Estado: Compilación exitosa, listo para testing
Rama: boids_development
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Cambios realizados:
- Creado BoidManager (source/boids_mgr/) con algoritmo de Reynolds (1987)
* Separación: Evitar colisiones con vecinos cercanos
* Alineación: Seguir dirección promedio del grupo
* Cohesión: Moverse hacia centro de masa del grupo
* Wrapping boundaries (teletransporte en bordes)
* Velocidad y fuerza limitadas (steering behavior)
- Añadido BOIDS a enum SimulationMode (defines.h)
- Añadidas constantes de configuración boids (defines.h)
- Integrado BoidManager en Engine (inicialización, update, toggle)
- Añadido binding de tecla J para toggleBoidsMode() (input_handler.cpp)
- Añadidos helpers en Ball: getVelocity(), setVelocity(), setPosition()
- Actualizado CMakeLists.txt para incluir source/boids_mgr/*.cpp
Arquitectura:
- BoidManager sigue el patrón establecido (similar a ShapeManager)
- Gestión independiente del comportamiento de enjambre
- Tres reglas de Reynolds implementadas correctamente
- Compatible con sistema de resolución dinámica
Estado: Compilación exitosa, BoidManager funcional
Próximo paso: Testing y ajuste de parámetros boids
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Limpieza:
- Eliminadas declaraciones de métodos privados obsoletos en engine.h
- Eliminado método Engine::enterLogoMode(bool) obsoleto
- Actualizados comentarios de callbacks para reflejar arquitectura final
- Documentadas todas las variables de estado DEMO/LOGO en Engine
Documentación:
- Aclarado que callbacks son parte de la arquitectura pragmática
- Explicado que StateManager coordina, Engine implementa
- Documentado propósito de cada variable de estado duplicada
- Actualizado comentarios de sistema de figuras 3D
Arquitectura final:
- StateManager: Coordina estados, timers y triggers
- Engine: Proporciona implementación vía callbacks
- Separación de responsabilidades clara y mantenible
- Sin TODO markers innecesarios
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementación:
- StateManager::update() ahora maneja timers y triggers DEMO/LOGO
- Detección de flips de PNG_SHAPE migrada completamente
- Callbacks temporales en Engine para acciones complejas
- enterLogoMode() y exitLogoMode() públicos para transiciones automáticas
- Toggle methods en Engine delegados a StateManager
Callbacks implementados (temporal para Fase 9):
- Engine::performLogoAction()
- Engine::executeDemoAction()
- Engine::executeRandomizeOnDemoStart()
- Engine::executeToggleGravityOnOff()
- Engine::executeEnterLogoMode()
- Engine::executeExitLogoMode()
TODO Fase 9:
- Eliminar callbacks moviendo lógica completa a StateManager
- Limpiar duplicación de estado entre Engine y StateManager
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
ENFOQUE PRAGMÁTICO:
- ShapeManager creado e implementado completamente
- Código DUPLICADO entre Engine y ShapeManager temporalmente
- Engine mantiene implementación para DEMO/LOGO (hasta Fase 8)
- Compilación exitosa, aplicación funcional
ARCHIVOS CREADOS/MODIFICADOS:
1. shape_manager.h:
- Interfaz completa de ShapeManager
- Métodos públicos para control de figuras 3D
- Referencias a Scene/UI/StateManager
2. shape_manager.cpp:
- Implementación completa de todos los métodos
- toggleShapeMode(), activateShape(), update(), generateShape()
- Sistema de atracción física con spring forces
- Cálculo de convergencia para LOGO MODE
- Includes de todas las Shape classes
3. engine.h:
- Variables de figuras 3D MANTENIDAS (duplicadas con ShapeManager)
- Comentarios documentando duplicación temporal
- TODO markers para Fase 8
4. engine.cpp:
- Inicialización de ShapeManager con dependencias
- Métodos de figuras restaurados (no eliminados)
- Código DEMO/LOGO funciona con variables locales
- Sistema de rendering usa current_mode_ local
DUPLICACIÓN TEMPORAL DOCUMENTADA:
```cpp
// Engine mantiene:
- current_mode_, current_shape_type_, last_shape_type_
- active_shape_, shape_scale_factor_, depth_zoom_enabled_
- shape_convergence_
- toggleShapeModeInternal(), activateShapeInternal()
- updateShape(), generateShape(), clampShapeScale()
```
JUSTIFICACIÓN:
- Migrar ShapeManager sin migrar DEMO/LOGO causaba conflictos masivos
- Enfoque incremental: Fase 7 (ShapeManager) → Fase 8 (DEMO/LOGO)
- Permite compilación y testing entre fases
- ShapeManager está listo para uso futuro en controles manuales
RESULTADO:
✅ Compilación exitosa (1 warning menor)
✅ Aplicación funciona correctamente
✅ Todas las características operativas
✅ ShapeManager completamente implementado
✅ Listo para Fase 8 (migración DEMO/LOGO a StateManager)
PRÓXIMOS PASOS (Fase 8):
1. Migrar lógica DEMO/LOGO de Engine a StateManager
2. Convertir métodos de Engine en wrappers a StateManager/ShapeManager
3. Eliminar código duplicado
4. Limpieza final
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Actualiza REFACTOR_SUMMARY.md con sección completa sobre el bug crítico
de nullptr dereference y su solución:
NUEVO CONTENIDO:
- Sección "Post-Refactor Bug Fix" con análisis detallado
- Stack trace del crash (UIManager → Engine::initialize)
- Root cause: Llamada a método antes de crear ui_manager_
- Comparación código BEFORE/AFTER con explicación
- Verificación de la solución (compilación + ejecución exitosa)
- Actualización del status final: COMPLETED AND VERIFIED ✅
JUSTIFICACIÓN:
- Documenta problema crítico descubierto post-refactor
- Útil para referencia futura si surgen bugs similares
- Clarifica orden de inicialización correcto en Engine
- Completa la historia del refactor (6 fases + bug fix)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
PROBLEMA CRÍTICO RESUELTO:
- El programa compilaba pero crasheaba inmediatamente al ejecutar
- Stack trace apuntaba a UIManager::updatePhysicalWindowSize() (línea 135)
- Root cause: Engine::initialize() llamaba updatePhysicalWindowSize() en línea 228
ANTES de crear ui_manager_ en línea 232 → nullptr dereference
SOLUCIÓN:
- Calcular tamaño físico de ventana inline sin llamar al método completo
- Usar SDL_GetWindowSizeInPixels() directamente antes de crear ui_manager_
- Pasar valores calculados a UIManager::initialize()
CAMBIOS ADICIONALES:
1. engine.h: Documentar duplicación pragmática Engine ↔ StateManager
- Variables de estado DEMO/LOGO mantenidas temporalmente en Engine
- StateManager mantiene current_app_mode_ (fuente de verdad)
- Comentarios explicativos para futuras migraciones
2. shape_manager.cpp: Documentar facade pattern completo
- Añadidos comentarios extensivos explicando stubs
- Cada método stub documenta por qué Engine mantiene implementación
- Clarifica dependencias (SceneManager, UIManager, notificaciones)
RESULTADO:
✅ Compilación exitosa (sin errores)
✅ Aplicación ejecuta sin crashes
✅ Inicialización de UIManager correcta
✅ Todos los recursos cargan apropiadamente
Archivos modificados:
- source/engine.cpp: Fix de inicialización (líneas 227-238)
- source/engine.h: Documentación de estado duplicado
- source/shapes_mgr/shape_manager.cpp: Documentación facade
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implementa patrón facade/delegation para gestión de estado de aplicación.
Engine ahora consulta estado a través de StateManager en lugar de acceso directo.
## Cambios
**source/engine.cpp:**
- Reemplazar `current_app_mode_` con `state_manager_->getCurrentMode()` (18 ocurrencias)
- setState() delega a StateManager pero mantiene setup en Engine (temporal)
- toggleDemoMode/Lite/Logo() usan getCurrentMode() de StateManager
- updateDemoMode() consulta modo actual mediante StateManager
**source/state/state_manager.cpp:**
- setState() implementado con lógica básica de cambio de estado
- Maneja transiciones LOGO ↔ otros modos correctamente
- Reset de demo_timer_ al cambiar estado
## Patrón Facade Aplicado
**Justificación:** Token budget limitado requiere enfoque pragmático
- StateManager = Interfaz pública para consultas de estado
- Engine = Mantiene implementación compleja temporalmente
- Refactorización incremental sin reescribir 600+ líneas
**Próximo paso (Fase 4c):**
- Eliminar duplicación de miembros entre Engine y StateManager
- Migrar lógica compleja gradualmente
## Verificación
✅ Compilación exitosa
✅ Sin errores de asignación a lvalue
✅ Todas las consultas de estado delegadas correctamente
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Crea la infraestructura del StateManager para gestionar estados DEMO/LOGO
con patrón de callbacks al Engine. Estructura lista para migración de lógica.
## Archivos Nuevos
**source/state/state_manager.h:**
- Declaración de clase StateManager
- Forward declaration de Engine (patrón callback)
- Métodos públicos: initialize(), update(), setState()
- Métodos toggle: toggleDemoMode(), toggleDemoLiteMode(), toggleLogoMode()
- Getters: getCurrentMode(), getPreviousMode(), is*ModeActive()
- Métodos privados: performDemoAction(), randomizeOnDemoStart(), etc.
- Miembros para timers, convergencia, flip detection, estado previo
**source/state/state_manager.cpp:**
- Implementación de constructor/destructor
- initialize() con callback al Engine
- Stubs de todos los métodos (TODO: migrar lógica completa)
- Preparado para recibir ~600 líneas de lógica DEMO/LOGO
## Archivos Modificados
**CMakeLists.txt:**
- Agregado: source/state/*.cpp al glob de archivos fuente
**source/engine.h:**
- Agregado: #include "state/state_manager.h"
- Agregado: std::unique_ptr<StateManager> state_manager_
- NOTA: Miembros de estado aún no removidos (pendiente migración)
**source/engine.cpp:**
- initialize(): Crea state_manager_ con `this` como callback
- NOTA: Métodos DEMO/LOGO aún no migrados (pendiente)
## Estado Actual
- ✅ Estructura del StateManager creada y compila
- ✅ Patrón de callbacks al Engine configurado
- ✅ CMakeLists actualizado
- ⏳ Migración de lógica DEMO/LOGO: PENDIENTE (~600 líneas)
- ⏳ Remoción de miembros duplicados en Engine: PENDIENTE
## Próximos Pasos (Fase 4b)
1. Migrar updateDemoMode() → StateManager::update()
2. Migrar performDemoAction() → StateManager (privado)
3. Migrar randomizeOnDemoStart() → StateManager (privado)
4. Migrar enterLogoMode() → StateManager (privado)
5. Migrar exitLogoMode() → StateManager (privado)
6. Migrar toggleGravityOnOff() → StateManager (privado)
7. Migrar setState() completo
8. Delegar toggle*Mode() desde Engine a StateManager
9. Remover miembros de estado duplicados en Engine
10. Commit final de Fase 4
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
