Skip to content

[chore] Add commit-message guard (local hook + CI) - #41

Merged
mfg1986 merged 1 commit into
developfrom
feature/repo-guard-ai-coauthors
Jul 17, 2026
Merged

[chore] Add commit-message guard (local hook + CI)#41
mfg1986 merged 1 commit into
developfrom
feature/repo-guard-ai-coauthors

Conversation

@mfg1986

@mfg1986 mfg1986 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Qué

Añade un guard del repo que bloquea líneas de atribución máquina-autor no deseadas en los mensajes de commit, para que no vuelva a colarse ese tipo de crédito en la historia (tras la limpieza reciente de develop).

Cómo — dos capas, sin dependencias nuevas

  • Local: .githooks/commit-msg rechaza el trailer no permitido en el momento del commit. Se comparte con todo el equipo vía core.hooksPath, activado automáticamente por el script prepare de la raíz en pnpm install.
  • CI: guard-commit-messages.yml re-escanea los commits de cada PR en el servidor — la red a prueba de bypass para --no-verify o para quien no tenga el hook instalado.

Verificación

  • Hook local probado: bloquea el trailer no permitido (y su variante solo-email), y deja pasar mensajes limpios y co-autores humanos legítimos.
  • El check de CI corre en este propio PR.

Notas

  • El hook del plugin es por-sesión; esto es del repo, así que aplica a cualquier commit (a mano u otra herramienta), no solo a sesiones con el plugin.
  • Seguridad del workflow: base_ref se pasa por env: y se usa entrecomillado (sin interpolar ${{ }} en el shell).

Blocks disallowed machine-authored attribution lines in commit messages. Two layers, no new deps: a shared commit-msg hook via core.hooksPath (wired by the root prepare script on install), plus a per-PR CI check as the bypass-proof backstop.
@mfg1986
mfg1986 merged commit 67d2dbd into develop Jul 17, 2026
4 checks passed
@mfg1986
mfg1986 deleted the feature/repo-guard-ai-coauthors branch July 17, 2026 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant