Skip to content

feat(cli): improve transcript visibility and resume history#12270

Closed
lumenradley wants to merge 1 commit into
NousResearch:mainfrom
lumenradley:feat/transcript-visibility
Closed

feat(cli): improve transcript visibility and resume history#12270
lumenradley wants to merge 1 commit into
NousResearch:mainfrom
lumenradley:feat/transcript-visibility

Conversation

@lumenradley

Copy link
Copy Markdown
Contributor

What does this PR do?

Improves CLI transcript visibility around resumed sessions and clarify prompts. Resume history now supports richer display modes, and clarify responses are echoed back into scrollback so the conversation transcript reflects the user’s actual choice.

Related Issue

Fixes #TBD

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 🔒 Security fix
  • 📝 Documentation update
  • ✅ Tests (adding or improving test coverage)
  • ♻️ Refactor (no behavior change)
  • 🎯 New skill (bundled or hub)

Changes Made

  • Added richer resume-history handling in cli.py.
  • Added clarify transcript echoing in cli.py.
  • Updated resume-display defaults/schema handling in hermes_cli/config.py and hermes_cli/web_server.py.
  • Added coverage in tests/cli/test_cli_clarify_history.py and tests/cli/test_resume_display.py.

How to Test

  1. Run source /Volumes/Shared/turbo/hermes-agent/venv/bin/activate && scripts/run_tests.sh.
  2. Run scripts/run_tests.sh tests/cli/test_cli_clarify_history.py tests/cli/test_resume_display.py -q.
  3. Resume a prior session and verify summary vs history rendering.
  4. Trigger a clarify prompt and verify the selected answer is echoed into scrollback.

Checklist

Code

  • I've read the Contributing Guide
  • My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
  • I searched for existing PRs to make sure this isn't a duplicate
  • My PR contains only changes related to this fix/feature (no unrelated commits)
  • I've run pytest tests/ -q and all tests pass
  • I've added tests for my changes (required for bug fixes, strongly encouraged for features)
  • I've tested on my platform: macOS 15

Documentation & Housekeeping

  • I've updated relevant documentation (README, docs/, docstrings) — or N/A
  • I've updated cli-config.yaml.example if I added/changed config keys — or N/A
  • I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
  • I've considered cross-platform impact (Windows, macOS) per the compatibility guide — or N/A
  • I've updated tool descriptions/schemas if I changed tool behavior — or N/A

Screenshots / Logs

  • Full suite on branch feat/transcript-visibility (d9e817f7506d): 35 failed, 12914 passed, 38 skipped.
  • Clean upstream/main baseline on this machine: 34 failed, 12909 passed, 38 skipped.
  • The branch still has a small net regression versus baseline; see tests-feat-transcript-visibility-d9e817f7506d.delta.txt.

@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard labels Apr 23, 2026
@teknium1

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

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants