Fix: Ignore GraphQL permission error when quickly toggling reactions

When users quickly click on a reaction to add/remove it, a race condition
can occur where the client tries to remove a reaction that the user doesn't
have permission to remove. This results in a GraphQL error:
"does not have the correct permissions to execute 'RemoveReaction'"

This change silently ignores this specific error as it's expected behavior
during rapid toggling of reactions.

Fixes microsoft/vscode#69321

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

Author: Copilot

src/view/pullRequestCommentController.ts

@@ -546,14 +546,25 @@ export class PullRequestCommentController extends CommentControllerBase implemen
 			return;
 		}
 
-		if (
-			comment.reactions &&
-			!comment.reactions.find(ret => ret.label === reaction.label && !!ret.authorHasReacted)
-		) {
-			// add reaction
-			await this.pullRequestModel.addCommentReaction(comment.rawComment.graphNodeId, reaction);
-		} else {
-			await this.pullRequestModel.deleteCommentReaction(comment.rawComment.graphNodeId, reaction);
+		try {
+			if (
+				comment.reactions &&
+				!comment.reactions.find(ret => ret.label === reaction.label && !!ret.authorHasReacted)
+			) {
+				// add reaction
+				await this.pullRequestModel.addCommentReaction(comment.rawComment.graphNodeId, reaction);
+			} else {
+				await this.pullRequestModel.deleteCommentReaction(comment.rawComment.graphNodeId, reaction);
+			}
+		} catch (e) {
+			// Ignore permission errors when removing reactions due to race conditions
+			// See: https://github.com/microsoft/vscode/issues/69321
+			const errorMessage = (e as Error).message || String(e);
+			if (errorMessage.includes('does not have the correct permissions to execute \'RemoveReaction\'')) {
+				// Silently ignore this error - it occurs when quickly toggling reactions
+				return;
+			}
+			throw e;
 		}
 	}
 

src/view/reviewCommentController.ts

@@ -985,7 +985,14 @@ ${suggestionInformation.suggestionContent}
 				);
 			}
 		} catch (e) {
-			throw new Error(formatError(e));
+			// Ignore permission errors when removing reactions due to race conditions
+			// See: https://github.com/microsoft/vscode/issues/69321
+			const errorMessage = formatError(e);
+			if (errorMessage.includes('does not have the correct permissions to execute \'RemoveReaction\'')) {
+				// Silently ignore this error - it occurs when quickly toggling reactions
+				return;
+			}
+			throw new Error(errorMessage);
 		}
 	}