Code review fixes: use makeKey, existing getNotificationsCount method, and repo getter

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

Author: Copilot

src/commands.ts

@@ -919,9 +919,8 @@ export function registerCommands(
 
 	context.subscriptions.push(
 		vscode.commands.registerCommand('pr.markAllCopilotNotificationsAsRead', node => {
-			if (node instanceof CategoryTreeNode && node.isCopilot && node.folderRepoManager.gitHubRepositories.length > 0) {
-				const repo = node.folderRepoManager.gitHubRepositories[0];
-				copilotRemoteAgentManager.clearAllNotifications(repo.remote.owner, repo.remote.repositoryName);
+			if (node instanceof CategoryTreeNode && node.isCopilot && node.repo) {
+				copilotRemoteAgentManager.clearAllNotifications(node.repo.owner, node.repo.repositoryName);
 				tree.refresh(node);
 			}
 		}),

src/github/copilotPrWatcher.ts

@@ -114,8 +114,9 @@ export class CopilotStateModel extends Disposable {
 			// If owner and repo are specified, only clear notifications for that repo
 			if (owner && repo) {
 				const keysToRemove: string[] = [];
+				const prefix = `${this.makeKey(owner, repo)}#`;
 				for (const key of this._showNotification.keys()) {
-					if (key.startsWith(`${owner}/${repo}#`)) {
+					if (key.startsWith(prefix)) {
 						const item = this._states.get(key)?.item;
 						if (item) {
 							items.push(item);

src/github/copilotRemoteAgent.ts

@@ -880,14 +880,7 @@ export class CopilotRemoteAgentManager extends Disposable {
 	}
 
 	getNotificationsCountForRepo(owner: string, repo: string): number {
-		let count = 0;
-		const prefix = `${owner}/${repo}#`;
-		for (const key of this._stateModel.notifications.keys()) {
-			if (key.startsWith(prefix)) {
-				count++;
-			}
-		}
-		return count;
+		return this._stateModel.getNotificationsCount(owner, repo);
 	}
 
 	public clearAllNotifications(owner?: string, repo?: string): void {

src/view/treeNodes/categoryNode.ts

@@ -195,6 +195,10 @@ export class CategoryTreeNode extends TreeNode implements vscode.TreeItem {
 		}
 	}
 
+	get repo(): RemoteInfo | undefined {
+		return this._repo;
+	}
+
 	private _getDescription(): string | undefined {
 		if (!this.isCopilot || !this._repo) {
 			return undefined;