fix: si ningún modelo responde, el bot dice que no revisó y el paso falla #14

Merged

julio.caicedo merged 1 commit from fix/sin-modelos-no-es-revision into main

2026-10-23 14:50:34 +00:00

julio.caicedo commented

2026-10-23 14:49:48 +00:00

Owner

Sugerencia de Joiner, tras el /revisar 35 en balanzio/api#20: los cuatro modelos respondieron HTTP 402 Insufficient credits y el bot publicó «0 propuestos». Eso se lee como una revisión limpia, aunque nadie miró el código.

Si ningún modelo responde, el programa escribe un aviso: «No se revisó: ningún modelo respondió. Esto no es una aprobación», con el motivo de cada modelo. Después sale con error.
Post review corre también si el paso del revisor falló (success() || failure()), así el aviso llega al PR mientras la ejecución queda en rojo. Si el fallo fue antes de escribir nada (tarea, tablero), el borrador está vacío y no se publica nada, como antes.
writeReview junta la escritura del JSON, que ahora tiene dos caminos.

TestNotReviewedIsNotAnApproval comprueba que el aviso diga que no se revisó y por qué, y que no traiga un conteo.

🤖 Generated with Claude Code

Sugerencia de Joiner, tras el `/revisar 35` en balanzio/api#20: los cuatro modelos respondieron `HTTP 402 Insufficient credits` y el bot publicó «0 propuestos». Eso se lee como una revisión limpia, aunque nadie miró el código. - Si ningún modelo responde, el programa escribe un aviso: **«No se revisó: ningún modelo respondió. Esto no es una aprobación»**, con el motivo de cada modelo. Después sale con error. - **Post review** corre también si el paso del revisor falló (`success() || failure()`), así el aviso llega al PR mientras la ejecución queda en rojo. Si el fallo fue antes de escribir nada (tarea, tablero), el borrador está vacío y no se publica nada, como antes. - `writeReview` junta la escritura del JSON, que ahora tiene dos caminos. `TestNotReviewedIsNotAnApproval` comprueba que el aviso diga que no se revisó y por qué, y que no traiga un conteo. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

julio.caicedo added 1 commit

2026-10-23 14:49:48 +00:00

fix: si ningún modelo responde, el bot dice que no revisó y el paso falla

Go / Test (pull_request) Successful in 34s

Details

2df4f362e2

Con los cuatro modelos sin créditos (HTTP 402) el borrador decía
«0 propuestos», que se lee como una revisión limpia. Ahora publica un
aviso con el motivo de cada modelo y la ejecución queda en rojo.
Sugerencia de Joiner.

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

julio.caicedo merged commit b50b485c8b into main

2026-10-23 14:50:34 +00:00

julio.caicedo deleted branch fix/sin-modelos-no-es-revision

2026-10-23 14:50:34 +00:00

julio.caicedo referenced this pull request from a commit

2026-10-23 14:50:35 +00:00

Merge pull request 'fix: si ningún modelo responde, el bot dice que no revisó y el paso falla' (#14) from fix/sin-modelos-no-es-revision into main