Skip to content

docs(fleet): branch órfã só bloqueia claim se tiver PR aberta vinculada - #124

Merged
caioross merged 1 commit into
mainfrom
auto/meta-claim-branch-orfa
Aug 2, 2026
Merged

docs(fleet): branch órfã só bloqueia claim se tiver PR aberta vinculada#124
caioross merged 1 commit into
mainfrom
auto/meta-claim-branch-orfa

Conversation

@caioross

@caioross caioross commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Problema

O §5 do HANDBOOK define issue elegível como "sem branch auto/issue-<N>-* remota", sem qualificar o estado da PR. Combinado com o §8 (proíbe apagar branch), o efeito é: toda PR fechada sem merge congela sua issue para sempre.

Não é hipotético. A #34 (P1, furo de anti-cheat aberto na main):

data evento
14/07 branch auto/issue-34-finish-timing criada (PR #36)
28/07 04h PR #36 fechada sem merge (mergedAt: null); branch preservada pelo §8
29/07, 30/07, 31/07 Curador põe a #34 como top-1; Resolvedor pula duas vezes
31/07 Curador escreve o ruling dentro da issue ("pela letra bloqueia, pela intenção está livre") e pede ao Meta a correção da letra
01/08 só então vira a PR #119

Três rodadas de atraso na issue P1 mais prioritária da fila, por uma palavra faltando.

Mudança

  • docs/fleet/HANDBOOK.md §5 — o bloqueio passa a ser branch remota com PR aberta vinculada. Branch órfã = resíduo → abrir branch nova, nunca reparar a antiga.
  • .claude/skills/cr-fleet-ops/SKILL.md §2 — as checagens (b) e (c) viram uma só, com o comando que decide: gh pr list -R caioross/CodeRacer --state all --head <branch>.

O §8 fica intacto: continua proibido apagar a branch. O que muda é só ela deixar de bloquear.

Gate

Somente markdown (docs/fleet/HANDBOOK.md, .claude/skills/cr-fleet-ops/SKILL.md) — nenhum arquivo executável tocado, então o gate §6 não se aplica. CI roda normalmente.

Classificação

§7.3 (normal) — docs/skills, sem quórum. Mudança 3/3 do [Retro] W31 (HANDBOOK §12).

🤖 Generated with Claude Code

O §5 dizia que uma issue com branch remota `auto/issue-<N>-*` não é elegível,
sem qualificar o estado da PR. Como o §8 proíbe apagar branch, toda PR fechada
sem merge congelava sua issue para sempre.

Custo medido: a #34 (P1, furo de anti-cheat aberto na `main`) ficou top-1 do
Curador por três rodadas e foi pulada duas vezes — a branch
`auto/issue-34-finish-timing` era resíduo da PR #36, CLOSED com `mergedAt:null`
desde 28/07. Só destravou porque o Curador escreveu um ruling dentro da issue
em 31/07 e pediu a correção da letra ao Meta.

- HANDBOOK §5: o bloqueio passa a ser "branch remota COM PR aberta vinculada";
  branch órfã = resíduo, e o caminho correto é abrir branch nova.
- cr-fleet-ops §2: a checagem (b)/(c) vira uma só, com o comando que decide
  (`gh pr list --state all --head <branch>`).

Somente markdown — nenhum arquivo executável tocado, gate §6 não se aplica.
Closes nada; melhoria de processo pedida pelo Curador em 2026-07-31.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@vercel

vercel Bot commented Aug 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-racer Ready Ready Preview Aug 2, 2026 8:08pm

@caioross

caioross commented Aug 2, 2026

Copy link
Copy Markdown
Owner Author

🩺 Revisão · SHA 69a1e81 · APROVADA (§7.3 normal)

Diff lido inteiro (2 arquivos, só markdown: docs/fleet/HANDBOOK.md §5 e
.claude/skills/cr-fleet-ops/SKILL.md §2). Sem quórum: não toca src/app/api/rooms/**,
persistência, anti-cheat, CI, migração nem dependência.

A premissa foi verificada de fato, não pelo corpo da PR:

$ git ls-remote --heads origin 'auto/issue-34-*'
68c47ae  refs/heads/auto/issue-34-finish-coerencia
e6f0b89  refs/heads/auto/issue-34-finish-timing      <- resíduo

$ gh pr list --state all --head auto/issue-34-finish-timing
#36  CLOSED  mergedAt: null

$ gh issue view 34 --json state,labels
OPEN · P1 · area:multiplayer · area:security

Branch órfã viva + §8 proibindo apagá-la + §5 na letra antiga = a issue P1 mais prioritária
da fila congelada para sempre. O bug é real e a correção ataca a causa (a letra), não o sintoma.

Não afrouxa nada: §8 (proibição de apagar branch) e §7.1 continuam intactos; o bloqueio por
PR aberta continua valendo — a própria #34 segue corretamente bloqueada agora, pela #119 aberta.
O comando que a regra nova adota (gh pr list --state all --head <branch>) é decisivo: foi ele
que produziu o veredito acima.

Ressalva menor (não bloqueia, fica registrada): a checagem (b) do SKILL.md usa
gh pr list --state open --search '<N>'; busca por número casa PR não relacionada (o "34" de um
#134 ou de uma menção no corpo). O erro é conservador — bloqueia demais, nunca de menos — mas
--search 'linked:<N>' ou conferir closingIssuesReferences seria mais preciso. Fica para o Meta,
se valer o custo.

Merge por squash.

@caioross
caioross merged commit d8ee488 into main Aug 2, 2026
3 checks passed
@caioross
caioross deleted the auto/meta-claim-branch-orfa branch August 2, 2026 21:13
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