Skip to content

docs: document clawsweeper self smoke target#18

Merged
clawsweeper[bot] merged 1 commit intomainfrom
smoke/clawsweeper-event-automerge-20260429131236
Apr 29, 2026
Merged

docs: document clawsweeper self smoke target#18
clawsweeper[bot] merged 1 commit intomainfrom
smoke/clawsweeper-event-automerge-20260429131236

Conversation

@steipete
Copy link
Copy Markdown
Contributor

Summary

  • documents openclaw/clawsweeper as the self-contained target for ClawSweeper event/review/comment-router/automerge smoke tests

Verification

  • pnpm exec oxfmt --check --threads=1 .github/workflows/sweep.yml docs/repair/operations.md

This PR is the end-to-end live smoke for default-branch event review, manual review, comment-router commands, and guarded automerge.

@steipete
Copy link
Copy Markdown
Contributor Author

/clawsweeper status

@steipete
Copy link
Copy Markdown
Contributor Author

/clawsweeper automerge

@clawsweeper clawsweeper Bot added the clawsweeper:automerge Maintainer opted this ClawSweeper PR into bounded ClawSweeper-reviewed automerge label Apr 29, 2026
@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

ClawSweeper automerge is enabled for this PR.

I added clawsweeper:automerge and asked ClawSweeper to review this head. If ClawSweeper emits a repair marker or requests changes, I will repair/rebase the branch and ask for another review, up to the configured round limit.

A maintainer can pause this with /clawsweeper stop.

@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

ClawSweeper status:

  • PR: docs: document clawsweeper self smoke target #18
  • Branch: smoke/clawsweeper-event-automerge-20260429131236
  • ClawSweeper PR: no
  • Automerge: not-enabled
  • Automerge job: jobs/openclaw/inbox/automerge-openclaw-clawsweeper-18.md
  • Merge state: UNKNOWN
  • Checks: SUCCESS:1

@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

Codex review: needs maintainer review before merge.

What this changes:

The PR adds three documentation lines to docs/repair/operations.md naming openclaw/clawsweeper as the self-contained target for ClawSweeper event, review, comment-router, and automerge smoke tests.

Maintainer follow-up before merge:

Review and land this docs-only PR if maintainers want the operations guide to explicitly call out openclaw/clawsweeper for self-contained event, review, comment-router, and automerge smoke tests; otherwise a maintainer can manually close it as redundant with the broader README self-review note.

Best possible solution:

Review and land this docs-only PR if maintainers want the operations guide to explicitly call out openclaw/clawsweeper for self-contained event, review, comment-router, and automerge smoke tests; otherwise a maintainer can manually close it as redundant with the broader README self-review note.

What I checked:

  • PR scope: The provided PR file patch changes only docs/repair/operations.md, adding three lines under Runner Strategy that document openclaw/clawsweeper as the self-contained smoke-test target for event, review, comment-router, and automerge checks. (docs/repair/operations.md:153, dbdf7be991a3)
  • Current operations docs lack exact guidance: On current main, Runner Strategy says to use ubuntu-latest for parity/correctness smoke tests and then describes Blacksmith runner usage; it does not name openclaw/clawsweeper as the target repository there. (docs/repair/operations.md:153, 945d38f8518a)
  • Adjacent README overlap: Current README says openclaw/clawsweeper is available for manual and event self-review smoke tests, which partially overlaps the PR but does not include the more specific operations-guide wording about comment-router and automerge smoke tests. (README.md:805, 945d38f8518a)
  • Self-review support provenance: Commit 77146710b1d18c470203925ea3069b104c8952df added the openclaw/clawsweeper repository profile and same-repo checkout isolation for self-review workflows, so the PR aligns with existing behavior rather than adding new code behavior. (src/repository-profiles.ts:64, 77146710b1d1)
  • Security review pass: The PR patch is prose-only in docs/repair/operations.md; it does not touch workflows, action refs, dependency sources, lockfiles, install/build/release scripts, permissions, secrets handling, or downloaded/generated code paths. (docs/repair/operations.md:153, dbdf7be991a3)

Likely related people:

  • steipete: Current-main history shows commit 77146710b1d18c470203925ea3069b104c8952df added ClawSweeper self-review workflow support, the openclaw/clawsweeper repository profile, isolated same-repo checkout handling, and the current README self-review smoke-test note. (role: introduced self-review support and adjacent docs; confidence: high; commits: 77146710b1d1; files: README.md, src/repository-profiles.ts, .github/workflows/sweep.yml)

Remaining risk / open question:

  • Auto-closing would drop a small operations-doc clarification because current main's README note is adjacent but does not contain the PR's specific comment-router/automerge wording in docs/repair/operations.md.

Codex review notes: model gpt-5.5, reasoning high; reviewed against 945d38f8518a.

@clawsweeper clawsweeper Bot added the clawsweeper:human-review ClawSweeper automerge is paused for maintainer review label Apr 29, 2026
@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

ClawSweeper is pausing automerge for human review.

Source: clawsweeper[bot]
Reason: structured ClawSweeper verdict: needs-human (sha=dbdf7be991a3e3a7098a6f93e48d5fe049689380)

I added clawsweeper:human-review and left the final call with a maintainer.

@steipete
Copy link
Copy Markdown
Contributor Author

/clawsweeper approve

@clawsweeper clawsweeper Bot removed the clawsweeper:human-review ClawSweeper automerge is paused for maintainer review label Apr 29, 2026
@clawsweeper clawsweeper Bot merged commit 2ca9477 into main Apr 29, 2026
1 check passed
@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

Maintainer-approved ClawSweeper automerge is complete.

Approver: steipete
Head: dbdf7be991a3e3a7098a6f93e48d5fe049689380
Merge status: merged by ClawSweeper automerge
Merged at: 2026-04-29T13:19:04Z

The automerge loop is complete.

@clawsweeper clawsweeper Bot added the clawsweeper:human-review ClawSweeper automerge is paused for maintainer review label Apr 29, 2026
@clawsweeper
Copy link
Copy Markdown
Contributor

clawsweeper Bot commented Apr 29, 2026

ClawSweeper is pausing automerge for human review.

Source: clawsweeper[bot]
Reason: structured ClawSweeper verdict: needs-human (sha=dbdf7be991a3e3a7098a6f93e48d5fe049689380)

I added clawsweeper:human-review and left the final call with a maintainer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clawsweeper:automerge Maintainer opted this ClawSweeper PR into bounded ClawSweeper-reviewed automerge clawsweeper:human-review ClawSweeper automerge is paused for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant