Skip to content

fix(gateway): add missing /compress case to command.dispatch in TUI - #52521

Closed
ethanhuang996-alt wants to merge 1 commit into
NousResearch:mainfrom
ethanhuang996-alt:fix/tui-compress-command-routing
Closed

fix(gateway): add missing /compress case to command.dispatch in TUI#52521
ethanhuang996-alt wants to merge 1 commit into
NousResearch:mainfrom
ethanhuang996-alt:fix/tui-compress-command-routing

Conversation

@ethanhuang996-alt

Copy link
Copy Markdown

Summary

The /compress slash command was unavailable in the TUI because command.dispatch lacked a compress case.

Fix

Added compress case routing to _mirror_slash_side_effects.

Verification

Syntax check passed. 9 related tests pass.

The /compress slash command was unavailable in the TUI because
command.dispatch lacked a compress case, falling through to the
error 'not a quick/plugin/skill command: compress'.

The compress handler already exists in _mirror_slash_side_effects
(line 11137) with full session-busy guards and before/after stats.
This fix routes the command from command.dispatch to that handler.

Bug reported: /compress returns error in TUI, blocking manual
compression for users at context limit.
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused routing fix.

This has already been implemented on current main with broader handling. Automated hermes-sweeper review found:

  • c0fbee990e90656fc0fe49d5c237b97958499acc adds command.dispatch support for both /compress and /compact in tui_gateway/server.py:12296, including busy protection, session-key synchronization, and a real compression summary.
  • 7e3986ae686977dd4a4dc6bb3080f0ce2fadb588 additionally routes /compress and /compact past the slash worker via _PENDING_INPUT_COMMANDS (tui_gateway/server.py:11652-11665), avoiding the failing fallback path.

The current implementation therefore provides the requested behavior and stronger safeguards than this PR's patch.

@teknium1 teknium1 closed this Jul 15, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tui Terminal UI (ui-tui/ + tui_gateway/) P2 Medium — degraded but workaround exists sweeper:implemented-on-main Sweeper: behavior already present on current main type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants