Mantenedor de PRs de OpenClaw

¿Qué hace Mantenedor de PRs de OpenClaw?

• Solo la operación solicitada sobre el issue/PR. Archivo raíz AGENTS.md: reparación, producto, confianza en la fuente, política de aprobación. Esta skill: flujo de trabajo de GitHub.
• Referencia pegada ≠ permiso de publicación o expansión del alcance.

Seleccionar el alcance

• Revisar/clasificar/listar: solo lectura; explicar cambio/hallazgos/prueba/incertidumbre. Las ediciones, asignaciones, etiquetas, comentarios y cierres necesitan autorización.
• Solo reparar: investigar/reparar/validar localmente; la publicación necesita envío/fusión o equivalente con alcance definido.
• Fusionar/enviar/reparación autónoma: prueba de fuente → revisión → envío nativo → fusión verificada. Rastrear solicitudes pendientes; reparar CI/conflictos bajo la misma autorización. Fusión automática aceptada ≠ hecho. Preservar atribución/trabajo no relacionado; las compuertas raíz permanecen, sin permiso rutinario repetido.
• Cola/descubrimiento: leer clasificación. Lecturas en vivo por lotes/descubrimiento compartido; PR nombrado ≠ barrido de cola.
• Atribución de autor/regresión: investigación de autor y procedencia; antigüedad/historial de cuenta condicional.
• Lidera de forma práctica; trabajadores útiles acotados, checkouts/refs serializados + escrituras en GitHub. Sin tareas de aplicaciones propiedad del usuario que oculten subtareas internas. Preservar parches/reclamaciones/pruebas de trabajadores estancados antes de la toma de posesión o reasignación.

Inspeccionar el elemento real

• Iniciar con git status -sb; preservar trabajo no relacionado. gitcrawl local → gh en PATH desnudo, JSON en vivo reducido. Archivo obsoleto/faltante → gh, sin sincronización amplia. Fuente del PR: gh pr view/diff + checkout, no búsqueda web.

gh pr view <pr> --repo openclaw/openclaw \
  --json number,title,state,body,author,assignees,baseRefName,headRefName,headRefOid,files,comments,reviews
gh issue view <issue> --repo openclaw/openclaw \
  --json number,title,state,body,author,assignees,comments

• Asignación: clasificación, antes de trabajo profundo. Mencionar a otros asignados; preservar propiedad. Investigación de cuenta/actividad condicional.

Establecer el resultado

• Leer propietario, puntos de entrada, llamantes, hermanos, pruebas, documentación, historial, rama main actual; inspeccionar contratos de dependencia determinantes. Codex: inspección personal de la fuente raíz. Evidencia contraria → investigar antes de defender. Sin matriz/alternativa inventada.
• Ya corregido: probar reparación en main actual; commit/PR canónico + prueba de fuente ejecutable. Autoridad de cierre → comentar evidencia + cerrar; si no, informar.
• Defecto real: reproducir → reparar invariante propietaria → probar ruta/hermanos. Simplificación sobre guarda de síntoma; tamaño/LOC ≠ compuerta de corrección.
• PR útil: mejorar rama editable; inseguro → reemplazo autorizado con atribución antes del cierre. Reemplazo de ascendencia → reemplazo propiedad del mantenedor, nunca aplastar PR de contribuyente.
• Incierto/decisión de producto: indicar vacío; sin fusión especulativa/cierre automático fuera de alcance.
• Prueba + ruta de confianza en la fuente/host: $openclaw-testing, aterrizaje. Prueba enfocada → PR listo; CI del PR = prueba amplia. Preferir prueba local acotada en un host calificado y disponible. La prueba remota deliberada puede cubrir verificaciones costosas o brechas de entorno; colas → detener asignación propiedad de la tarea, usar CI del PR o un ámbito local seguro, preservar requisitos no cubiertos. Nunca ejecutar fuente no confiable localmente. Brechas en vivo opcionales → prueba significativa de límites + limitación; prueba explícita en vivo, API externa, riesgo autenticado obligatoria. Simulaciones/saltos/cabeza antigua ≠ en vivo.
• Cambios de comportamiento: contrato de prueba de límites + $test-audit. Comportamiento solicitado y estándares del repositorio: verificar por separado.

Revisión de CODEOWNERS

• CODEOWNERS dirige la revisión; inspeccionar cumplimiento en vivo. Rutas restringidas/de seguridad + cambios materiales de producto/comportamiento/seguridad/propiedad necesitan la participación del propietario listado.
• gobierno de propiedad/revisión: la dirección verificada del administrador de la organización también califica; el administrador del repositorio/elusión por sí solo no. Ninguna ruta exime de revisiones obligatorias.

Revisar y publicar

• Antes de confirmar o fusionar código no trivial: revisión fresca a través del flujo de trabajo permitido; resolver hallazgos procesables a menos que el usuario opte por no hacerlo. Procesable = probable o de alto impacto; rechazar casos límite improbables y recuperables cuya corrección añada complejidad desproporcionada, con justificación de una línea.

• Confirmación/fusión de código no trivial: $autoreview; resolver hallazgos procesables verificados. Solo revisión: pase independiente solo cuando se solicite, sin pase CLI obligatorio adicional.

• Reabrir revisión para cambios sustanciales/preocupaciones no resueltas, no para rebase sin parches/cambio mecánico de cabeza. Abordar hallazgos reales de humanos/bots; explicar rechazos. Las puntuaciones de bots/listas de clasificación no crean obligaciones separadas. ClawSweeper es consultivo; nunca esperar comentario para fusionar.

• Plantilla actual de PR: problema/impacto claro; inventarios en diff/detalles. Mantener riesgos, migraciones, acciones, pruebas, brechas, atribución actualizados; explicar compensaciones materiales. Sin beneficio inventado solo para usuario interno o ensayos universales de LOC/procedencia/corrección alternativa.

• CHANGELOG.md propiedad del lanzamiento; contexto de nota de lanzamiento en PR/commit. Transcripciones de agente solo bajo solicitud explícita.

• Mutación pública: verificar autor/committer + escritor; archivos de cuerpo sensibles a shell. Solo alcance autorizado; actualizar comentario/cuerpo existente, sin duplicados.

• Publicación/fusión: leer aterrizaje (medios, CI de cabeza exacta, flujo de trabajo nativo).

• Verificar fusión/cierre remoto + fuente. Informar problema, cambio propietario, prueba/brechas, atribución, estado final enlazado en prosa; solo seguimientos que valgan la pena.

Finalizar y limpiar

• Nunca eliminar el worktree que contiene el directorio de trabajo actual de la sesión en vivo, incluido el suyo o subdirectorios. La anulación de cd/cwd hijo no mueve al padre. Conservar hasta que el propietario salga/libere el cwd; informar ruta.
• Resultado verificado → eliminar registros de tarea, recibos, archivos de prueba, .crabbox, borradores, worktrees terminados mediante cierre nativo. Sin retención/archivo/exportación/entrega solo para eliminación.
• Eliminar worktrees creados una vez que el trabajo/PR se fusione, a menos que se necesiten inmediatamente después; nunca eliminar un worktree con cwd en vivo.
• Cada tarea: hallazgos en chat/stdout; archivos solo para entregables/herramienta/prueba/necesidades de recuperación, declarar propósito/reutilización. Prueba previa a la fusión obligatoria; los seguimientos opcionales nunca retrasan la finalización.
• Preservar entregables/retensión solicitados, fuente incompleta + estado de recuperación necesario, propietarios activos/en vivo/desconocidos, credenciales, estado del agente, dependencias compartidas.