From cdb5ac14718f3df6b0f94f4e04ac0ccb404c1d4a Mon Sep 17 00:00:00 2001
From: Giulio V <vannini.gv@gmail.com>
Date: Sat, 21 Mar 2026 18:49:35 +0100
Subject: [PATCH] fix(tools): fix remove_reaction_success test

The output format used "{action}ed" which produced "removeed" for the
remove action. Use explicit past-tense mapping instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 src/tools/reaction.rs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/tools/reaction.rs b/src/tools/reaction.rs
index 22d3cbd2e..59681dea9 100644
--- a/src/tools/reaction.rs
+++ b/src/tools/reaction.rs
@@ -158,11 +158,17 @@ impl Tool for ReactionTool {
                 .await
         };
 
+        let past_tense = if action == "remove" {
+            "removed"
+        } else {
+            "added"
+        };
+
         match result {
             Ok(()) => Ok(ToolResult {
                 success: true,
                 output: format!(
-                    "Reaction {action}ed: {emoji} on message {message_id} in {channel_name}"
+                    "Reaction {past_tense}: {emoji} on message {message_id} in {channel_name}"
                 ),
                 error: None,
             }),