Adds uniqueEditId to editTelemetry.reportEditArc

Author: hediet

src/vs/workbench/contrib/editTelemetry/browser/telemetry/arcTelemetrySender.ts

@@ -17,6 +17,7 @@ import { ArcTracker } from './arcTracker.js';
 import { IDocumentWithAnnotatedEdits, EditSourceData, createDocWithJustReason } from '../helpers/documentWithAnnotatedEdits.js';
 import type { ScmRepoBridge } from './editSourceTrackingImpl.js';
 import { ITextModelEditSourceMetadata } from '../../../../../editor/common/textModelEditSource.js';
+import { generateUuid } from '../../../../../base/common/uuid.js';
 
 export class InlineEditArcTelemetrySender extends Disposable {
 	constructor(
@@ -118,6 +119,8 @@ export class ChatArcTelemetrySender extends Disposable {
 			}
 			const data = edit.replacements[0].data.editSource;
 
+			const uniqueEditId = generateUuid();
+
 			const docWithJustReason = createDocWithJustReason(docWithAnnotatedEdits, this._store);
 			const reporter = this._instantiationService.createInstance(ArcTelemetryReporter, [0, 60, 300].map(s => s * 1000), _prev, docWithJustReason, scmRepoBridge, edit, res => {
 				res.telemetryService.publicLog2<{
@@ -130,6 +133,7 @@ export class ChatArcTelemetrySender extends Disposable {
 					modelId: string | undefined;
 					languageId: string | undefined;
 					mode: string | undefined;
+					uniqueEditId: string | undefined;
 
 					didBranchChange: number;
 					timeDelayMs: number;
@@ -152,6 +156,7 @@ export class ChatArcTelemetrySender extends Disposable {
 					modelId: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; comment: 'The model id.' };
 					languageId: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; comment: 'The language id of the document.' };
 					mode: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; comment: 'The mode chat was in.' };
+					uniqueEditId: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; comment: 'The unique identifier for the edit.' };
 
 					didBranchChange: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'Indicates if the branch changed in the meantime. If the branch changed (value is 1); this event should probably be ignored.' };
 					timeDelayMs: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The time delay between the user accepting the edit and measuring the survival rate.' };
@@ -178,6 +183,7 @@ export class ChatArcTelemetrySender extends Disposable {
 					modelId: data.props.$modelId,
 					languageId: data.props.$$languageId,
 					mode: data.props.$$mode,
+					uniqueEditId,
 
 					didBranchChange: res.didBranchChange ? 1 : 0,
 					timeDelayMs: res.timeDelayMs,