Skip to content

Commit e995ca9

Browse files
Copilotalexr00
andcommitted
Add ChangeReviewersReply type for addReviewerCopilot reply
- Created ChangeReviewersReply interface in views.ts - Updated addReviewerCopilot to use typed reply instead of inline object - Fixed copilotUser reference instead of using removed COPILOT_REVIEWER_ACCOUNT - Removed unused imports Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
1 parent b2dfd46 commit e995ca9

3 files changed

Lines changed: 13 additions & 4 deletions

File tree

src/@types/vscode.proposed.chatSessionsProvider.d.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,11 @@ declare module 'vscode' {
9595
*/
9696
description?: string | MarkdownString;
9797

98+
/**
99+
* An optional badge that provides additional context about the chat session.
100+
*/
101+
badge?: string | MarkdownString;
102+
98103
/**
99104
* An optional status indicating the current state of the session.
100105
*/

src/github/pullRequestOverview.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,9 @@ import { isCopilotOnMyBehalf, PullRequestModel } from './pullRequestModel';
2525
import { PullRequestReviewCommon, ReviewContext } from './pullRequestReviewCommon';
2626
import { pickEmail, reviewersQuickPick } from './quickPicks';
2727
import { parseReviewers } from './utils';
28-
import { CancelCodingAgentReply, DeleteReviewResult, MergeArguments, MergeResult, PullRequest, ReviewType } from './views';
28+
import { CancelCodingAgentReply, ChangeReviewersReply, DeleteReviewResult, MergeArguments, MergeResult, PullRequest, ReviewType } from './views';
2929
import { COPILOT_ACCOUNTS, IComment } from '../common/comment';
30-
import { COPILOT_REVIEWER, COPILOT_REVIEWER_ACCOUNT, COPILOT_SWE_AGENT, copilotEventToStatus, CopilotPRStatus, mostRecentCopilotEvent } from '../common/copilot';
30+
import { COPILOT_SWE_AGENT, copilotEventToStatus, CopilotPRStatus, mostRecentCopilotEvent } from '../common/copilot';
3131
import { commands, contexts } from '../common/executeCommands';
3232
import { disposeAll } from '../common/lifecycle';
3333
import Logger from '../common/logger';
@@ -757,10 +757,10 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
757757
try {
758758
const copilotUser = this._assignableUsers[this._item.remote.remoteName]?.find(user => COPILOT_ACCOUNTS[user.login]);
759759
if (copilotUser) {
760-
await this._item.requestReview([COPILOT_REVIEWER_ACCOUNT], []);
760+
await this._item.requestReview([copilotUser], []);
761761
const newReviewers = await this._item.getReviewRequests();
762762
this._existingReviewers = parseReviewers(newReviewers!, await this._item.getTimelineEvents(), this._item.author);
763-
const reply = {
763+
const reply: ChangeReviewersReply = {
764764
reviewers: this._existingReviewers
765765
};
766766
this._replyMessage(message, reply);

src/github/views.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,10 @@ export interface ChangeAssigneesReply {
121121
events: TimelineEvent[];
122122
}
123123

124+
export interface ChangeReviewersReply {
125+
reviewers: ReviewState[];
126+
}
127+
124128
export interface SubmitReviewReply {
125129
events?: TimelineEvent[];
126130
reviewedEvent: ReviewEvent | CommentEvent;

0 commit comments

Comments
 (0)