Skip to content

fix(cli): show history after resume#16496

Closed
sparklesea wants to merge 2 commits into
NousResearch:mainfrom
sparklesea:fix/resume-history-display
Closed

fix(cli): show history after resume#16496
sparklesea wants to merge 2 commits into
NousResearch:mainfrom
sparklesea:fix/resume-history-display

Conversation

@sparklesea

@sparklesea sparklesea commented Apr 27, 2026

Copy link
Copy Markdown

Summary

  • show the Previous Conversation panel after using /resume mid-session
  • replay the current session history when /resume targets the active session
  • hide pure tool-call assistant messages from resume history recap while preserving assistant text with tool summaries

Test Plan

  • ./venv/bin/python -m pytest tests/cli/test_resume_display.py -q

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/cli CLI entry point, hermes_cli/, setup wizard labels Apr 27, 2026
@alt-glitch alt-glitch mentioned this pull request Apr 29, 2026
Closed
@teknium1 teknium1 mentioned this pull request May 24, 2026
Merged
@teknium1

teknium1 commented May 24, 2026

Copy link
Copy Markdown
Contributor

Closing as superseded by PR #31695 (merged), which bundles this fix and several sibling PRs in the /resume recap cluster.

PR #31695 wires _display_resumed_history() into in-session /resume (the missing 1-line bug fix) AND exposes the recap truncation limits + tool-call-only skip as configurable display.* keys. Eight contributors independently filed PRs against this same UX gap — thanks for being one of them.

Authorship for the cherry-picked commits is preserved in git log via rebase-merge. If your PR included specific behavior that PR #31695 missed, please reopen with a follow-up scoped to that delta.

(Bulk-closed during a CLI PR triage sweep.)

@teknium1 teknium1 closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sparklesea @teknium1 @alt-glitch