Skip to content

feat(cli): improve resumed session recap#6458

Closed
xinbenlv wants to merge 3 commits into
NousResearch:mainfrom
xinbenlv:feat/resume-summary-tail
Closed

feat(cli): improve resumed session recap#6458
xinbenlv wants to merge 3 commits into
NousResearch:mainfrom
xinbenlv:feat/resume-summary-tail

Conversation

@xinbenlv

@xinbenlv xinbenlv commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • improve the resumed-session recap shown in the CLI
  • make resume output more useful by surfacing clearer recent context
  • keep the final branch free of the temporary demo artifacts used during iteration

Downstream carry

  • Carried patch: patch-upstream-pr-6458

Previous PR

Test Plan

  • Updated tests/test_resume_display.py

@teknium1

teknium1 commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Thanks for the effort here — the idea of improving the resume recap is solid and something we want to do.

We're going to take a different approach for this though. When context compression fires, we already generate a structured LLM summary (Goal, Progress, Decisions, Files, Next Steps) that lives in the conversation history. The plan is to surface that existing summary in the resume panel rather than reconstructing one from truncated message text — it gives a much richer and more accurate recap for free.

A v2 of the CLI resume UX is coming soon. If it doesn't cover what you were going for here, happy to re-review. Appreciate the contribution.

@teknium1 teknium1 closed this Apr 9, 2026
xinbenlv added a commit to xinbenlv/zn-hermes-agent that referenced this pull request Apr 10, 2026
## Why carried
- Makes resumed sessions less opaque by showing a clearer recap and recent context.
- Fixes a real CLI annoyance instead of forcing the user to infer what happened before the resume.

## Upstream
- PR: NousResearch#6458
- Status: open

## Summary
- Improves the recap shown after resuming a session.
- Keeps the resumed-session UX focused on useful context instead of noisy filler.

## Drop condition
- Drop once PR NousResearch#6458 lands upstream and local `main` rebases onto the upstream commit.

## Files
- `cli.py`
- `tests/cli/test_resume_display.py`
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