chore: move Motivation section above Change Type in PR template by clawsweeper[bot] · Pull Request #84098 · openclaw/openclaw

clawsweeper · 2026-05-19T10:59:58Z

Makes #84003 merge-ready for the ClawSweeper automerge loop.
The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

Cluster: automerge-openclaw-openclaw-84003
Source PRs: chore: move Motivation section above Change Type in PR template #84003
Credit: Source PR: chore: move Motivation section above Change Type in PR template #84003
Validation: pnpm check:changed
Replacement reason: ClawSweeper could not update the source PR branch directly, so it opened a writable replacement PR instead.
Automerge requested by: @Takhoffman

Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Inherited issue-closing references from the source PR:
Closes #83939

Co-author credit kept:

@seraphjiang: Co-authored-by: Huan Jiang 550437+seraphjiang@users.noreply.github.com

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

#83939)

clawsweeper · 2026-05-19T11:01:15Z

Codex review: passed.

Workflow note: Future ClawSweeper reviews update this same comment in place.

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.

Summary
This PR moves the existing Motivation section in .github/pull_request_template.md from below Linked Issue/PR to immediately after Summary without changing the section text.

Reproducibility: not applicable. this is a PR-template ordering cleanup, not a runtime bug. Source inspection of current main and the PR head verifies the before/after section order.

PR rating
Overall: 🐚 platinum hermit
Proof: 🐚 platinum hermit
Patch quality: 🦞 diamond lobster
Summary: Clean template-only patch with proof override and no blocking findings; confidence is normal-good rather than exceptional because no live GitHub UI artifact was available to inspect.

Rank-up moves:

none

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.

PR egg
✨ Hatched: 🥚 common Frosted Proofling

       _..------.._          
    .-'  .-.  .-.  '-.       
   /    ( * )( * )    \      
  |        .--.        |     
  |   <\   ====   />   |     
   \    '.______.'    /      
    '-._   ____   _.-'       
        `-.____.-'           
       __/|_||_|\__          
      /__.'    '.__\         
       .-----------.         
      '-------------'

Rarity: 🥚 common.
Trait: stacks clean commits.
Share on X: post this hatch
Copy: My PR egg hatched a 🥚 common Frosted Proofling 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.
How to hatch it: reach status: 👀 ready for maintainer look or status: 🚀 automerge armed; that usually means sufficient real-behavior proof, no blocking P0/P1/P2 findings, no security attention needed, and clean correctness.
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.

Real behavior proof
Override: A maintainer applied proof: override for this PR.

Next step before merge
No repair job is needed; the active automerge path can gate this exact head with the existing review and checks.

Security
Cleared: The diff only reorders Markdown in the PR template and does not change workflows, permissions, dependencies, secrets handling, or code execution paths.

Review details

Best possible solution:

Land this narrow template reorder if required checks stay green, then let the closing reference resolve the linked template-order issue.

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

Not applicable; this is a PR-template ordering cleanup, not a runtime bug. Source inspection of current main and the PR head verifies the before/after section order.

Is this the best way to solve the issue?

Yes; moving the existing Motivation block without changing wording or checklists is the narrow maintainable solution, and the proof parser is heading-based rather than position-based.

Label justifications:

P3: This is a low-risk PR-template ergonomics change with no runtime behavior, config, security, or migration surface.

What I checked:

Current main still has the old order: On current main, Change Type begins before Motivation, with Motivation still at line 38 after the linked-issue checklist. (.github/pull_request_template.md:38, 5e0850fc548c)
PR head implements the requested order: At the PR head, Motivation is line 12 and Change Type follows at line 18, matching the requested section order. (.github/pull_request_template.md:12, 6c68583faccb)
Diff is limited to moving one Markdown block: The patch changes only .github/pull_request_template.md with 6 insertions and 6 deletions, preserving the moved text verbatim. (.github/pull_request_template.md:12, 6c68583faccb)
Proof automation is heading-based: The real-behavior-proof parser locates the Real behavior proof section by heading, so moving earlier template sections does not create an obvious parser-order dependency. (scripts/github/real-behavior-proof-policy.mjs:145, 5e0850fc548c)
Proof gate is overridden/skipped for this replacement context: Current policy treats proof: override as a passing override and skips bot-authored PRs; the PR context includes proof: override and author clawsweeper[bot]. (scripts/github/real-behavior-proof-policy.mjs:286, 5e0850fc548c)
Template history and routing: Feature-history checks show the current template content attributed to Peter Steinberger, while Tak Hoffman has several recent PR-template commits and requested the automerge replacement in the PR body. (.github/pull_request_template.md:1, 8aff1807fa65)

Likely related people:

Takhoffman: Recent history shows multiple PR-template changes by Tak Hoffman, and the replacement PR body says automerge was requested by this login. (role: recent area contributor and requested follow-up owner; confidence: high; commits: c19321ed9e02, a4e5b23dc37f, 9f0305420aad; files: .github/pull_request_template.md)
Peter Steinberger: Current checkout blame attributes the existing PR template block to commit 8aff180. (role: current template introducer; confidence: medium; commits: 8aff1807fa65; files: .github/pull_request_template.md)

Codex review notes: model gpt-5.5, reasoning high; reviewed against 5e0850fc548c.

clawsweeper · 2026-05-19T11:06:02Z

🦞✅
ClawSweeper merged this PR after the passing review.

Source: clawsweeper[bot]
Feedback: structured ClawSweeper verdict: pass (sha=6c68583faccb029c44eb2dd216a0782817a642de)
Merge status: merged by ClawSweeper automerge
Merged at: 2026-05-19T11:05:58Z
Merge commit: d7083bab4c6e

What merged:

This PR moves the existing Motivation section in .github/pull_request_template.md from below Linked Issue/PR to immediately after Summary without changing the section text.
Reproducibility: not applicable. this is a PR-template ordering cleanup, not a runtime bug. Source inspection of current main and the PR head verifies the before/after section order.

Automerge notes:

PR branch already contained follow-up commit before automerge: chore: move Motivation section above Change Type in PR template

The automerge loop is complete.

Automerge progress:

2026-05-19 11:05:44 UTC review passed 6c68583faccb (structured ClawSweeper verdict: pass (sha=6c68583faccb029c44eb2dd216a0782817a64...)

2026-05-19 11:06:01 UTC merged 6c68583faccb (merged by ClawSweeper automerge)

Summary: - This PR moves the existing Motivation section in `.github/pull_request_template.md` from below Linked Issue/PR to immediately after Summary without changing the section text. - Reproducibility: not applicable. this is a PR-template ordering cleanup, not a runtime bug. Source inspection of current main and the PR head verifies the before/after section order. Automerge notes: - PR branch already contained follow-up commit before automerge: chore: move Motivation section above Change Type in PR template Validation: - ClawSweeper review passed for head 6c68583faccb029c44eb2dd216a0782817a642de. - Required merge gates passed before the squash merge. Prepared head SHA: 6c68583faccb029c44eb2dd216a0782817a642de Review: openclaw/openclaw#84098 (comment) Co-authored-by: Huan Jiang <seraphjiang@gmail.com> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

…claw#84098) Summary: - This PR moves the existing Motivation section in `.github/pull_request_template.md` from below Linked Issue/PR to immediately after Summary without changing the section text. - Reproducibility: not applicable. this is a PR-template ordering cleanup, not a runtime bug. Source inspection of current main and the PR head verifies the before/after section order. Automerge notes: - PR branch already contained follow-up commit before automerge: chore: move Motivation section above Change Type in PR template Validation: - ClawSweeper review passed for head 6c68583. - Required merge gates passed before the squash merge. Prepared head SHA: 6c68583 Review: openclaw#84098 (comment) Co-authored-by: Huan Jiang <seraphjiang@gmail.com> Co-authored-by: clawsweeper <274271284+clawsweeper[bot]@users.noreply.github.com> Co-authored-by: clawsweeper[bot] <274271284+clawsweeper[bot]@users.noreply.github.com> Approved-by: takhoffman Co-authored-by: takhoffman <781889+takhoffman@users.noreply.github.com>

seraphjiang and others added 2 commits May 19, 2026 10:55

chore: move Motivation section above Change Type in PR template (closes

cef536e

#83939)

chore: move Motivation section above Change Type in PR template

6c68583

clawsweeper Bot mentioned this pull request May 19, 2026

chore: move Motivation section above Change Type in PR template #84003

Closed

2 tasks

clawsweeper Bot merged commit d7083ba into main May 19, 2026
76 of 85 checks passed

clawsweeper Bot deleted the clawsweeper/automerge-openclaw-openclaw-84003 branch May 19, 2026 11:06

github-actions Bot mentioned this pull request May 19, 2026

📡 Upstream Digest — 2026-05-19 14:40 UTC curtismercier/openclaw-mods#897

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

chore: move Motivation section above Change Type in PR template#84098

chore: move Motivation section above Change Type in PR template#84098
clawsweeper[bot] merged 2 commits into
mainfrom
clawsweeper/automerge-openclaw-openclaw-84003

clawsweeper Bot commented May 19, 2026

Uh oh!

clawsweeper Bot commented May 19, 2026 •

edited

Loading

Uh oh!

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

clawsweeper Bot commented May 19, 2026

Uh oh!

clawsweeper Bot commented May 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

clawsweeper Bot commented May 19, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

clawsweeper Bot commented May 19, 2026 •

edited

Loading