Skip to content

fix(tui): /history shows the TUI's own transcript, scrollable#13728

Merged
OutThisLife merged 1 commit into
mainfrom
bb/tui-history-local
Apr 21, 2026
Merged

fix(tui): /history shows the TUI's own transcript, scrollable#13728
OutThisLife merged 1 commit into
mainfrom
bb/tui-history-local

Conversation

@OutThisLife

@OutThisLife OutThisLife commented Apr 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • `/history` in the TUI ran through `slash.exec` → `_SlashWorker` (a detached `HermesCLI` subprocess) whose `conversation_history` starts empty, so the output only surfaced whatever was persisted from earlier CLI sessions
  • The modal also had no scrolling
  • Intercept `/history` as a local slash command that dumps `ctx.local.getHistoryItems()` (the TUI's actual transcript) through the pager, which now scrolls after fix(tui): pager supports scrolling (up/down/page/top/bottom) #13591
  • Accepts an optional preview-length argument (`/history 600`) — defaults to 400 chars per message

Issue

Row 4 from the TUI v2 blitz Confluence: "/history command doesn't register the latest inputs that occur in TUI mode (it only shows prompts from non-TUI Hermes) and also doesn't show the responses to the prompts."

Test plan

Reported during TUI v2 blitz retest: `/history` in the TUI only shows
prompts from non-TUI Hermes runs and can't scroll the window.  Root
cause is the slash-worker subprocess: it's a detached HermesCLI that
never sees the TUI's turns, so its `conversation_history` starts empty
and `show_history` surfaces whatever was persisted from earlier CLI
sessions — not what the user just did inside the TUI.

Intercept `/history` as a local slash command so it dumps
`ctx.local.getHistoryItems()` — the TUI's own transcript — routed
through the pager (which scrolls after #13591).  Accepts an optional
preview-length argument (default 400 chars per message).

Adds createSlashHandler coverage.
@alt-glitch alt-glitch added type/bug Something isn't working comp/tui Terminal UI (ui-tui/ + tui_gateway/) labels Apr 21, 2026
@OutThisLife OutThisLife merged commit 1e5daa4 into main Apr 21, 2026
9 of 10 checks passed
@OutThisLife OutThisLife deleted the bb/tui-history-local branch April 21, 2026 23:59
ulasbilgen pushed a commit to ulasbilgen/hermes-adhd-agent that referenced this pull request May 1, 2026
…y-local

fix(tui): /history shows the TUI's own transcript, scrollable
aj-nt pushed a commit to aj-nt/hermes-agent that referenced this pull request May 1, 2026
…y-local

fix(tui): /history shows the TUI's own transcript, scrollable
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…y-local

fix(tui): /history shows the TUI's own transcript, scrollable
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
…y-local

fix(tui): /history shows the TUI's own transcript, scrollable
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
…y-local

fix(tui): /history shows the TUI's own transcript, scrollable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tui Terminal UI (ui-tui/ + tui_gateway/) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants