TUI /q alias conflict: quit vs queue

Fixes on record

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

• #10538 fix: remove 'q' alias from /quit so /queue's 'q' alias works

  merged to main

  files: hermes_cli/commands.py, tests/hermes_cli/test_commands.py

  Show the diff

  • hermes_cli/commands.py

    @@ -164,7 +164,7 @@ class CommandDef:
     
         # Exit
         CommandDef("quit", "Exit the CLI", "Exit",
    -               cli_only=True, aliases=("exit", "q")),
    +               cli_only=True, aliases=("exit",)),
     ]
     
     

  • tests/hermes_cli/test_commands.py

    @@ -97,7 +97,7 @@ def test_canonical_name_resolves(self):
         def test_alias_resolves_to_canonical(self):
             assert resolve_command("bg").name == "background"
             assert resolve_command("reset").name == "new"
    -        assert resolve_command("q").name == "quit"
    +        assert resolve_command("q").name == "queue"
             assert resolve_command("exit").name == "quit"
             assert resolve_command("gateway").name == "platforms"
             assert resolve_command("set-home").name == "sethome"

• #32074 fix(tui): remap /q alias from quit to queue (salvage #14681 + #31985)

  merged to main

  files: scripts/release.py, ui-tui/src/__tests__/slashParity.test.ts, ui-tui/src/app/slash/commands/core.ts

  Show the diff

  • scripts/release.py

    @@ -1308,6 +1308,8 @@
         "[redacted:email]": "wanwan2qq",  # PR #10215 (strip brackets/quotes from /resume; gateway session-ID lookup)
         "[redacted:email]": "SimoKiihamaki",  # PR #30773 (Windows /reset+/new freeze; stdin fallback for modal)
         "[redacted:email]": "Tranquil-Flow",  # PR #27518 (bracketed-paste timeout)
    +    "[redacted:email]": "8bit64k",  # PR #14681 (TUI /q alias from quit to queue)
    +    "[redacted:email]": "hclsys",  # PR #31985 (TUI /q alias regression test)
     }
     
     

  • ui-tui/src/__tests__/slashParity.test.ts

    @@ -4,7 +4,7 @@ import { fileURLToPath } from 'node:url'
     
     import { describe, expect, it } from 'vitest'
     
    -import { SLASH_COMMANDS } from '../app/slash/registry.js'
    +import { findSlashCommand, SLASH_COMMANDS } from '../app/slash/registry.js'
     
     type CommandRoute = 'fallback' | 'local' | 'native'
     
    @@ -110,4 +110,14 @@ describe('slash parity matrix', () => {
           expect(routes[name], `mutating command must not fallback: ${name}`).not.toBe('fallback')
         }
       })
    +
    +  it('/q alias resolves to queue, not quit (#31983)', () => {
    +    // Regression for #31983: the TUI `quit` command used to carry alias `q`,
    +    // which collided with the Python-side `/queue` alias. TUI-local commands
    +    // dispatch before the backend, so `/q` resolved to /quit (session.die)
    +    // instead of queueing a prompt.
    +    const cmd = findSlashCommand('q')
    +    expect(cmd, '/q must resolve to a command').toBeDefined()
    +    expect(cmd!.name).toBe('queue')
    +  })
     })

  • ui-tui/src/app/slash/commands/core.ts

    @@ -110,7 +110,7 @@ export const coreCommands: SlashCommand[] = [
       },
     
       {
    -    aliases: ['exit', 'q'],
    +    aliases: ['exit'],
         help: 'exit hermes',
         name: 'quit',
         run: (_arg, ctx) => ctx.session.die()
    @@ -547,6 +547,7 @@ export const coreCommands: SlashCommand[] = [
       },
     
       {
    +    aliases: ['q'],
         help: 'inspect or enqueue a message',
         name: 'queue',
         run: (arg, ctx) => {