Skip to content

Enable local AH by default in Insiders#317667

Merged
roblourens merged 1 commit into
mainfrom
roblou/surviving-pig
May 21, 2026
Merged

Enable local AH by default in Insiders#317667
roblourens merged 1 commit into
mainfrom
roblou/surviving-pig

Conversation

@roblourens
Copy link
Copy Markdown
Member

and include AH-related settings in stable builds

and include AH-related settings in stable builds
Copilot AI review requested due to automatic review settings May 21, 2026 00:16
@roblourens roblourens enabled auto-merge (squash) May 21, 2026 00:16
@roblourens roblourens self-assigned this May 21, 2026
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 adjusts Chat Agent Host (AH) configuration defaults so the local agent host is enabled by default in Insiders, while also making AH-related settings available in Stable builds (with Stable-safe defaults).

Changes:

  • Sets chat.agentHost.enabled default to true for non-stable product qualities (Insiders), and false on Stable.
  • Removes included: product.quality !== 'stable' from several AH-related settings so they appear in Stable builds.
  • Switches several OpenTelemetry setting descriptions to markdownDescription to support richer formatting.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/chat.shared.contribution.ts Updates Agent Host settings visibility/defaults across Stable vs Insiders and improves OTel setting descriptions via markdown.

Copilot's findings

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

@roblourens roblourens merged commit f4bd7b0 into main May 21, 2026
29 checks passed
@roblourens roblourens deleted the roblou/surviving-pig branch May 21, 2026 00:38
@vs-code-engineering vs-code-engineering Bot added this to the 1.122.0 milestone May 21, 2026
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.

3 participants