Checklist de revisão de código no Odoo

No artigo anterior, falamos sobre como a revisão de código no Odoo é um diferencial invisível que reduz riscos e acelera a evolução do ERP.

Agora vem a parte mais importante: como saber se isso está realmente acontecendo no seu projeto.

Porque “a gente revisa” pode significar duas coisas bem diferentes:

• existe um processo técnico consistente, com padrão e rastreabilidade
• alguém dá uma olhada rápida e aprova porque “parece ok”

E no Odoo, onde o sistema sustenta fiscal, financeiro, estoque, vendas e integrações, essa diferença separa projetos previsíveis de projetos que vivem em modo emergência.

No Brasil isso fica ainda mais sensível, porque o ERP costuma estar conectado à localização brasileira e rotinas fiscais como NF-e e NFS-e. Se o processo é fraco, o problema aparece no pior lugar possível: produção.

Neste artigo, você vai ter um checklist prático para avaliar revisão de código em projetos Odoo — seja para governar seu time interno, seja para validar a qualidade de uma consultoria.

Por que um checklist importa (especialmente em Odoo)

Odoo é um ERP modular e extensível. Isso é uma vantagem enorme… e também uma armadilha quando não existe governança.

Sem revisão técnica consistente, o padrão do projeto vira:

• cada dev faz de um jeito
• customizações se acumulam sem controle
• correções rápidas viram dívida técnica
• upgrades ficam cada vez mais caros

Com checklist, você cria uma regra simples:

✅ Só entra no Odoo o que consegue sobreviver ao futuro.

Checklist de revisão de código no Odoo (padrão profissional)

1) A entrega é feita via Pull Request (PR)?

Se o time trabalha com Odoo e não usa PR, normalmente acontece uma destas situações:

• mudanças entram direto na branch principal
• não existe histórico claro do que foi feito
• revisar vira opcional e “na conversa”

O que esperar num projeto saudável:

• PR por demanda ou tarefa
• descrição clara do objetivo
• referência ao ticket ou requisito

Sinal de alerta: a gente sobe direto no servidor e testa lá.

2) O PR explica o porquê, não só o o quê?

Um PR bom não é só código. Ele explica intenção.

O que deve estar claro:

• qual problema foi resolvido
• impacto no processo do usuário
• regras de negócio envolvidas
• cenários de exceção (edge cases)

Sinal de alerta: PR sem descrição ou com “ajustes”.

3) O código segue padrões Odoo e boas práticas?

Odoo tem padrões conhecidos para manter consistência.

Na revisão, precisa validar:

• herança correta (_inherit, _name)
• uso adequado de @api.depends, @api.constrains, @api.onchange
• cuidado com sudo() (segurança e permissões)
• evitar lógica em views quando deveria estar no backend
• respeito ao fluxo padrão do Odoo (sem hack desnecessário)

Sinal de alerta: funciona, então tá bom.

4) A mudança é segura para upgrades e migração Odoo?

Esse é o ponto onde a revisão paga o projeto inteiro.

A revisão precisa perguntar:

• isso vai quebrar em upgrade?
• está sobrescrevendo método core sem necessidade?
• tem monkey patch, gambiarra ou dependência frágil?
• está criando acoplamento que vai travar a evolução?

Sinal de alerta: alteração grande sem justificativa e sem estratégia de compatibilidade.

5) Existe preocupação real com performance?

Odoo pode ficar lento de repente — e geralmente é culpa de código customizado.

Na revisão, checar:

• buscas com search() sem limite ou sem filtro
• loops com queries repetidas (N+1)
• uso excessivo de compute sem necessidade
• campos compute sem store=True quando deveria
• queries pesadas em horários críticos

Sinal de alerta: ficou lento, mas depois a gente vê.

6) O PR inclui testes ou ao menos um plano de validação?

Nem todo projeto tem testes automatizados (infelizmente), mas todo projeto deveria ter validação mínima.

O que a revisão deve exigir:

• passo a passo de teste (cenário real)
• quais módulos e processos foram afetados
• como reproduzir o caso

Sinal de alerta: validação baseada só em abri aqui e parece ok.

7) Logs, rastreabilidade e debug foram tratados?

Quando dá erro em produção, você precisa de rastreabilidade, não de achismo.

A revisão deve avaliar:

• logs suficientes para diagnóstico
• mensagens de erro úteis (sem vazar informação sensível)
• exceções tratadas de forma correta

Sinal de alerta: erro genérico, sem contexto, ou try/except que engole problema.

8) Segurança e permissões foram consideradas?

Odoo é multiusuário e cheio de regras de acesso.

A revisão precisa validar:

• regras de acesso (ir.model.access)
• record rules quando necessário
• uso responsável de sudo()
• risco de usuário acessar dado que não deveria

Sinal de alerta: coloquei sudo pra funcionar.

9) O PR evita duplicação e respeita modularidade?

No Odoo, modularidade é tudo.

A revisão deve procurar:

• reaproveitamento de lógica existente
• evitar duplicação de código
• evitar hardcode de empresa, imposto, produto
• configuração via dados e parâmetros quando possível

Sinal de alerta: copiei esse trecho em 4 lugares.

10) Existe padrão de commit e versionamento?

Não é frescura: é governança.

O mínimo aceitável:

• commits com mensagem clara
• histórico que permite rollback
• branches organizadas

Sinal de alerta: commit “fix”, “teste”, “agora vai”.

Como usar esse checklist na prática

Você pode usar esse checklist de 3 formas:

1) Como empresa (cliente)

Peça para sua consultoria mostrar:

• exemplos de PRs revisados
• como o time aprova mudanças
• como eles evitam retrabalho e bugs recorrentes

2) Como gestor de TI

Transforme isso em critério de aceite:

Sem PR e revisão, não vai pra produção.

3) Como time interno

Use como padrão para treinar devs e reduzir dependência técnica.

Conclusão: revisão é um sistema de proteção do seu ERP

Odoo é um ERP poderoso.

Mas poder sem governança vira risco.

Revisão de código é um mecanismo simples que protege:

• estabilidade
• performance
• segurança
• capacidade de upgrade
• evolução contínua

E o melhor: quando esse processo está bem implementado, ele não atrapalha.

Ele faz o projeto andar mais rápido no longo prazo.

Sobre a Escodoo

A Escodoo aplica padrões de engenharia usados pela comunidade global do Odoo e pela OCA, trazendo governança técnica para projetos no Brasil.

Se você quer um Odoo que evolui com previsibilidade — sem improviso e sem sustos — fale com a Escodoo.

Entre em contato e entenda como podemos apoiar sua implantação, suporte e evolução do Odoo.