Close outdated multidiff editors when PR is reopened (#8256)

* Initial plan

* Initial plan for closing outdated multidiff editors

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Close outdated multidiff editors when PR is reopened

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Refactor multidiff editor closing based on code review feedback

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Add documentation comments based on code review feedback

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Revert formatting changes, keep only functional changes

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Add TabInputTextMultiDiff API proposal

* Use TabInputTextMultiDiff API instead of workaround

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

* Clean up

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

Author: Copilot

package.json

@@ -36,6 +36,7 @@
 		"quickDiffProvider",
 		"remoteCodingAgents",
 		"shareProvider",
+		"tabInputMultiDiff",
 		"tokenInformation",
 		"treeItemMarkdownLabel",
 		"treeViewMarkdownMessage"

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

@@ -0,0 +1,22 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the MIT License. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+
+// https://github.com/microsoft/vscode/issues/206411
+
+declare module 'vscode' {
+
+	export class TabInputTextMultiDiff {
+
+		readonly textDiffs: TabInputTextDiff[];
+
+		constructor(textDiffs: TabInputTextDiff[]);
+	}
+
+	export interface Tab {
+
+		readonly input: TabInputText | TabInputTextDiff | TabInputTextMultiDiff | TabInputCustom | TabInputWebview | TabInputNotebook | TabInputNotebookDiff | TabInputTerminal | unknown;
+
+	}
+}

src/view/reviewManager.ts

@@ -675,7 +675,27 @@ export class ReviewManager extends Disposable {
 		}
 	}
 
+	private async _closeOutdatedMultiDiffEditors(pullRequest: PullRequestModel): Promise<void> {
+		// Close any multidiff editors for this PR that may be outdated
+		const multiDiffLabel = vscode.l10n.t('Changes in Pull Request #{0}', pullRequest.number);
+
+		const closePromises: Promise<boolean>[] = [];
+		for (const tabGroup of vscode.window.tabGroups.all) {
+			for (const tab of tabGroup.tabs) {
+				// Check if this is a TabInputTextMultiDiff with matching label
+				if (tab.input instanceof vscode.TabInputTextMultiDiff && tab.label.startsWith(multiDiffLabel)) {
+					Logger.appendLine(`Closing outdated multidiff editor for PR #${pullRequest.number}`, this.id);
+					closePromises.push(Promise.resolve(vscode.window.tabGroups.close(tab)));
+				}
+			}
+		}
+		await Promise.all(closePromises);
+	}
+
 	public async _upgradePullRequestEditors(pullRequest: PullRequestModel) {
+		// Close any outdated multidiff editors first
+		await this._closeOutdatedMultiDiffEditors(pullRequest);
+
 		// Go through all open editors and find pr scheme editors that belong to the active pull request.
 		// Close the editors, and reopen them from the pull request.
 		const reopenFilenames: Set<[PRUriParams, PRUriParams]> = new Set();