Verificação Pré-PR
Faça todos os itens e reporte em uma tabela: Aprovado / Reprovado, com evidências (comandos e resultados resumidos).
-
make checkdeve passar completamente (docs-check, lint + SAST, test, build, license) — exceto 2 casos:- PR somente de documentação (
docs(design): …, ADR, docs):make docs-checkpassar é suficiente. - Repositório que ainda não escolheu stack (target exibe
not configured— ADR-0001 ainda não aceito): reporte esse target como N/A (ainda não configurado), não como aprovado, e não é motivo para impedir — não modifique a recipe paraexit 0a fim de forçar aprovação.
E
make e2ese alterar o que o usuário vê (adicione/modifique e2e também) emake dastse mexer em middleware, header, cookie, página de erro ou nova rota. - PR somente de documentação (
-
git diff origin/main...HEAD --stat— verifique se mexeu apenas nos arquivos relacionados ao trabalho, sem arquivos extras (build output, .env, arquivos pessoais). -
Logs temporários esquecidos:
git diff origin/main...HEAD | grep -n '\[DBG-'não deve encontrar (tópicofix-bugitem 3). -
Procure por segredos no diff:
git diff origin/main...HEAD | grep -nEi '(api[_-]?key|secret|token|password|BEGIN .*PRIVATE KEY)'— se encontrar, pare e avise. -
Novas variáveis de ambiente estão presentes em
.env.example. -
Dependências adicionadas/atualizadas (veja diff de lockfile, go.mod, Dockerfile
FROM): versão e licença conformedocs/dependencies.mde especifique na descrição do PR. -
Migrações novas (se houver) devem ser executáveis e reversíveis.
-
Documentação: README/docs/ADR (
docs/adr/) atualizada conforme as mudanças; novos termos de domínio em CONTEXT.md e nomes no código consistentes com CONTEXT.md. -
Revise conforme a skill
review(Claude Code: subagentereviewer; outros agentes: leia.agents/skills/review/SKILL.mde revise seu próprio diff) e corrija os itens "deve corrigir" antes. -
Mensagem de commit seguindo Conventional Commits e branch não sendo
main. -
Descrição do PR contém
Closes #<issue>(se não houver issue, crie uma antes) e seção## CHANGELOG(categoria + 1 linha em português na perspectiva do usuário ou-) — não editeCHANGELOG.mddiretamente (docs/release.md). -
Se mexer na UI: sem textos hardcoded, chaves completas para todos os idiomas, sem valores de cor diretos, visual consistente em
th/en× light/dark (docs/ui.md).
Se algum item não passar, não abra o PR. Corrija antes ou reporte o que está travando.