From a54d4b0e46429eb2d13bd41145c74c5e863d1e49 Mon Sep 17 00:00:00 2001 From: Quarkex <2879008+Quarkex@users.noreply.github.com> Date: Tue, 12 May 2026 17:11:50 -0700 Subject: [PATCH] fix(send_message): recognize XMPP JIDs as explicit targets _parse_target_ref() has no handler for XMPP JIDs (user@server or room@conference.server), so they fall through to the final `return None, None, False`. This causes send_message to fail when targeting an XMPP chat by JID, since the JID is not numeric and doesn't match any other platform pattern. Add an explicit check for XMPP targets containing '@', matching the existing Matrix pattern above it. --- tools/send_message_tool.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/send_message_tool.py b/tools/send_message_tool.py index c8d84fdf213..664c8736a12 100644 --- a/tools/send_message_tool.py +++ b/tools/send_message_tool.py @@ -355,6 +355,9 @@ def _parse_target_ref(platform_name: str, target_ref: str): # Matrix room IDs (start with !) and user IDs (start with @) are explicit if platform_name == "matrix" and (target_ref.startswith("!") or target_ref.startswith("@")): return target_ref, None, True + # XMPP JIDs (user@server or room@conference.server) are explicit + if platform_name == "xmpp" and "@" in target_ref: + return target_ref, None, True return None, None, False