Report when the client LSP is fully initialized

Author: dibarbet

src/lsptoolshost/server/roslynLanguageServer.ts

@@ -183,6 +183,7 @@ export class RoslynLanguageServer {
                     state: ServerState.Started,
                     workspaceLabel: this.workspaceDisplayName(),
                 });
+                this._telemetryReporter.sendTelemetryEvent(TelemetryEventNames.ClientServerReady);
             } else if (state.newState === State.Stopped) {
                 this._languageServerEvents.onServerStateChangeEmitter.fire({
                     state: ServerState.Stopped,
@@ -338,7 +339,7 @@ export class RoslynLanguageServer {
      * Returns whether or not the underlying LSP server is running or not.
      */
     public isRunning(): boolean {
-        return this._languageClient.state === State.Running;
+        return this._languageClient.isRunning();
     }
 
     /**

src/shared/telemetryEventNames.ts

@@ -11,9 +11,17 @@ export enum TelemetryEventNames {
     CSharpActivated = 'CSharpActivated',
 
     // Events related to the roslyn language server.
+
+    // Roslyn client has started initialization process.
     ClientInitialize = 'roslyn/clientInitialize',
+    // Roslyn client has started the server initialization process.
     ClientServerStart = 'roslyn/clientServerInitialize',
+    // Roslyn client has acquired the runtime needed to start the server.
     AcquiredRuntime = 'roslyn/acquiredRuntime',
+    // Roslyn client has successfully started the server process.
     LaunchedServer = 'roslyn/launchedServer',
+    // Roslyn client has connected to the server process named pipe.
     ClientConnected = 'roslyn/clientConnected',
+    // Roslyn client and server have fully initialized via LSP.
+    ClientServerReady = 'roslyn/clientServerReady',
 }