From ba57c99699b40f166cdfa774898b2b8a0dc8ba5e Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 1 Apr 2026 08:08:13 +0000
Subject: [PATCH] fix(whatsapp): apply group_trigger filtering so bot doesn't
 respond to every group message
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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
---
 pkg/channels/whatsapp/whatsapp.go | 30 +++++++++++++-----------------
 1 file changed, 13 insertions(+), 17 deletions(-)

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