fix(tui): drop /quit alias 'q' so /q routes to backend /queue by hclsys · Pull Request #31985 · NousResearch/hermes-agent

hclsys · 2026-05-25T09:35:32Z

Fixes #31983.

Summary

In the TUI, typing /q exited Hermes instead of queuing a prompt. The quit command carried alias q (ui-tui/src/app/slash/commands/core.ts), which collides with the Python-side /queue alias q (hermes_cli/commands.py). TUI-local commands dispatch before the backend, so /q always resolved to /quit → ctx.session.die().

Fix

Remove q from the quit aliases (now just ['exit']). /quit and /exit are explicit enough for exiting, so /q falls through to the backend where it correctly maps to /queue.

Test

Added a regression test to src/__tests__/slashParity.test.ts asserting q is not a TUI-local name/alias (so it can't shadow the backend /queue). The existing parity suite only deduped an alias set, so this collision was previously silent.

$ cd ui-tui && npm run build --prefix packages/hermes-ink && ./node_modules/.bin/vitest run src/__tests__/slashParity.test.ts
 Test Files  1 passed (1)
      Tests  3 passed (3)

Verified the test guards the regression: with the q alias restored, the new test fails (1 failed); with the fix, it passes.

Note: npm run type-check reports pre-existing errors in packages/hermes-ink/src/utils/execFileNoThrow.ts and src/app/useMainApp.ts that are present on main independent of this change (confirmed by stashing the diff); this PR touches only the quit alias array and the parity test, both of which lint clean.

The TUI 'quit' command carried alias 'q', which collides with the Python-side /queue alias (hermes_cli/commands.py). TUI-local commands dispatch before the backend, so typing /q exited Hermes (session.die) instead of queueing a prompt for the next turn. Remove 'q' from the quit aliases — /quit and /exit are explicit enough — so /q falls through to the backend where it maps to /queue. Add a slash-parity regression test asserting 'q' is not a TUI-local name/alias. Fixes NousResearch#31983

alt-glitch · 2026-05-25T09:39:11Z

Duplicate of PR #14681 which makes the same change (remove q from quit aliases in core.ts). #14681 has been open since April and is the canonical fix for this alias collision.

hclsys · 2026-05-25T09:41:05Z

Closing in favor of #14681 by @8bit64k, which makes the same q-alias removal and has been open since April (and goes further by also registering /queue as a TUI-local command). Theirs is the canonical fix — I missed it in my dup-scan since it predates the issue #31983 by a month. One thing from here that might be worth folding into #14681 if useful: I added a slash-parity regression test (expect(LOCAL_COMMAND_NAMES.has('q')).toBe(false)) that fails-without / passes-with the alias removal, since the existing parity suite deduped the alias set and so never caught this collision. Happy to PR just that test against #14681's branch or separately if a maintainer wants it. Thanks @8bit64k.

@hclsys

Adapted from @hclsys's test in PR #31985. Asserts findSlashCommand('q') resolves to the queue command, not quit.

@hclsys