Show all comments in the comments view from a non-checked out PR when the description is opened. (#6575)

Fixes #6167

Author: alexr00

src/github/pullRequestModel.ts

@@ -1263,17 +1263,9 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 	}
 
 	static async openChanges(folderManager: FolderRepositoryManager, pullRequestModel: PullRequestModel) {
-		const isCurrentPR = folderManager.activePullRequest?.number === pullRequestModel.number;
-		const changes = pullRequestModel.fileChanges.size > 0 ? pullRequestModel.fileChanges.values() : await pullRequestModel.getFileChangesInfo();
+		const changeModels = await PullRequestModel.getChangeModels(folderManager, pullRequestModel);
 		const args: [vscode.Uri, vscode.Uri | undefined, vscode.Uri | undefined][] = [];
-
-		for (const change of changes) {
-			let changeModel;
-			if (change instanceof SlimFileChange) {
-				changeModel = new RemoteFileChangeModel(folderManager, change, pullRequestModel);
-			} else {
-				changeModel = new InMemFileChangeModel(folderManager, pullRequestModel as (PullRequestModel & IResolvedPullRequestModel), change, isCurrentPR, pullRequestModel.mergeBase!);
-			}
+		for (const changeModel of changeModels) {
 			args.push([changeModel.filePath, changeModel.parentFilePath, changeModel.filePath]);
 		}
 
@@ -1399,6 +1391,22 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 		return parsed;
 	}
 
+	public static async getChangeModels(folderManager: FolderRepositoryManager, pullRequestModel: PullRequestModel): Promise<(RemoteFileChangeModel | InMemFileChangeModel)[]> {
+		const isCurrentPR = folderManager.activePullRequest?.number === pullRequestModel.number;
+		const changes = pullRequestModel.fileChanges.size > 0 ? pullRequestModel.fileChanges.values() : await pullRequestModel.getFileChangesInfo();
+		const changeModels: (RemoteFileChangeModel | InMemFileChangeModel)[] = [];
+		for (const change of changes) {
+			let changeModel;
+			if (change instanceof SlimFileChange) {
+				changeModel = new RemoteFileChangeModel(folderManager, change, pullRequestModel);
+			} else {
+				changeModel = new InMemFileChangeModel(folderManager, pullRequestModel as (PullRequestModel & IResolvedPullRequestModel), change, isCurrentPR, pullRequestModel.mergeBase!);
+			}
+			changeModels.push(changeModel);
+		}
+		return changeModels;
+	}
+
 	/**
 	 * List the changed files in a pull request.
 	 */

src/view/pullRequestCommentController.ts

@@ -12,6 +12,7 @@ import Logger from '../common/logger';
 import { ITelemetry } from '../common/telemetry';
 import { fromPRUri, Schemes } from '../common/uri';
 import { groupBy } from '../common/utils';
+import { PULL_REQUEST_OVERVIEW_VIEW_TYPE } from '../common/webview';
 import { FolderRepositoryManager } from '../github/folderRepositoryManager';
 import { GitHubRepository } from '../github/githubRepository';
 import { GHPRComment, GHPRCommentThread, TemporaryComment } from '../github/prComment';
@@ -186,18 +187,35 @@ export class PullRequestCommentController extends CommentControllerBase implemen
 		return tabs.filter(tab => tab.input instanceof vscode.TabInputText || tab.input instanceof vscode.TabInputTextDiff).map(tab => tab.input as vscode.TabInputText | vscode.TabInputTextDiff);
 	}
 
+	private prDescriptionOpened(tabs: readonly vscode.Tab[]): boolean {
+		return tabs.some(tab => tab.input instanceof vscode.TabInputWebview && tab.label.includes(`#${this.pullRequestModel.number}`) && tab.input.viewType.includes(PULL_REQUEST_OVERVIEW_VIEW_TYPE));
+	}
+
 	private async cleanClosedPrs() {
 		// Remove comments for which no editors belonging to the same PR are open
 		const allPrEditors = await this.getPREditors(this.allTabs());
-		if (allPrEditors.length === 0) {
+		const prDescriptionOpened = this.prDescriptionOpened(vscode.window.tabGroups.all.map(group => group.tabs).flat());
+		if (allPrEditors.length === 0 && !prDescriptionOpened) {
 			this.removeAllCommentsThreads();
 		}
 	}
 
+	private async openAllTextDocuments(): Promise<vscode.TextDocument[]> {
+		const files = await PullRequestModel.getChangeModels(this._folderRepoManager, this.pullRequestModel);
+		const textDocuments: vscode.TextDocument[] = [];
+		for (const file of files) {
+			textDocuments.push(await vscode.workspace.openTextDocument(file.filePath));
+		}
+		return textDocuments;
+	}
+
 	private async onDidChangeOpenTabs(e: vscode.TabChangeEvent): Promise<void> {
 		const added = await this.getPREditors(this.filterTabsToPrTabs(e.opened));
 		if (added.length) {
 			await this.addThreadsForEditors(added);
+		} else if (this.prDescriptionOpened(e.opened)) {
+			const textDocuments = await this.openAllTextDocuments();
+			await this.addThreadsForEditors(textDocuments);
 		}
 		if (e.closed.length > 0) {
 			// Delay cleaning closed editors to handle the case where a preview tab is replaced