Skip to content

Commit 21263f0

Browse files
committed
reorder fields to group them
1 parent fc61452 commit 21263f0

File tree

1 file changed

+18
-14
lines changed

1 file changed

+18
-14
lines changed

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

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,12 @@ export class InlineEditArcTelemetrySender extends Disposable {
5252
languageId: string;
5353
didBranchChange: number;
5454
timeDelayMs: number;
55-
arc: number;
55+
5656
originalCharCount: number;
5757
originalLineCount: number;
58-
currentLineCount: number;
5958
originalDeletedLineCount: number;
59+
arc: number;
60+
currentLineCount: number;
6061
currentDeletedLineCount: number;
6162
}, {
6263
owner: 'hediet';
@@ -69,11 +70,12 @@ export class InlineEditArcTelemetrySender extends Disposable {
6970

7071
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.' };
7172
timeDelayMs: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The time delay between the user accepting the edit and measuring the survival rate.' };
72-
arc: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The accepted and restrained character count.' };
73+
7374
originalCharCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original character count before any edits.' };
7475
originalLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original line count before any edits.' };
75-
currentLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current line count after edits.' };
7676
originalDeletedLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original deleted line count before any edits.' };
77+
arc: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The accepted and restrained character count.' };
78+
currentLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current line count after edits.' };
7779
currentDeletedLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current deleted line count after edits.' };
7880
}>('editTelemetry.reportInlineEditArc', {
7981
extensionId: data.$extensionId ?? '',
@@ -82,11 +84,12 @@ export class InlineEditArcTelemetrySender extends Disposable {
8284
languageId: data.$$languageId,
8385
didBranchChange: res.didBranchChange ? 1 : 0,
8486
timeDelayMs: res.timeDelayMs,
85-
arc: res.arc,
87+
8688
originalCharCount: res.originalCharCount,
8789
originalLineCount: res.originalLineCount,
88-
currentLineCount: res.currentLineCount,
8990
originalDeletedLineCount: res.originalDeletedLineCount,
91+
arc: res.arc,
92+
currentLineCount: res.currentLineCount,
9093
currentDeletedLineCount: res.currentDeletedLineCount,
9194

9295
...forwardToChannelIf(isCopilotLikeExtension(data.$extensionId)),
@@ -140,12 +143,12 @@ export class ChatArcTelemetrySender extends Disposable {
140143

141144
didBranchChange: number;
142145
timeDelayMs: number;
143-
arc: number;
144-
originalCharCount: number;
145146

147+
originalCharCount: number;
146148
originalLineCount: number;
147-
currentLineCount: number;
148149
originalDeletedLineCount: number;
150+
arc: number;
151+
currentLineCount: number;
149152
currentDeletedLineCount: number;
150153
}, {
151154
owner: 'hediet';
@@ -164,11 +167,12 @@ export class ChatArcTelemetrySender extends Disposable {
164167

165168
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.' };
166169
timeDelayMs: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The time delay between the user accepting the edit and measuring the survival rate.' };
167-
arc: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The accepted and restrained character count.' };
170+
168171
originalCharCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original character count before any edits.' };
169172
originalLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original line count before any edits.' };
170-
currentLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current line count after edits.' };
171173
originalDeletedLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The original deleted line count before any edits.' };
174+
arc: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The accepted and restrained character count.' };
175+
currentLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current line count after edits.' };
172176
currentDeletedLineCount: { classification: 'SystemMetaData'; purpose: 'FeatureInsight'; isMeasurement: true; comment: 'The current deleted line count after edits.' };
173177
}>('editTelemetry.reportEditArc', {
174178
sourceKeyCleaned: data.toKey(Number.MAX_SAFE_INTEGER, {
@@ -192,12 +196,12 @@ export class ChatArcTelemetrySender extends Disposable {
192196

193197
didBranchChange: res.didBranchChange ? 1 : 0,
194198
timeDelayMs: res.timeDelayMs,
195-
arc: res.arc,
196-
originalCharCount: res.originalCharCount,
197199

200+
originalCharCount: res.originalCharCount,
198201
originalLineCount: res.originalLineCount,
199-
currentLineCount: res.currentLineCount,
200202
originalDeletedLineCount: res.originalDeletedLineCount,
203+
arc: res.arc,
204+
currentLineCount: res.currentLineCount,
201205
currentDeletedLineCount: res.currentDeletedLineCount,
202206

203207
...forwardToChannelIf(isCopilotLikeExtension(data.props.$extensionId)),

0 commit comments

Comments
 (0)