"""Parser-only and lightweight routing tests for send_message targets. These stay separate from ``test_send_message_tool.py`` because that module skips wholesale when optional Telegram dependencies are not installed. """ import asyncio import json from types import SimpleNamespace from unittest.mock import AsyncMock, patch from gateway.config import Platform from tools.send_message_tool import _parse_target_ref, send_message_tool def _run_async_immediately(coro): return asyncio.run(coro) def test_photon_e164_target_is_explicit() -> None: chat_id, thread_id, is_explicit = _parse_target_ref("photon", "+15551234567") assert chat_id == "+15551234567" assert thread_id is None assert is_explicit is True def test_e164_target_still_requires_phone_platform() -> None: assert _parse_target_ref("matrix", "+15551234567")[2] is False def test_send_message_routes_whatsapp_group_jid_without_home_fallback() -> None: whatsapp_cfg = SimpleNamespace(enabled=True, token=None, extra={"api_url": "http://bridge"}) config = SimpleNamespace( platforms={Platform.WHATSAPP: whatsapp_cfg}, get_home_channel=lambda _platform: SimpleNamespace(chat_id="15551234567@s.whatsapp.net"), ) with patch("gateway.config.load_gateway_config", return_value=config), \ patch("tools.interrupt.is_interrupted", return_value=False), \ patch("gateway.channel_directory.resolve_channel_name", side_effect=AssertionError("raw JID should not resolve via directory")), \ patch("model_tools._run_async", side_effect=_run_async_immediately), \ patch("tools.send_message_tool._send_to_platform", new=AsyncMock(return_value={"success": True})) as send_mock, \ patch("gateway.mirror.mirror_to_session", return_value=True): result = json.loads( send_message_tool( { "action": "send", "target": "whatsapp:120363408391911677@g.us", "message": "hello group", } ) ) assert result["success"] is True assert "note" not in result send_mock.assert_awaited_once_with( Platform.WHATSAPP, whatsapp_cfg, "120363408391911677@g.us", "hello group", thread_id=None, media_files=[], force_document=False, ) def test_resolved_opaque_plugin_target_uses_directory_id() -> None: from gateway.platform_registry import PlatformEntry, platform_registry platform_name = "opaque-resolved-test" entry = PlatformEntry( name=platform_name, label="Opaque resolved test", adapter_factory=lambda cfg: None, check_fn=lambda: True, ) platform_registry.register(entry) platform = Platform(platform_name) pconfig = SimpleNamespace(enabled=True, token=None, extra={}) config = SimpleNamespace( platforms={platform: pconfig}, get_home_channel=lambda _platform: None, ) try: with patch("gateway.config.load_gateway_config", return_value=config), \ patch("tools.interrupt.is_interrupted", return_value=False), \ patch( "gateway.channel_directory.resolve_channel_name", return_value="opaque:directory-id", ), \ patch("model_tools._run_async", side_effect=_run_async_immediately), \ patch( "tools.send_message_tool._send_to_platform", new=AsyncMock(return_value={"success": True}), ) as send_mock, \ patch("gateway.mirror.mirror_to_session", return_value=True): result = json.loads( send_message_tool( { "action": "send", "target": f"{platform_name}:Friendly name", "message": "hello", } ) ) finally: platform_registry.unregister(platform_name) assert result["success"] is True send_mock.assert_awaited_once_with( platform, pconfig, "opaque:directory-id", "hello", thread_id=None, media_files=[], force_document=False, ) def test_unresolved_plugin_target_requires_explicit_parser() -> None: from gateway.platform_registry import PlatformEntry, platform_registry platform_name = "opaque-verbatim-test" entry = PlatformEntry( name=platform_name, label="Opaque verbatim test", adapter_factory=lambda cfg: None, check_fn=lambda: True, ) platform_registry.register(entry) platform = Platform(platform_name) # Simulate a fresh `hermes send` process: the dynamic Platform member # is known from config, but plugin discovery has not registered its # adapter entry yet. platform_registry.unregister(platform_name) pconfig = SimpleNamespace(enabled=True, token=None, extra={}) config = SimpleNamespace( platforms={platform: pconfig}, get_home_channel=lambda _platform: None, ) try: with patch("gateway.config.load_gateway_config", return_value=config), \ patch("tools.interrupt.is_interrupted", return_value=False), \ patch( "gateway.channel_directory.resolve_channel_name", return_value=None, ), \ patch( "hermes_cli.plugins.discover_plugins", side_effect=lambda: platform_registry.register(entry), ) as discover_mock, \ patch("model_tools._run_async", side_effect=_run_async_immediately), \ patch( "tools.send_message_tool._send_to_platform", new=AsyncMock(return_value={"success": True}), ) as send_mock, \ patch("gateway.mirror.mirror_to_session", return_value=True): result = json.loads( send_message_tool( { "action": "send", "target": f"{platform_name}:dm:panyaozhen", "message": "hello", } ) ) finally: platform_registry.unregister(platform_name) assert result == { "error": f"Could not resolve 'dm:panyaozhen' on {platform_name}. " "The plugin parser did not recognize it and no channel-directory entry matched." } discover_mock.assert_called_once_with() send_mock.assert_not_awaited() def test_unresolved_builtin_target_keeps_directory_error() -> None: telegram_cfg = SimpleNamespace(enabled=True, token="***", extra={}) config = SimpleNamespace( platforms={Platform.TELEGRAM: telegram_cfg}, get_home_channel=lambda _platform: None, ) with patch("gateway.config.load_gateway_config", return_value=config), \ patch("gateway.channel_directory.resolve_channel_name", return_value=None), \ patch( "tools.send_message_tool._send_to_platform", new=AsyncMock(return_value={"success": True}), ) as send_mock: result = json.loads( send_message_tool( { "action": "send", "target": "telegram:missing-room", "message": "hello", } ) ) assert result == { "error": "Could not resolve 'missing-room' on telegram. " "Use send_message(action='list') to see available targets." } send_mock.assert_not_awaited()