Buenas prácticas
Equipos sin revisión de código
Si los cambios llegan a main sin que nadie los lea, porque los agentes hacen commit directamente en main o los pull requests se fusionan en cuanto pasa la CI, deja que HeyDeer sea la revisión. Así, un pull request se fusiona por sí solo en cuanto HeyDeer lo aprueba.
El riesgo
La CI solo detecta lo que cubren tus pruebas. Sin una revisión, se cuelan dos tipos de problemas:
- Sesgo del modelo. El modelo que escribió el código tiende a dar por bueno su propio razonamiento y a pasar por alto sus propios errores, incluso cuando le pides que revise su trabajo.
- Errores por ediciones repetidas. Cuando un agente modifica el mismo código una y otra vez, cada cambio puede romper sin que nadie lo note algo que uno anterior había dejado bien.
Qué aporta HeyDeer
- Una opinión independiente. HeyDeer revisa cada pull request por separado del agente que lo escribió y verifica cada hallazgo con el código antes de publicarlo.
- Un proceso de revisión dedicado. Cada push recibe una revisión, las revisiones de seguimiento vuelven a comprobar los problemas anteriores y la comprobación HeyDeer Review registra el resultado en cada commit.
Configuración recomendada
Un agente abre un pull request
HeyDeer revisa cada push
La comprobación se supera y HeyDeer aprueba
GitHub fusiona el pull request
Si HeyDeer encuentra problemas, la comprobación falla y no se fusiona nada. El agente hace push de una corrección y HeyDeer revisa los nuevos commits.
- Abre un pull request para cada cambio. Haz que los agentes hagan push a una rama y abran un pull request en lugar de hacer commit directamente en main.
- Haz que HeyDeer revise cada push. En Configuración del espacio de trabajo → Reglas de revisión, establece “Frecuencia de revisión” en “En cada push”. Consulta Revisiones automáticas.
- Haz que la comprobación falle si hay problemas. En Publicar resultados, activa “Marcar la comprobación como fallida si se detectan problemas”. Consulta La comprobación HeyDeer Review.
- Activa la aprobación automática. En Publicar resultados, activa “Aprobación automática”. HeyDeer aprueba un pull request cuando una revisión completa no deja ningún problema. Los cambios importantes de arquitectura y comportamiento clave siguen esperando a una persona; para aprobarlos también, establece “Cambios que requieren aprobación humana” en “Sin restricciones adicionales”. Consulta Aprobación automática.
- Protege main en GitHub. En las reglas de rama de main, activa Require a pull request before merging con 1 aprobación requerida, y Dismiss stale pull request approvals when new commits are pushed. Después, activa Require status checks to pass before merging y añade HeyDeer Review junto a tus comprobaciones de CI.
- Fusiona automáticamente. Activa Allow auto-merge en la configuración de GitHub del repositorio. Después, activa la fusión automática (auto-merge) en cada pull request o haz que el agente ejecute
gh pr merge --auto --squashdespués de abrirlo.
HeyDeer aprobó estos cambios
Se superaron todas las comprobaciones
HeyDeer Review — Revisión completada: no se encontraron problemasObligatoria
CI / test — CorrectaObligatoria
La fusión automática está activada: GitHub fusionará este pull request cuando se cumplan todos los requisitos.
Escribir tus estándares como comprobaciones
Los agentes siguen una regla con más fiabilidad cuando la revisión la hace cumplir. Añade un archivo Markdown a .agents/checks/ por cada regla, y HeyDeer la aplicará a cada pull request que modifique los archivos coincidentes. Consulta Comprobaciones del repositorio.
.agents/checks/tenant-scope.md
---
files:
- "src/db/**/*.ts"
- "!**/*.test.ts"
---
Every query that reads or writes customer data must be
scoped to one tenant, taken from the request context and
never from request parameters.