metyatech/skill-pr-review-workflow

Flujo de trabajo de retroalimentación de revisión de PR

Abordar la retroalimentación de revisión

• Después de abordar la retroalimentación de una revisión de PR, el agente DEBE resolver los hilos de revisión correspondientes antes de concluir. Si el agente no tiene permiso, DEBE indicarlo explícitamente.
• Antes de volver a solicitar una revisión, el agente DEBE ejecutar el conjunto de verificación relevante y DEBE resumir los resultados (comandos y resultados) en el comentario o descripción del PR.
• Para la retroalimentación causada por un error originado por el agente o una brecha de razonamiento reutilizable, el agente DEBE evaluar si la prevención de recurrencia pertenece a pruebas, verificaciones, habilidades o reglas. Cuando una actualización de regla o habilidad esté justificada, el agente DEBE realizar la actualización como parte del mismo conjunto de cambios. Cuando no esté justificada, el agente DEBE indicar brevemente por qué la puerta reutilizable más temprana está en otro lugar.

Volver a solicitar revisiones

• Después de enviar correcciones, el agente DEBE volver a solicitar revisión solo a los revisores que publicaron retroalimentación abordada en la ronda actual.
• El agente NO DEBE volver a solicitar revisión a revisores (incluyendo revisores de IA) que no publicaron retroalimentación abordada o que ya indicaron que no hay problemas accionables.
• Si no queda ningún revisor aplicable, el agente DEBE preguntar quién debería revisar a continuación.

Re-revisión de bots de revisión de IA

Cuando los bots de revisión de Codex o Copilot estén configurados para el repositorio, el agente DEBE activar la re-revisión solo para el/los bot(s) que publicaron retroalimentación abordada.

• Para re-revisión de Codex: comenta @codex review en el PR.
• Para re-revisión de Copilot: usa la API de GitHub para eliminar y volver a solicitar el revisor bot copilot-pull-request-reviewer[bot]. El agente NO DEBE basarse en la asignación de revisores basada en la interfaz de usuario.
  • Eliminar: DELETE /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers con cuerpo {"reviewers":["copilot-pull-request-reviewer[bot]"]}.
  • Agregar: POST /repos/{owner}/{repo}/pulls/{pr}/requested_reviewers con cuerpo {"reviewers":["copilot-pull-request-reviewer[bot]"]}.

Gestión de notificaciones de GitHub

• El agente DEBE usar DELETE /notifications/threads/{id} (HTTP 204) para marcar notificaciones como Hecho (elimina la notificación de la bandeja de entrada y la mueve a la pestaña Hecho).
• El agente NO DEBE usar PATCH /notifications/threads/{id}, que marca como leído pero deja la notificación en la bandeja de entrada.
• Después de procesar notificaciones, el agente DEBE eliminar en lote cualquier notificación restante leída pero no hecha usando la misma API DELETE.

Limpieza posterior a la fusión

• Después de completar un PR, el agente DEBE fusionarlo, sincronizar la rama objetivo y eliminar la rama del PR local y remotamente.