fix: clawtributor claims ping Community Team instead of moderation subscribers - #42
Merged
Merged
Conversation
Use the canonical moderation notification role for the claim card and its allowed mentions. Leave review configuration and existing authorization behavior unchanged. Co-authored-by: hannesrudolph <49103247+hannesrudolph@users.noreply.github.com>
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
…bscribers OpenClaw-Publication: 78cbd5da-f9dd-45db-8926-2c0021688ee5 Co-authored-by: hannesrudolph <49103247+hannesrudolph@users.noreply.github.com>
roboclaw-bot
force-pushed
the
openclaw/fix-claim-moderation-pings-20260911
branch
from
September 11, 2026 14:32
c13cd4c to
9b8cede
Compare
roboclaw-bot
marked this pull request as ready for review
September 11, 2026 14:32
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: fix: clawtributor claims ping Community Team instead of moderation subscribers This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Problem This Solves
Resolves a problem where submitting a Clawtributor claim notified the general Community Team instead of Community Moderation Pings.
Why This Change Was Made
Claim notifications now use the existing canonical moderation-ping setting for both the visible Carbon v2 mention and allowed mentions. Review configuration, claim controls, and existing authorization behavior are unchanged. This changes only claim notifications: no Barnacle changes, Discord settings changes, edits to existing messages, or live test pings.
User Impact
New claim requests notify Community Moderation Pings (
1546936406272778271), not Community Team (1477360613125787678). The Community Team reviewer setting remains unchanged. This patch does not add or claim to enforce a reviewer-role gate in the existing claim handlers.Evidence
bun run typecheck: passed.bun run typecheck: passed.bun run deploy:dry-run: passed; no production deployment run.git diff --check, and focused read-only review: passed/no blocking findings.Worked on by: @hannesrudolph