[cherry-pick] sessions: surface archived state in session header#320440
Merged
Conversation
Contributor
Author
sandy081
approved these changes
Jun 8, 2026
dmitrivMS
approved these changes
Jun 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry-pick of #320392 from
main.What
In the Agents window, an archived ("done") session is currently almost invisible — the only signals are missing actions and the Restore button hidden away in the title bar's command center.
This PR consolidates the archived-session UI on the session header:
getStatusIconnow returnsCodicon.passFilled(in the muted read-indicator color) for archived sessions, instead of reusing the plain read dot. Applies wherever the shared status icon is used (header, sessions list, picker).Menus.CommandCenter(title bar pill area); now contributed toMenus.SessionBarToolbar,navigationgroup (order 5), so it sits where New Chat would otherwise be. Gated by the session-scopedSessionIsArchivedContext, which is the right scope for a per-session header toolbar.SessionIsArchivedContext.negate()added to bothwhenclauses.Why
A done session now reads as done at a glance (check glyph in the header), and the Restore affordance lives next to the session it acts on, rather than across the window in the title bar. The header toolbar stays focused on actions that are actually meaningful for the current state.
Notes for reviewers
SessionIsArchivedContextalready bound insessionView.ts(and used elsewhere for header context-menu gating).compile-check-ts-native) and layers check both pass.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com