Mainteneur de PR OpenClaw

Que fait Mainteneur de PR OpenClaw ?

• Opération demandée sur issue/PR uniquement. Racine AGENTS.md : réparation, produit, confiance-sourcée, politique d'approbation. Ce skill : workflow GitHub.
• Référence collée ≠ permission de publication ou extension de périmètre.

Sélectionner le périmètre

• Review/triage/liste : lecture seule ; expliquer le changement/constatations/preuve/incertitude. Les modifications, affectation, labels, commentaires, fermetures nécessitent une autorité.
• Réparation uniquement : investiguer/réparer/valider localement ; la publication nécessite un ship/land ou un équivalent dans le périmètre.
• Land/ship/réparation autonome : preuve source → revue → soumission native → fusion vérifiée. Suivre les demandes en attente ; réparer CI/conflits sous la même autorité. Auto-merge accepté ≠ terminé. Préserver le crédit/les travaux non liés ; les barrières racines restent, pas de permission de routine répétée.
• File d'attente/découverte : lire triage. Lectures en direct par lots/découverte partagée ; PR nommée ≠ balayage de file.
• Attribution auteur/régression : recherche d'auteur et de provenance ; âge/historique du compte conditionnel.
• Meneur pratique ; workers utiles limités, checkout/réfs sérialisés + écritures GitHub. Pas de tâches d'applications utilisateur cachant des sous-tâches internes. Préserver les patches/réclamations/preuves des workers bloqués avant reprise/réaffectation.

Inspecter l'élément réel

• Démarrer git status -sb ; préserver les travaux non liés. gitcrawl local → chemin nu gh, JSON en direct limité. Archive obsolète/manquante → gh, pas de synchronisation large. Source PR : gh pr view/diff + checkout, pas de recherche 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

• Affectation : triage, avant travail approfondi. Mentionner les autres assignés ; préserver la propriété. Recherche de compte/activité conditionnelle.

Établir le résultat

• Lire propriétaire, points d'entrée, appelants, frères, tests, docs, historique, main actuel ; inspecter les contrats de dépendance déterminants. Codex : inspection personnelle de la source racine. Preuve contraire → investiguer avant de défendre. Aucune matrice/alternative inventée.
• Déjà corrigé : prouver la réparation dans le main actuel ; commit/PR canonique + preuve source/exécution correspondante. Autorité de fermeture → commenter la preuve + fermer ; sinon rapporter.
• Vrai défaut : reproduire → réparer l'invariant propriétaire → prouver le chemin/frères. Simplification plutôt que garde de symptôme ; taille/LOC ≠ porte de correction.
• PR utile : améliorer la branche éditable ; non sûr → remplacement autorisé crédité avant fermeture. Ancêtre remplacé → remplacement appartenant au mainteneur, ne jamais squasher une PR de contributeur.
• Incertain/décision produit : indiquer le manque ; pas de fusion spéculative/fermeture automatique hors périmètre.
• Preuve + chemin confiance-sourcée/hôte : $openclaw-testing, landing. Preuve ciblée → PR prête ; CI de PR = preuve large. Préférer une preuve locale limitée sur un hôte qualifié et disponible. Une preuve distante délibérée peut couvrir des vérifications coûteuses ou des lacunes d'environnement ; files d'attente → arrêter l'allocation propre à la tâche, utiliser CI de PR ou un périmètre local sûr, préserver les exigences non couvertes. Ne jamais exécuter de source non fiable localement. Lacunes optionnelles en direct → preuve significative aux limites + limitation ; preuve explicite en direct, API externe, risque authentifié obligatoire. Mocks/sauts/ancienne tête ≠ direct.
• Changements comportementaux : contrat de preuve aux limites + $test-audit. Comportement demandé et normes du dépôt : vérifier séparément.

Review CODEOWNERS

• CODEOWNERS achemine la revue ; inspecter l'application en direct. Chemins restreints/sécurité + changements matériels de produit/comportement/sécurité/propriété nécessitent l'implication du propriétaire listé.
• gouvernance de propriété/review : une directive vérifiée d'un administrateur d'organisation actif est également qualifiée ; l'administrateur de dépôt/le contournement seul ne l'est pas. Aucune des deux voies ne renonce aux revues imposées.

Review et publication

• Avant de commiter ou fusionner du code non trivial : nouvelle revue via le workflow autorisé ; résoudre les conclusions exploitables sauf si l'utilisateur s'y oppose. Actionnable = probable ou à fort impact ; refuser les cas limites improbables et récupérables dont la correction ajoute une complexité disproportionnée, avec une raison d'une ligne.

• Commit/fusion de code non trivial : $autoreview ; résoudre les conclusions exploitables vérifiées. Review uniquement : passage indépendant seulement si demandé, pas de passage CLI obligatoire supplémentaire.

• Rouvrir la revue pour des changements substantiels/préoccupations non résolues, pas pour un rebase identique au patch/changement mécanique d'en-tête. Répondre aux conclusions humaines/réelles du bot ; expliquer les rejets. Les scores de bot/classements ne créent pas d'obligations distinctes. Avis de ClawSweeper ; ne jamais attendre un commentaire pour fusionner.

• Modèle de PR actuel : problème/impact simple ; inventaires dans diff/détails. Maintenir les risques, migrations, actions, preuve, lacunes, crédit à jour ; expliquer les compromis matériels. Pas d'avantage utilisateur interne inventé ou d'essais universels sur LOC/provenance/correction alternative.

• CHANGELOG.md appartient à la release ; contexte de note de version dans PR/commit. Transcriptions d'agent uniquement sur demande explicite.

• Mutation publique : vérifier auteur/committeur + rédacteur ; fichiers de corps sensibles au shell. Périmètre autorisé uniquement ; mettre à jour un commentaire/corps existant, pas de doublons.

• Publication/fusion : lire landing (média, CI sur tête exacte, workflow natif).

• Vérifier la fusion/fermeture distante + source. Rapporter le problème, le changement propriétaire, la preuve/lacunes, le crédit, l'état final lié en prose ; seulement les suivis utiles.

Finaliser et nettoyer

• Ne jamais supprimer l'arbre de travail contenant le répertoire de travail courant de la session, y compris le vôtre ou ses sous-répertoires. Un cd/remplacement de cwd enfant ne déplace pas le parent. Conserver jusqu'à ce que le propriétaire quitte/libère le cwd ; rapporter le chemin.
• Résultat vérifié → supprimer les journaux de tâche, reçus, archives de preuve, .crabbox, brouillons, arbres de travail terminés via closeout natif. Pas de conservation/archive/export/transfert uniquement pour suppression.
• Supprimer les arbres de travail créés une fois que le travail/PR est fusionné, sauf nécessité immédiate ; ne jamais supprimer un arbre de travail avec cwd actif.
• Chaque tâche : conclusions dans le chat/stdout ; fichiers uniquement pour les livrables/outils/preuve/besoins de récupération, indiquer l'état/la réutilisation. Preuve avant fusion obligatoire ; les suivis optionnels ne bloquent jamais l'achèvement.
• Préserver les livrables/la rétention demandés, la source non terminée + l'état de récupération nécessaire, les propriétaires actifs/en direct/inconnus, les identifiants, l'état de l'agent, les dépendances partagées.