[Proyecto I] Il Gusto Giusto

[joserafael0160]

Nombre: José Rafael Pérez Rivero
Grupo: C-122
Repositorio: https://github.com/joserafael0160/Il-Gusto-Giusto

Descripción

Sistema de gestión de restaurante italiano. Planifica comandas como eventos que consumen recursos (mesas, chefs, ingredientes) respetando restricciones culinarias. Incluye control de inventario, gestión de personal, contabilidad y exportación/importación de datos. Desarrollado con Python, Streamlit, Pandas y pytest.

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/joserafael0160/Il-Gusto-Giusto

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

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

3. Proyecto Python

✅ pyproject.toml — proyecto estructurado correctamente.
✅ requirements.txt encontrado.
Dependencias declaradas: streamlit, pandas, pytest

Archivos Python (25, 2,421 líneas en total):

• __init__.py
• conftest.py
• main.py
• src/__init__.py
• tests/__init__.py
• tests/test_scheduler.py
• src/core/__init__.py
• src/core/constraints.py
• src/core/scheduler.py
• src/persistence/__init__.py
• src/persistence/repository.py
• src/persistence/json_handler.py
• src/models/__init__.py
• src/models/events.py
• src/models/restaurant.py
• (y 10 más)

4. Punto de Entrada

✅ Punto de entrada detectado: main.py

5. Ejecución

✅ El proyecto se ejecuta.

2026-06-25 22:02:41.097 WARNING streamlit.runtime.scriptrunner_utils.script_run_context: Thread 'MainThread': missing ScriptRunContext! This warning can be ignored when running in bare mode.
2026-06-25 22:02:41.098 WARNING streamlit.runtime.scriptrunner_utils.script_run_context: Thread 'MainThread': missing ScriptRunContext! This warning can be ignored when running in bare mode.
2026-06-25 22:02:4

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.

[apiad]

🔍 Revisión de Código — Claude Code

Repositorio: https://github.com/joserafael0160/Il-Gusto-Giusto

Qué hace tu proyecto

Construiste un sistema de gestión para un restaurante italiano donde las comandas se modelan como eventos que consumen recursos compartidos en el tiempo (mesas, chefs, ingredientes), y cada pedido pasa por una tubería de validación: colisión de horario, restricciones culinarias, stock y asignación de chef con la especialidad correcta. Lo envolviste en una app Streamlit con seis vistas (dashboard, menú, staff, compras, contabilidad, configuración) y le sumaste cancelación con reembolso, contabilidad y export/import de estado. Es un proyecto ambicioso y muy bien pensado.

Ejecución

Lo corrí de tres maneras. Tu suite de tests: 26/26 pasan en 0.13s. py_compile sobre tus 25 módulos: sin errores. Y como tu lógica de negocio está separada de la GUI, cargué data/default_config.json (8 platos, 16 ingredientes, 5 mesas) y ejecuté los flujos reales:

• Pedido válido (Pizza Margherita): agendado, balance $15000 → $15014.50, chef asignado correcto.
• Colisión de mesa, mesa inexistente, plato fantasma, cantidad cero → todos rechazados con mensajes claros.
• Cancelación → mesa y chef liberados, balance de vuelta exacto a $15000.00.
• Exclusión mutua (mariscos + queso), co-requisito de trufa, stock insuficiente → todos rechazados con el mensaje correcto.
• Trufa (d7) requiere especialidad truffle_specialty; el motor asignó al único chef que la tiene. Correcto.

También arranqué la GUI headless (streamlit run --server.headless true): levantó limpia, respondió HTTP 200, sin Traceback. No encontré ni un solo bug.

Lo que está bien

• La arquitectura. Separaste modelos, núcleo, servicios, persistencia y UI en capas reales. Eso hizo tu lógica testeable y evaluable sin la GUI — una decisión de diseño que pocos toman en primer año.
• El sistema de restricciones es un patrón Strategy genuino (constraints.py): Constraint(ABC) con subclases. Agregar una regla nueva es heredar, sin tocar el validador. Extensibilidad real, no decorativa.
• La detección de colisiones por solape de intervalos (start < evt.end_time and end > evt.start_time) está perfectamente implementada, incluyendo el caso borde exacto fin==inicio.
• Uso maduro de dataclass, Enum, ABC y hasta Protocol para el repositorio. Manejo defensivo de I/O que atrapa JSON corrupto y devuelve (None, []).
• 26 tests que pasan y cubren de verdad los casos importantes, incluido el reembolso exacto de ingredientes con personalizaciones.

Qué mejorar

• Evita duplicar la serialización. En settings.py:22-70 reescribiste a mano casi la misma lógica que ya tienes en JSONHandler.save. Reutiliza tu handler para exportar: un solo lugar donde vive la serialización significa que si cambias el formato, no tienes que acordarte de dos sitios. (De paso, tu informe dice "se evita la duplicación" — aquí es el único punto donde no se cumple.)
• find_next_available_slot avanza de 5 en 5 minutos buscando hueco. Funciona bien, pero cuando el conflicto es un evento concreto podrías saltar directo a su end_time en vez de iterar. Es una optimización opcional, no un error.

Sobre el informe

Muy bueno: 2341 palabras, bien estructurado, y fiel al código. Tu sección de SOLID no es relleno — el Strategy de restricciones y el Protocol del repositorio respaldan de verdad lo que afirmas. La sección de tests describe con exactitud lo que cubren, sin exagerar. La única discrepancia es la frase "se evita la duplicación", que la duplicación de settings.py contradice. Detalle mínimo en un informe honesto.

Enhorabuena: es un trabajo de una madurez arquitectónica poco común para primer año. Sigue diseñando así.