opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(whatsapp): apply group_trigger filtering so bot doesn't respond to every group message

Browse source 
WhatsAppConfig was missing the GroupTrigger field, NewWhatsAppChannel wasn't
passing WithGroupTrigger to the base channel, and handleIncomingMessage never
called ShouldRespondInGroup for group chats — so the group_trigger config was
completely ignored and the bot responded to every message.

- Add GroupTrigger GroupTriggerConfig to WhatsAppConfig
- Pass WithGroupTrigger when constructing the channel
- Detect group messages (chatID != senderID) and call ShouldRespondInGroup
- Detect bot mentions from bridge "mentioned" bool or non-empty "mentions" array

Users can now set group_trigger.mention_only: true (or configure prefixes) to
control when the bot responds in WhatsApp groups.

https://claude.ai/code/session_01PfpBofWnnVaNf7uyBEJA4T
This commit is contained in:
Claude 2026-04-01 08:08:13 +00:00 • committed by github-actions[bot]
parent c0b2aa3d52
commit ba57c99699
1 changed files with 13 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
pkg/channels/whatsapp/whatsapp.go
View file

@ -224,16 +224,19 @@ func (c *WhatsAppChannel) handleIncomingMessage(msg map[string]any) {
metadata["user_name"] = userName metadata["user_name"] = userName
} }
isGroup := chatID != senderID
var peer bus.Peer var peer bus.Peer
if chatID == senderID { if !isGroup {
peer = bus.Peer{Kind: "direct", ID: senderID} peer = bus.Peer{Kind: "direct", ID: senderID}
} else { } else {
peer = bus.Peer{Kind: "group", ID: chatID} peer = bus.Peer{Kind: "group", ID: chatID}
} }
logger.InfoCF("whatsapp", "WhatsApp message received", map[string]any{ logger.InfoCF("whatsapp", "WhatsApp message received", map[string]any{
"sender": senderID, "sender": senderID,
"preview": utils.Truncate(content, 50), "preview": utils.Truncate(content, 50),
"is_group": isGroup,
}) })
sender := bus.SenderInfo{ sender := bus.SenderInfo{
@ -249,21 +252,14 @@ func (c *WhatsAppChannel) handleIncomingMessage(msg map[string]any) {
return return
} }
// In group chats, apply unified group trigger filtering. // In group chats, apply group trigger filtering.
// Mention detection: check if the "mentions" field in the message payload // The bridge may signal a bot mention via a "mentioned" bool or a non-empty "mentions" array.
// contains the sender's own JID (proxy for being mentioned by others), if isGroup {
// or use false when no mention info is available.
if peer.Kind == "group" {
isMentioned := false isMentioned := false
if mentionList, ok := msg["mentions"].([]any); ok { if v, ok := msg["mentioned"].(bool); ok {
for _, m := range mentionList { isMentioned = v
if jid, ok := m.(string); ok && jid != "" { } else if mentions, ok := msg["mentions"].([]any); ok && len(mentions) > 0 {
// Any mention in the payload counts as the bot being addressed isMentioned = true
_ = jid
isMentioned = true
break
}
}
} }
respond, cleaned := c.ShouldRespondInGroup(isMentioned, content) respond, cleaned := c.ShouldRespondInGroup(isMentioned, content)
if !respond { if !respond {