Agent status indicators react to chat.viewSessions.enabled

[joshspicer]

fixes #290874

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

• src/vs/workbench/contrib/chat/browser/agentSessions/experiments/agentTitleBarStatusWidget.ts

[Copilot]

Pull request overview

This pull request makes the agent status indicators (unread and in-progress session counts) respect the chat.viewSessions.enabled configuration setting, addressing user feedback that these indicators persist even when sessions are disabled.

Changes:

• Updated documentation for chat.agentsControl.enabled setting to explain the dependency on chat.viewSessions.enabled
• Modified the agent title bar status widget to conditionally render session indicators based on the chat.viewSessions.enabled setting
• Added the new configuration to the widget's state tracking and re-render logic

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/vs/workbench/contrib/chat/browser/chat.contribution.ts	Updated configuration description to document that session indicators require chat.viewSessions.enabled
src/vs/workbench/contrib/chat/browser/agentSessions/experiments/agentTitleBarStatusWidget.ts	Added logic to hide unread and in-progress indicators when sessions are disabled, updated configuration change listener, and included the setting in render state tracking

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/chat/browser/agentSessions/experiments/agentTitleBarStatusWidget.ts:262

• Inconsistent boolean configuration check pattern. Other boolean configurations in this file (lines 260-261) use === true, but this uses !== false. For consistency with the surrounding code, consider using === true instead of !== false.

			const viewSessionsEnabled = this.configurationService.getValue<boolean>(ChatConfiguration.ChatViewSessionsEnabled) !== false;