Code review checklist

Efektivní code review není o stylu kódu. Tady je na co se opravdu zaměřit.

Funkčnost

• ☐ Řeší PR skutečný problém?
• ☐ Edge cases ošetřeny
• ☐ Error handling kompletní
• ☐ Backward kompatibilita zachována

Čitelnost

• ☐ Kód je srozumitelný bez vysvětlování
• ☐ Naming jasný a konzistentní
• ☐ Komentáře tam, kde jsou potřeba (ne všude)
• ☐ Funkce/metody krátké a jednoúčelové

Bezpečnost

• ☐ Žádné hardcoded secrets
• ☐ Input validace
• ☐ SQL injection prevence
• ☐ XSS prevence

Performance

• ☐ Žádné N+1 queries
• ☐ Vhodné indexy pro nové queries
• ☐ Caching kde má smysl
• ☐ Neblokující operace pro I/O

Testy

• ☐ Unit testy pro novou logiku
• ☐ Integration testy pro nové endpointy
• ☐ Edge case testy
• ☐ Test coverage neklesá

Ostatní

• ☐ Dokumentace aktualizována
• ☐ Migrace reversibilní
• ☐ Feature flag pro rizikové změny
• ☐ Changelog entry