Skip to content

[Fix] Clear review eyes reaction on completion#71

Open
samzong wants to merge 1 commit into
openclaw:mainfrom
samzong:fix/review-reaction-cleanup
Open

[Fix] Clear review eyes reaction on completion#71
samzong wants to merge 1 commit into
openclaw:mainfrom
samzong:fix/review-reaction-cleanup

Conversation

@samzong
Copy link
Copy Markdown
Contributor

@samzong samzong commented May 14, 2026

What's changed?

  • Keep the existing successful review +1 reaction on event item completion.
  • Remove ClawSweeper-owned eyes reactions from the reviewed issue or pull request after successful completion.
  • Add a workflow regression test for the completion reaction cleanup path.

Why

  • eyes is the in-progress signal; leaving it after a successful review makes completed PRs look like they are still being reviewed.
  • The cleanup filters by ClawSweeper bot logins and issue reaction ids, so user reactions and issue comment reactions are left alone.
  • Verification: pnpm run build, pnpm run test:unit, pnpm run format:check, git diff --check, and the pre-ship committed-diff gate passed. pnpm run check was attempted and reached unrelated existing repair git fixture failures with fatal: Not a valid object name origin/main.

Signed-off-by: samzong <samzong.lu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant