From 78122c52cf9c66f67e499ae80774b27adce3d86d Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Fri, 5 Jun 2026 17:53:42 -0700 Subject: [PATCH] test(slack): drop /q alias assertion now displaced by /version cap clamp Slack's native-slash manifest hard-caps at 50 (_SLACK_MAX_SLASH_COMMANDS). Adding the /version canonical claims a pass-1 slot, so the lowest-priority pass-2 alias (/q for /quit) clamps off the end. /q stays reachable via /hermes q. Surviving aliases (/btw /bg /reset) still prove alias parity. --- tests/hermes_cli/test_commands.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/tests/hermes_cli/test_commands.py b/tests/hermes_cli/test_commands.py index b6e11e4c517..562f54a5e0a 100644 --- a/tests/hermes_cli/test_commands.py +++ b/tests/hermes_cli/test_commands.py @@ -336,13 +336,19 @@ class TestSlackNativeSlashes: ) def test_includes_aliases_as_first_class_slashes(self): - """Aliases (/btw, /bg, /reset, /q) must be registered as standalone - slashes — this is the whole point of native-slashes parity.""" + """Aliases (/btw, /bg, /reset) must be registered as standalone + slashes — this is the whole point of native-slashes parity. + + Note: Slack's manifest hard-caps slash commands at 50 + (``_SLACK_MAX_SLASH_COMMANDS``). Canonical names win slots first, + then aliases, so the lowest-priority aliases can be clamped off + once the registry fills the cap (e.g. ``/q`` once ``/version`` + landed). The surviving aliases below still prove alias parity; + anything dropped remains reachable via ``/hermes ``.""" names = {n for n, _d, _h in slack_native_slashes()} assert "btw" in names assert "bg" in names assert "reset" in names - assert "q" in names def test_telegram_parity(self): """Every Telegram bot command must be registerable on Slack too.