startcodex/ci
Template
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: el revisor comenta la revisión en el pull request #8

Merged
julio.caicedo merged 1 commit from feat/revisor-comenta-en-el-pr into main 2026-10-23 12:37:58 +00:00
Conversation 0 Commits 1 Files changed 2 +39 -3
julio.caicedo commented 2026-10-23 12:36:45 +00:00
Owner
Copy link

Hasta ahora el revisor dejaba el borrador solo en el log y en el artefacto review-draft, y nadie lo veía en el PR.

  • Paso nuevo Comment on pull request: publica el borrador como comentario con github.token. El JSON lo arma node (ya está por actions/checkout), así que el texto de los modelos nunca pasa por el shell. Si Forgejo rechaza el comentario, se imprime solo el código de respuesta.
  • Sin número de HU no hay borrador y no se comenta nada; queda el aviso de siempre.
  • set -o pipefail en Run review. Sin eso, go run ... | tee tapaba los errores del revisor: el paso salía verde aunque fallara al leer la tarea o al llamar a los modelos.

Probado localmente con un servidor falso: POST a /issues/20/comments con el cuerpo intacto (comillas, $HOME); con borrador vacío no llama.

Pendiente de ver en Forgejo de verdad: que github.token tenga permiso para comentar. Se prueba con /revisar 35 en balanzio/api#20 después del merge, porque balanzio usa la plantilla de @main.

🤖 Generated with Claude Code

Hasta ahora el revisor dejaba el borrador solo en el log y en el artefacto `review-draft`, y nadie lo veía en el PR. - Paso nuevo **Comment on pull request**: publica el borrador como comentario con `github.token`. El JSON lo arma node (ya está por `actions/checkout`), así que el texto de los modelos nunca pasa por el shell. Si Forgejo rechaza el comentario, se imprime solo el código de respuesta. - Sin número de HU no hay borrador y no se comenta nada; queda el aviso de siempre. - **`set -o pipefail` en Run review.** Sin eso, `go run ... | tee` tapaba los errores del revisor: el paso salía verde aunque fallara al leer la tarea o al llamar a los modelos. Probado localmente con un servidor falso: POST a `/issues/20/comments` con el cuerpo intacto (comillas, `$HOME`); con borrador vacío no llama. Pendiente de ver en Forgejo de verdad: que `github.token` tenga permiso para comentar. Se prueba con `/revisar 35` en balanzio/api#20 después del merge, porque balanzio usa la plantilla de `@main`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
feat: el revisor comenta la revisión en el pull request
All checks were successful
Go / Test (pull_request) Successful in 34s
Details
f3bddd3296 
El borrador se publica como comentario con el token del job. Además, el
paso del revisor corre con pipefail: el tee tapaba sus errores y el paso
quedaba en verde aunque la revisión hubiera fallado.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
julio.caicedo deleted branch feat/revisor-comenta-en-el-pr 2026-10-23 12:37:58 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
startcodex/ci!8
No description provided.