Skip to content

Browser: Match ExP value for setting workbench.browser.enableChatTools#319465

Merged
jruales merged 6 commits into
mainfrom
jruales/chat-tools-exp
Jun 5, 2026
Merged

Browser: Match ExP value for setting workbench.browser.enableChatTools#319465
jruales merged 6 commits into
mainfrom
jruales/chat-tools-exp

Conversation

@jruales
Copy link
Copy Markdown
Contributor

@jruales jruales commented Jun 1, 2026

ExP is at 100%, so setting it explicitly in the code too

ExP is at 100%, so setting it explicitly in the code too
Copilot AI review requested due to automatic review settings June 1, 2026 22:06
@jruales jruales requested a review from kycutler June 1, 2026 22:07
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 updates the Integrated Browser configuration to match the current experiment (ExP) rollout by setting workbench.browser.enableChatTools to be enabled by default.

Changes:

  • Set workbench.browser.enableChatTools default from false to true.
Show a summary per file
File Description
src/vs/workbench/contrib/browserView/electron-browser/features/browserEditorChatFeatures.ts Changes the default value of the browser chat tools setting to true.

Copilot's findings

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

Co-authored-by: jruales <1588988+jruales@users.noreply.github.com>
@jruales jruales marked this pull request as draft June 3, 2026 00:06
@jruales jruales marked this pull request as ready for review June 3, 2026 01:09
@jruales jruales enabled auto-merge (squash) June 3, 2026 01:48
@jruales jruales requested review from Copilot and removed request for kycutler June 5, 2026 00:11
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.

Copilot's findings

  • Files reviewed: 4/4 changed files
  • Comments generated: 3

Comment thread src/vs/workbench/contrib/chat/browser/chatTipCatalog.ts Outdated
Comment thread src/vs/workbench/contrib/chat/test/browser/chatTipService.test.ts
Comment thread src/vs/workbench/contrib/chat/test/browser/chatTipService.test.ts
@jruales jruales marked this pull request as draft June 5, 2026 00:34
auto-merge was automatically disabled June 5, 2026 00:34

Pull request was converted to draft

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.

Copilot's findings

  • Files reviewed: 4/4 changed files
  • Comments generated: 0 new

@jruales jruales marked this pull request as ready for review June 5, 2026 05:58
@jruales jruales enabled auto-merge (squash) June 5, 2026 05:58
@vs-code-engineering
Copy link
Copy Markdown
Contributor

📬 CODENOTIFY

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

@kycutler

Matched files:

  • src/vs/workbench/contrib/browserView/electron-browser/features/browserEditorChatFeatures.ts

@jruales jruales merged commit 628f6fe into main Jun 5, 2026
26 checks passed
@jruales jruales deleted the jruales/chat-tools-exp branch June 5, 2026 06:28
@vs-code-engineering vs-code-engineering Bot added this to the 1.124.0 milestone Jun 5, 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.

4 participants