Skip to content

docs: add ClawSweeper review policy to AGENTS#86197

Merged
Takhoffman merged 3 commits into
mainfrom
clawsweeper/issue-openclaw-openclaw-86185
May 24, 2026
Merged

docs: add ClawSweeper review policy to AGENTS#86197
Takhoffman merged 3 commits into
mainfrom
clawsweeper/issue-openclaw-openclaw-86185

Conversation

@clawsweeper

@clawsweeper clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a near-top ClawSweeper Review Policy section to root AGENTS.md.
  • Moves OpenClaw-specific review expectations into OpenClaw-owned repository policy so ClawSweeper can keep generic prompt policy out of repo-specific details.
  • Keeps the change intentionally narrow to AGENTS.md; the companion ClawSweeper prompt cleanup remains out of scope for this repo.

Linked context

Closes #86185

Requested by @Takhoffman in the source issue.

Real behavior proof

  • Behavior addressed: root AGENTS.md now gives ClawSweeper review agents a near-top OpenClaw-specific policy section, including the full-read/scoped-AGENTS instruction.
  • Real environment tested: repository checkout on the PR branch.
  • Exact steps or command run after this patch: pnpm check:changed
  • Evidence after fix: command output from pnpm check:changed in CI or local executor log.
  • Observed result after fix: check:changed passes for the AGENTS.md-only policy change.
  • What was not tested: runtime OpenClaw behavior, because this change only updates repository policy text.
  • Proof limitations or environment constraints: this worker was read-only and emitted the implementation artifact; the executor should apply the patch and capture command output.

Tests and validation

  • pnpm check:changed

Reviewer notes

Please review the exact policy wording closely. The intended placement is immediately after the existing Start section and before Map so review agents encounter it before later sections can be truncated or skimmed.

ClawSweeper 🐠 replacement reef notes:

  • Cluster: issue-openclaw-openclaw-86185
  • Source PRs: none
  • Credit: Source issue and requested policy move: Move OpenClaw-specific ClawSweeper review policy into AGENTS.md #86185 by @Takhoffman.; No contributor PR is being replaced; preserve issue credit in the PR body.
  • Validation: pnpm check:changed
  • Replacement reason: ClawSweeper could not update the source PR branch directly, so it opened a writable replacement PR instead.

fish notes: model gpt-5.5, reasoning high; reviewed against f914e4c.

@clawsweeper clawsweeper Bot added clawsweeper Tracked by ClawSweeper automation clawsweeper:autogenerated PR created automatically by ClawSweeper labels May 24, 2026
@clawsweeper

clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor Author

Codex review: needs maintainer review before merge. Reviewed May 24, 2026, 5:16 PM ET / 21:16 UTC.

Summary
Adds a near-top ClawSweeper Review Policy section to root AGENTS.md with OpenClaw-specific review, routing, compatibility, validation, changelog, and docs-link rules.

PR surface: Docs +15. Total +15 across 1 file.

Reproducibility: not applicable. this is a docs/policy PR rather than a runtime bug. Source inspection shows current main lacks the section and the PR head adds it near the top of AGENTS.md.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🌊 off-meta tidepool
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

  • Have a maintainer or the source policy requester approve the exact wording and undraft the PR before merge.

Risk before merge

  • AGENTS.md is consumed as review policy, so merging imprecise wording could alter future ClawSweeper review behavior; the draft state and linked maintainer-labeled issue indicate the wording still needs maintainer approval.

Maintainer options:

  1. Approve The Policy Wording (recommended)
    A maintainer or the source policy requester should confirm the latest terse section is the intended repository policy before this draft PR merges.
  2. Tighten Before Merge
    If the wording is still short of the linked issue, update the AGENTS.md section in this PR rather than relying on future prompt text to carry OpenClaw-specific policy.
  3. Pause If Policy Ownership Changed
    If maintainers no longer want this policy in OpenClaw AGENTS.md, keep the PR paused or close it with that decision and leave the companion ClawSweeper prompt unchanged.

Next step before merge
This draft policy PR needs maintainer/requester wording approval; there is no narrow automation repair left after the latest head.

Security
Cleared: The diff only changes repository policy text in AGENTS.md and does not touch code execution, dependencies, CI, secrets, packaging, or release surfaces.

Review details

Best possible solution:

Land this concise AGENTS.md policy after explicit maintainer approval, then use it as the OpenClaw-owned basis for the companion ClawSweeper prompt cleanup.

Do we have a high-confidence way to reproduce the issue?

Not applicable; this is a docs/policy PR rather than a runtime bug. Source inspection shows current main lacks the section and the PR head adds it near the top of AGENTS.md.

Is this the best way to solve the issue?

Yes, subject to maintainer approval; a near-top AGENTS.md section is the requested ownership boundary and the latest text covers the main OpenClaw-specific review rules without changing runtime code.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 3e275a53dc9e.

Label changes

Label changes:

  • add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Not applicable: This is a bot-authored policy-only AGENTS.md PR; the external-contributor real-behavior-proof gate does not apply.
  • remove rating: 🦐 gold shrimp: Current PR rating is rating: 🐚 platinum hermit, so this older rating label is no longer current.
  • remove status: 🛠️ actively grinding: Current PR status label is status: 👀 ready for maintainer look.

Label justifications:

  • P3: This is a narrow repository-policy documentation change with no runtime behavior impact.
  • merge-risk: 🚨 automation: The added AGENTS.md text is intended to steer future ClawSweeper review automation, and incorrect wording could change review gates despite green checks.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🌊 off-meta tidepool and patch quality is 🐚 platinum hermit.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Not applicable: This is a bot-authored policy-only AGENTS.md PR; the external-contributor real-behavior-proof gate does not apply.
Evidence reviewed

PR surface:

Docs +15. Total +15 across 1 file.

View PR surface stats
Area Files Added Removed Net
Source 0 0 0 0
Tests 0 0 0 0
Docs 1 15 0 +15
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 1 15 0 +15

Acceptance criteria:

What I checked:

  • Current main lacks the new section: Current main goes from the Start bullets directly to Map, so the requested ClawSweeper Review Policy is not already present on main. (AGENTS.md:20, 3e275a53dc9e)
  • PR head adds the requested near-top policy: The PR head inserts ClawSweeper Review Policy at line 20 and covers full-file reading, scoped AGENTS.md, ClawHub/plugin routing, compatibility risk, sibling-surface review, changelog handling, public docs links, evidence, validation, and Telegram/Desktop proof expectations. (AGENTS.md:20, 6cc8d117fffd)
  • Diff scope is narrow: GitHub API reports one modified file, AGENTS.md, with 15 additions and no deletions. (AGENTS.md:20, 6cc8d117fffd)
  • Linked issue defines the policy acceptance criteria: The linked source issue asks for a near-top AGENTS.md ClawSweeper Review Policy that preserves lean-core/ClawHub routing, compatibility-sensitive APIs, upgrade-safe defaults, sibling-surface review, maintainer-owned changelog work, and public docs-link preference.
  • Draft and maintainer-review signal: The live PR is open, draft, mergeable, and the timeline shows Takhoffman converted it to draft; the linked issue carries the maintainer and needs-maintainer-review labels. (6cc8d117fffd)
  • Whitespace validation passes: A read-only diff check between the base SHA and PR head produced no whitespace errors for AGENTS.md. (AGENTS.md:20, 6cc8d117fffd)

Likely related people:

  • steipete: git blame and git show identify Peter Steinberger's commit c91c3c6 as the commit that added the current AGENTS.md file this PR extends. (role: current root policy introducer; confidence: high; commits: c91c3c6e5abc; files: AGENTS.md)
  • Takhoffman: The linked issue contains the detailed policy request, and the PR timeline shows Takhoffman converted this PR to draft for wording review. (role: source policy requester and draft reviewer; confidence: medium; files: AGENTS.md)
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

@Takhoffman Takhoffman marked this pull request as draft May 24, 2026 20:49
@clawsweeper clawsweeper Bot added rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. merge-risk: 🚨 automation 🚨 May affect CI, automerge, proof capture, label sync, or maintainer automation. labels May 24, 2026
@clawsweeper

clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor Author

ClawSweeper PR egg

✨ Hatched: 🥚 common Brave Branchling

Hatch command

Comment @clawsweeper hatch when this PR is hatchable.

Hatchability rules:

  • Merged PRs are hatchable.
  • Open PRs are hatchable when they are status: 👀 ready for maintainer look, status: 🚀 automerge armed, or labeled clawsweeper:automerge.
  • Closed unmerged PRs are hatchable only when one of those hatchable labels is still present in the durable record.

Rarity: 🥚 common.
Trait: keeps receipts.
Image traits: location CI tidepool; accessory commit compass; palette cobalt, lime, and pearl; mood curious; pose leaning over a miniature review desk; shell starlit enamel shell; lighting soft studio lighting; background tiny artifact crates.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Brave Branchling in ClawSweeper.

What is this egg doing here?
  • Eggs appear after the PR passes real-behavior proof. It is here for vibes, not verdicts: it does not change labels, ratings, merge decisions, or automation.
  • The shell reacts to review momentum: open follow-up work warms it up, re-review makes it wobble, and a clean final review lets it hatch.
  • Hatchability usually comes from sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness. A merged PR is already final, so merge makes the egg hatchable independently.
  • The hatch is seeded from this repository and PR number, so the same PR keeps the same creature; the reviewed head SHA can only change safe visual details.
  • Rarity is just collectible sparkle: 🥚 common, 🌱 uncommon, 💎 rare, ✨ glimmer, and 🌈 legendary.

@clawsweeper

clawsweeper Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor Author

🦞🔧
ClawSweeper applied a repair to this PR branch.

Repair: kept the fix on this contributor branch instead of opening a replacement PR.
Validation: pnpm check:changed; pnpm lint; pnpm check:test-types
Updated head: 942c2996c3aa
Run: https://github.com/openclaw/clawsweeper/actions/runs/26372609352

Current state: exact-head review queued immediately; GitHub checks and the review verdict gate final merge.

Automerge progress:

  • 2026-05-24 20:58:46 UTC review requested repair f914e4c5aad8 (structured ClawSweeper needs-human verdict with repairable P-severity findings...)
  • 2026-05-24 21:06:50 UTC review queued 942c2996c3aa (after repair)

@clawsweeper clawsweeper Bot force-pushed the clawsweeper/issue-openclaw-openclaw-86185 branch from f914e4c to 942c299 Compare May 24, 2026 21:06
clawsweeper Bot added 3 commits May 24, 2026 21:12
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>
@clawsweeper clawsweeper Bot force-pushed the clawsweeper/issue-openclaw-openclaw-86185 branch from e1692c8 to 6cc8d11 Compare May 24, 2026 21:12
@clawsweeper clawsweeper Bot added status: 🛠️ actively grinding The PR author has acted after the latest ClawSweeper review and work remains. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. and removed status: ⏳ waiting on author ClawSweeper has contributor-facing work open and is waiting for author action. rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. status: 🛠️ actively grinding The PR author has acted after the latest ClawSweeper review and work remains. labels May 24, 2026
@Takhoffman Takhoffman marked this pull request as ready for review May 24, 2026 21:21
@Takhoffman Takhoffman merged commit 242e876 into main May 24, 2026
76 of 84 checks passed
@Takhoffman Takhoffman deleted the clawsweeper/issue-openclaw-openclaw-86185 branch May 24, 2026 21:22
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 25, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
SebTardif pushed a commit to SebTardif/openclaw that referenced this pull request May 26, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
SYU8384 pushed a commit to SYU8384/openclaw that referenced this pull request Jun 3, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
* docs: add ClawSweeper review policy to AGENTS

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

* docs: add ClawSweeper review policy to AGENTS

* docs: move ClawSweeper review policy into AGENTS.md

Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com>

---------

Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clawsweeper:autogenerated PR created automatically by ClawSweeper clawsweeper Tracked by ClawSweeper automation merge-risk: 🚨 automation 🚨 May affect CI, automerge, proof capture, label sync, or maintainer automation. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: XS status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move OpenClaw-specific ClawSweeper review policy into AGENTS.md

1 participant