Skip to content

Commit f281364

Browse files
authored
Assigning Copilot for a code review fails (#7475)
Fixes #7027
1 parent c6fa091 commit f281364

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/github/quickPicks.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import { Buffer } from 'buffer';
88
import * as vscode from 'vscode';
99
import { COPILOT_ACCOUNTS } from '../common/comment';
10+
import { COPILOT_LOGINS } from '../common/copilot';
1011
import { emojify, ensureEmojis } from '../common/emoji';
1112
import Logger from '../common/logger';
1213
import { DataUri } from '../common/uri';
@@ -137,8 +138,11 @@ async function getReviewersQuickPickItems(folderRepositoryManager: FolderReposit
137138
const allTeamReviewers = isInOrganization ? await folderRepositoryManager.getTeamReviewers(refreshKind) : [];
138139
const teamReviewers: ITeam[] = allTeamReviewers[remoteName] ?? [];
139140
const assignableUsers: (IAccount | ITeam)[] = [...teamReviewers];
140-
if (allAssignableUsers[remoteName]) {
141-
assignableUsers.push(...allAssignableUsers[remoteName]);
141+
142+
// Remove the swe agent as it can't do reviews
143+
const assignableUsersForRemote = allAssignableUsers[remoteName].filter(user => user.login !== COPILOT_LOGINS[1]);
144+
if (assignableUsersForRemote) {
145+
assignableUsers.push(...assignableUsersForRemote);
142146
}
143147

144148
// used to track logins that shouldn't be added to pick list

0 commit comments

Comments
 (0)