수정된 메시지에는 반응하지 않는다

슬랙은 이미 올라온 메시지를 고쳐도 이벤트를 보낸다. app_mention 경로에는
이걸 걸러내는 곳이 없어서, 멘션 명령이 담긴 글을 고치면 같은 명령이 한 번 더
실행됐다 (리마인더 DM 재발송 등). 답글도 중복으로 달린다.

수정 사실이 이벤트마다 다른 자리에 실려오기 때문에 (message 는 subtype
'message_changed' + 본문이 event['message'] 아래로, app_mention 은 최상위
'edited') 판별을 util.slack_event.is_edited_message 로 모으고, 두 이벤트
핸들러 진입점에서 바로 끊는다. 핸들러 안이 아니라 진입점에서 막아야
loading 이모지도 붙었다 떨어지지 않는다.

message 이벤트는 SubinLikeResponse 의 subtype 검사가 이미 막고 있었지만,
같은 판별을 한곳에 두고 진입점에서 함께 끊도록 했다.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Pko9c11HWMkm9zEPfCAPGc

Author: claude

src/anna.py

@@ -33,6 +33,7 @@
     verify_calendar_token,
 )
 from util.daily_scheduler import DailyScheduler
+from util.slack_event import is_edited_message
 
 init_logger()
 
@@ -63,6 +64,12 @@ def handle_reaction_removed_event(ack, event, say, client):
 @app.event("app_mention")
 def handle_app_mention_event(ack, event, say, client):
     ack()
+    # 이미 있던 멘션을 고쳐도 이벤트가 다시 온다 — 명령이 두 번 실행되지 않도록 여기서 끊는다
+    if is_edited_message(event):
+        logging.getLogger(__name__).info(
+            "Ignoring app_mention from an edited message (ts=%s)", event.get("ts")
+        )
+        return
     mention_response(say=say, event=event, client=client)
 
 
@@ -75,6 +82,12 @@ def handle_channel_created_event(ack, event, say, client):
 @app.event("message")
 def handle_message_event(ack, event, say, client):
     ack()
+    # 수정은 새 글이 아니다 (자동 답글 대상 채널은 수정이 잦아 DEBUG 로만 남긴다)
+    if is_edited_message(event):
+        logging.getLogger(__name__).debug(
+            "Ignoring edited message event (ts=%s)", event.get("ts")
+        )
+        return
     subin_like_response(event=event, say=say, client=client)
 
 

src/util/slack_event.py

@@ -0,0 +1,22 @@
+"""슬랙 이벤트 판별 헬퍼."""
+
+
+def is_edited_message(event) -> bool:
+    """이미 올라온 메시지를 고쳐서 온 이벤트인지.
+
+    슬랙은 메시지 '수정'에도 이벤트를 보내는데, 새 글과 똑같이 처리하면
+    같은 멘션 명령이 한 번 더 실행되고(리마인더 재발송 등) 답글도 중복으로 달린다.
+    수정 사실은 이벤트 종류마다 다른 자리에 실려온다:
+
+    - message: subtype 이 'message_changed' 이고, 본문은 event['message'] 아래로 내려간다
+    - app_mention: 최상위에 'edited' 가 붙는다 (이미 있던 멘션을 고치면 다시 발생)
+    """
+    if not isinstance(event, dict):
+        return False
+    if event.get("subtype") == "message_changed":
+        return True
+    if event.get("edited"):
+        return True
+    if (event.get("message") or {}).get("edited"):
+        return True
+    return False

test/util/test_slack_event.py

@@ -0,0 +1,67 @@
+import unittest
+
+from util.slack_event import is_edited_message
+
+# 새로 올라온 멘션 (실제 payload 형태)
+NEW_APP_MENTION = {
+    "type": "app_mention",
+    "text": "<@UANNA> 빅챗 리마인더 테스트 <@U0001>",
+    "user": "UQJ8HQJG5",
+    "ts": "1689403771.805849",
+    "channel": "C03SZTDEDK3",
+    "event_ts": "1689403771.805849",
+}
+
+# 그 멘션을 고쳤을 때 다시 오는 이벤트 — 최상위에 edited 가 붙는다
+EDITED_APP_MENTION = {
+    **NEW_APP_MENTION,
+    "edited": {"user": "UQJ8HQJG5", "ts": "1689403800.000000"},
+}
+
+# 메시지 수정 — 본문이 message 아래로 내려가고 subtype 이 붙는다
+MESSAGE_CHANGED = {
+    "type": "message",
+    "subtype": "message_changed",
+    "channel": "C03SZTDEDK3",
+    "ts": "1689403800.000200",
+    "message": {
+        "type": "message",
+        "text": "고친 글",
+        "user": "UQJ8HQJG5",
+        "ts": "1689403771.805849",
+        "edited": {"user": "UQJ8HQJG5", "ts": "1689403800.000000"},
+    },
+    "previous_message": {"text": "원래 글", "ts": "1689403771.805849"},
+}
+
+NEW_MESSAGE = {
+    "type": "message",
+    "channel": "C03SZTDEDK3",
+    "text": "새로 쓴 글",
+    "user": "UQJ8HQJG5",
+    "ts": "1689403771.805849",
+}
+
+# 이미지 첨부 글은 subtype 이 있어도 수정이 아니다 (자동 답글 대상)
+FILE_SHARE_MESSAGE = {
+    **NEW_MESSAGE,
+    "subtype": "file_share",
+    "files": [{"mimetype": "image/png", "url_private": "https://..."}],
+}
+
+
+class TestIsEditedMessage(unittest.TestCase):
+    def test_edited_events(self):
+        assert is_edited_message(EDITED_APP_MENTION)
+        assert is_edited_message(MESSAGE_CHANGED)
+
+    def test_new_events(self):
+        assert not is_edited_message(NEW_APP_MENTION)
+        assert not is_edited_message(NEW_MESSAGE)
+        assert not is_edited_message(FILE_SHARE_MESSAGE)
+
+    def test_ignores_falsy_and_malformed_payloads(self):
+        assert not is_edited_message({})
+        assert not is_edited_message(None)
+        assert not is_edited_message({"edited": None})
+        assert not is_edited_message({"message": None})