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])

Referências

• Diretrizes de Revisão de Código Sentry