Reuses a function that prepares issue data for telemetry

(#3774, #3775)

Author: sergeibbb

src/plus/startWork/startWork.ts

@@ -176,19 +176,9 @@ export class StartWorkCommand extends QuickCommand<State> {
 							'startWork/issue/chosen',
 							{
 								...context.telemetryContext!,
+								...buildItemTelemetryData(result),
 								connected: true,
 								type: state.type,
-								'item.id': getStartWorkItemIdHash(result),
-								'item.type': result.item.issue.type,
-								'item.provider': result.item.issue.provider.id,
-								'item.assignees.count': result.item.issue.assignees?.length ?? undefined,
-								'item.createdDate': result.item.issue.createdDate.getTime(),
-								'item.updatedDate': result.item.issue.updatedDate.getTime(),
-
-								'item.comments.count': result.item.issue.commentsCount ?? undefined,
-								'item.upvotes.count': result.item.issue.thumbsUpCount ?? undefined,
-
-								'item.issue.state': result.item.issue.state,
 							},
 							this.source,
 						);
@@ -536,27 +526,13 @@ export class StartWorkCommand extends QuickCommand<State> {
 		state: StepState<State>,
 		context: Context,
 	) {
-		this.container.telemetry.sendEvent(
-			'startWork/issue/action',
-			{
-				...context.telemetryContext!,
-				action: action,
-				connected: true,
-				type: state.type,
-				'item.id': getStartWorkItemIdHash(item),
-				'item.type': item.item.issue.type,
-				'item.provider': item.item.issue.provider.id,
-				'item.assignees.count': item.item.issue.assignees?.length ?? undefined,
-				'item.createdDate': item.item.issue.createdDate.getTime(),
-				'item.updatedDate': item.item.issue.updatedDate.getTime(),
-
-				'item.comments.count': item.item.issue.commentsCount ?? undefined,
-				'item.upvotes.count': item.item.issue.thumbsUpCount ?? undefined,
-
-				'item.issue.state': item.item.issue.state,
-			},
-			this.source,
-		);
+		this.container.telemetry.sendEvent('startWork/issue/action', {
+			...context.telemetryContext!,
+			...buildItemTelemetryData(item),
+			action: action,
+			connected: true,
+			type: state.type,
+		});
 	}
 
 	private async getConnectedIntegrations(): Promise<Map<SupportedStartWorkIntegrationIds, boolean>> {
@@ -615,6 +591,22 @@ export function getStartWorkItemIdHash(item: StartWorkItem) {
 	return md5(item.item.issue.id);
 }
 
+function buildItemTelemetryData(item: StartWorkItem) {
+	return {
+		'item.id': getStartWorkItemIdHash(item),
+		'item.type': item.item.issue.type,
+		'item.provider': item.item.issue.provider.id,
+		'item.assignees.count': item.item.issue.assignees?.length ?? undefined,
+		'item.createdDate': item.item.issue.createdDate.getTime(),
+		'item.updatedDate': item.item.issue.updatedDate.getTime(),
+
+		'item.comments.count': item.item.issue.commentsCount ?? undefined,
+		'item.upvotes.count': item.item.issue.thumbsUpCount ?? undefined,
+
+		'item.issue.state': item.item.issue.state,
+	};
+}
+
 function getOpenOnWebQuickInputButton(integrationId: string): QuickInputButton | undefined {
 	switch (integrationId) {
 		case HostingIntegrationId.GitHub: