Was macht OpenClaw-PR-Wartung?
- Nur angeforderte Issue/PR-Operation ausführen. Root
AGENTS.md: Reparatur, Produkt, Quellenvertrauen, Genehmigungsrichtlinie. Dieses Skill: GitHub-Workflow und Maintainer-Befugnis. - Eingefügte Referenz ≠ Veröffentlichungserlaubnis oder Bereichserweiterung.
Maintainer-Befugnis
- Rot
mainist ein Notfall, keine Voraussetzung für ein unabhängiges Einbringen. Nachgewiesene rot-main-Korrekturen im Reparaturbereich direkt aufmainpushen. Bei einer abgegrenzten Maintainer-Anweisung, trotz unabhängiger/vorhandener CI einzubringen, die Ausnahme für geerbte Fehler wählen; keine spezielle Formulierung, erneute Genehmigung oder Reparatur/Neuausführung der unabhängigen Haupt-CI verlangen. Menschliche Autorisierung, Fehlernachweise und die Ausführungsfähigkeit des geschützten Autors sind separate Prüfungen. - Reparatur-und-Einbringen-Befugnis umfasst interne Terminplanung, Datenbankaufnahme, Lebenszyklus-Design, Risiko, Verifikation. In-Task-Bugfixes (einschließlich kompatibler SDK-Fixes) und gesicherte Reparaturen, die nur ungültige oder nicht erreichbare Daten entfernen, benötigen keine erneute Genehmigung.
- Massenschließen/Wiederöffnen von mehr als 50 Elementen benötigt explizite Anzahl und Umfang.
- Beratungs-Workflows benötigen explizite Sicherheitsanfrage.
- Von Peter angewiesene Team-Updates verwenden den Release-Deployer; niemals Live-Zustand außerhalb seines Besitzers bearbeiten. Nachtwache über jeden Deploy/Restart informieren; sie koordiniert den Neustartzeitpunkt, nicht die Update-Genehmigung.
Umfang auswählen
- Review/Triage/Liste: schreibgeschützt; Änderung/Ergebnisse/Nachweise/Unsicherheit erklären. Bearbeitungen, Zuweisung, Labels, Kommentare, Schließen benötigen Autorisierung.
- Nur beheben: lokal untersuchen/reparieren/validieren; Veröffentlichung benötigt Ship/Land oder einen abgegrenzten Äquivalent.
- Einbringen/Versenden/Autonome Reparatur: Quellennachweis → Review → native Einreichung → verifizierte Zusammenführung. Ausstehende Anfragen verfolgen; PR-verursachte CI/Konflikte unter derselben Autorisierung beheben, zugeschriebene geerbte CI über die obige Ausnahme leiten. Akzeptiertes Auto-Merge ≠ erledigt. Anerkennung/unabhängige Arbeit bewahren; Root-Gates bleiben bestehen, keine wiederholte Routineerlaubnis.
- Warteschlange/Entdeckung: Triage lesen. Live-Lese-/gemeinsame Entdeckung stapeln; benannter PR ≠ Warteschlangendurchlauf.
- Autor/Regression-Zuschreibung: Autor- und Herkunftsforschung; Kontoname/Verlauf bedingt.
- Leitender Praktiker; begrenzte nützliche Arbeiter, serialisierter Checkout/Refs + GitHub-Schreibvorgänge. Keine benutzereigenen App-Aufgaben, die interne Unteraufgaben verbergen. Stillgelegte Arbeiter-Patches/Ansprüche/Nachweise vor Übernahme/Neuzuweisung bewahren.
Das tatsächliche Element inspizieren
- Mit
git status -sbbeginnen; unabhängige Arbeit bewahren. Lokalesgitcrawl→ PATHghx, enge Live-JSON. Veraltetes/fehlendes Archiv →ghx, keine breite Synchronisation. PR-Quelle:ghx pr view/diff+ Checkout, keine Websuche.
ghx pr view <pr> --repo openclaw/openclaw \
--json number,title,state,body,author,assignees,baseRefName,headRefName,headRefOid,files,comments,reviews
glx issue view <issue> --repo openclaw/openclaw \
--json number,title,state,body,author,assignees,comments
- Zuweisung: Triage, vor tiefer Arbeit. Andere Zugewiesene erwähnen; Eigentum bewahren. Konto-/Aktivitätsforschung bedingt.
Ergebnis festlegen
- Owner, Einstiegspunkte, Aufrufer, Geschwister, Tests, Docs, Historie, aktuellen main lesen; bestimmende Abhängigkeitsverträge inspizieren. Codex: persönliche Quelleninspektion des Roots. Gegenbeweise → untersuchen, bevor verteidigt wird. Keine erfundene Matrix/Alternative.
- Bereits behoben: aktuelle main-Reparatur nachweisen; kanonischer Commit/PR + passender Quell-/ausführbarer Nachweis. Schließbefugnis → Kommentar mit Nachweis + schließen; sonst Bericht.
- Echter Fehler: reproduzieren → besitzende Invariante reparieren → Pfad/Geschwister nachweisen. Vereinfachung vor Symptomwächter; Größe/LOC ≠ Korrektheits-Gate.
- Nützlicher PR: bearbeitbaren Branch verbessern; unsicher → autorisierte anerkannte Ersetzung vor dem Schließen. Ersetzte Abstammung → vom Maintainer besitzte Ersetzung, niemals Contributor-PR squashen.
- Unsicher/Produktentscheidung: Lücke benennen; kein spekulatives Mergen/automatisches Schließen außerhalb des Umfangs.
- Nachweis + Quellenvertrauen/Host-Route:
$openclaw-testing, Einbringen. Fokussierter Nachweis → fertiger PR; PR-CI = breiter Nachweis. Begrenzten lokalen Nachweis auf einem qualifizierten, verfügbaren Host bevorzugen. Bewusster Remote-Nachweis kann teure Prüfungen oder Umgebungslücken abdecken; Warteschlangen → task-eigene Zuweisung stoppen, PR-CI oder sicheren lokalen Umfang verwenden, nicht abgedeckte Anforderungen bewahren. Niemals unvertrauten Quellcode lokal ausführen. Optionale Live-Lücken → aussagekräftiger Grenznachweis + Einschränkung; expliziter Live-, externer API-, authentifizierter Risikonachweis zwingend. Mocks/Skips/alter Kopf ≠ Live. - Verhaltensänderungen: Grenznachweis-Vertrag +
$test-audit. Angefordertes Verhalten und Repository-Standards: getrennt prüfen.
CODEOWNERS-Review
CODEOWNERSleitet Review; Live-Erzwingung inspizieren. Eingeschränkte/Sicherheitspfade + materielle Produkt-/Verhalten-/Sicherheits-/Eigentumsänderungen benötigen Beteiligung des aufgeführten Owners.- Eigentums-/Review-Governance: verifizierte aktive Organisationsadministrator-Anweisung qualifiziert ebenfalls; Repository-Administrator/Umgehung allein nicht. Keine Route verzichtet auf erzwungene Reviews.
Review und Veröffentlichung
-
Vor dem Commit oder Einbringen von nichttrivialem Code: frischer Review über genehmigten Workflow; umsetzbare Ergebnisse auflösen, es sei denn, der Benutzer widerspricht. Umsetzbar = wahrscheinlich oder hohe Auswirkung; unwahrscheinliche, behebbare Grenzfälle ablehnen, deren Behebung unverhältnismäßige Komplexität hinzufügt, mit einzeiliger Begründung.
-
Nichttrivialer Code-Commit/Einbringen: die ausgewählte lokale Review-Route verwenden und verifizierte umsetzbare Ergebnisse auflösen. Erforderlicher frischer unabhängiger Review gilt weiterhin. Nur-Review: unabhängiger Durchlauf nur auf Anfrage, kein zusätzlicher zwingender CLI-Durchlauf.
-
Review für substanzielle Änderungen/nicht gelöste Bedenken wieder öffnen, nicht für Patch-identisches Rebase/mechanische Kopfänderung. Echte menschliche/Bot-Ergebnisse adressieren; Ablehnungen erklären. Bot-Bewertungen/Ranglisten erzeugen keine separaten Verpflichtungen. ClawSweeper-Beratung; niemals auf Kommentar warten, um einzubringen.
-
Aktuelle PR-Vorlage: klares Problem/Auswirkung; Bestandsverzeichnisse in Diff/Details. Risiken, Migrationen, Aktionen, Nachweise, Lücken, Anerkennung aktuell halten; materielle Kompromisse erklären. Keinen erfundenen internen Benutzernutzen oder universelle LOC/Provenienz/Alternativ-Fix-Essays.
-
CHANGELOG.mdRelease-eigen; Release-Notiz-Kontext in PR/Commit. Agententranskripte nur auf explizite Anfrage. -
Öffentliche Mutation: Author/Committer + Writer verifizieren.
ghxmit echtem--body-file <path>-Payloads verwenden, niemals stdin (--body-file -,--input -oder/dev/stdin); der Proxy leitet stdin nicht weiter. Geänderte Bodies mitghx --no-cachezurücklesen und exakte nichtleere Bytes verifizieren. Autorisierter Umfang nur; vorhandenen Kommentar/Body aktualisieren, keine Duplikate. -
Veröffentlichung/Zusammenführen: Einbringen lesen (Medium, exakter Kopf-CI, nativer Workflow).
-
Remote-Merge/Schließen + Quelle verifizieren. Problem, besitzende Änderung, Nachweise/Lücken, Anerkennung, verlinkten Endzustand in Prosa melden; nur lohnenswerte Follow-ups.
Finalisieren und Bereinigen
- Niemals Worktree entfernen, das das aktuelle Sitzungs-CWD hält, einschließlich Ihres oder Unterverzeichnisse. Kind-
cd/CWD-Überschreibung verschiebt nicht das Eltern. Behalten, bis Owner CWD verlässt/freigibt; Pfad melden. - Verifiziertes Ergebnis → Aufgabenlogs, Belege, Nachweisarchive,
.crabbox, Scratch, fertiggestellte Worktrees via nativem Closeout entfernen. Keine Aufbewahrung/Archivierung/Export/Übergabe nur zur Entfernung. - Erstellte Worktrees entfernen, sobald Arbeit/PR eingebracht ist, es sei denn, sofort danach benötigt; niemals ein Worktree mit Live-CWD entfernen.
- Jede Aufgabe: Ergebnisse in Chat/Stdout; Dateien nur für Liefergegenstände/Werkzeug/Nachweise/Wiederherstellungsbedarf, Zweck/Wiederverwendung angeben. Pre-Merge-Nachweis zwingend; optionale Follow-ups halten niemals die Fertigstellung auf.
- Angeforderte Liefergegenstände/Aufbewahrung, unvollendete Quelle + notwendigen Wiederherstellungszustand, aktive/Live/unbekannte Owner, Anmeldeinformationen, Agentenzustand, gemeinsame Abhängigkeiten bewahren.