openclaw/imsg
openclaw/imsg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 7 Wiki Activity

fix: isGroupHandle now correctly detects group chats

Browse Source 
The previous implementation preferred `identifier` over `guid` when
checking for group chat prefixes. For group chats, `identifier` contains
the bare GUID (e.g. `70012e07fc54...`) which never has the `;+;` prefix,
while `guid` contains the prefixed form (e.g. `any;+;70012e07fc54...`).

Since `identifier` is never empty (ChatInfo always populates it), the
`guid` field was never checked, causing `is_group` to always return
`false` for group chats in RPC output.

Also removed the `;-;` check since that prefix indicates DMs, not groups.

Fix: always check both `guid` and `identifier` for the `;+;` group prefix.
This commit is contained in:
safetynotgauranteed 2026-02-06 12:52:50 -08:00 committed by Peter Steinberger
parent deb01a0ef7
commit f849fcab14
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Sources/imsg/RPCPayloads.swift
View File

@ -96,8 +96,7 @@ func reactionPayload(_ reaction: Reaction) -> [String: Any] {
}
func isGroupHandle(identifier: String, guid: String) -> Bool {
let handle = identifier.isEmpty ? guid : identifier
return handle.contains(";+;") || handle.contains(";-;")
return guid.contains(";+;") || identifier.contains(";+;")
}
func stringParam(_ value: Any?) -> String? {