Unify dismiss notification commands (#7978)

Fixes #7893

Author: alexr00

package.json

@@ -967,7 +967,7 @@
 			},
 			{
 				"command": "pr.markAllCopilotNotificationsAsRead",
-				"title": "Mark All as Read",
+				"title": "%command.pr.markAllCopilotNotificationsAsRead.title%",
 				"category": "%command.pull.request.category%",
 				"enablement": "viewItem == copilot-query-with-notifications"
 			},

package.nls.json

@@ -191,6 +191,7 @@
 	"command.pr.pickOnVscodeDev.title": "Checkout Pull Request on vscode.dev",
 	"command.pr.exit.title": "Checkout Default Branch",
 	"command.pr.dismissNotification.title": "Dismiss Notification",
+	"command.pr.markAllCopilotNotificationsAsRead.title": "Dismiss All Copilot Notifications",
 	"command.pr.merge.title": "Merge Pull Request",
 	"command.pr.readyForReview.title": "Mark Pull Request Ready For Review",
 	"command.pr.openPullRequestOnGitHub.title": "Open Pull Request on GitHub",

src/commands.ts

@@ -900,6 +900,7 @@ export function registerCommands(
 		vscode.commands.registerCommand('pr.dismissNotification', node => {
 			if (node instanceof PRNode) {
 				notificationManager.markPrNotificationsAsRead(node.pullRequestModel);
+				copilotRemoteAgentManager.clearNotification(node.pullRequestModel.remote.owner, node.pullRequestModel.remote.repositoryName, node.pullRequestModel.number);
 			}
 		}),
 	);

src/github/copilotRemoteAgent.ts

@@ -994,10 +994,14 @@ export class CopilotRemoteAgentManager extends Disposable {
 	}
 
 
-	public clearAllNotifications(owner?: string, repo?: string): void {
+	clearAllNotifications(owner?: string, repo?: string): void {
 		this._stateModel.clearAllNotifications(owner, repo);
 	}
 
+	clearNotification(owner: string, repo: string, pullRequestNumber: number): void {
+		this._stateModel.clearNotification(owner, repo, pullRequestNumber);
+	}
+
 	hasNotification(owner: string, repo: string, pullRequestNumber?: number): boolean {
 		if (pullRequestNumber !== undefined) {
 			const key = this._stateModel.makeKey(owner, repo, pullRequestNumber);

src/view/treeNodes/pullRequestNode.ts

@@ -307,7 +307,7 @@ export class PRNode extends TreeNode implements vscode.CommentingRangeProvider2
 		}
 		const login = author.specialDisplayName ?? author.login;
 
-		const hasNotification = this._notificationProvider.hasNotification(this.pullRequestModel);
+		const hasNotification = this._notificationProvider.hasNotification(this.pullRequestModel) || this._codingAgentManager.hasNotification(this.pullRequestModel.remote.owner, this.pullRequestModel.remote.repositoryName, this.pullRequestModel.number);
 
 		const formattedPRNumber = number.toString();
 		let labelPrefix = currentBranchIsForThisPR ? '✓ ' : '';