Skip to content

fix(github): permitir reviews disparados por bots autorizados - #1

Merged
AdolfoVillalobos merged 3 commits into
fintocfrom
trusted-app-review
Sep 22, 2026
Merged

AdolfoVillalobos merged 3 commits into
fintocfrom
trusted-app-review

Conversation

@AdolfoVillalobos

@AdolfoVillalobos AdolfoVillalobos commented Sep 21, 2026 •

Copy link
Copy Markdown
Collaborator

Contexto

Los agents deben poder recibir el AI review al actualizar un PR y volver a pedirlo con fin review. Hoy OpenCode consulta sus permisos como colaboradores del repo y recibe none, aunque sean GitHub Apps autorizadas.

Qué hay de nuevo?

  • Se agrega allowed_bots: una lista de nombres como fin-tank-agent[bot], configurada desde Hermes. Se valida que GitHub identifique al actor como bot y que esté en la lista. Cubre eventos de PR y comentarios; humanos y otros bots mantienen el chequeo actual. Por default la lista está vacía.
  • El action instala una versión fija de nuestro CLI y verifica checksum y versión. El original descarga el último CLI oficial: cambiar solo el action no basta porque el rechazo ocurre dentro del CLI.
  • Se agregan workflows para publicar el binario y proponer actualizaciones semanales desde OpenCode. Las actualizaciones pasan tests y compilación antes de abrir un PR; no se mergean ni publican automáticamente.

Tests

  • 56 tests de autorización, instalación y comportamiento existente de GitHub; typecheck y lint locales.
  • Review real solicitado por una App al conectar Hermes.

Consideraciones

El motor, prompt y flujo de review siguen siendo los originales. La diferencia con upstream está en la autorización y distribución del CLI. Distribuimos Linux X64, sin UI web.

Falta mergear, publicar el binario y apuntar Hermes al nuevo action con allowed_bots. Este PR no cambia los triggers ni activa el fork en producción.

@github-actions

Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No issue referenced. Please add Closes #<number> linking to the relevant issue.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@AdolfoVillalobos AdolfoVillalobos changed the title fix(github): allow trusted apps to request AI reviews fix(github): permitir reviews solicitados por GitHub Apps Sep 21, 2026
@AdolfoVillalobos AdolfoVillalobos changed the title fix(github): permitir reviews solicitados por GitHub Apps fix(github): permitir reviews disparados por bots autorizados Sep 22, 2026
@AdolfoVillalobos
AdolfoVillalobos marked this pull request as ready for review September 22, 2026 20:04
@AdolfoVillalobos
AdolfoVillalobos merged commit 327a105 into fintoc Sep 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant