fix: rely on processedMessageList for calculations

Author: khushal87

package/src/components/MessageList/__tests__/useMessageDateSeparator.test.ts

@@ -28,7 +28,7 @@ describe('useMessageDateSeparator', () => {
         text: 'Hello World',
       },
     ] as LocalMessage[];
-    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage(messages);
+    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage({ messages });
   });
 
   it('should return undefined if no message is passed', () => {
@@ -70,7 +70,7 @@ describe('useMessageDateSeparator', () => {
       },
     ] as LocalMessage[];
     const messageListPreviousAndNextMessageStore = new MessagePreviousAndNextMessageStore();
-    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage(messages);
+    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage({ messages });
     const { result: resultOfFirstMessage } = renderHook(() =>
       useMessageDateSeparator({ message: messages[0], messageListPreviousAndNextMessageStore }),
     );

package/src/components/MessageList/hooks/useMessageList.ts

@@ -62,7 +62,7 @@ export const shouldIncludeMessageInList = (
 };
 
 export const useMessageList = (params: UseMessageListParams) => {
-  const { noGroupByUser, threadList, isLiveStreaming, isFlashList } = params;
+  const { noGroupByUser, threadList, isLiveStreaming, isFlashList = false } = params;
   const { client } = useChatContext();
   const { hideDateSeparators, maxTimeBetweenGroupedMessages } = useChannelContext();
   const { deletedMessagesVisibilityType, getMessagesGroupStyles = getGroupStyles } =
@@ -95,10 +95,10 @@ export const useMessageList = (params: UseMessageListParams) => {
   }, [messageList, deletedMessagesVisibilityType, client.userID, isFlashList]);
 
   useEffect(() => {
-    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage(
-      processedMessageList,
+    messageListPreviousAndNextMessageStore.setMessageListPreviousAndNextMessage({
       isFlashList,
-    );
+      messages: processedMessageList,
+    });
   }, [processedMessageList, messageListPreviousAndNextMessageStore, isFlashList]);
 
   /**

package/src/state-store/message-list-prev-next-state.ts

@@ -21,7 +21,14 @@ export class MessagePreviousAndNextMessageStore {
     this.state = new StateStore<MessagePreviousAndNextMessageStoreType>(INITIAL_STATE);
   }
 
-  public setMessageListPreviousAndNextMessage(messages: LocalMessage[], isFlashList?: boolean) {
+  // The default value of isFlashList is true as the logic in the function makes more sense when the list is not reversed.
+  public setMessageListPreviousAndNextMessage({
+    messages,
+    isFlashList = true,
+  }: {
+    messages: LocalMessage[];
+    isFlashList?: boolean;
+  }) {
     const currentValue = this.state.getLatestValue();
     const prevMessageList: MessagePreviousAndNextMessageStoreType['messageList'] =
       currentValue.messageList;