Skip to content

agent sessions - fix chat title sidebar action visibility - #283338

Merged
Benjamin Pasero (bpasero) merged 3 commits into
mainfrom
ben/leading-sloth
Dec 13, 2025
Merged

agent sessions - fix chat title sidebar action visibility#283338
Benjamin Pasero (bpasero) merged 3 commits into
mainfrom
ben/leading-sloth

Conversation

@bpasero

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the visibility of the "Show Agent Sessions" action in the chat title sidebar toolbar by introducing a new context key agentSessionsViewerVisible that tracks whether the agent sessions viewer is currently visible. The action should be shown when the viewer is either hidden or in stacked orientation.

Key Changes

  • Adds a new agentSessionsViewerVisible context key to track agent sessions viewer visibility state
  • Updates the menu conditions for the chat view session title toolbar to show the "Show Agent Sessions" button when the viewer is not visible OR when it's in stacked orientation

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/vs/workbench/contrib/chat/common/chatContextKeys.ts Defines new agentSessionsViewerVisible context key
src/vs/workbench/contrib/chat/browser/chatViewPane.ts Binds the new context key and updates it when sessions container visibility changes
src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.contribution.ts Updates menu item conditions to check visibility state using the new context key

Comment thread src/vs/workbench/contrib/chat/browser/chatViewPane.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.contribution.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.contribution.ts Outdated
@bpasero
Benjamin Pasero (bpasero) merged commit e43b4d6 into main Dec 13, 2025
28 checks passed
@bpasero
Benjamin Pasero (bpasero) deleted the ben/leading-sloth branch December 13, 2025 17:01
@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Jan 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants