Skip to content

fix: Implement /resume slash command (Issue #2869)#2893

Open
ten-jampa wants to merge 2 commits intoNousResearch:mainfrom
ten-jampa:fix-resume-command
Open

fix: Implement /resume slash command (Issue #2869)#2893
ten-jampa wants to merge 2 commits intoNousResearch:mainfrom
ten-jampa:fix-resume-command

Conversation

@ten-jampa
Copy link
Copy Markdown
Contributor

Summary

Fixes Issue #2869: Session history not persisting between restarts

Problem

When Hermes was closed and reopened, showed empty and showed "Unknown command" - session history was not persisting between restarts.

Root Cause

The command was registered in the command registry but not implemented as a slash command handler - it only worked via the CLI flag when starting Hermes.

Solution

  1. Add slash command handler in :

    • — Lists available saved sessions
    • — Resumes that session and loads history from the database
  2. Improve command:

    • Now loads history from database if not in current session
    • Shows saved sessions when no history is available
    • Provides helpful prompt to use
  3. Fix NoneType crash:

    • Handle explicitly null field in session listing to prevent error

Testing

hermes
/resume              # Lists saved sessions
/resume <session-id> # Resumes session with full history
/history             # Shows history from database

Files Changed

  • — Added /resume handler, improved /history, fixed NoneType handling

Prevents workflow failure when posting PR comment fails (e.g., fork PRs)
- Implement /resume slash command with session listing
- Add saved sessions display in /history when empty
- Remove [SILENT] system prompt that caused cron output truncation
- Resolve merge conflicts with origin/main
@ten-jampa ten-jampa force-pushed the fix-resume-command branch from 1f60b7b to 5be005f Compare April 4, 2026 14:20
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.

1 participant