Wrap completions core events to avoid tweaking their names (#827)

Author: alexdima

package.json

@@ -5,7 +5,7 @@
 	"version": "0.31.0",
 	"build": "1",
 	"internalAIKey": "1058ec22-3c95-4951-8443-f26c1f325911",
-	"completionsCore": "952221f2120a4babb940c51a57af102fe782a37e",
+	"completionsCore": "ab8607db78e22157003a2d128b0c198630a1b923",
 	"completionsCoreVersion ": "1.363.1764",
 	"internalLargeStorageAriaKey": "ec712b3202c5462fb6877acae7f1f9d7-c19ad55e-3e3c-4f99-984b-827f6d95bd9e-6917",
 	"ariaKey": "0c6ae279ed8443289764825290e4f9e2-1a736e7c-1324-4338-be46-fc2a58ae4d14-7255",

src/platform/telemetry/node/azureInsightsReporter.ts

@@ -13,6 +13,17 @@ import { ICAPIClientService } from '../../endpoint/common/capiClient';
 import { IEnvService } from '../../env/common/envService';
 import { TelemetryProperties } from '../common/telemetry';
 
+export function wrapEventNameForPrefixRemoval(eventName: string): string {
+	return `wrapped-telemetry-event-name-${eventName}-wrapped-telemetry-event-name`;
+}
+function isWrappedEventName(eventName: string): boolean {
+	return eventName.includes('wrapped-telemetry-event-name-') && eventName.endsWith('-wrapped-telemetry-event-name');
+}
+function unwrapEventNameFromPrefix(eventName: string): string {
+	const match = eventName.match(/wrapped-telemetry-event-name-(.*?)-wrapped-telemetry-event-name/);
+	return match ? match[1] : eventName;
+}
+
 export class AzureInsightReporter implements TelemetrySender {
 	private readonly client: appInsights.TelemetryClient;
 	constructor(capiClientService: ICAPIClientService, envService: IEnvService, private readonly namespace: string, key: string) {
@@ -41,7 +52,7 @@ export class AzureInsightReporter implements TelemetrySender {
 	sendEventData(eventName: string, data?: Record<string, any> | undefined): void {
 		const { properties, measurements } = this.separateData(data || {});
 		this.client.trackEvent({
-			name: this.qualifyEventName(eventName),
+			name: this.massageEventName(eventName),
 			properties,
 			measurements,
 		});
@@ -66,7 +77,10 @@ export class AzureInsightReporter implements TelemetrySender {
 		});
 	}
 
-	private qualifyEventName(eventName: string): string {
+	private massageEventName(eventName: string): string {
+		if (isWrappedEventName(eventName)) {
+			return unwrapEventNameFromPrefix(eventName);
+		}
 		return eventName.includes(this.namespace) ? eventName : `${this.namespace}/${eventName}`;
 	}
 }