Suppress RemoveReaction permission errors during rapid reaction toggling

[Copilot]

Race condition when rapidly toggling reactions causes permission errors as the client attempts to remove reactions that don't belong to the user or are already removed server-side.

Changes

• Error suppression in toggleReaction: Catch and silently ignore RemoveReaction permission errors in both reviewCommentController.ts and pullRequestCommentController.ts
• Pattern matching: Check error message for 'does not have the correct permissions to execute \'RemoveReaction\'' (consistent with existing GraphQL error handling throughout the codebase)
• Consistent error handling: Use formatError() and re-wrap in both implementations

try {
  // add or remove reaction
} catch (e) {
  const errorMessage = formatError(e);
  if (errorMessage.includes('does not have the correct permissions to execute \'RemoveReaction\'')) {
    return; // Silently ignore race condition
  }
  throw new Error(errorMessage);
}

All other errors remain unhandled to preserve existing behavior.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>GraphQL error while quickly adding and removing reaction</issue_title>
<issue_description>microsoft/vscode#69321

Repo

1. open a PR with reaction comments
2. Quickly click multiple times on an existing reaction that someone else has added

Bug
See the error: Deleting the comment reaction failed: GraphQL error: mjbvz does not have the correct permissions to execute 'RemoveReaction'

</issue_description>

<agent_instructions>Just ignore this specific error.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

• Fixes GraphQL error while quickly adding and removing reaction #1001

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.