name	code-review
description	Code review following project conventions. Activated when the user asks for a review, CR, or code revision.
allowed tools	Read, Grep, Glob

Code Review

Correctness: does the code do what it should?
Tests: coverage for happy path + edge cases
Naming: descriptive variables and functions
Security: no hardcoded secrets, inputs validated
Performance: no N+1 queries, no unnecessary loops
Docs: docstrings on public functions

[SPEC] Add checklist items for each active spec module:

Racionalização	Realidade
"É só um refactor pequeno, não precisa de review"	Refactors pequenos introduzem bugs sutis. Review sempre.
"Os testes passam, então o código está correto"	Testes cobrem cenários escritos. Review encontra cenários não testados.
"Esse padrão é idiomático, não precisa comentar"	Idiomático pra quem? Docstring em função pública é obrigatório.
"Performance não importa agora, é MVP"	N+1 query em MVP vira incidente em produção. Revise sempre.

name	code-review
description	Code review following project conventions. Activated when the user asks for a review, CR, or code revision.
allowed tools	Read, Grep, Glob

[SPEC] Add checklist items for each active spec module:

Racionalização	Realidade
"É só um refactor pequeno, não precisa de review"	Refactors pequenos introduzem bugs sutis. Review sempre.
"Os testes passam, então o código está correto"	Testes cobrem cenários escritos. Review encontra cenários não testados.
"Esse padrão é idiomático, não precisa comentar"	Idiomático pra quem? Docstring em função pública é obrigatório.
"Performance não importa agora, é MVP"	N+1 query em MVP vira incidente em produção. Revise sempre.

code-review