fixup! In Start Work switches to issue list before waiting for the result

Moves telemetry (#3741, #3751)

Author: chivorotkiv

src/plus/startWork/startWork.ts

@@ -161,21 +161,8 @@ export class StartWorkCommand extends QuickCommand<State> {
 					}
 				}
 
-				if (this.container.telemetry.enabled) {
-					this.container.telemetry.sendEvent(
-						opened ? 'startWork/steps/issue' : 'startWork/opened',
-						{
-							...context.telemetryContext!,
-							connected: true,
-							type: state.type,
-						},
-						this.source,
-					);
-				}
-
+				const result = yield* this.pickIssueStep(state, context, opened);
 				opened = true;
-
-				const result = yield* this.pickIssueStep(state, context);
 				if (result === StepResultBreak) continue;
 				if (!isStartWorkTypeItem(result)) {
 					state.item = result;
@@ -419,6 +406,7 @@ export class StartWorkCommand extends QuickCommand<State> {
 	private *pickIssueStep(
 		state: StepState<State>,
 		context: Context,
+		opened: boolean,
 	): StepResultGenerator<StartWorkItem | StartWorkTypeItem> {
 		const buildIssueItem = (i: StartWorkItem) => {
 			const buttons = i.item.issue.url ? [OpenOnGitHubQuickInputButton] : [];
@@ -476,6 +464,18 @@ export class StartWorkCommand extends QuickCommand<State> {
 				const { items, placeholder } = getItemsAndPlaceholder();
 				quickpick.placeholder = placeholder;
 				quickpick.items = items;
+
+				if (this.container.telemetry.enabled) {
+					this.container.telemetry.sendEvent(
+						opened ? 'startWork/steps/issue' : 'startWork/opened',
+						{
+							...context.telemetryContext!,
+							connected: true,
+							type: state.type,
+						},
+						this.source,
+					);
+				}
 			} finally {
 				quickpick.busy = false;
 			}