🕵️ Estándar de Code Review

Este documento detalla el checklist y las reglas de oro para las revisiones de código en este repositorio. Una revisión de código efectiva previene bugs, mantiene el estándar y comparte el conocimiento entre el equipo.

Reglas de Oro

[REQUIRED] Ningún código llega a main sin al menos un Approve.
[REQUIRED] Las PRs deben ser pequeñas y enfocadas (menos de 400 líneas cambiadas idealmente).
[RECOMMENDED] Revisa la lógica y arquitectura, no el estilo. El estilo se delega a las herramientas de linting (ESLint, Prettier).

Checklist del Autor (Antes de pedir review)

[ ] Mi código pasa todos los tests y el linter local (npm run lint, npm run test).
[ ] No he dejado console.log() innecesarios ni código comentado sin un TODO.
[ ] He documentado con TSDoc las funciones públicas y casos complejos.
[ ] He revisado mi propio diff (Files changed) antes de solicitar la revisión.

Checklist del Revisor

[ ] Arquitectura: ¿Sigue los estándares definidos en el handbook? (ej. separación de responsabilidades).
[ ] Rendimiento: ¿Hay bucles ineficientes o renders innecesarios en React?
[ ] Seguridad: ¿Se exponen datos sensibles? ¿Hay validación de inputs?
[ ] Testing: ¿Se han incluido tests para cubrir la nueva lógica?

Código de Conducta en Reviews

Haz preguntas en lugar de dar órdenes: "¿Consideraste usar un Set aquí en lugar de un Array?" en vez de "Cambia esto por un Set".
Elogia el buen trabajo. Los comentarios no son solo para criticar.