O que OpenClaw PR Maintainer faz?
- Operação apenas no issue/PR solicitado.
AGENTS.mdraiz: reparo, produto, confiança na fonte, política de aprovação. Esta habilidade: fluxo de trabalho do GitHub e autoridade de mantenedor. - Referência colada não é permissão de publicação ou expansão de escopo.
Autoridade do mantenedor
mainvermelho é uma emergência, não um pré-requisito de integração não relacionado. Envie correções comprovadas demainvermelho diretamente paramaindentro do escopo de reparo. Para uma instrução de mantenedor com escopo definido para integrar apesar de CI não relacionada/pré-existente, selecione a exceção de falha herdada; não exija redação especial, renovação de aprovação ou reparo/re-execução da CI principal não relacionada primeiro. Autorização humana, evidência de falha e capacidade de execução do escritor protegido são verificações separadas.- A autoridade de reparo e integração cobre agendamento interno, admissão em banco de dados, design de ciclo de vida, risco, verificação. Correções de bugs dentro da tarefa (incluindo correções compatíveis de SDK) e reparos com backup removendo apenas dados inválidos ou inacessíveis não precisam de renovação de aprovação.
- Fechar/reabrir em lote acima de 50 itens precisa de contagem e escopo explícitos.
- Fluxos de trabalho consultivos precisam de solicitação de segurança explícita.
- Atualizações de equipe instruídas por Peter usam o deployer de release; nunca edite estado ativo fora de seu proprietário. Notifique a Vigília Noturna a cada deploy/reinicialização; ela coordena o tempo de reinicialização, não a aprovação da atualização.
Selecione o escopo
- Revisão/Triagem/Listagem: somente leitura; explique mudança/descobertas/prova/incerteza. Edições, atribuição, rótulos, comentários, fechamentos precisam de autoridade.
- Apenas correção: investigue/repare/valide localmente; a publicação precisa de 'ship'/'land' ou equivalente com escopo.
- Integrar/Enviar/Reparo autônomo: prova da fonte → revisão → submissão nativa → merge verificado. Acompanhe solicitações pendentes; repare CI/conflitos causados pelo PR sob a mesma autoridade, direcione CI herdada atribuída através da exceção acima. Auto-merge aceito não é igual a concluído. Preserve crédito/trabalho não relacionado; portões raiz permanecem, nenhuma permissão de rotina repetida.
- Fila/Descoberta: leia triagem. Leituras ativas em lote/descoberta compartilhada; PR nomeado não é varredura de fila.
- Atribuição de autor/regressão: pesquisa de autor e proveniência; idade/histórico da conta condicional.
- Líder prático; trabalhadores úteis limitados, checkout/referências serializados + escritas no GitHub. Nenhuma tarefa de aplicativo de propriedade do usuário escondendo subtarefas internas. Preserve patches/reivindicações/prova de trabalhadores travados antes de assumir/reatribuir.
Inspecione o item real
- Comece com
git status -sb; preserve trabalho não relacionado.gitcrawllocal → PATHghx, JSON ativo estreito. Arquivo desatualizado/ausente →ghx, sem sincronização ampla. Fonte do PR:ghx pr view/diff+ checkout, não pesquisa web.
ghx pr view <pr> --repo openclaw/openclaw \
--json number,title,state,body,author,assignees,baseRefName,headRefName,headRefOid,files,comments,reviews
ghx issue view <issue> --repo openclaw/openclaw \
--json number,title,state,body,author,assignees,comments
- Atribuição: triagem, antes de trabalho profundo. Mencione outros responsáveis; preserve a propriedade. Pesquisa de conta/atividade condicional.
Estabeleça o resultado
- Leia proprietário, pontos de entrada, chamadores, irmãos, testes, docs, histórico, main atual; inspecione contratos de dependência determinantes. Codex: inspeção pessoal da fonte raiz. Evidência contrária → investigue antes de defender. Nenhuma matriz/alternativa inventada.
- Já corrigido: prove reparo no main atual; commit/PR canônico + fonte correspondente/prova executável. Autoridade de fechamento → comente evidência + feche; caso contrário, reporte.
- Defeito real: reproduza → repare invariante proprietário → prove caminho/irmãos. Simplificação em vez de guarda de sintoma; tamanho/LOC não é portão de correção.
- PR útil: melhore branch editável; inseguro → substituição creditada autorizada antes do fechamento. Ancestralidade substituída → substituição de propriedade do mantenedor, nunca esmague PR de contribuidor.
- Incerteza/Decisão de produto: declare lacuna; nenhum merge especulativo/fechamento automático fora de escopo.
- Prova + rota de confiança na fonte/anfitrião:
$openclaw-testing, integração. Prova focada → PR pronto; CI do PR = prova ampla. Prefira prova local limitada em um anfitrião qualificado e disponível. Prova remota deliberada pode cobrir verificações caras ou lacunas de ambiente; filas → pare alocação de tarefa própria, use CI do PR ou escopo local seguro, preserve requisitos descobertos. Nunca execute fonte não confiável localmente. Lacunas ativas opcionais → prova de limite significativo + limitação; prova ativa, API externa, risco autenticado obrigatória. Mocks/skips/head antigo não é igual a ativo. - Mudanças comportamentais: contrato de prova de limite +
$test-audit. Comportamento solicitado e padrões do repositório: verifique separadamente.
Revisão de CODEOWNERS
CODEOWNERSroteia revisão; inspecione a aplicação ativa. Caminhos restritos/de segurança + mudanças materiais de produto/comportamento/segurança/propriedade precisam de envolvimento do proprietário listado.- Governança de propriedade/revisão: direção verificada de administrador de organização ativa também qualifica; administrador de repositório/bypass sozinho não. Nenhuma rota dispensa revisões aplicadas.
Revise e publique
-
Antes de commitar ou integrar código não trivial: revisão nova através de fluxo de trabalho permitido; resolva descobertas acionáveis a menos que o usuário opte por não fazê-lo. Acionável = provável ou de alto impacto; recuse casos de borda improváveis e recuperáveis cuja correção adiciona complexidade desproporcional, com justificativa de uma linha.
-
Commit/integração de código não trivial: use a rota de revisão local selecionada e resolva descobertas acionáveis verificadas. Revisão nova independente obrigatória ainda se aplica. Apenas revisão: passagem independente somente quando solicitada, sem passagem CLI extra obrigatória.
-
Reabra revisão para mudanças substantivas/preocupações não resolvidas, não rebase idêntica de patch/mudança mecânica de head. Aborde descobertas reais de humanos/bots; explique rejeições. Pontuações de bot/listas de Rank-up não criam obrigações separadas. ClawSweeper consultivo; nunca espere comentário para integrar.
-
Template de PR atual: problema/impacto simples; inventários em diff/detalhes. Mantenha riscos, migrações, ações, prova, lacunas, crédito atualizados; explique trade-offs materiais. Nenhum benefício de usuário interno inventado ou ensaios universais de LOC/proveniência/correção alternativa.
-
CHANGELOG.mdde propriedade da release; contexto de nota de release em PR/commit. Transcrições de agente apenas mediante solicitação explícita. -
Mutação pública: verifique autor/committer + escritor. Use
ghxcom payloads reais--body-file <caminho>, nunca stdin (--body-file -,--input -ou/dev/stdin); o proxy não encaminha stdin. Leia os corpos alterados de volta comghx --no-cachee verifique bytes exatos não vazios. Escopo autorizado apenas; atualize comment/corpo existente, sem duplicatas. -
Publicação/merge: leia integração (mídia, CI de head exato, fluxo de trabalho nativo).
-
Verifique merge/fechamento remoto + fonte. Reporte problema, mudança proprietária, prova/lacunas, crédito, estado final vinculado em prosa; apenas acompanhamentos válidos.
Finalize e limpe
- Nunca remova worktree que contém o cwd da sessão ativa, incluindo o seu ou subdiretórios. Sobrescrita
cd/cwd filho não move o pai. Retenha até que o proprietário saia/libère o cwd; reporte o caminho. - Resultado verificado → remova logs de tarefa, recibos, arquivos de prova,
.crabbox, rascunho, worktrees finalizados via encerramento nativo. Nenhuma retenção/arquivo/exportação/transferência apenas para remoção. - Remova worktrees criados quando o trabalho/PR for integrado, a menos que imediatamente necessário em seguida; nunca remova um worktree com cwd ativo.
- Cada tarefa: descobertas no chat/stdout; arquivos apenas para necessidades de entregáveis/ferramenta/prova/recuperação, declare propósito/reúso. Prova pré-merge obrigatória; acompanhamentos opcionais nunca seguram a conclusão.
- Preserve entregáveis/retenção solicitados, fonte inacabada + estado de recuperação necessário, proprietários ativos/ativos/desconhecidos, credenciais, estado do agente, dependências compartilhadas.