Mattermost slash command handling and registration improvements

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #59127 fix(mattermost): accept leading-space slash commands

  merged to main

  files: plugins/platforms/mattermost/adapter.py, scripts/release.py, tests/gateway/test_mattermost.py

  Show the diff

  • plugins/platforms/mattermost/adapter.py

    @@ -878,6 +878,8 @@ async def _handle_ws_event(self, event: Dict[str, Any]) -> None:
             # Determine message type.
             file_ids = post.get("file_ids") or []
             msg_type = MessageType.TEXT
    +        if message_text[:1].isspace() and message_text.lstrip().startswith("/"):
    +            message_text = message_text.lstrip()
             if message_text.startswith("/"):
                 msg_type = MessageType.COMMAND
     

  • scripts/release.py

    @@ -45,6 +45,7 @@
     
     # Auto-extracted from noreply emails + manual overrides
     AUTHOR_MAP = {
    +    "[redacted:email]": "l0h1nth",  # PR #32210 salvage (mattermost: accept leading-space slash commands from mobile clients; #25184)
         "[redacted:email]": "devatnull",  # PR #58704 salvage (whatsapp: native Baileys polls, clarify-as-poll, location pins, structured quoted replies, PTT/audio split, bridge_helpers extraction)
         "[redacted:email]": "tjp2021",  # PR #4097 salvage (copilot: per-turn x-initiator header so user prompts bill as premium requests; #3040)
         "[redacted:email]": "kevinrajaram",  # PR #3850 salvage (gateway: add POSIX system dirs to PATH so launchctl/systemctl resolve under UV's minimal-PATH bundled Python; #3849)

  • tests/gateway/test_mattermost.py

    @@ -6,6 +6,7 @@
     from unittest.mock import MagicMock, patch, AsyncMock
     
     from gateway.config import Platform, PlatformConfig
    +from gateway.platforms.base import MessageType
     from gateway.run import (
         _resolve_gateway_display_bool,
         _resolve_progress_thread_id,
    @@ -588,6 +589,55 @@ async def test_channel_type_mapping(self):
             msg_event = self.adapter.handle_message.call_args[0][0]
             assert msg_event.source.chat_type == "dm"
     
    +    @pytest.mark.asyncio
    +    async def test_leading_space_slash_command_is_command(self):
    +        """Mattermost mobile suggests leading-space slash commands."""
    +        post_data = {
    +            "id": "post_cmd",
    +            "user_id": "user_123",
    +            "channel_id": "chan_dm",
    +            "message": " /new",
    +        }
    +        event = {
    +            "event": "posted",
    +            "data": {
    +                "post": json.dumps(post_data),
    +                "channel_type": "D",
    +                "sender_name": "@bob",
    +            },
    +        }
    +
    +        await self.adapter._handle_ws_event(event)
    +        assert self.adapter.handle_message.called
    +        msg_event = self.adapter.handle_message.call_args[0][0]
    +        assert msg_event.text == "/new"
    +        assert msg_event.message_type is MessageType.COMMAND
    +        assert msg_event.get_command() == "new"
    +
    +    @pytest.mark.asyncio
    +    async def test_leading_space_normal_text_is_preserved(self):
    +        """Only command-shaped mobile messages should be normalized."""
    +        post_data = {
    +            "id": "post_text",
    +            "user_id": "user_123",
    +            "channel_id": "chan_dm",
    +            "message": " hello",
    +        }
    +        event = {
    +            "event": "posted",
    +            "data": {
    +                "post": json.dumps(post_data),
    +                "channel_type": "D",
    +                "sender_name": "@bob",
    +            },
    +        }
    +
    +        await self.adapter._handle_ws_event(event)
    +        assert self.adapter.handle_message.called
    +        msg_event = self.adapter.handle_message.call_args[0][0]
    +        assert msg_event.text == " hello"
    +        assert msg_event.message_type is MessageType.TEXT
    +
         @pytest.mark.asyncio
         async def test_thread_id_from_root_id(self):
             """Post with root_id should have thread_id set."""