Pular para conteúdo

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:

  1. o PR, reaproveitando a revisão anterior (é incremental);
  2. CI — só lê, nunca re-executa (ci-api, ci-crm);
  3. blast radius via skill-graphify — quem depende do que mudou (o passo central);
  4. revisão sênior do diff; design se toca UI; e o Linear: o código bate com o PRD?
  5. Sentry, check do skill-testsprite-e2e (lê, não executa), governança de telemetria, sanidade de migration nova;
  6. 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.