feat: nuevo dashboard de seguimiento nutricional v2

src/components/PetDashboard/NutritionChart.tsx línea 47

El hook usePetNutrition se llama en cada render sin memoizar. Esto causa recalculaciones innecesarias cada vez que el padre re-renderiza. Envuelve el valor de retorno en useMemo.

✅ Fix aplicado — useMemo añadido con deps [petId, dateRange]

src/api/nutrition.ts línea 23

La función fetchNutritionData no maneja el caso de error del fetch. Si la API devuelve 500, el componente se queda cargando indefinidamente. Añade try/catch y propaga el error.

23−const data = await fetch(`/api/pets/${petId}/nutrition`);
24−return data.json();
23+const res = await fetch(`/api/pets/${petId}/nutrition`);
24+if (!res.ok) throw new Error(`API error: ${res.status}`);
25+return res.json();

✅ Fix aplicado — error handling añadido con throw Error

src/components/PetDashboard/WeightTracker.tsx líneas 88-95

El componente WeightTracker hace su propia llamada a la API en lugar de usar el contexto PetDataContext que ya tenemos. Esto duplica requests al backend. Refactoriza para consumir el contexto.

86 export function WeightTracker({ petId }: Props) {
87−  const [weights, setWeights] = useState([]);
88−  useEffect(() => { fetch(`/api/pets/${petId}/weights`).then(...) }, [petId]);
87+  const { petData } = usePetDataContext();
88+  const weights = petData?.weightHistory ?? [];

⚠️

Acción requerida: Reemplazar llamada directa a API por usePetDataContext() y eliminar el useEffect de fetch.

src/pages/dashboard/[petId].tsx línea 12

Falta la propiedad aria-label en el botón de "Ver historial completo". Esto rompe accesibilidad para usuarios con lector de pantalla. Es un blocker de producción.

12−  <button onClick={handleViewHistory}>Ver historial completo</button>
12+  <button onClick={handleViewHistory} aria-label="Ver historial nutricional completo de la mascota">
13+    Ver historial completo
14+  </button>

♿

Acción requerida: Añadir aria-label descriptivo al botón. Blocker de accesibilidad.

Buen trabajo en general. ¿Podéis añadir tests unitarios para NutritionChart? El coverage actual de ese componente es 0%. Con la lógica de cálculo que tiene, necesitamos al menos los casos edge de mascota sin datos y rango de fechas vacío.

✅ Resuelto — Tests añadidos en NutritionChart.test.tsx (coverage 78%)

¡El nuevo diseño del dashboard se ve muy bien! Los clientes beta de la semana pasada preguntaron específicamente por poder exportar el PDF del historial nutricional. ¿Lo ponemos en el roadmap para el siguiente sprint? No es blocker para este PR.

💬 Comentario informativo — no requiere cambios de código en este PR.

feat: nuevo dashboard de seguimiento nutricional v2

PR listo para re-revisión