Skip to content

Commit e comentario passam pelo comando que os escreve, como a PR - #142

Merged
RicardoAlbuquerquet merged 1 commit into
mainfrom
publicar-passa-pelo-comando
Sep 22, 2026
Merged

RicardoAlbuquerquet merged 1 commit into
mainfrom
publicar-passa-pelo-comando

Conversation

@RicardoAlbuquerquet

Copy link
Copy Markdown
Owner

O que resolve

Mesmo com o estilo ligado, a sessão escreve o commit e o comentário de review
direto, sem passar pelo comando que lê o diff, a linha ou a thread antes. A PR
já era barrada uma vez por sessão; o commit e o comentário não eram.

O que foi feito

  • Primeiro git commit da sessão: barrado uma vez, com o /concise:commit
    no motivo.
  • Primeiro comentário da sessão: o mesmo com o /concise:comment, em
    gh pr comment, gh issue comment e gh pr review --body.
  • Uma negativa por comando: gastar a do commit não cala a da PR na mesma
    sessão.
  • A sessão que já rodou o comando passa direto, como já valia para a PR.
  • Onde muda: skills/concise/hooks/route-hint.sh passa a receber regras
    regex::comando::motivo, e skills/concise/hooks/hooks.json traz as três.
  • Versão: 1.92.0 nos dois manifestos, com entrada no CHANGELOG.md.

Fica de fora: o que é publicado pelo navegador, porque o hook só vê chamada
de ferramenta. Cards de quadro por MCP também ficam fora desta.

Não medido: os evals não rodaram.

Como testar

  1. Rode a suíte dos hooks. Imprime 155 ok, 0 falhas.
bash scripts/test-hooks.sh
  1. Troque os hooks pelos do main e rode de novo. Imprime 148 ok, 7 falhas: comentário, commit e a marca por comando.
git checkout origin/main -- skills/concise/hooks
bash scripts/test-hooks.sh
  1. Volte os hooks da branch.
git checkout HEAD -- skills/concise/hooks

@RicardoAlbuquerquet RicardoAlbuquerquet self-assigned this Sep 22, 2026
@RicardoAlbuquerquet
RicardoAlbuquerquet merged commit a20a6d9 into main Sep 22, 2026
5 of 6 checks passed
@RicardoAlbuquerquet
RicardoAlbuquerquet deleted the publicar-passa-pelo-comando branch September 22, 2026 19:14
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