@droppii-org/chat-mobile 0.2.79 → 0.2.80

package/src/utils/message.ts

@@ -6,12 +6,30 @@ import {
 } from '@droppii/openim-rn-client-sdk';
 import { getConversationID } from './conversation';
 
-const getMessageTime = (message: MessageItem) =>
+/**
+ * Canonical timestamp for a message. `sendTime` is 0 until the SDK stamps an
+ * outgoing message, so `createTime` covers the in-flight window. This is the
+ * key the list is sorted by — anything reasoning about message order must use
+ * it too, or it will disagree with the order actually rendered.
+ */
+export const getMessageTime = (message: MessageItem) =>
   message.sendTime || message.createTime || 0;
 
 const getMessageKey = (message: MessageItem) =>
   message.clientMsgID || message.serverMsgID;
 
+/**
+ * Stable id for a message inside a rendered list.
+ *
+ * A message that has not round-tripped through the server yet may carry
+ * neither id, so position and timestamp stand in. Every consumer that looks a
+ * message up by id must derive it the same way — the precompute map is keyed
+ * on this, so a divergent formula silently misses and the message renders
+ * without its day divider or cluster position.
+ */
+export const getMessageRenderId = (message: MessageItem, index: number) =>
+  getMessageKey(message) || `temp-${getMessageTime(message)}-${index}`;
+
 export const mergeMessages = (
   current: MessageItem[],
   incoming: MessageItem[]

package/src/utils/messageUtils.ts

@@ -92,6 +92,14 @@ export const GROUP_NOTIFICATION_TEXT: Partial<
   },
 };
 
+// The content types above, as a plain list. Both the chat list (to decide
+// whether a message renders as a system banner) and the grouping precompute
+// (system messages never join a sender cluster) need this, so it is derived
+// once here rather than in each consumer.
+export const GROUP_NOTIFICATION_TYPES = Object.keys(
+  GROUP_NOTIFICATION_TEXT
+).map(Number) as MessageType[];
+
 const parseNotificationDetail = (message: MessageItem): any => {
   if (!message.notificationElem?.detail) return undefined;
   try {