Add telemetry for when Copilot is assigned to an issue (#7463)

Fixes #7445

Author: alexr00

src/github/folderRepositoryManager.ts

@@ -1304,7 +1304,7 @@ export class FolderRepositoryManager extends Disposable {
 			};
 			for (const issue of data.items) {
 				const githubRepository = await this.getRepoForIssue(issue);
-				mappedData.items.push(new IssueModel(githubRepository, githubRepository.remote, issue));
+				mappedData.items.push(new IssueModel(this.telemetry, githubRepository, githubRepository.remote, issue));
 			}
 			return mappedData;
 		} catch (e) {
@@ -1601,7 +1601,7 @@ export class FolderRepositoryManager extends Disposable {
 			// Create PR
 			const { data } = await repo.octokit.call(repo.octokit.api.issues.create, params);
 			const item = convertRESTIssueToRawPullRequest(data, repo);
-			const issueModel = new IssueModel(repo, repo.remote, item);
+			const issueModel = new IssueModel(this.telemetry, repo, repo.remote, item);
 
 			/* __GDPR__
 				"issue.create.success" : {

src/github/githubRepository.ts

@@ -1104,7 +1104,7 @@ export class GitHubRepository extends Disposable {
 			}
 			Logger.debug(`Fetch issue ${id} - done`, this.id);
 
-			return new IssueModel(this, remote, await parseGraphQLIssue(data.repository.issue, this));
+			return new IssueModel(this.telemetry, this, remote, await parseGraphQLIssue(data.repository.issue, this));
 		} catch (e) {
 			Logger.error(`Unable to fetch issue: ${e}`, this.id);
 			return;

src/github/issueModel.ts

@@ -8,6 +8,7 @@ import { COPILOT_ACCOUNTS, IComment } from '../common/comment';
 import { Disposable } from '../common/lifecycle';
 import Logger from '../common/logger';
 import { Remote } from '../common/remote';
+import { ITelemetry } from '../common/telemetry';
 import { ClosedEvent, CrossReferencedEvent, EventType, TimelineEvent } from '../common/timelineEvent';
 import { compareIgnoreCase, formatError } from '../common/utils';
 import { OctokitCommon } from './common';
@@ -57,6 +58,7 @@ export class IssueModel<TItem extends Issue = Issue> extends Disposable {
 	public updatedAt: string;
 	public milestone?: IMilestone;
 	public readonly githubRepository: GitHubRepository;
+	protected readonly _telemetry: ITelemetry;
 	public readonly remote: Remote;
 	public item: TItem;
 	public body: string;
@@ -67,8 +69,9 @@ export class IssueModel<TItem extends Issue = Issue> extends Disposable {
 	protected _onDidChange = this._register(new vscode.EventEmitter<IssueChangeEvent>());
 	public onDidChange = this._onDidChange.event;
 
-	constructor(githubRepository: GitHubRepository, remote: Remote, item: TItem, skipUpdate: boolean = false) {
+	constructor(telemetry: ITelemetry, githubRepository: GitHubRepository, remote: Remote, item: TItem, skipUpdate: boolean = false) {
 		super();
+		this._telemetry = telemetry;
 		this.githubRepository = githubRepository;
 		this.remote = remote;
 		this.item = item;
@@ -579,6 +582,15 @@ export class IssueModel<TItem extends Issue = Issue> extends Disposable {
 
 		try {
 			if (schema.ReplaceActorsForAssignable) {
+				const assignToCopilot = allAssignees.find(assignee => COPILOT_ACCOUNTS[assignee.login]);
+				const alreadyHasCopilot = this.assignees?.find(assignee => COPILOT_ACCOUNTS[assignee.login]) !== undefined;
+				if (assignToCopilot && !alreadyHasCopilot) {
+					/* __GDPR__
+						"pr.assignCopilot" : {}
+					*/
+					this._telemetry.sendTelemetryEvent('pr.assignCopilot');
+				}
+
 				const assigneeIds = allAssignees.map(assignee => assignee.id);
 				await mutate({
 					mutation: schema.ReplaceActorsForAssignable,

src/github/pullRequestModel.ts

@@ -156,7 +156,6 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 		this._isActive = isActive;
 	}
 
-	_telemetry: ITelemetry;
 
 	constructor(
 		private readonly credentialStore: CredentialStore,
@@ -166,9 +165,8 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 		item: PullRequest,
 		isActive?: boolean,
 	) {
-		super(githubRepository, remote, item, true);
+		super(telemetry, githubRepository, remote, item, true);
 
-		this._telemetry = telemetry;
 		this.isActive = !!isActive;
 
 		this._showChangesSinceReview = false;