extend to add all reactions to the review context

Author: HungKNguyen

claudecode/format_pr_comments.py

@@ -14,6 +14,18 @@
 # Bot comment marker pattern
 BOT_COMMENT_MARKER = "🤖 **Code Review Finding:"
 
+# GitHub reaction content to emoji mapping
+REACTION_EMOJI_MAP = {
+    '+1': '👍',
+    '-1': '👎',
+    'laugh': '😄',
+    'confused': '😕',
+    'heart': '❤️',
+    'hooray': '🎉',
+    'rocket': '🚀',
+    'eyes': '👀',
+}
+
 
 def is_bot_comment(comment: Dict[str, Any]) -> bool:
     """Check if a comment was posted by this bot.
@@ -223,17 +235,11 @@ def _format_threads_for_prompt(threads: List[Dict[str, Any]]) -> str:
 
         # Add user reactions (excluding bot's own reactions)
         if reactions:
-            thumbs_up = reactions.get('+1', 0)
-            thumbs_down = reactions.get('-1', 0)
-            other_reactions = {k: v for k, v in reactions.items() if k not in ['+1', '-1']}
-
             reaction_parts = []
-            if thumbs_up > 0:
-                reaction_parts.append(f"👍 {thumbs_up}")
-            if thumbs_down > 0:
-                reaction_parts.append(f"👎 {thumbs_down}")
-            for reaction, count in other_reactions.items():
-                reaction_parts.append(f"{reaction} {count}")
+            for reaction, count in reactions.items():
+                if count > 0:
+                    emoji = REACTION_EMOJI_MAP.get(reaction, reaction)
+                    reaction_parts.append(f"{emoji} {count}")
 
             if reaction_parts:
                 lines.append(f"  User Reactions: {', '.join(reaction_parts)}")

claudecode/test_format_pr_comments.py

@@ -146,6 +146,33 @@ def test_reactions_included(self):
         # Should show 2 thumbs down
         assert "👎 2" in result
 
+    def test_all_reaction_types(self):
+        """Test that all GitHub reaction types are converted to emojis."""
+        bot_comment = _make_bot_comment(1)
+        reactions = {
+            '+1': 3,
+            '-1': 1,
+            'laugh': 2,
+            'confused': 1,
+            'heart': 4,
+            'hooray': 2,
+            'rocket': 1,
+            'eyes': 3,
+        }
+        threads = [_make_thread(bot_comment, reactions=reactions)]
+
+        result = format_pr_comments_for_prompt(threads)
+
+        # All reactions should be converted to emojis
+        assert "👍 3" in result
+        assert "👎 1" in result
+        assert "😄 2" in result
+        assert "😕 1" in result
+        assert "❤️ 4" in result
+        assert "🎉 2" in result
+        assert "🚀 1" in result
+        assert "👀 3" in result
+
     def test_truncate_long_replies(self):
         """Test that very long reply threads are truncated."""
         bot_comment = _make_bot_comment(1)

claudecode/test_workflow_integration.py

@@ -159,7 +159,7 @@ def test_full_workflow_with_real_pr_structure(self, mock_get, mock_run):
             }
         ]
 
-        # Reactions response for the bot comment - one +1 from a human
+        # Reactions response for the bot comment - multiple reactions from humans
         reactions_response = Mock()
         reactions_response.status_code = 200
         reactions_response.json.return_value = [
@@ -173,6 +173,28 @@ def test_full_workflow_with_real_pr_structure(self, mock_get, mock_run):
                 },
                 'content': '+1',
                 'created_at': '2024-01-15T14:00:00Z'
+            },
+            {
+                'id': 2,
+                'node_id': 'MDg6UmVhY3Rpb24y',
+                'user': {
+                    'login': 'teammate',
+                    'id': 67890,
+                    'type': 'User'
+                },
+                'content': 'eyes',
+                'created_at': '2024-01-15T14:05:00Z'
+            },
+            {
+                'id': 3,
+                'node_id': 'MDg6UmVhY3Rpb24z',
+                'user': {
+                    'login': 'lead',
+                    'id': 11111,
+                    'type': 'User'
+                },
+                'content': 'rocket',
+                'created_at': '2024-01-15T14:10:00Z'
             }
         ]