From cbe091ad90325932e724e4ba4b2a9c818581a211 Mon Sep 17 00:00:00 2001 From: Jack Kingsman Date: Mon, 16 Feb 2026 18:32:45 -0800 Subject: [PATCH] Add clarifying comment for missing DM pathing info --- app/event_handlers.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/app/event_handlers.py b/app/event_handlers.py index a968b35..f1f893e 100644 --- a/app/event_handlers.py +++ b/app/event_handlers.py @@ -245,6 +245,9 @@ async def on_ack(event: "Event") -> None: logger.info("ACK received for message %d", message_id) ack_count = await MessageRepository.increment_ack_count(message_id) + # DM ACKs don't carry path data, so paths is intentionally omitted. + # The frontend's mergePendingAck handles the missing field correctly, + # preserving any previously known paths. broadcast_event("message_acked", {"message_id": message_id, "ack_count": ack_count}) else: logger.debug("ACK code %s does not match any pending messages", ack_code)