L’interface est localisée ; les présentations et documents des Skills sont traduits automatiquement par IA dans votre langue. Le code, les commandes, chemins et termes techniques restent en version originale, et le document de l’auteur fait foi.
Explorer les Skills
Développement · SKILL.md

openclaw-pr-maintainer

openclaw-pr-maintainer

Review, triage, repair, or land OpenClaw issues and pull requests with current-source evidence and the native maintainer workflow.

Independent GitHub snapshot · Automated classification · Original documentation governs

Documentation de la Skill

GitHub

name: openclaw-pr-maintainer description: "Review, triage, repair, or land OpenClaw issues and pull requests with current-source evidence and the native maintainer workflow."

OpenClaw PR Maintainer

  • Requested issue/PR operation only. Root AGENTS.md: repair, product, source-trust, approval policy. This skill: GitHub workflow.
  • Pasted ref ≠ publication permission or scope expansion.

Select the scope

  • Review/triage/list: read-only; explain change/findings/proof/uncertainty. Edits, assignment, labels, comments, closes need authority.
  • Fix only: investigate/repair/validate locally; publication needs ship/land or scoped equivalent.
  • Land/ship/autonomous repair: source proof → review → native submission → verified merge. Track pending requests; repair CI/conflicts under same authority. Accepted auto-merge ≠ done. Preserve credit/unrelated work; root gates remain, no repeat routine permission.
  • Queue/discovery: read triage. Batch live reads/shared discovery; named PR ≠ queue sweep.
  • Author/regression attribution: author and provenance research; account age/history conditional.
  • Lead hands-on; bounded useful workers, serialized checkout/refs + GitHub writes. No user-owned app tasks hiding internal subtasks. Preserve stalled worker patches/claims/proof before takeover/reassignment.

Inspect the actual item

  • Start git status -sb; preserve unrelated work. Local gitcrawl → bare PATH gh, narrow live JSON. Stale/missing archive → gh, no broad sync. PR source: gh pr view/diff + checkout, not web search.
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
  • Assignment: triage, before deep work. Mention other assignees; preserve ownership. Account/activity research conditional.

Establish the outcome

  • Read owner, entry points, callers, siblings, tests, docs, history, current main; inspect determining dependency contracts. Codex: root's personal source inspection. Contrary evidence → investigate before defending. No invented matrix/alternative.
  • Already fixed: prove current-main repair; canonical commit/PR + matching source/runnable proof. Closure authority → comment evidence + close; otherwise report.
  • Real defect: reproduce → repair owning invariant → prove path/siblings. Simplification over symptom guard; size/LOC ≠ correctness gate.
  • Useful PR: improve editable branch; unsafe → authorized credited replacement before closure. Replaced ancestry → maintainer-owned replacement, never squash contributor PR.
  • Uncertain/product decision: state gap; no speculative merge/automatic out-of-scope close.
  • Proof + source-trust/host route: $openclaw-testing, landing. Local focused proof → ready PR; PR CI = broad proof. Boxes only for uncovered environment needs, immediate grant; queues → local trusted proof, never untrusted local execution. Optional live gaps → meaningful boundary proof + limitation; explicit live, external API, authenticated-risk proof mandatory. Mocks/skips/old head ≠ live.
  • Behavioral changes: boundary-proof contract + $test-audit. Requested behavior and repo standards: check separately.

CODEOWNERS review

  • CODEOWNERS routes review; inspect live enforcement. Restricted/security paths + material product/behavior/security/ownership changes need listed-owner involvement.
  • ownership/review governance: verified active organization-admin direction also qualifies; repository admin/bypass alone does not. Neither route waives enforced reviews.

Review and publish

  • Nontrivial code commit/landing: $autoreview; resolve verified actionable findings. Review-only: independent pass only when requested, no extra mandatory CLI pass.
  • Reopen review for substantive changes/unresolved concerns, not patch-identical rebase/mechanical head change. Address real human/bot findings; explain rejections. Bot scores/Rank-up lists create no separate obligations. ClawSweeper advisory; never wait for comment to land.
  • Current PR template: plain problem/impact; inventories in diff/details. Keep risks, migrations, actions, proof, gaps, credit current; explain material tradeoffs. No invented internal-only user benefit or universal LOC/provenance/alternate-fix essays.
  • CHANGELOG.md release-owned; release-note context in PR/commit. Agent transcripts only on explicit request.
  • Public mutation: verify author/committer + writer; shell-sensitive body files. Authorized scope only; update existing comment/body, no duplicates.
  • Publication/merge: read landing (media, exact-head CI, native workflow).
  • Verify remote merge/closure + source. Report problem, owning change, proof/gaps, credit, linked final state in prose; worthwhile follow-ups only.

Finalize and clean up

  • Never remove worktree holding live session cwd, including yours or subdirectories. Child cd/cwd override does not move parent. Retain until owner exits/releases cwd; report path.
  • Verified outcome → remove task logs, receipts, proof archives, .crabbox, scratch, finished worktrees via native closeout. No retention/archive/export/handoff just for removal.
  • Every task: findings in chat/stdout; files only for deliverables/tool/proof/recovery needs, state purpose/reuse. Pre-merge proof mandatory; optional follow-ups never hold completion.
  • Preserve requested deliverables/retention, unfinished source + necessary recovery state, active/live/unknown owners, credentials, agent state, shared dependencies.
Skills enregistrées