chat: simplify Agent Host Copilot label#322179
Merged
roblourens merged 6 commits intoJun 20, 2026
Merged
Conversation
Use the shorter Copilot label for the Agent Host Copilot session when the local editor agent is disabled, while keeping the Agent Host-disambiguated name when both implementations can appear. (Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Agent Host “Copilot” provider naming so it can display as “Copilot” when chat.editor.localAgent.enabled is disabled, while preserving the disambiguated label when both Copilot implementations may appear. The change is applied across key UI surfaces (picker, hover, filter, Agents window) and verified with additional unit tests.
Changes:
- Extend
getAgentSessionProviderNameto optionally consult configuration and conditionally simplify the Agent Host Copilot label. - Thread
IConfigurationServicethrough relevant workbench UI components so labels remain consistent (picker, hover, filter, model). - Update agent-host session providers and tests to use the conditional naming behavior.
Show a summary per file
| File | Description |
|---|---|
| src/vs/workbench/contrib/chat/test/browser/agentSessions/agentSessionViewModel.test.ts | Adds a unit test ensuring Agent Host Copilot name simplifies to “Copilot” when the editor local agent is disabled. |
| src/vs/workbench/contrib/chat/test/browser/agentSessions/agentHostChatContribution.test.ts | Adds coverage that the Agent Host contribution advertises the simplified display name under the disabled setting. |
| src/vs/workbench/contrib/chat/electron-browser/chat.contribution.ts | Passes configuration into provider-name resolution when opening new Agent Host sessions. |
| src/vs/workbench/contrib/chat/browser/widget/input/sessionTargetPickerActionItem.ts | Uses configuration-aware provider naming for session-type picker labels. |
| src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatSuggestNextWidget.ts | Uses configuration-aware provider naming for “Continue in …” action labels. |
| src/vs/workbench/contrib/chat/browser/chatSessions/chatSessions.contribution.ts | Adjusts in-place session action labels to prefer contribution displayName for Agent Host Copilot. |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionsModel.ts | Injects configuration and uses it to compute provider labels for known providers. |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionsFilter.ts | Injects configuration and uses it to compute provider labels in the filter menu. |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessions.ts | Adds optional configuration parameter to getAgentSessionProviderName and introduces the simplified “Copilot” label for Agent Host Copilot when configured. |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionHoverWidget.ts | Uses configuration-aware provider naming in hover UI. |
| src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostChatContribution.ts | Centralizes Agent Host agent display name logic and aligns handler fullName/displayName with conditional provider naming for Copilot. |
| src/vs/workbench/contrib/chat/browser/actions/chatContinueInAction.ts | Ensures the continue-in dropdown label selection prefers contribution displayName for Agent Host Copilot. |
| src/vs/sessions/contrib/providers/remoteAgentHost/browser/remoteAgentHostSessionsProvider.ts | Updates override signature for session type label formatting to include provider id. |
| src/vs/sessions/contrib/providers/agentHost/browser/localAgentHostSessionsProvider.ts | Special-cases copilotcli provider to use configuration-aware Agent Host Copilot naming. |
| src/vs/sessions/contrib/providers/agentHost/browser/baseAgentHostSessionsProvider.ts | Extends session-type label formatting to include provider id and wires it through session-type sync. |
| .github/skills/sessions/SKILL.md | Documents a review guideline to avoid live config listeners for one-time label reads. |
Copilot's findings
- Files reviewed: 16/16 changed files
- Comments generated: 0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
(Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
(Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
(Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dmitrivMS
approved these changes
Jun 20, 2026
roblourens
deleted the
agents/copilot-agent-naming-clarification-44d30d22
branch
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.
Fix #322193
Summary
copilotcli) so routing and stored sessions do not changeTesting
npm run typecheck-clientnpm run valid-layers-checknpm run gulp compile./scripts/test.sh --run src/vs/platform/agentHost/test/node/copilotAgent.test.ts --run src/vs/workbench/contrib/chat/test/browser/agentSessions/agentHostChatContribution.test.ts --run src/vs/sessions/contrib/providers/agentHost/test/browser/localAgentHostSessionsProvider.test.ts./scripts/test.sh --run src/vs/workbench/contrib/chat/test/browser/agentSessions/agentHostChatContribution.test.ts --run src/vs/sessions/contrib/providers/agentHost/test/browser/localAgentHostSessionsProvider.test.ts --run src/vs/sessions/contrib/providers/remoteAgentHost/test/browser/remoteAgentHostSessionsProvider.test.tsnpm run precommit(Written by Copilot)