feat(tui): describe the highlighted session in the sessions panel header #71

Open
Taha wants to merge 1 commit from taha/tui-sessions-header into main AGit
Contributor

Single commit on main (86c3dfc6). While browsing the /resume selector (/the sessions panel), the brand header now describes the highlighted session instead of always describing this client: the highlighted session creation date, its working directory (the endpoint directory for remote sessions), its model, cost, message count and endpoint replace the endpoints summary line; the cwd line shows the highlighted session directory instead of yours. The current session, an empty roster and a search with no match keep the client's own header, the height never changes so the roster does not jump, and the highlighted row is part of the header memo key so a cursor move repaints it. agents.view.list rows gain CreatedAt from the session store; the generated plugin surface is unchanged (the method result is typed as a generic array and web reads fields tolerantly), verified byte-identical after running the generator. Docs: new section in docs/keybindings.md. Validation: tui2 + targeted app tests green; five behavioural tests fail against the old code (header follows the cursor, client cwd kept for the current session, repaint on cursor move, remote cwd, CreatedAt on the wire); parity goldens unchanged; render line-budget raised per its own convention (measured 21099/21000). check.sh --fast green.

Single commit on main (86c3dfc6). While browsing the /resume selector (/the sessions panel), the brand header now describes the highlighted session instead of always describing this client: the highlighted session creation date, its working directory (the endpoint directory for remote sessions), its model, cost, message count and endpoint replace the endpoints summary line; the cwd line shows the highlighted session directory instead of yours. The current session, an empty roster and a search with no match keep the client's own header, the height never changes so the roster does not jump, and the highlighted row is part of the header memo key so a cursor move repaints it. agents.view.list rows gain CreatedAt from the session store; the generated plugin surface is unchanged (the method result is typed as a generic array and web reads fields tolerantly), verified byte-identical after running the generator. Docs: new section in docs/keybindings.md. Validation: tui2 + targeted app tests green; five behavioural tests fail against the old code (header follows the cursor, client cwd kept for the current session, repaint on cursor move, remote cwd, CreatedAt on the wire); parity goldens unchanged; render line-budget raised per its own convention (measured 21099/21000). check.sh --fast green.
While browsing /resume the header above the roster always described this
client - its cwd and the endpoints summary - so it said nothing about the
session the cursor was about to open. When the cursor rests on another
session, the endpoints line now carries that session's creation date, model,
cost, message count and endpoint, and the cwd line shows its working
directory (the endpoint's directory for a session on an SSH endpoint). The
current session, an empty roster and a search with no match keep the
client's own header. The height never changes, so the roster does not jump,
and the highlighted row is part of the header memo key so a cursor move
repaints it.

agents.view.list rows gain CreatedAt from the session store. The generated
plugin surface is unchanged: it types the result as a generic array.

header_session_test.go and agents_view_created_test.go fail against the old
header and roster.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin '+refs/pull/71/head:taha/tui-sessions-header'
git switch 'taha/tui-sessions-header'

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch 'main'
git merge --no-ff 'taha/tui-sessions-header'
git switch 'taha/tui-sessions-header'
git rebase 'main'
git switch 'main'
git merge --ff-only 'taha/tui-sessions-header'
git switch 'taha/tui-sessions-header'
git rebase 'main'
git switch 'main'
git merge --no-ff 'taha/tui-sessions-header'
git switch 'main'
git merge --squash 'taha/tui-sessions-header'
git switch 'main'
git merge --ff-only 'taha/tui-sessions-header'
git switch 'main'
git merge 'taha/tui-sessions-header'
git push origin 'main'
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kreato/yazi!71
No description provided.