Skip to content

Gate AgentHostTerminalContribution on chat.agentHost.enabled#311309

Merged
connor4312 merged 2 commits intomainfrom
anthonykim1/whyAgentLocalShowsUpWithoutSetting
Apr 20, 2026
Merged

Gate AgentHostTerminalContribution on chat.agentHost.enabled#311309
connor4312 merged 2 commits intomainfrom
anthonykim1/whyAgentLocalShowsUpWithoutSetting

Conversation

@anthonykim1
Copy link
Copy Markdown
Contributor

@anthonykim1 anthonykim1 added this to the 1.117.0 milestone Apr 20, 2026
@anthonykim1 anthonykim1 self-assigned this Apr 20, 2026
Copilot AI review requested due to automatic review settings April 20, 2026 05:40
@anthonykim1 anthonykim1 marked this pull request as ready for review April 20, 2026 05:41
Copy link
Copy Markdown
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 prevents “Agent Host Terminal (Local)” from appearing in the terminal dropdown when the local agent host feature is disabled via the chat.agentHost.enabled setting, matching the intended UX for the agent host feature flag.

Changes:

  • Gate AgentHostTerminalContribution initialization logic on chat.agentHost.enabled.
  • Add configuration service dependency to read the setting at contribution startup.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostTerminalContribution.ts Early-return from the contribution when chat.agentHost.enabled is false so the local agent host terminal entry isn’t registered/shown.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 20, 2026

Screenshot Changes

Base: abdc25ae Current: f58b5a72

Changed (3)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark
Before After
before after
agentSessionsViewer/CompletedUnread/Dark
Before After
before after
agentSessionsViewer/CompletedUnread/Light
Before After
before after

@alexr00
Copy link
Copy Markdown
Member

alexr00 commented Apr 20, 2026

Moving to 1.118.0

@alexr00 alexr00 modified the milestones: 1.117.0, 1.118.0 Apr 20, 2026
Copilot AI requested a review from connor4312 April 20, 2026 16:12
@connor4312 connor4312 enabled auto-merge (squash) April 20, 2026 16:12
@connor4312 connor4312 merged commit ef2d27c into main Apr 20, 2026
26 checks passed
@connor4312 connor4312 deleted the anthonykim1/whyAgentLocalShowsUpWithoutSetting branch April 20, 2026 16:48
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.

Agent Host (Local) shows up even when agent host setting is disabled

6 participants