[Proyecto I] #251

Activity

1. JulioEIppo

   changed the title [-][SchoolManager][/-] [+][Proyecto I][/+] on Jun 21, 2026
2. apiad commented on Jul 6, 2026

   

   apiad

   on Jul 6, 2026

   Member

   More actions

   🔧 El título ya tiene el formato correcto pero el CI no se lanzó al renombrar el issue. Disparando la verificación manualmente.

   /revisar

   👀React with 👀1Reacted by github-actions
3. github-actions commented on Jul 6, 2026

   

   github-actionsBot

   on Jul 6, 2026 – with GitHub Actions

   More actions

   Verificación Automática — Proyecto I 🤖

   Repositorio analizado: https://github.com/JulioEIppo/SchoolManagerProject1

   1. Repositorio

   ✅ El repositorio es accesible y fue clonado correctamente.

   2. Informe (report.md)

   ✅ report.md — 2,730 palabras (mínimo: 2,000).

   3. Proyecto Python

   ⚠️ Sin pyproject.toml.
   ✅ requirements.txt encontrado.
   Dependencias declaradas: streamlit

   Archivos Python (7, 1,582 líneas en total):

   • main.py
   • src/core/__init__.py
   • src/core/EventManager.py
   • src/core/Exceptions.py
   • src/core/Classes.py
   • src/core/default_config.py
   • src/core/FileManager.py

   4. Punto de Entrada

   ✅ Punto de entrada detectado: main.py

   5. Ejecución

   ✅ El proyecto se ejecuta.

   2026-07-06 14:42:17.467 WARNING streamlit.runtime.scriptrunner_utils.script_run_context: Thread 'MainThread': missing ScriptRunContext! This warning can be ignored when running in bare mode.
   2026-07-06 14:42:17.467 WARNING streamlit.runtime.scriptrunner_utils.script_run_context: Thread 'MainThread': missing ScriptRunContext! This warning can be ignored when running in bare mode.
   2026-07-06 14:42:1
   

   Resumen

   • ✅ Repositorio accesible
   • ✅ report.md con ≥2000 palabras
   • ✅ Archivos Python
   • ✅ Configuración de dependencias
   • ✅ Punto de entrada
   • ✅ Ejecución

   6/6 verificaciones pasadas.

   ---

   Verificación automática — los resultados son orientativos. El profesor revisará en detalle.
4. apiad

   added a commit that references this issue on Jul 6, 2026

   docs(reviews): add Proyecto I code reviews (batch 4: #250,#251,#255,#256

   ...

   f7f74d3
5. apiad commented on Jul 6, 2026

   

   apiad

   on Jul 6, 2026

   Member

   More actions

   🔍 Revisión de Código — Claude Code

   Repositorio: https://github.com/JulioEIppo/SchoolManagerProject1

   Qué hace tu proyecto

   Es una aplicación web hecha con Streamlit para gestionar eventos escolares. Organiza el trabajo en cinco pestañas: una vista general con métricas, un formulario para crear eventos (con opción de evento especial, recurrente y sugerencia automática de horario libre), un listado con filtros y borrado, una pestaña de configuración para guardar y cargar el estado en JSON, y otra para gestionar las reglas del dominio. Toda la lógica de negocio (eventos, recursos, co-requisitos, exclusiones, conflictos de horario) vive separada de la interfaz, en un paquete core bien organizado. Cumple exactamente lo que planteaste: planificar un evento con día, horario y recursos, sujeto a una serie de reglas.

   Ejecución

   Levanté la app con streamlit run main.py en un entorno aislado y arrancó sin problemas (respondió con estado saludable). Además ejercité toda la lógica de dominio con datos: crear eventos válidos, nombres duplicados, conflictos de horario, exclusiones, co-requisitos, eventos recurrentes y el guardado/carga en JSON — todo funcionó correctamente. El único fallo real apareció al cargar uno de los archivos de estado incluidos (state.json): en vez de rechazarlo con un mensaje, el programa lanzó un error inesperado (Traceback).

   Lo que está bien

   • Organización sobresaliente para primer año: separaste la interfaz de la lógica en módulos con responsabilidades claras. Eso hace el código fácil de leer y de extender.
   • Buen uso de herramientas de Python: dataclass(frozen=True) para recursos inmutables (correcto, porque los usas en conjuntos y diccionarios), un Enum para los tipos, y excepciones propias que devuelven varios mensajes de error a la vez.
   • Modelo de reglas bien pensado: los co-requisitos y las exclusiones (estas últimas bidireccionales) están bien resueltos y funcionan tal como los describes.
   • Persistencia correcta: el guardado y la carga en JSON conservan el estado íntegro, y descartas la carga completa si detectas cualquier inconsistencia.

   Qué mejorar

   • Un error real en el manejo de fallos al cargar: cuando un archivo de estado tiene un esquema antiguo (una clave con nombre distinto), el mensaje de error que preparaste está mal escrito y, en lugar de avisar, rompe el programa. Es un fallo de una sola línea: revisa cómo construyes ese texto de error (mezcla de comillas y un : que Python interpreta mal) y verás que el resto de la carga es sólido.
   • Código muerto: quedan bloques grandes comentados. Bórralos para que el archivo respire.
   • Idioma mezclado: hay comentarios y variables en español dentro de código en inglés. Elige uno y sé consistente.

   Sobre el informe

   El informe es excelente y honesto: describe con precisión que es una app Streamlit (no de consola), explica la arquitectura módulo a módulo y las decisiones de diseño reales, sin inventar funcionalidades. Solo hay una pequeña contradicción interna (en un punto dices que se omite el evento con recurso faltante, en otro que se descarta toda la carga — esto último es lo que hace el código, y es lo correcto), y no menciona el error de carga que encontré.

   Valoración general

   Un trabajo claramente por encima del nivel esperado en primer año: arquitectura limpia, buen dominio de Python y un modelo de reglas bien diseñado. Tu principal fortaleza es la organización y separación de responsabilidades; tu principal área de mejora es la robustez del manejo de errores al cargar archivos. Muy buen proyecto.

   ---

   Revisión automática de código con Claude Code — orientativa. La nota final la pone el profesor.

   📄 Reporte detallado (código, archivo y línea): ver análisis completo
6. JulioEIppo commented on Jul 15, 2026

   

   JulioEIppo

   on Jul 15, 2026

   Author

   More actions

   Revisado el reporte de claude y arreglado errores pertinentes