Skip to content

Skip RoslynCopilot install when ai features not enabled#9196

Merged
dibarbet merged 2 commits into
dotnet:mainfrom
dibarbet:copilot_skip_install
May 5, 2026
Merged

Skip RoslynCopilot install when ai features not enabled#9196
dibarbet merged 2 commits into
dotnet:mainfrom
dibarbet:copilot_skip_install

Conversation

@dibarbet
Copy link
Copy Markdown
Member

@dibarbet dibarbet commented Apr 15, 2026

Only install and load the RoslynCopilot component when ai features are enabled. Prompts for reload when the setting changes.

Issue: microsoft/vscode-dotnettools#2979

Comment thread src/lsptoolshost/server/roslynLanguageServer.ts Outdated
@dibarbet dibarbet force-pushed the copilot_skip_install branch from 4603807 to 2f1bf30 Compare April 15, 2026 23:13
@dibarbet dibarbet marked this pull request as ready for review April 15, 2026 23:50
@dibarbet dibarbet requested a review from a team as a code owner April 15, 2026 23:50
@claudiaregio
Copy link
Copy Markdown
Contributor

@dibarbet I think we may need to update logic for this, you can disable AI features with chat.disableAIFeatures setting now

dibarbet and others added 2 commits May 4, 2026 12:36
Only install and load the RoslynCopilot component when a GitHub Copilot
extension (github.copilot or github.copilot-chat) is detected, in
addition to the existing C# Dev Kit requirement.

When the user installs a Copilot extension after startup while Dev Kit
is active, detect the change and prompt for an extension host reload to
enable Copilot integration.

Fixes microsoft/vscode-dotnettools#2979

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add disableAIFeatures as a CommonOption backed by chat.disableAIFeatures.
Gate the Roslyn Copilot component download and server loading on this
setting. The existing option change infrastructure handles prompting
for a window reload when the setting changes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dibarbet dibarbet force-pushed the copilot_skip_install branch from 2f1bf30 to 795c626 Compare May 4, 2026 20:00
@dibarbet
Copy link
Copy Markdown
Member Author

dibarbet commented May 5, 2026

@JoeRobich / @akhera99 for re-review (using the setting instead of extension presence)

Copy link
Copy Markdown
Member

@JoeRobich JoeRobich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was looking to see if you added it to the restart options and you did. Looks good

@dibarbet dibarbet merged commit 467d5b9 into dotnet:main May 5, 2026
33 checks passed
@dibarbet dibarbet changed the title Skip RoslynCopilot install when Copilot extension is not present Skip RoslynCopilot install when ai features not enabled May 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