metyatech/skill-pr-review-workflow

Fluxo de trabalho de feedback de revisão de PR

Tratamento do feedback de revisão

• Após tratar o feedback de uma revisão de PR, o agente DEVE resolver os respectivos tópicos de revisão antes de concluir. Se o agente não tiver permissão, ele DEVE declarar isso explicitamente.
• Antes de solicitar uma nova revisão, o agente DEVE executar a suíte de verificação relevante e DEVE resumir os resultados (comandos e resultados) no comentário ou descrição do PR.
• Para feedback causado por um erro originado pelo agente ou uma lacuna de raciocínio reutilizável, o agente DEVE avaliar se a prevenção de recorrência pertence a testes, verificações, habilidades ou regras. Quando uma atualização de regra ou habilidade for justificada, o agente DEVE fazer a atualização como parte do mesmo conjunto de alterações. Quando não for justificada, o agente DEVE declarar brevemente por que a barreira reutilizável mais próxima está em outro lugar.

Solicitação de novas revisões

• Após enviar correções, o agente DEVE solicitar uma nova revisão apenas dos revisores que postaram feedback tratado na rodada atual.
• O agente NÃO DEVE solicitar nova revisão de revisores (incluindo revisores de IA) que não postaram feedback tratado ou que já indicaram não ter problemas acionáveis.
• Se não restar nenhum revisor aplicável, o agente DEVE perguntar quem deve revisar em seguida.

Re-revisão de bot de revisão de IA

Quando bots de revisão Codex ou Copilot estiverem configurados para o repositório, o agente DEVE acionar a re-revisão apenas para o(s) bot(s) que postou(aram) feedback tratado.

• Para re-revisão do Codex: comente @codex review no PR.
• Para re-revisão do Copilot: use a API do GitHub para remover e solicitar novamente o revisor bot copilot-pull-request-reviewer[bot]. O agente NÃO DEVE depender de atribuição de revisor baseada em interface.
  • Remover: DELETE /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers com corpo {"reviewers":["copilot-pull-request-reviewer[bot]"]}.
  • Adicionar: POST /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers com corpo {"reviewers":["copilot-pull-request-reviewer[bot]"]}.

Gerenciamento de notificações do GitHub

• O agente DEVE usar DELETE /notifications/threads/{id} (HTTP 204) para marcar notificações como concluídas (remove a notificação da caixa de entrada e a move para a aba Concluído).
• O agente NÃO DEVE usar PATCH /notifications/threads/{id}, que marca como lida mas deixa a notificação na caixa de entrada.
• Após processar notificações, o agente DEVE excluir em massa quaisquer notificações restantes marcadas como lidas mas não concluídas, usando a mesma API DELETE.

Limpeza pós-merge

• Após concluir um PR, o agente DEVE fazer o merge, sincronizar a branch de destino e excluir a branch do PR local e remotamente.