Updated start event trigger point and added workspaceChange event re-trigger

ensure workspaceChangeEvent interval is triggered only once

Author: Achal1607

vscode/src/lsp/initializer.ts

@@ -108,9 +108,7 @@ export const clientInit = () => {
     const client = NbLanguageClient.build(connection, LOGGER);
 
     LOGGER.log('Language Client: Starting');
-    client.start().then(() => {
-        Telemetry.enqueueStartEvent();
-        
+    client.start().then(() => {        
         registerListenersAfterClientInit();
         registerNotificationListeners(client);
         registerRequestListeners(client);

vscode/src/telemetry/events/start.ts

@@ -59,6 +59,7 @@ export class ExtensionStartEvent extends BaseEvent<StartEventData> {
     }
 
     onSuccessPostEventCallback = async (): Promise<void> => {
+        LOGGER.debug(`Start event sent successfully`);
         this.addEventToCache();
     }
 

vscode/src/telemetry/events/workspaceChange.ts

@@ -13,6 +13,8 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 */
+import { LOGGER } from "../../logger";
+import { Telemetry } from "../telemetry";
 import { BaseEvent } from "./baseEvent";
 
 interface ProjectInfo {
@@ -30,11 +32,22 @@ export interface WorkspaceChangeData {
     projInitTimeTaken: number;
 }
 
+let workspaceChangeEventTimeout: NodeJS.Timeout | null = null;
+
 export class WorkspaceChangeEvent extends BaseEvent<WorkspaceChangeData> {
     public static readonly NAME = "workspaceChange";
     public static readonly ENDPOINT = "/workspaceChange";
 
     constructor(payload: WorkspaceChangeData) {
         super(WorkspaceChangeEvent.NAME, WorkspaceChangeEvent.ENDPOINT, payload);
     }
+
+    public onSuccessPostEventCallback = async (): Promise<void> => {
+        LOGGER.debug(`WorkspaceChange event sent successfully`);
+        if (workspaceChangeEventTimeout != null) {
+            clearTimeout(workspaceChangeEventTimeout);
+            workspaceChangeEventTimeout = null;
+        }
+        workspaceChangeEventTimeout = setTimeout(() => Telemetry.sendTelemetry(this), 60 * 60 * 24 * 1000);
+    };
 }

vscode/src/telemetry/telemetry.ts

@@ -55,10 +55,6 @@ export namespace Telemetry {
 		enqueueEvent((reporter) => reporter.addEventToQueue(event));
 	}
 
-	export const enqueueStartEvent = (): void => {
-		enqueueEvent((reporter) => reporter.startEvent());
-	}
-
 	export const enqueueCloseEvent = (): void => {
 		enqueueEvent((reporter) => reporter.closeEvent());
 	}

vscode/src/telemetry/telemetryManager.ts

@@ -69,6 +69,7 @@ export class TelemetryManager {
         }
         if (this.settings.isExtTelemetryEnabled) {
             this.telemetryRetryManager.startTimer();
+            this.reporter?.startEvent();
         }
     }
 };