codex-review

Codex Review

Execute a revisão de código integrada do Codex como verificação de encerramento. Esta é a revisão de código (codex review), não a roteamento de aprovação auto_review do Guardian.

Use quando:

• o usuário solicitar revisão Codex / autoreview / revisão de segundo modelo
• após edições de código não triviais, antes de final/commit/envio
• revisando um branch local ou branch de PR após correções

Contrato

• Trate a saída da revisão como consultiva. Nunca a aplique cegamente.
• Verifique cada constatação lendo o caminho real do código e arquivos adjacentes.
• Leia a documentação/fontes/tipos de dependências quando a constatação depender de comportamento externo.
• Rejeite casos extremos irreais, riscos especulativos, reescritas amplas e correções que compliquem demais a base de código.
• Prefira correções pequenas no limite de propriedade correto; sem refatoração a menos que melhore claramente a classe de bug.
• Continue até que a revisão Codex não retorne constatações aceitas/acrionáveis.
• Se uma correção acionada pela revisão alterar o código, reexecute os testes focados e reexecute a revisão Codex.
• Nunca altere ou substitua o modelo de revisão. Se a revisão atingir a capacidade do modelo, repita o mesmo comando algumas vezes com o mesmo modelo. O helper executa a revisão aninhada em modo yolo/acesso total por padrão; use --no-yolo apenas ao testar intencionalmente o comportamento da sandbox.
• Pare assim que o comando/helper de revisão sair com 0 e sem constatações aceitas/acrionáveis. Não execute uma codex review direta extra apenas para obter uma linha "limpa" mais bonita, uma segunda opinião ou um texto de encerramento mais claro.
• Trate a saída bem-sucedida do helper mais a ausência de constatações acionáveis como o resultado de revisão limpo, mesmo se a saída subjacente do CLI do Codex for concisa.
• Se rejeitar uma constatação como intencional/não vale a pena corrigir, adicione um breve comentário inline de código apenas quando explicar uma invariante real ou decisão de propriedade que revisores futuros devam saber.
• Não faça push apenas para revisar. Faça push apenas quando o usuário solicitar push/envio/atualização de PR.

Escolher Alvo

Trabalho local sujo:

codex review --uncommitted

Use isto apenas quando o patch estiver realmente não staged/staged/não rastreado no checkout atual. Para trabalho commitado, enviado ou de PR, aponte o Codex para o diff do commit ou branch; não force --mode local / --uncommitted apenas porque a documentação do helper menciona trabalho sujo primeiro. Uma revisão limpa --uncommitted apenas prova que não há patch local.

Trabalho de Branch/PR:

git fetch origin
codex review --base origin/main

Não passe um prompt inline com --base; o CLI atual rejeita --base + [PROMPT] mesmo que o texto de ajuda seja ambíguo. Se instruções personalizadas forem necessárias, execute a revisão base simples primeiro, depois faça uma passagem de acompanhamento local/manual.

Se existir um PR aberto, use sua base real:

base=$(gh pr view --json baseRefName --jq .baseRefName)
codex review --base "origin/$base"

Alteração commitada única:

codex review --commit HEAD

ou com o helper:

/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --mode commit --commit HEAD

Use revisão de commit para trabalho já publicado ou já enviado em main. Revisar main limpo contra origin/main geralmente é um diff vazio após o push. Para uma pequena pilha, revise cada commit explicitamente ou revise o branch antes de mesclar com --base.

Encerramento Paralelo

Formate primeiro se a formatação puder alterar as localizações das linhas. Então está OK executar testes e revisão em paralelo:

scripts/codex-review --parallel-tests "<comando de teste focado>"

Compensação: os testes podem forçar alterações de código que tornam a revisão obsoleta. Se os testes ou a revisão levarem a edições de código, reexecute os testes afetados e reexecute a revisão até que nenhuma constatação aceita/acrionável permaneça. Assim que essa reexecução sair limpa, pare; não gaste outro ciclo longo de revisão em confirmação redundante.

Eficiência de Contexto

A revisão Codex é geralmente ruidosa. Padrão para um filtro de subagente quando subagentes estiverem disponíveis. Peça para executar a revisão e retornar apenas:

• constatações acionáveis que aceita
• constatações que rejeita, com motivo de uma linha
• arquivos/testes exatos para reexecutar

Execute inline apenas para alterações muito pequenas ou quando subagentes estiverem indisponíveis.

Helper

Helper integrado:

~/.codex/skills/codex-review/scripts/codex-review --help

Se instalado de agent-scripts, o caminho é:

/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --help

O helper:

• escolhe --uncommitted sujo primeiro
• caso contrário usa a base atual do PR se gh pr view funcionar
• caso contrário usa origin/main para branches não-main
• use --mode commit --commit <ref> para trabalho já commitado, especialmente main limpo após publicação
• deve ser deixado em --mode auto ou forçado para --mode branch para trabalho de PR/branch; não force --mode local após commitar
• escreve apenas na stdout a menos que --output ou CODEX_REVIEW_OUTPUT esteja definido
• suporta --dry-run, --parallel-tests e refs de commit
• executa revisão aninhada com --dangerously-bypass-approvals-and-sandbox por padrão
• continua aceitando --full-access; use --no-yolo ou CODEX_REVIEW_YOLO=0 para optar por não participar
• imprime codex-review clean: no accepted/actionable findings reported quando o comando de revisão selecionado sai com 0

Relatório Final

Inclua:

• comando de revisão usado
• testes/prova executados
• constatações aceitas/rejeitadas, brevemente por quê
• o resultado limpo da revisão da execução final do helper/revisão, ou por que uma constatação restante foi conscientemente rejeitada

Não execute outra revisão Codex apenas para melhorar a redação do relatório final. Se a execução final do helper saiu com 0 e não produziu constatações aceitas/acrionáveis, relate essa execução exata como limpa.