Commenting on a coding agent PR doesn't refresh timeline (#7195)

Fixes #7193

Author: alexr00

src/github/activityBarViewProvider.ts

@@ -364,8 +364,9 @@ export class PullRequestViewProvider extends WebviewViewBase implements vscode.W
 			this.updateReviewers(review);
 			const reviewMessage: SubmitReviewReply & { command: string } = {
 				command: 'pr.append-review',
-				event: review,
-				reviewers: this._existingReviewers
+				events: [],
+				reviewers: this._existingReviewers,
+				reviewedEvent: review,
 			};
 			await this._postMessage(reviewMessage);
 		} catch (e) {
@@ -380,7 +381,8 @@ export class PullRequestViewProvider extends WebviewViewBase implements vscode.W
 			const review = await action(message.args);
 			this.updateReviewers(review);
 			const reviewMessage: SubmitReviewReply = {
-				event: review,
+				events: [],
+				reviewedEvent: review,
 				reviewers: this._existingReviewers,
 			};
 			this._replyMessage(message, reviewMessage);

src/github/issueOverview.ts

@@ -19,7 +19,7 @@ import { GithubItemStateEnum, IAccount, ILabel, IMilestone, IProject, IProjectIt
 import { IssueModel } from './issueModel';
 import { getAssigneesQuickPickItems, getLabelOptions, getMilestoneFromQuickPick, getProjectFromQuickPick } from './quickPicks';
 import { isInCodespaces, vscodeDevPrLink } from './utils';
-import { ChangeAssigneesReply, Issue, ProjectItemsReply } from './views';
+import { ChangeAssigneesReply, Issue, ProjectItemsReply, SubmitReviewReply } from './views';
 
 export class IssueOverviewPanel<TItem extends IssueModel = IssueModel> extends WebviewBase {
 	public static ID: string = 'IssueOverviewPanel';
@@ -603,14 +603,17 @@ export class IssueOverviewPanel<TItem extends IssueModel = IssueModel> extends W
 	}
 
 	private createComment(message: IRequestMessage<string>) {
-		return this._item.createIssueComment(message.args).then(comment => {
+		return this._item.createIssueComment(message.args).then(async (comment) => {
 			const commentedEvent: CommentEvent = {
 				...comment,
 				event: EventType.Commented
 			};
-			return this._replyMessage(message, {
-				event: commentedEvent,
-			});
+			const allEvents = await this._item.githubRepository.getIssueTimelineEvents(this._item);
+			const reply: SubmitReviewReply = {
+				events: allEvents,
+				reviewedEvent: commentedEvent,
+			};
+			return this._replyMessage(message, reply);
 		});
 	}
 

src/github/pullRequestOverview.ts

@@ -655,9 +655,11 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 		try {
 			const review = await action(context.body);
 			this.updateReviewers(review);
+			const allEvents = await this._item.getTimelineEvents();
 			const reviewMessage: SubmitReviewReply & { command: string } = {
 				command: 'pr.append-review',
-				event: review,
+				reviewedEvent: review,
+				events: allEvents,
 				reviewers: this._existingReviewers
 			};
 			await this._postMessage(reviewMessage);
@@ -673,8 +675,10 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 		try {
 			const review = await action(message.args);
 			this.updateReviewers(review);
+			const allEvents = await this._item.getTimelineEvents();
 			const reply: SubmitReviewReply = {
-				event: review,
+				reviewedEvent: review,
+				events: allEvents,
 				reviewers: this._existingReviewers,
 			};
 			this._replyMessage(message, reply);

src/github/views.ts

@@ -3,7 +3,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-import { ReviewEvent, TimelineEvent } from '../common/timelineEvent';
+import { CommentEvent, ReviewEvent, TimelineEvent } from '../common/timelineEvent';
 import {
 	GithubItemStateEnum,
 	IAccount,
@@ -110,7 +110,8 @@ export interface ChangeAssigneesReply {
 }
 
 export interface SubmitReviewReply {
-	event?: ReviewEvent | TimelineEvent;
+	events?: TimelineEvent[];
+	reviewedEvent: ReviewEvent | CommentEvent;
 	reviewers?: ReviewState[];
 }
 

webviews/common/context.tsx

@@ -176,25 +176,19 @@ export class PRContext {
 	};
 
 	private appendReview(reply: SubmitReviewReply) {
-		const { event, reviewers } = reply;
+		const { events, reviewers, reviewedEvent } = reply;
 		const state = this.pr;
 		state.busy = false;
-		if (!event) {
+		if (!events) {
 			this.updatePR(state);
 			return;
 		}
-		const events = state.events.filter(e => e.event !== EventType.Reviewed || e.state?.toLowerCase() !== 'pending');
-		events.forEach(event => {
-			if (event.event === EventType.Reviewed) {
-				event.comments.forEach(c => (c.isDraft = false));
-			}
-		});
 		if (reviewers) {
 			state.reviewers = reviewers;
 		}
-		state.events = [...state.events.filter(e => (e.event === EventType.Reviewed ? e.state !== 'PENDING' : e)), event];
-		if (event.event === EventType.Reviewed) {
-			state.currentUserReviewState = event.state;
+		state.events = events;
+		if (reviewedEvent.event === EventType.Reviewed) {
+			state.currentUserReviewState = reviewedEvent.state;
 		}
 		state.pendingCommentText = '';
 		state.pendingReviewType = undefined;