| name | pre-pr-review |
| description | Checklist técnico final antes de abrir ou atualizar um PR, com veredito GO/NO-GO. Use quando o usuário disser "revise antes do PR", "posso abrir o PR?" ou "checklist de PR". |
Skill: Pre-PR Review
Objetivo
Consolidar um veredito GO / NO-GO antes de abrir ou atualizar um Pull Request,
verificando diff, testes, secrets e qualidade em uma única passada.
Quando usar
Antes de abrir ou atualizar um PR, como último gate da branch.
Quando NÃO usar
Durante o loop de desenvolvimento — para análise pontual de código use a skill
analise-de-codigo; para revisão de segurança profunda, o subagente security-reviewer.
Inputs esperados
- Branch/diff alvo (padrão: diff da branch atual contra
main).
Workflow
- Diff:
git diff main...HEAD --stat — confira se o escopo bate com a intenção
do PR (arquivos inesperados = achado).
- Gates: rode
bash .claude/scripts/quality/verify-all.sh --fast
(em projetos Java) ou o comando de testes do projeto (ver CLAUDE.md).
Falha = NO-GO automático.
- Secrets: varra o diff por credenciais (
git diff main...HEAD | grep -iE 'password|token|secret|api[_-]?key|BEGIN.*PRIVATE'). Achado = NO-GO automático.
- Revisão: delegue o diff ao subagente
code-reviewer.
- Consolidação: monte o relatório GO/NO-GO por item.
Saída esperada
## Pre-PR Review — <branch>
| Item | Status | Observação |
|---|---|---|
| Escopo do diff coeso | ✅/❌ | |
| Testes (bugfix tem regressão?) | ✅/❌ | |
| Secrets | ✅/❌ | |
| Contrato de API sem breaking change não versionado | ✅/❌/n.a. | |
| Documentação atualizada | ✅/❌/n.a. | |
| Code review | ✅/⚠️/❌ | |
**Veredito: GO | NO-GO** — <justificativa por item bloqueante>
Achados classificados pela matriz references/review-severity-matrix.md:
somente CRÍTICO bloqueia sozinho; ALTO bloqueia se não houver plano de correção.
Segurança
- Nunca rodar comandos destrutivos (rm -rf, git reset --hard, terraform apply,
kubectl delete).
- Nunca expor o valor de secrets encontrados — apenas
arquivo:linha mascarado.