Repository navigation
Conversation
…ovar O STATUS do review é calculado só com os findings da rodada, então uma rodada que traz apenas question e suggestion sai approved-with-* e o 8b oferecia Postar e aprovar mesmo com um request-change ou breaking-change de rodada anterior ainda aberto na PR. Novo Passo 4c calcula OPEN_BLOCKERS (blockers desta rodada mais threads abertas com banner request-change ou breaking-change, exceto as reverificadas como PROCEDE). Com OPEN_BLOCKERS não vazio, o 8b trata a PR como de veredito não-aprovar e o Step 7 avisa o motivo. Atualiza os cinco manifests para 1.46.1. Co-Authored-By: Claude <noreply@anthropic.com>
…bloqueio aberto Sem bloqueio aberto (nesta rodada ou em anterior, de qualquer autor ou bot, após reverificação), o veredito aprovar passa a terminar toda opção de postagem do 8b com a aprovação, em vez de exigir a opção especial Postar e aprovar. O menu vira Prioridades + praise e aprovar, Tudo e aprovar, Só prioridades e aprovar e Não postar; postar sem aprovar é pelo campo livre. O Passo 4c passa a se chamar elegibilidade de aprovação: thread bloqueante de terceiro ou de bot também é reverificada, e só continua bloqueando a que não foi corrigida. Severidade média ou baixa de bot não bloqueia. Co-Authored-By: Claude <noreply@anthropic.com>
grippado
changed the title
3 tasks done
Pagine reviewThreads e comments até o fim e descarte coletas parciais. Falhe fechada a elegibilidade de aprovação quando a coleta não estiver completa. Atualize a CLI, o contrato do review e os cinco manifests para 1.46.2. Co-Authored-By: Codex <noreply@openai.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problema
O
STATUSdoflux:reviewé calculado só com os findings da rodada, e a aprovação só saía pela opção especialPostar e aprovar. Dois efeitos ruins:questionesuggestion, oSTATUSsaiapproved-with-*e o 8b oferecia aprovar mesmo comrequest-changeoubreaking-changede rodada anterior ainda aberto.Mudança
plugins/flux/skills/review/SKILL.md), agora "Elegibilidade de aprovação": calculaOPEN_BLOCKERScom os blockers da rodada mais as threads abertas de rodadas anteriores, de qualquer autor, inclusive bot, com bannerrequest-changeoubreaking-changeou severidade crítica (🔴,critical,high,blocker). Severidade média ou baixa de bot não bloqueia.reviewThreadse cada conexão internacommentssão paginadas até o fim. Se alguma página falhar ou estiver inválida, a coleta parcial é descartada,OPEN_BLOCKERSfica desconhecido eAPPROVE_ELIGIBLEé falso.APPROVE_ELIGIBLEexige coleção completa, PR de terceiros,STATUSaprovável eOPEN_BLOCKERSvazio. Elegibilidade segue como regra normativa no Passo 4c;review-legend.mdaponta para ele.threads indisponiveis; o Passo 7 e o menu do Passo 8 deixam a aprovação indisponível.cli/src/gather.tspagina ambas as conexões e retornathreads: nullcom a degradação canônica se alguma página não puder ser coletada.cli/src/gather.test.tscobre paginação e descarte de resultados parciais.Plano de testes
scripts/check-manifests.sh: 5 manifests em flux@1.46.2.scripts/check-codex-agent-contract.sh: contrato Codex íntegro.node --check cli/src/gather.tsenode --check cli/src/gather.test.ts.git diff --check.bun test src/gather.test.ts, pois Bun não está instalado neste ambiente.claude plugin validate .eclaude plugin validate ./plugins/flux, pois Claude CLI não está instalado.cli/package.jsonnão declara esses scripts.Summary-en: Review paginates all review threads and comments, and withholds approval when collection is incomplete.
Summary-pt: Review pagina threads e comentários integralmente, e bloqueia aprovação quando a coleta está incompleta.
🤖 Generated with Claude Code