Commands to install/uninstall roslyn-language-server for the Copilot CLI#9344

Open

phil-allen-msft wants to merge 20 commits into

mainfrom

dev/phil-allen-msft/installLsp

Commits on May 20, 2026

Initial implementation
Phil Allen
committed
Please author a command in this vscode extension that will make the above fullConfig changes to the lsp-config.json file if it exists, and write a new lsp-config.json file if it doesn't exist, assu…
Phil Allen
committed
Change it so that #sym:csharpLspServerConfig is read from a file included in the vsix. Copy the file from the dotnet/skills repo now.
Phil Allen
committed
As part of the update code, check and see if the lsp-config.json file on the user's machine already contains content for the roslyn-language-server. If so, make no changes.
Phil Allen
committed
Now refactor as necessary and add a test so that if the 'install roslyn-language-server for Copilot CLI' runs multiple times, the resulting lsp-config.json has exactly the same contents as the copy…
Phil Allen
committed
Fixup: Add tests
Phil Allen
committed
Add a command to the npm build system as 'npm run ingest' which will do the same copy that you did earlier of the lsp-config.json from the dotnet/skills repo, but do it again when 'npm run ingest' …
Phil Allen
committed
Code review the changes that have been committed but are 'ahead' of main.
Phil Allen
committed
Change the name of the folder that contains lsp-config.json in this repo and in the eventual extension to be 'redist'. Update all references.
Phil Allen
committed
Add a new command to uninstall the CSharp LSP for Copilot CLI use. It should find the lsp-config.json in the customer's HOME/.copilot directory; if it exists it should parse the file, and if the "l…
Phil Allen
committed
Rename the configure command to be 'install' so that it mirrors the naming for the uninstall command
Phil Allen
committed

Commits on May 21, 2026

Commits on May 22, 2026

Local Code Review feedback taken
Phil Allen
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commands to install/uninstall roslyn-language-server for the Copilot CLI#9344

Commands to install/uninstall roslyn-language-server for the Copilot CLI#9344
phil-allen-msft wants to merge 20 commits into
mainfrom
dev/phil-allen-msft/installLsp

Commits on May 20, 2026

Initial implementation

Please author a command in this vscode extension that will make the above fullConfig changes to the lsp-config.json file if it exists, and write a new lsp-config.json file if it doesn't exist, assu…

Change it so that #sym:csharpLspServerConfig is read from a file included in the vsix. Copy the file from the dotnet/skills repo now.

As part of the update code, check and see if the lsp-config.json file on the user's machine already contains content for the roslyn-language-server. If so, make no changes.

Now refactor as necessary and add a test so that if the 'install roslyn-language-server for Copilot CLI' runs multiple times, the resulting lsp-config.json has exactly the same contents as the copy…

Fixup: Add tests

Add a command to the npm build system as 'npm run ingest' which will do the same copy that you did earlier of the lsp-config.json from the dotnet/skills repo, but do it again when 'npm run ingest' …

Code review the changes that have been committed but are 'ahead' of main.

Change the name of the folder that contains lsp-config.json in this repo and in the eventual extension to be 'redist'. Update all references.

Add a new command to uninstall the CSharp LSP for Copilot CLI use. It should find the lsp-config.json in the customer's HOME/.copilot directory; if it exists it should parse the file, and if the "l…

Rename the configure command to be 'install' so that it mirrors the naming for the uninstall command

Commits on May 21, 2026

Change how to determine if the lsp is installed. Instead of looking for an object named 'csharp', look for an object that lists ".cs" in the "fileExtensions" collection. Use lsp-config.json in th…

Change the method #sym:getUninstalledCopilotLspConfigContent to also search for content to delete by looking for an lspServer object that has a "fileExtensions" with ".cs"

Factor together both loops that look for an existing lspServer content into one method and use from both existing implementation sites.

Change all of the methods that are new in this branch as compared to origin/main such that if they contained #sym:RoslynLanguageServer , they instead contain CSharpLsp

Code review all code on the branch as compared to origin/main

Remove "%configuration.dotnet.experiments.targetPopulation...% keys," from the package.json file

Fix 4 from the list above

Change the ingest code so that if the lspServers.csharp element does not exist, it reports an error

Commits on May 22, 2026

Local Code Review feedback taken