Skip to content

fix(commands): make /sessions CLI-only and functional - #23680

Closed
rylena wants to merge 1 commit into
NousResearch:mainfrom
rylena:fix/issue-23533-sessions-slash-command
Closed

fix(commands): make /sessions CLI-only and functional#23680
rylena wants to merge 1 commit into
NousResearch:mainfrom
rylena:fix/issue-23533-sessions-slash-command

Conversation

@rylena

@rylena rylena commented May 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • mark /sessions as CLI-only so gateway platforms no longer advertise or silently swallow it
  • wire the classic CLI slash dispatcher to show recent sessions inline when /sessions is used
  • add regression tests for command registration and CLI slash handling

Verification

  • scripts/run_tests.sh tests/hermes_cli/test_commands.py tests/cli/test_cli_init.py
  • independent delegated code review: passed

Notes

  • Existing authored PR fix(cli): report passwordless sudo in status #18315 was inspected first; its current red checks reproduce on upstream/main and were not actionable in this run.
  • Codex was attempted first, but it failed with the known sandbox / CA-bundle permission errors and a usage-limit stop, so Hermes handled the fallback implementation.

Closes #23533

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have labels May 11, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Overlaps with #23577 which also marks /sessions as cli_only. This PR additionally wires the handler. Related issue: #23533.

@rylena

rylena commented May 13, 2026

Copy link
Copy Markdown
Contributor Author

Closing this as overlapping with #23577, which already carries the CLI-only /sessions change for #23533. I am not refreshing or pushing this duplicate branch further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: /sessions command is registered but has no handler — silent no-op in gateway and CLI

2 participants