feat: /revisar a secas saca la tarea de la rama del PR comentado #11

Merged

julio.caicedo merged 1 commit from feat/revisar-sin-numero into main

2026-10-23 12:45:40 +00:00

julio.caicedo commented

2026-10-23 12:44:54 +00:00

Owner

Hasta ahora había que comentar /revisar 35: en issue_comment Forgejo no manda la rama del PR, que es de donde sale el número de la tarea. El número es el de la tarea, no el de otro PR: la revisión siempre corre sobre el PR comentado.

Ahora, si no hay rama en el payload, el paso le pregunta a Forgejo (GET /pulls/{número del PR comentado}, con github.token) y saca el número de la rama con la misma expresión de siempre. Así basta con /revisar.

La rama no se imprime: solo se le sacan los dígitos.
Si la consulta falla, o la rama no tiene número, siguen valiendo /revisar 48 y la línea Tarea: #48.

Probado contra un Forgejo falso: feat/35-… → 35; fix/arreglar-404, una rama con ;rm -rf /, un PR inexistente y la API caída → sin número, y el paso no se rompe; en pull_request sigue saliendo de head.ref.

🤖 Generated with Claude Code

Hasta ahora había que comentar `/revisar 35`: en `issue_comment` Forgejo no manda la rama del PR, que es de donde sale el número de la tarea. El número es el de la **tarea**, no el de otro PR: la revisión siempre corre sobre el PR comentado. Ahora, si no hay rama en el payload, el paso le pregunta a Forgejo (`GET /pulls/{número del PR comentado}`, con `github.token`) y saca el número de la rama con la misma expresión de siempre. Así basta con `/revisar`. - La rama no se imprime: solo se le sacan los dígitos. - Si la consulta falla, o la rama no tiene número, siguen valiendo `/revisar 48` y la línea `Tarea: #48`. Probado contra un Forgejo falso: `feat/35-…` → 35; `fix/arreglar-404`, una rama con `;rm -rf /`, un PR inexistente y la API caída → sin número, y el paso no se rompe; en `pull_request` sigue saliendo de `head.ref`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

julio.caicedo added 1 commit

2026-10-23 12:44:54 +00:00

feat: /revisar a secas saca la tarea de la rama del PR comentado

Go / Test (pull_request) Successful in 34s

Details

335adb8117

En issue_comment el payload no trae la rama, y por eso había que escribir
/revisar 35. Ahora se le pregunta a Forgejo por el PR comentado y el
número sale de su rama, igual que al abrirlo.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

julio.caicedo merged commit bfb46c41f6 into main

2026-10-23 12:45:40 +00:00

julio.caicedo deleted branch feat/revisar-sin-numero

2026-10-23 12:45:40 +00:00

julio.caicedo referenced this pull request from a commit

2026-10-23 12:45:41 +00:00

Merge pull request 'feat: /revisar a secas saca la tarea de la rama del PR comentado' (#11) from feat/revisar-sin-numero into main