Skip to content

hide chat participants @ contributed via chat sessions api#292953

Merged
joshspicer merged 1 commit intomainfrom
josh/at
Feb 4, 2026
Merged

hide chat participants @ contributed via chat sessions api#292953
joshspicer merged 1 commit intomainfrom
josh/at

Conversation

@joshspicer
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 4, 2026 22:33
@joshspicer joshspicer enabled auto-merge (squash) February 4, 2026 22:33
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 4, 2026
Copy link
Contributor

Copilot AI left a comment

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 hides chat participants (agents) that are contributed via the chat sessions API from appearing in the agent completion list and removes their @ prefix description from the delegation session picker dropdown.

Changes:

  • Filter out chat session-contributed agents from the @ agent completions list
  • Remove the @ prefix description display for chat session contributions in the delegation session picker

Reviewed changes

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

File Description
src/vs/workbench/contrib/chat/browser/widget/input/editor/chatInputCompletions.ts Added filter to exclude agents whose IDs match chat session contribution types from the agent completions list
src/vs/workbench/contrib/chat/browser/widget/input/delegationSessionPickerActionItem.ts Removed override of _getSessionDescription method that was displaying @${contribution.name} for chat session contributions, falling back to base class implementation that returns undefined

@joshspicer joshspicer merged commit 1ee5226 into main Feb 4, 2026
27 of 28 checks passed
@joshspicer joshspicer deleted the josh/at branch February 4, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants