workspace/projects/gamilit/orchestration/agentes/architecture-analyst/audit-database-2025-12-14/09-AUDITORIA-DUPLICACION-FUNCIONAL.md

AUDITORÍA DE DUPLICACIÓN FUNCIONAL - BASE DE DATOS GAMILIT

Fecha: 2025-12-14 Auditor: Architecture Analyst Agent Objetivo: Detectar funciones SQL, triggers y validadores con lógica duplicada o redundante Alcance: 118 funciones SQL, 49 triggers, 26 validadores de ejercicios

---

RESUMEN EJECUTIVO

Hallazgos Principales

Categoría	Duplicaciones Críticas (P0)	Duplicaciones Importantes (P1)	Duplicaciones Menores (P2)
Funciones timestamp	3 funciones duplicadas	-	-
Triggers de misiones	-	8 funciones con 80% código duplicado	-
Validadores de ejercicios	-	26 validadores con patrones comunes	-
Triggers updated_at	-	23 triggers usando función común	Correcto
Notificaciones	-	-	2 schemas (sin conflicto)

Métricas de Duplicación

• Total funciones analizadas: 118
• Total triggers analizados: 49
• Código duplicado estimado: ~1,200 líneas (65% de 8 funciones de misiones)
• Potencial de consolidación: 3 funciones críticas + 8 funciones importantes
• Impacto en mantenibilidad: ALTO (cambios requieren modificar 8+ archivos)

---

1. FUNCIONES DE TIMESTAMP (P0 - CRÍTICO)

1.1 Análisis de Duplicación

Se identificaron 4 funciones que actualizan updated_at, de las cuales 3 son duplicadas:

Función	Schema	Implementación	Líneas
update_updated_at_column	gamilit	NEW.updated_at = gamilit.now_mexico()	17
update_notifications_updated_at	gamification_system	NEW.updated_at = NOW()	17
update_missions_updated_at	gamification_system	NEW.updated_at = NOW()	17
update_exercise_submissions_updated_at	progress_tracking	NEW.updated_at = gamilit.now_mexico()	87

Archivos:

• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/gamilit/functions/15-update_updated_at_column.sql
• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/gamification_system/functions/07-update_notifications_updated_at.sql
• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/gamification_system/functions/06-update_missions_updated_at.sql
• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/progress_tracking/functions/07-update_exercise_submissions_updated_at.sql

1.2 Código Duplicado

Función genérica (gamilit schema):

CREATE OR REPLACE FUNCTION gamilit.update_updated_at_column()
RETURNS trigger
LANGUAGE plpgsql
AS $function$
BEGIN
    NEW.updated_at = gamilit.now_mexico();
    RETURN NEW;
END;
$function$;

Funciones duplicadas (gamification_system schema):

-- update_notifications_updated_at
BEGIN
  NEW.updated_at = NOW();  -- ❌ Usa NOW() en lugar de now_mexico()
  RETURN NEW;
END;

-- update_missions_updated_at
BEGIN
  NEW.updated_at = NOW();  -- ❌ Usa NOW() en lugar de now_mexico()
  RETURN NEW;
END;

1.3 Problema de Inconsistencia

Inconsistencia de timezone:

• gamilit.update_updated_at_column() usa gamilit.now_mexico() (timezone México)
• gamification_system.update_*_updated_at() usan NOW() (timezone sistema)
• Riesgo: Timestamps inconsistentes entre tablas

1.4 Uso en Triggers

Función genérica (usada en 23 triggers):

23 triggers → gamilit.update_updated_at_column()
1 trigger  → gamification_system.update_notifications_updated_at()
1 trigger  → gamification_system.update_missions_updated_at()
1 trigger  → progress_tracking.update_exercise_submissions_updated_at()

Triggers usando función genérica:

• educational_content.exercises → gamilit.update_updated_at_column()
• auth_management.profiles → gamilit.update_updated_at_column()
• social_features.classrooms → gamilit.update_updated_at_column()
• ... (20 triggers más)

Triggers usando funciones duplicadas:

• gamification_system.notifications → gamification_system.update_notifications_updated_at()
• gamification_system.missions → gamification_system.update_missions_updated_at()
• progress_tracking.exercise_submissions → progress_tracking.update_exercise_submissions_updated_at()

1.5 Recomendación (P0)

ACCIÓN REQUERIDA (Prioridad: CRÍTICA)

1. Eliminar funciones duplicadas:

   • Borrar gamification_system.update_notifications_updated_at()
   • Borrar gamification_system.update_missions_updated_at()
   • Evaluar si consolidar progress_tracking.update_exercise_submissions_updated_at() (tiene documentación extensa)

2. Migrar triggers a función genérica:

   -- ANTES
   CREATE TRIGGER notifications_updated_at
   BEFORE UPDATE ON gamification_system.notifications
   FOR EACH ROW EXECUTE FUNCTION gamification_system.update_notifications_updated_at();
   
   -- DESPUÉS
   CREATE TRIGGER notifications_updated_at
   BEFORE UPDATE ON gamification_system.notifications
   FOR EACH ROW EXECUTE FUNCTION gamilit.update_updated_at_column();
   

3. Archivos a modificar:

   • /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/gamification_system/triggers/18-notifications_updated_at.sql
   • /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/gamification_system/triggers/17-missions_updated_at.sql

4. Beneficios:

   • ✅ Consistencia de timezone en toda la BD
   • ✅ Reducción de 2-3 funciones redundantes
   • ✅ Mantenimiento en un solo punto
   • ✅ Mejor documentación centralizada

Esfuerzo estimado: 1-2 horas Riesgo: BAJO (cambio simple, bien probado) Impacto: ALTO (mejora consistencia de datos)

---

2. FUNCIONES DE ACTUALIZACIÓN DE MISIONES (P1 - IMPORTANTE)

2.1 Análisis de Código Duplicado

Se identificaron 8 funciones de actualización de misiones con ~80% de código duplicado:

Función	Archivo	Líneas	Diferencia
update_missions_on_exercise_complete	17-update_missions_on_exercise_complete.sql	196	Filtra por complete_exercises
update_missions_on_correct_streak	19-update_missions_on_correct_streak.sql	218	Filtra por correct_streak
update_missions_on_use_comodines	21-update_missions_on_use_comodines.sql	194	Filtra por use_comodines
update_missions_on_earn_xp	22-update_missions_on_earn_xp.sql	225	Filtra por earn_xp, calcula delta XP
update_missions_on_daily_streak	23-update_missions_on_daily_streak.sql	185	Filtra por daily_streak
update_missions_on_perfect_scores	24-update_missions_on_perfect_scores.sql	209	Filtra por perfect_scores
update_missions_on_complete_modules	25-update_missions_on_complete_modules.sql	-	Filtra por complete_modules
update_missions_on_explore_modules	26-update_missions_on_explore_modules.sql	-	Filtra por explore_modules

Total líneas duplicadas: ~1,227 líneas Código compartido: ~80% (iteración objetivos, cálculo progreso, update statement)

2.2 Patrón Común (Código Duplicado)

Todas las funciones siguen el mismo patrón:

-- 1. Buscar misiones activas con objetivo específico
FOR v_mission IN
    SELECT id, objectives, status
    FROM gamification_system.missions
    WHERE user_id = NEW.user_id
      AND status IN ('active', 'in_progress')
      AND end_date > gamilit.now_mexico()
      AND objectives @> '[{"type": "OBJETIVO_TIPO"}]'::jsonb  -- ← ÚNICA DIFERENCIA
LOOP
    -- 2. Iterar sobre objetivos (CÓDIGO IDÉNTICO)
    FOR v_i IN 0..v_obj_count-1 LOOP
        v_objective := v_objectives->v_i;

        -- 3. Incrementar current si es del tipo correcto
        IF v_objective->>'type' = 'OBJETIVO_TIPO' THEN  -- ← ÚNICA DIFERENCIA
            v_new_current := LEAST(
                (v_objective->>'current')::INTEGER + 1,
                (v_objective->>'target')::INTEGER
            );
            v_objective := jsonb_set(v_objective, '{current}', to_jsonb(v_new_current));
        END IF;

        -- 4. Calcular progreso (CÓDIGO IDÉNTICO)
        v_obj_progress := ((v_objective->>'current')::FLOAT / (v_objective->>'target')::FLOAT) * 100;
        v_total_progress := v_total_progress + v_obj_progress;
        v_new_objectives := v_new_objectives || jsonb_build_array(v_objective);
    END LOOP;

    -- 5. Actualizar misión (CÓDIGO IDÉNTICO)
    UPDATE gamification_system.missions
    SET
        objectives = v_new_objectives,
        progress = ROUND(v_total_progress::NUMERIC, 2),
        status = CASE
            WHEN v_total_progress >= 100 THEN 'completed'
            WHEN v_total_progress > 0 AND status = 'active' THEN 'in_progress'
            ELSE status
        END,
        completed_at = CASE
            WHEN v_total_progress >= 100 AND completed_at IS NULL THEN gamilit.now_mexico()
            ELSE completed_at
        END,
        updated_at = gamilit.now_mexico()
    WHERE id = v_mission.id;
END LOOP;

2.3 Diferencias Clave

Solo 2 aspectos cambian entre funciones:

1. Tipo de objetivo: 'complete_exercises', 'use_comodines', 'earn_xp', etc.
2. Incremento: +1 (mayoría) vs +v_xp_gained (solo earn_xp)

2.4 Triggers que Invocan Estas Funciones

Trigger	Tabla	Evento	Función
trg_update_missions_on_exercise	progress_tracking.exercise_attempts	AFTER INSERT	update_missions_on_exercise_complete
trg_update_missions_on_submission	progress_tracking.exercise_submissions	AFTER UPDATE	update_missions_on_exercise_complete
trg_update_missions_on_streak	progress_tracking.exercise_attempts	AFTER INSERT	update_missions_on_correct_streak
trg_update_missions_on_use_comodines	gamification_system.comodin_usage_log	AFTER INSERT	update_missions_on_use_comodines
trg_update_missions_on_earn_xp	gamification_system.user_stats	AFTER UPDATE	update_missions_on_earn_xp
trg_update_missions_on_daily_streak	gamification_system.user_stats	AFTER UPDATE	update_missions_on_daily_streak
trg_update_missions_on_perfect_scores	progress_tracking.exercise_attempts	AFTER INSERT	update_missions_on_perfect_scores
trg_update_missions_on_complete_modules	progress_tracking.module_progress	AFTER UPDATE	update_missions_on_complete_modules

2.5 Recomendación (P1)

OPCIÓN A: Función Genérica Parametrizada (Recomendado)

Crear una función maestra que recibe el tipo de objetivo:

CREATE OR REPLACE FUNCTION gamilit.update_missions_progress(
    p_user_id UUID,
    p_objective_type TEXT,
    p_increment INTEGER DEFAULT 1
)
RETURNS void
LANGUAGE plpgsql
SECURITY DEFINER
AS $$
-- Código genérico que recibe el tipo de objetivo como parámetro
$$;

Wrapper functions:

-- Funciones específicas se vuelven simples wrappers
CREATE OR REPLACE FUNCTION gamilit.update_missions_on_exercise_complete()
RETURNS TRIGGER AS $$
BEGIN
    PERFORM gamilit.update_missions_progress(NEW.user_id, 'complete_exercises', 1);
    RETURN NEW;
END;
$$ LANGUAGE plpgsql;

Beneficios:

• ✅ Código duplicado: 1,200 líneas → ~150 líneas (92% reducción)
• ✅ Cambios en lógica: 1 archivo vs 8 archivos
• ✅ Testing: 1 función vs 8 funciones
• ✅ Bugs: Fix en un solo lugar

Esfuerzo estimado: 4-6 horas Riesgo: MEDIO (requiere testing exhaustivo de misiones) Impacto: ALTO (mejora mantenibilidad crítica)

OPCIÓN B: Mantener Status Quo

Si el equipo prefiere no refactorizar por ahora:

• ✅ Documentar el patrón común en README
• ✅ Crear template para nuevas funciones de misiones
• ✅ Agregar tests de regresión
• ⚠️ Aceptar deuda técnica

---

3. VALIDADORES DE EJERCICIOS (P1 - IMPORTANTE)

3.1 Análisis de Funciones Validadoras

Se identificaron 26 funciones de validación de ejercicios con patrones comunes:

Categorías de validadores:

Módulo	Funciones	Archivos
Módulo 1 (Comprensión Literal)	5	validate_crucigrama, validate_timeline, validate_word_search, validate_fill_in_blank, validate_true_false
Módulo 2 (Inferencia)	6	validate_detective_textual, validate_rueda_inferencias, validate_mapa_conceptual, etc.
Módulo 3 (Análisis Crítico)	5	validate_analisis_fuentes, validate_construccion_hipotesis, etc.
Módulo 4-5 (Argumentación)	6	validate_module4_module5, validate_podcast_argumentativo, etc.
Función Maestra	1	validate_answer (dispatcher)
Helpers	3	validate_and_audit, recalculate_exercise

Total: 26 funciones

3.2 Patrones Comunes Identificados

Patrón 1: Normalización de Texto (27 usos)

-- Código repetido en casi todas las validaciones
IF p_normalize_text THEN
    v_correct_answer := gamilit.normalize_text(v_correct_answer);
    v_submitted_answer := gamilit.normalize_text(COALESCE(v_submitted_answer, ''));
END IF;

IF NOT p_case_sensitive THEN
    v_correct_answer := UPPER(TRIM(v_correct_answer));
    v_submitted_answer := UPPER(TRIM(v_submitted_answer));
END IF;

Usado en: validate_crucigrama, validate_detective_textual, validate_fill_in_blank, etc.

Patrón 2: Cálculo de Score Parcial (24 usos)

-- Código repetido para calcular puntaje
IF p_allow_partial_credit THEN
    score := ROUND((v_correct_count::NUMERIC / v_total_count) * p_max_points);
ELSE
    score := CASE WHEN is_correct THEN p_max_points ELSE 0 END;
END IF;

Usado en: Mayoría de validadores (24 de 26)

Patrón 3: Iteración sobre JSONB keys (13 usos)

-- Código repetido para iterar objetos JSONB
FOR v_key IN SELECT jsonb_object_keys(v_solution_data)
LOOP
    v_correct := v_solution_data->>v_key;
    v_submitted := v_submitted_data->>v_key;
    -- Comparar...
END LOOP;

Usado en: validate_crucigrama, validate_detective_textual, validate_emparejamiento, etc.

Patrón 4: Construcción de Feedback (26 usos)

-- Construcción de mensajes de feedback
IF is_correct THEN
    feedback := format('¡Excelente! Todas las respuestas correctas.');
ELSE
    feedback := format('Tienes %s/%s respuestas correctas (%s%%).',
                      v_correct, v_total,
                      ROUND((v_correct::NUMERIC / v_total) * 100));
END IF;

Usado en: Todos los validadores (26 funciones)

3.3 Arquitectura Actual

Función Maestra (Dispatcher):

validate_answer(exercise_id, submitted_answer)
    ↓
    Consulta exercise_validation_config
    ↓
    CASE validation_function
        WHEN 'validate_crucigrama' → validate_crucigrama(...)
        WHEN 'validate_timeline' → validate_timeline(...)
        WHEN 'validate_detective_textual' → validate_detective_textual(...)
        ... (26 casos)

Archivo: /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/educational_content/functions/02-validate_answer.sql

3.4 Recomendación (P1)

OPCIÓN A: Extraer Helpers Comunes (Recomendado)

Crear funciones helper para patrones repetidos:

-- 1. Helper: Normalizar y comparar texto
CREATE FUNCTION educational_content.compare_text_normalized(
    p_text1 TEXT,
    p_text2 TEXT,
    p_case_sensitive BOOLEAN,
    p_normalize BOOLEAN
) RETURNS BOOLEAN;

-- 2. Helper: Calcular score con crédito parcial
CREATE FUNCTION educational_content.calculate_partial_score(
    p_correct INTEGER,
    p_total INTEGER,
    p_max_points INTEGER,
    p_allow_partial BOOLEAN
) RETURNS INTEGER;

-- 3. Helper: Generar feedback estándar
CREATE FUNCTION educational_content.generate_validation_feedback(
    p_correct INTEGER,
    p_total INTEGER,
    p_is_correct BOOLEAN,
    p_exercise_type TEXT
) RETURNS TEXT;

-- 4. Helper: Iterar y comparar JSONB
CREATE FUNCTION educational_content.validate_jsonb_answers(
    p_solution JSONB,
    p_submitted JSONB,
    p_case_sensitive BOOLEAN
) RETURNS TABLE (key TEXT, is_correct BOOLEAN, details JSONB);

Refactorización de validadores:

-- ANTES (validate_crucigrama.sql - 150 líneas)
-- Código con normalización, cálculo score, feedback...

-- DESPUÉS (validate_crucigrama.sql - 60 líneas)
CREATE FUNCTION validate_crucigrama(...) AS $$
DECLARE
    v_results RECORD;
BEGIN
    -- Usar helper para comparación
    v_results := educational_content.validate_jsonb_answers(
        p_solution->'clues',
        p_submitted_answer->'clues',
        p_case_sensitive
    );

    -- Usar helper para score
    score := educational_content.calculate_partial_score(
        v_results.correct_count,
        v_results.total_count,
        p_max_points,
        p_allow_partial_credit
    );

    -- Usar helper para feedback
    feedback := educational_content.generate_validation_feedback(
        v_results.correct_count,
        v_results.total_count,
        is_correct,
        'crucigrama'
    );
END;
$$;

Beneficios:

• ✅ Reducción ~40% líneas de código en validadores
• ✅ Lógica de normalización/scoring centralizada
• ✅ Testing más fácil (4 helpers vs 26 validadores)
• ✅ Cambios en UX de feedback: 1 función vs 26

Esfuerzo estimado: 6-8 horas Riesgo: MEDIO-BAJO (helpers aislados, no rompen validadores existentes) Impacto: ALTO (mejora mantenibilidad de validaciones)

OPCIÓN B: Status Quo con Mejoras

Si no se refactoriza:

• ✅ Documentar patrones comunes en guía de desarrollo
• ✅ Crear plantilla para nuevos validadores
• ✅ Code review estricto para evitar más duplicación
• ⚠️ Aceptar deuda técnica acotada

---

4. TRIGGERS DE UPDATED_AT (P2 - CORRECTO)

4.1 Análisis

Estado actual: ✅ BIEN IMPLEMENTADO

• 23 triggers usan la función genérica gamilit.update_updated_at_column()
• 3 triggers usan funciones específicas (ya documentado en sección 1)

Patrón correcto:

CREATE TRIGGER trg_[tabla]_updated_at
BEFORE UPDATE ON [schema].[tabla]
FOR EACH ROW
EXECUTE FUNCTION gamilit.update_updated_at_column();

Ejemplos:

• educational_content.exercises → trg_exercises_updated_at
• auth_management.profiles → trg_profiles_updated_at
• social_features.classrooms → trg_classrooms_updated_at

4.2 Recomendación (P2)

Acción: Mantener patrón actual, consolidar 3 funciones duplicadas (ver sección 1)

✅ Este patrón es el correcto, solo falta migrar los 3 triggers restantes.

---

5. FUNCIONES DE NOTIFICACIONES (P2 - CORRECTO)

5.1 Análisis

Se encontraron funciones de notificaciones en 2 schemas diferentes:

Schema	Función	Propósito
notifications	send_notification	Sistema de notificaciones multi-canal (EXT-003)
notifications	queue_batch_notifications	Notificaciones masivas
social_features	cleanup_old_notifications	Limpieza de notificaciones antiguas

Archivos:

• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/notifications/functions/01-send_notification.sql
• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/notifications/functions/03-queue_batch_notifications.sql
• /home/isem/workspace/projects/gamilit/apps/database/ddl/schemas/social_features/functions/cleanup_old_notifications.sql

5.2 Evaluación

NO HAY DUPLICACIÓN:

• send_notification → Crea notificaciones en tabla notifications.notifications
• cleanup_old_notifications → Limpia tabla social_features.notifications (tabla diferente)

Observación: Existen 2 tablas de notificaciones:

1. notifications.notifications (sistema nuevo - EXT-003)
2. social_features.notifications (sistema legacy)

5.3 Recomendación (P2)

Acción sugerida (futura):

• Evaluar si consolidar ambas tablas de notificaciones
• Si no, documentar claramente la diferencia entre ambos sistemas
• Por ahora: NO requiere acción

---

6. OTROS PATRONES DUPLICADOS

6.1 Uso de gamilit.now_mexico() (121 veces)

Análisis: ✅ CORRECTO - Función centralizada bien usada

Beneficio: Timezone consistente en toda la aplicación

6.2 Uso de SECURITY DEFINER (55 funciones)

Análisis: ⚠️ REVISAR NECESIDAD

• 55 funciones con SECURITY DEFINER
• Revisar si todas realmente necesitan permisos elevados
• Riesgo de seguridad si se usa innecesariamente

Recomendación: Auditoría de seguridad separada

6.3 Funciones de Cálculo (Nivel/Rango)

Funciones identificadas:

• calculate_level_from_xp (gamification_system)
• calculate_user_rank (gamification_system)
• calculate_maya_rank_helpers (gamification_system)
• recalculate_level_on_xp_change (trigger function)

Análisis: ✅ NO DUPLICADAS - Cada una tiene propósito específico

---

7. PLAN DE CONSOLIDACIÓN

Prioridad P0 (CRÍTICO - Hacer AHORA)

Tarea	Archivos Afectados	Esfuerzo	Riesgo	Impacto
P0-001: Eliminar funciones timestamp duplicadas	2 funciones + 2 triggers	1-2h	BAJO	ALTO

Archivos a modificar:

DELETE:
  - apps/database/ddl/schemas/gamification_system/functions/06-update_missions_updated_at.sql
  - apps/database/ddl/schemas/gamification_system/functions/07-update_notifications_updated_at.sql

MODIFY:
  - apps/database/ddl/schemas/gamification_system/triggers/17-missions_updated_at.sql
  - apps/database/ddl/schemas/gamification_system/triggers/18-notifications_updated_at.sql

Prioridad P1 (IMPORTANTE - Planificar)

Tarea	Archivos Afectados	Esfuerzo	Riesgo	Impacto
P1-001: Consolidar funciones de misiones	8 funciones + 8 triggers	4-6h	MEDIO	ALTO
P1-002: Crear helpers para validadores	4 helpers nuevos + 26 validadores	6-8h	MEDIO-BAJO	ALTO

Validación requerida:

• ✅ Suite de tests para misiones (antes de refactorizar)
• ✅ Suite de tests para validadores (antes de refactorizar)

Prioridad P2 (MENOR - Futuro)

Tarea	Descripción	Esfuerzo
P2-001: Auditoría de SECURITY DEFINER	Revisar 55 funciones	2-3h
P2-002: Consolidar tablas de notificaciones	Migrar social_features → notifications	8-12h

---

8. MÉTRICAS DE MEJORA ESPERADAS

Antes de Consolidación

Funciones timestamp:           4 (3 duplicadas)
Funciones de misiones:         8 (1,227 líneas, 80% duplicadas)
Funciones validadoras:        26 (patrones comunes en 100%)
Líneas de código duplicadas:  ~1,400
Archivos a modificar (cambio típico): 8-10

Después de Consolidación (P0 + P1)

Funciones timestamp:           1 (consolidada)
Funciones de misiones:         1 maestra + 8 wrappers (~300 líneas)
Funciones validadoras:        26 + 4 helpers (reducción 40% código)
Líneas de código duplicadas:  ~200
Archivos a modificar (cambio típico): 1-2

Beneficios Cuantificables

• Reducción de código: -85% duplicación (~1,200 líneas)
• Mantenibilidad: 8-10 archivos → 1-2 archivos por cambio
• Testing: 34 funciones → 5 funciones críticas
• Bugs: Fixes en 1 lugar vs 8-10 lugares
• Onboarding: Curva de aprendizaje más simple

---

9. CONCLUSIONES

Hallazgos Clave

1. Duplicación CRÍTICA (P0):

   • 3 funciones timestamp duplicadas con inconsistencia de timezone
   • Acción requerida inmediata

2. Duplicación IMPORTANTE (P1):

   • 8 funciones de misiones con 80% código duplicado (1,200+ líneas)
   • 26 validadores con patrones comunes repetidos
   • Alto impacto en mantenibilidad

3. Patrones CORRECTOS:

   • Triggers updated_at usando función genérica (23 de 26)
   • Uso consistente de gamilit.now_mexico()
   • Funciones de cálculo bien separadas

Recomendaciones Prioritarias

Hacer AHORA (P0):

1. ✅ Consolidar funciones timestamp (1-2 horas)
2. ✅ Migrar triggers a función genérica

Planificar PRONTO (P1):

1. 🔄 Refactorizar funciones de misiones con patrón maestro
2. 🔄 Crear helpers para validadores comunes
3. 🔄 Suite de tests antes de refactorizar

Evaluar FUTURO (P2):

1. 📋 Auditoría de SECURITY DEFINER
2. 📋 Consolidación de tablas de notificaciones

Impacto en Calidad de Código

Antes:

• ❌ Cambios requieren modificar 8-10 archivos
• ❌ 1,400+ líneas de código duplicado
• ❌ Inconsistencias de timezone
• ❌ Difícil mantener coherencia

Después (implementando P0+P1):

• ✅ Cambios en 1-2 archivos
• ✅ ~200 líneas de código duplicado
• ✅ Timezone consistente
• ✅ Lógica centralizada y testeada

---

10. ANEXOS

A. Inventario Completo de Funciones Analizadas

Total funciones: 118 Total triggers: 49 Schemas analizados: 9

1. gamilit - 26 funciones
2. educational_content - 34 funciones
3. gamification_system - 18 funciones
4. progress_tracking - 12 funciones
5. auth_management - 8 funciones
6. social_features - 9 funciones
7. notifications - 4 funciones
8. content_management - 3 funciones
9. system_configuration - 4 funciones

B. Comandos Útiles para Auditoría

# Buscar funciones de timestamp
find apps/database/ddl/schemas -name "*updated_at*.sql" -path "*/functions/*"

# Contar triggers de updated_at
grep -rh "CREATE.*TRIGGER.*updated_at" apps/database/ddl/schemas | wc -l

# Buscar funciones de misiones
ls apps/database/ddl/schemas/gamilit/functions/*mission*.sql

# Listar validadores
find apps/database/ddl/schemas/educational_content/functions -name "validate*.sql"

# Contar uso de normalize_text
grep -rh "gamilit.normalize_text" apps/database/ddl/schemas/educational_content/functions/*.sql | wc -l

C. Referencias

• Documentación misiones: Triggers 24-31 en progress_tracking/triggers/
• Documentación validadores: ET-EDU-001 (especificaciones mecánicas)
• Sistema notificaciones: EXT-003 (Notificaciones Multi-Canal)
• Timezone estándar: gamilit.now_mexico() para toda la aplicación

---

FIN DEL REPORTE

---

Auditor: Architecture Analyst Agent Fecha: 2025-12-14 Versión: 1.0 Próxima revisión: Después de implementar P0 + P1