Skip to content

fix(gateway): show full session id and title in /status#5912

Closed
xinbenlv wants to merge 1 commit into
NousResearch:mainfrom
xinbenlv:feat/status-full-session-id-title
Closed

fix(gateway): show full session id and title in /status#5912
xinbenlv wants to merge 1 commit into
NousResearch:mainfrom
xinbenlv:feat/status-full-session-id-title

Conversation

@xinbenlv

@xinbenlv xinbenlv commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

Bug Description

/status currently truncates the session id to the first 12 characters, which makes copy/paste and exact session identification annoying. It also omits the session title even when one has already been set.

Root Cause

GatewayRunner._handle_status_command() slices session_entry.session_id[:12] directly and never consults the session database for an existing title.

Fix

  • show the full session id in /status
  • include **Title:** ... when a session title exists
  • keep the output unchanged when no title is set
  • add regression tests for both behaviors

How to Verify

  1. Start a gateway session and run /status
  2. Confirm the full session id is shown, not a truncated prefix
  3. Set a title with /title My Session Name, then run /status again
  4. Confirm the title is displayed in the status output

Test Plan

  • Added regression test for this bug
  • Existing targeted tests still pass
  • Manual verification of the fix

Targeted test run:

  • ~/.hermes/hermes-agent/venv/bin/python -m pytest tests/gateway/test_status_command.py -q -o addopts=''

Risk Assessment

Low — this only changes /status presentation and reads an already-existing title field from the session DB.

@teknium1

teknium1 commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Merged via PR #5942. Your commit was cherry-picked onto current main with authorship preserved in git log. Thanks for the improvement!

@xinbenlv xinbenlv deleted the feat/status-full-session-id-title branch April 9, 2026 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants