Skip to content

Enable agent host by default for insiders#318007

Merged
roblourens merged 3 commits into
mainfrom
agents/i-previously-had-the-agent-host-setting-enabled-f0bd2fe9
May 22, 2026
Merged

Enable agent host by default for insiders#318007
roblourens merged 3 commits into
mainfrom
agents/i-previously-had-the-agent-host-setting-enabled-f0bd2fe9

Conversation

@roblourens
Copy link
Copy Markdown
Member

@roblourens roblourens commented May 22, 2026

Summary

  • enable chat.agentHost.enabled by default for non-stable, non-web product quality
  • ignore the local/process-backed agent host setting in web runtimes even if the raw setting is true
  • register an explicit unavailable agentHostProxy channel on server-web when no agent host bridge endpoint exists
  • add coverage for the unavailable proxy channel path

Validation

  • npm run compile-check-ts-native
  • node --experimental-strip-types build/hygiene.ts src/vs/platform/agentHost/common/agentService.ts src/vs/workbench/contrib/chat/browser/chat.shared.contribution.ts src/vs/code/electron-main/app.ts src/vs/platform/agentHost/electron-browser/localAgentHostService.ts src/vs/workbench/services/agentHost/browser/editorRemoteAgentHostServiceClient.ts src/vs/sessions/contrib/providers/agentHost/browser/localAgentHost.contribution.ts src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostTerminalContribution.ts src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostChatContribution.ts src/vs/workbench/contrib/chat/browser/actions/exportAgentHostDebugLogsAction.ts
  • npm run gulp -- compile-client --silent
  • ./scripts/test.sh --run src/vs/server/test/node/agentHostChannel.test.ts

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 22, 2026 16:04
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 changes the default behavior of the chat agent host feature for non-stable builds and improves remote-server behavior by always registering an agentHostProxy IPC channel, even when no upstream agent host endpoint is configured, so clients fail with a clear error instead of “Unknown channel”.

Changes:

  • Enable chat.agentHost.enabled by default when product.quality !== 'stable'.
  • Register an UnavailableAgentHostChannel as the agentHostProxy IPC channel when no bridge endpoint is configured.
  • Add server-side unit coverage for the unavailable proxy channel behavior.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/chat.shared.contribution.ts Switches the default for chat.agentHost.enabled to be on for non-stable qualities.
src/vs/server/node/serverServices.ts Registers agentHostProxy unconditionally (real proxy when configured, otherwise an unavailable proxy).
src/vs/server/node/agentHostChannel.ts Adds UnavailableAgentHostChannel implementation and the explicit failure message.
src/vs/server/test/node/agentHostChannel.test.ts Adds a unit test validating the unavailable-channel path and error messaging.

Copilot's findings

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

Comment thread src/vs/workbench/contrib/chat/browser/chat.shared.contribution.ts
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

Base: 7b57c955 Current: 22389334

No screenshot changes.

Keep shared browser-layer contributions gated on the raw setting so desktop/browser tests continue to exercise their behavior, while the web runtime ignore remains in the service/client paths.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@roblourens roblourens marked this pull request as ready for review May 22, 2026 16:59
@roblourens roblourens enabled auto-merge (squash) May 22, 2026 16:59
@vs-code-engineering
Copy link
Copy Markdown
Contributor

📬 CODENOTIFY

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

@deepak1556

Matched files:

  • src/vs/code/electron-main/app.ts

@roblourens roblourens merged commit b4b5c20 into main May 22, 2026
31 of 37 checks passed
@roblourens roblourens deleted the agents/i-previously-had-the-agent-host-setting-enabled-f0bd2fe9 branch May 22, 2026 17:07
@vs-code-engineering vs-code-engineering Bot added this to the 1.122.0 milestone May 22, 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