Skip to content

Conversation

Copy link

Copilot AI commented Dec 8, 2025

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original prompt

This section details on the original issue you should resolve

<issue_title>Deprecated rootUri is used for lsp-multi-server-sample</issue_title>
<issue_description>The example server is still using params.rootUri https://github.com/microsoft/vscode-extension-samples/blob/master/lsp-multi-server-sample/server/src/server.ts#L25

Which makes sense in the usecase, but as it says, that it's deprecated and we should use params.workspaceFolders which does not help us to figure out which workspace folder this server was started for.

So is starting a server for every workspace folder still what's recommended? https://github.com/microsoft/vscode-extension-samples/blob/master/lsp-multi-server-sample/README.md#L5

Or should we handle this different nowadays?</issue_description>

Comments on the Issue (you are @copilot in this section)

@dbaeumer This is a fair point and needs to be addressed the LSP libs. Things to do is to only send the single workspace folder to the server if it is set in the client. @dbaeumer Fix the LSP libs. Moving back to extension sample to adapt when new version is released.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

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.

Deprecated rootUri is used for lsp-multi-server-sample

2 participants