artubss Revisão de Código Sentry
Siga estas diretrizes ao revisar código de projetos Sentry.
Checklist de Revisão
Identificando Problemas
Procure por estes problemas nas alterações de código:
- Erros em runtime: Exceções potenciais, problemas de null pointer, acesso fora dos limites
- Performance: Operações O(n²) ilimitadas, consultas N+1, alocações desnecessárias
- Efeitos colaterais: Mudanças comportamentais não intencionais afetando outros componentes
- Compatibilidade com versões anteriores: Alterações de API que quebram compatibilidade sem caminho de migração
- Consultas ORM: Django ORM complexo com performance inesperada de consultas
- Vulnerabilidades de segurança: Injeção, XSS, lacunas no controle de acesso, exposição de secrets
Avaliação de Design
- As interações entre componentes fazem sentido lógico?
- A alteração está alinhada com a arquitetura atual do projeto?
- Há conflitos com requisitos ou objetivos atuais?
Cobertura de Testes
Todo PR deve ter cobertura de testes apropriada:
- Testes funcionais para lógica de negócio
- Testes de integração para interações entre componentes
- Testes end-to-end para caminhos críticos do usuário
Verifique se os testes cobrem requisitos reais e casos extremos. Evite ramificações ou loops excessivos no código de teste.
Impacto a Longo Prazo
Sinalize para revisão de engenheiro sênior quando as alterações envolverem:
- Modificações de schema de banco de dados
- Alterações de contrato de API
- Adoção de novo framework ou biblioteca
- Caminhos de código críticos para performance
- Funcionalidade sensível a segurança
Diretrizes de Feedback
Tom
- Seja educado e empático
- Forneça sugestões acionáveis, não críticas vagas
- Formule como perguntas quando incerto: "Você considerou...?"
Aprovação
- Aprove quando apenas problemas menores permanecerem
- Não bloqueie PRs por preferências estilísticas
- Lembre-se: o objetivo é reduzir riscos, não obter código perfeito
Padrões Comuns para Sinalizar
Python/Django
# Bad: N+1 query
for user in users:
print(user.profile.name) # Separate query per user
# Good: Prefetch related
users = User.objects.prefetch_related('profile')
TypeScript/React
// Bad: Missing dependency in useEffect
useEffect(() => {
fetchData(userId);
}, []); // userId not in deps
// Good: Include all dependencies
useEffect(() => {
fetchData(userId);
}, [userId]);
Segurança
# Bad: SQL injection risk
cursor.execute(f"SELECT * FROM users WHERE id = {user_id}")
# Good: Parameterized query
cursor.execute("SELECT * FROM users WHERE id = %s", [user_id])