@chaterafrikang/sdk 0.1.0-beta.3 → 0.1.0-beta.4

package/dist/index.cjs

@@ -445,15 +445,19 @@ var MessageFactory = class {
     };
   }
   /**
-   * Create a message from server payload (confirmed message)
+   * Create a message from server payload (confirmed message).
+   * Normalizes sender_id/senderId and sender_type/senderType so either snake or camelCase from server works.
    */
   static fromServerPayload(conversationId, payload) {
+    const senderId = payload.sender_id ?? payload.senderId;
+    const senderType = payload.sender_type ?? payload.senderType;
     return {
       id: payload.message_id,
       conversationId,
       content: payload.content,
-      senderId: payload.sender_id,
-      senderType: payload.sender_type,
+      senderId,
+      senderName: payload.sender_name,
+      senderType,
       messageType: payload.message_type === "system" ? "system" : "text",
       optimistic: false,
       receiptState: "sent",
@@ -461,14 +465,18 @@ var MessageFactory = class {
     };
   }
   /**
-   * Convert an optimistic message to confirmed (reconciliation)
+   * Convert an optimistic message to confirmed (reconciliation).
+   * Normalizes sender_id/senderId and sender_type/senderType from server payload.
    */
   static confirmOptimistic(optimisticMessage, serverPayload) {
+    const senderId = serverPayload.sender_id ?? serverPayload.senderId;
+    const senderType = serverPayload.sender_type ?? serverPayload.senderType;
     return {
       ...optimisticMessage,
       optimistic: false,
-      senderId: serverPayload.sender_id,
-      senderType: serverPayload.sender_type,
+      senderId,
+      senderName: serverPayload.sender_name,
+      senderType,
       receiptState: optimisticMessage.receiptState ?? "sent",
       createdAt: serverPayload.sent_at ? new Date(serverPayload.sent_at) : optimisticMessage.createdAt
     };
@@ -619,6 +627,10 @@ var MessageManager = class extends eventemitter3.EventEmitter {
      * Map<conversationId, Map<message_id, Message>>
      */
     this.confirmedMessages = /* @__PURE__ */ new Map();
+    /**
+     * Conversations we've already warned about missing sender_id (warn once per conversation)
+     */
+    this.warnedMissingSenderId = /* @__PURE__ */ new Set();
     this.logger = new Logger(debug);
     this.receiptManager = new ReceiptManager(debug);
     this.receiptManager.on("receipt", (event) => {
@@ -659,6 +671,12 @@ var MessageManager = class extends eventemitter3.EventEmitter {
     const optimisticMessage = this.getOptimisticMessage(conversationId, messageId);
     if (optimisticMessage) {
       const reconciledMessage = this.reconcileOptimisticMessage(optimisticMessage, serverPayload);
+      if (reconciledMessage.senderId == null && !this.warnedMissingSenderId.has(conversationId)) {
+        this.warnedMissingSenderId.add(conversationId);
+        console.warn(
+          '[ChatAfrika SDK] Message payload missing sender_id (and sender_type). Clients cannot show "You" vs other participant. The realtime backend MUST include sender_id and sender_type in every type:"message" payload. See docs/REALTIME_WEBSOCKET_CONTRACT.md.'
+        );
+      }
       this.removeOptimisticMessage(conversationId, messageId);
       this.trackMessageId(conversationId, messageId);
       this.trackConfirmedMessage(conversationId, messageId, reconciledMessage);
@@ -670,6 +688,12 @@ var MessageManager = class extends eventemitter3.EventEmitter {
     }
     this.trackMessageId(conversationId, messageId);
     const message = MessageFactory.fromServerPayload(conversationId, serverPayload);
+    if (message.senderId == null && !this.warnedMissingSenderId.has(conversationId)) {
+      this.warnedMissingSenderId.add(conversationId);
+      console.warn(
+        '[ChatAfrika SDK] Message payload missing sender_id (and sender_type). Clients cannot show "You" vs other participant. The realtime backend MUST include sender_id and sender_type in every type:"message" payload. See docs/REALTIME_WEBSOCKET_CONTRACT.md.'
+      );
+    }
     this.trackConfirmedMessage(conversationId, messageId, message);
     this.emit("message", message);
     this.logger.debug("Handled incoming message:", messageId);
@@ -723,17 +747,14 @@ var MessageManager = class extends eventemitter3.EventEmitter {
     this.logger.warn("Receipt for unknown message:", event.messageId);
   }
   /**
-   * Reconcile an optimistic message with a server confirmation
-   * 
-   * This is called when we receive a server message with the same message_id
-   * as an optimistic message we previously created.
-   * 
+   * Reconcile an optimistic message with a server confirmation.
+   * Does not emit again so the app receives at most one event per message_id.
+   *
    * @returns The reconciled (confirmed) message
    */
   reconcileOptimisticMessage(optimisticMessage, serverPayload) {
     const confirmedMessage = MessageFactory.confirmOptimistic(optimisticMessage, serverPayload);
-    this.emit("message", confirmedMessage);
-    this.logger.debug("Reconciled optimistic message:", optimisticMessage.id);
+    this.logger.debug("Reconciled optimistic message (no second emit):", optimisticMessage.id);
     return confirmedMessage;
   }
   /**
@@ -743,6 +764,15 @@ var MessageManager = class extends eventemitter3.EventEmitter {
     const seenIds = this.seenMessageIds.get(conversationId);
     return seenIds?.has(messageId) ?? false;
   }
+  /**
+   * Track message IDs from REST history so the same message is not emitted again via WebSocket.
+   * Call after fetchMessages() so history and real-time stay deduped.
+   */
+  trackMessageIdsFromHistory(conversationId, messageIds) {
+    for (const id of messageIds) {
+      this.trackMessageId(conversationId, id);
+    }
+  }
   /**
    * Track a message ID for a conversation (for deduplication)
    */
@@ -1037,11 +1067,16 @@ var Conversation = class extends eventemitter3.EventEmitter {
         "REST not configured. Set apiUrl in ChatAfrika config for fetchMessages() (token, API keys, or getRestHeaders)."
       );
     }
-    return this.fetchMessagesCallback({
+    const result = await this.fetchMessagesCallback({
       externalUserId: options?.externalUserId ?? "",
       cursor: options?.cursor,
       limit: options?.limit
     });
+    this.messageManager.trackMessageIdsFromHistory(
+      this.id,
+      result.messages.map((m) => m.id)
+    );
+    return result;
   }
   /**
    * Mark conversation as joined