Merge pull request #21 from CodinGame/send-capabilities-one-by-one

Send capabilities one by one
CodinGame · Oct 10, 2023 · da76ba6 · da76ba6
2 parents 310e20e + bba2bf2
commit da76ba6
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/capabilities.ts b/src/capabilities.ts
@@ -96,11 +96,15 @@ export function synchronizeLanguageServerCapabilities (
   try {
     // Synchronizing dynamic server capabilities
     const sendRegistrationRequest = (registrations: readonly Registration[]) => {
-      clientConnection.sendRequest(RegistrationRequest.type, {
-        registrations: registrations.filter(r => !IGNORED_REGISTRATION_METHOD.has(r.method)).map(adaptRegistration)
-      }).catch(error => {
-        logger?.error('Unable to send registration requestion to client', { error })
-      })
+      // Send registration one by one even if it's possible to send them all at once
+      // Because C# LSP sends duplicated capabilities that make the client ignore the next capabilities in the list
+      for (const registration of registrations.filter(r => !IGNORED_REGISTRATION_METHOD.has(r.method)).map(adaptRegistration)) {
+        clientConnection.sendRequest(RegistrationRequest.type, {
+          registrations: [registration]
+        }).catch(error => {
+          logger?.error('Unable to send registration requestion to client', { error })
+        })
+      }
     }
     const registrationRequests = watchableServerCapabilities.getRegistrationRequests()
     if (registrationRequests.length > 0) {