CommunityProgramación y desarrollogithub.com

openclaw-landable-bug-sweep

Find or repair small high-confidence non-SDK-boundary OpenClaw bugfix PRs until five are landable.

Compatible con~Claude Code~Codex CLI~Cursor
npx add-skill https://github.com/clawdbot/clawdbot/tree/main/.agents/skills/openclaw-landable-bug-sweep

OpenClaw Landable Bug Sweep

Autonomous maintainer workflow for producing five landable OpenClaw bugfix PR URLs. Use for broad issue/PR sweeps where the bar is high and the output is PRs, not notes. Do not use for plugin SDK/API boundary work; those need separate architecture review.

Target

Return exactly five PR URLs, each with:

  • bug summary
  • why the fix is low-risk
  • proof: rebased-head local/Testbox/live commands or run IDs
  • autoreview: clean result on the exact head being shown
  • CI green on the exact pushed PR head
  • issue/duplicate cleanup done or still pending

The five URLs may be existing PRs that were reviewed/fixed, or new PRs created from issues/clusters. Do not present a PR URL to the maintainer until it has been refreshed on current main, left-tested, autoreviewed clean, pushed, and verified green in live GitHub CI. If code, tests, changelog, PR body, or branch base changes after autoreview, rerun autoreview before showing the URL.

Companion Skills

Use $gitcrawl for discovery/clustering, $openclaw-pr-maintainer for live GitHub mutation rules, $github-author-context when contributor trust matters, $openclaw-testing for proof choice, $autoreview before publishing/landing, and $crabbox for broad/E2E/live proof.

Candidate Bar

Accept only when all are true:

  • bug or paper cut, not feature/product/support/docs-only
  • root cause is proven in current code
  • dependency behavior checked via upstream docs/source/types when relevant
  • production/runtime diff is small, ideally much smaller than 500 LOC and always below 500 LOC
  • tests may be larger, but focused
  • no new dependency
  • no new config option
  • no backward-incompatible behavior
  • no security/product/owner-boundary decision needed
  • no plugin SDK, public plugin API, or src/plugin-sdk/** boundary change
  • no broad refactor smell
  • focused proof is feasible
  • branch can be rebased/refreshed and pushed, or a replacement PR can be created

Good examples:

  • provider parameter mismatch proven against dependency/API contract
  • CLI command diverges from adjacent command behavior
  • narrow runtime state/serialization bug with failing test
  • issue already fixed on current main, with proof and closeable duplicates

Reject:

  • feature requests, new knobs, migrations, release work, workflow policy, support
  • plugin SDK/API boundary changes, including compatibility shims, new SDK methods, SDK exports, or plugin-facing channel/provider seams
  • auth/security boundary changes unless explicitly assigned
  • bugs needing live credentials that are unavailable
  • PRs with red CI unless you fix, rebase, push, and recheck them green
  • PRs you only reviewed locally but did not refresh/push/check live
  • PRs whose final head has not passed $autoreview
  • fixes whose clean shape is a larger architecture move
  • speculative reports without reproducible/provable cause
  • UI/UX changes requiring product judgment

Sweep Loop

  1. Start clean:
    • git status -sb
    • git pull --ff-only
    • verify branch is expected, usually main
  2. Build candidate clusters:
    • gitcrawl open issues/PRs, neighbors, and search
    • live gh issue/pr view
    • include PRs linked from issues and duplicates
  3. For each cluster:
    • read issue/PR body, comments, labels, linked refs, current source, adjacent tests
    • suppress maintainer-owned queue noise unless it is the best fix path
    • identify opener/author and preserve credit
    • decide: repair-existing-pr, create-new-pr, close-fixed-on-main, close-duplicate, or reject
  4. Prove before patching:
    • failing test, focused repro, log/source proof, or dependency contract proof
    • if already fixed on main, prove with current source/test/commit and close kindly
  5. Patch:
    • prefer existing PR when good and writable
    • if unwritable or wrong shape, create own PR and preserve useful contributor credit
    • if no PR exists, create one
    • add regression test when it fits
    • release-note context for user-facing fixes in PR body or commit message; credit human reporter/contributor when known
  6. Review, refresh, and publish:
    • rebase or otherwise refresh the PR branch on current origin/main
    • resolve drift, including newly exposed CI failures, rather than counting the PR as ready
    • do not add CHANGELOG.md during normal sweep PRs; release automation generates it from PRs and commits
    • left-test the rebased head with the smallest meaningful local/Testbox/live command that proves the bug
    • run $autoreview until no accepted/actionable findings remain before creating, updating, or presenting the PR URL
    • create/update PR with real body and proof fields
    • push the exact reviewed head
    • verify live GitHub CI is green for that pushed head; do not count pending, red, dirty, conflicting, or externally blocked PRs in the five
  7. Hygiene:
    • close duplicates and fixed-on-main issues/PRs with proof as soon as you notice them during the sweep
    • never mutate more than five associated items in one cluster without explicit confirmation
    • comments must be kind, concrete, and include proof/PR/commit links
  8. Repeat until five landable PR URLs are ready.

PR Body Proof

Use the repo PR template. Include these exact labels:

Behavior addressed:
Real environment tested:
Exact steps or command run after this patch:
Evidence after fix:
Observed result after fix:
What was not tested:

Existing PR Rules

  • Review code path beyond the diff before trusting it.
  • If PR is good: rebase/refresh on current main, fix small issues, left-test, autoreview clean, push, and get CI green before showing or counting it.
  • If PR is not good but has a useful idea: recreate locally, co-author when warranted, close original with thanks and explanation.
  • If PR is duplicate or fixed on main: comment proof, close.
  • If maintainer cannot push to contributor branch: create own branch/PR, preserve useful commits or credit.
  • If CI turns red after local proof, treat that as normal work: inspect the failing job, fix or reject, rerun, and only count the PR once green.

Output Ledger

Maintain a running ledger:

accepted:
- PR URL:
  source refs:
  bug:
  root cause:
  fix:
  risk:
  rebase/head:
  left-test:
  autoreview:
  CI:
  credit/thanks:
  cleanup:

rejected:
- ref:
  reason:

closed:
- ref:
  reason:
  proof/comment:

Final answer:

  • exactly five accepted PR URLs
  • 2-4 sentence explainer per PR
  • proof/CI state per PR
  • closed duplicates/fixed-on-main refs
  • current branch/status

Individual skills in this repo

This repo contains 20 individual skills — each has its own dedicated page.

1password

Set up and use 1Password CLI for sign-in, desktop integration, and reading or injecting secrets.

acp-router

Route plain-language requests for Claude Code, Cursor, Copilot, OpenClaw ACP, OpenCode, Gemini CLI, Qwen, Kiro, Kimi, iFlow, Factory Droid, Kilocode, or explicit ACP harness work into either OpenClaw ACP runtime sessions or direct acpx-driven sessions ("telephone game" flow). For coding-agent thread requests, read this skill first, then use only `sessions_spawn` for thread creation. Codex chat binding defaults to the native Codex app-server plugin unless ACP is explicit or background spawn needs ACP.

agent-transcript

Add a redacted agent transcript section to GitHub PR or issue bodies during OpenClaw agent-created PR/issue workflows.

apple-notes

Create, view, edit, delete, search, move, or export Apple Notes via the memo CLI on macOS.

apple-reminders

List, add, edit, complete, or delete Apple Reminders and reminder lists via remindctl.

autoreview

Auto Review closeout. Codex review is the default when no engine is set and is the recommended reviewer.

bear-notes

Create, search, and manage Bear notes via grizzly CLI.

blacksmith-testbox

Run Blacksmith Testbox for CI-parity checks, secrets, hosted services, migrations, or builds local cannot reproduce.

blogwatcher

Monitor blogs and RSS/Atom feeds for updates using the blogwatcher CLI.

blucli

BluOS CLI (blu) for discovery, playback, grouping, and volume.

bluebubbles

Send and manage iMessages via BlueBubbles, including attachments, tapbacks, edits, replies, and groups.

browser-automation

Use when controlling web pages with the OpenClaw browser tool, especially multi-step flows, login checks, tab management, or recovery from stale refs/timeouts.

camsnap

Capture frames or clips from RTSP/ONVIF cameras.

canvas

Present HTML on connected OpenClaw node canvases, navigate/eval/snapshot, and debug canvas host URLs.

channel-message-flows

Use when previewing local channel message flow fixtures.

clawdtributor

Use for OpenClaw clawtributors PR/issue triage: Discrawl discovery, live-open rechecks, deep review, topic grouping, and compact @handle/LOC/type/blast/verification summaries.

clawhub

Search, install, update, sync, or publish agent skills with the ClawHub CLI and registry.

clawsweeper

Use for all ClawSweeper work: OpenClaw issue/PR sweep reports, commit-review reports, repair jobs, cloud fix PRs, @clawsweeper maintainer mention commands, trusted ClawSweeper-reviewed autofix/automerge, GitHub Actions monitoring, permissions, gates, and manual backfills.

clownfish-cloud-pr

Use when launching Clownfish in GitHub Actions to create or update one guarded GitHub implementation PR from issue/PR refs, a ClawSweeper report, a custom maintainer prompt, or to opt an existing Clownfish PR into ClawSweeper-reviewed cloud automerge.

codex-review

Codex code review closeout: local dirty changes, PR branch vs main, parallel tests.

Skills relacionados