skill pr-guardian — o gate de merge¶
| Roda em | model/effort |
Dispara quando |
|---|---|---|
| agente-monitor | opus / high |
PR opened/synchronize em menosjuros/menos-juros |
A pergunta central: "isso é seguro pra mergear, e entrega o que o PRD/issue do Linear pediu?" — tudo que precisa ser visto antes do merge.
O que ela olha, na ordem:
- o PR, reaproveitando a revisão anterior (é incremental);
- CI — só lê, nunca re-executa (
ci-api,ci-crm); - blast radius via skill-graphify — quem depende do que mudou (o passo central);
- revisão sênior do diff; design se toca UI; e o Linear: o código bate com o PRD?
- Sentry, check do skill-testsprite-e2e (lê, não executa), governança de telemetria, sanidade de migration nova;
- veredito explícito no PR + log de auditoria — sempre, inclusive quando aprova.
Duas escolhas que parecem detalhe¶
Não re-executa CI porque re-executar esconde flake: o resultado que importa é o que o time viu. E checa o Linear porque código correto que não faz o que o PRD pediu passa em qualquer review de diff — é o defeito mais caro e o mais invisível.
Migration nova: confere sanidade, não gera rollback.
O par pós-merge dela é o skill-prod-guardian.