Fix marking channel read when scrolling to bottom without unread count (#3881)

* Fix marking channel read when scrolling to bottom without unread count

* Update CHANGELOG.md

Author: nuno-vieira

CHANGELOG.md

@@ -3,7 +3,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 # Upcoming
 
-### 🔄 Changed
+## StreamChatUI
+### 🐞 Fixed
+- Fix marking channel read when scrolling to the bottom without unread counts [#3881](https://github.com/GetStream/stream-chat-swift/pull/3881)
 
 # [4.93.0](https://github.com/GetStream/stream-chat-swift/releases/tag/4.93.0)
 _November 18, 2025_

Sources/StreamChatUI/ChatChannel/ChatChannelVC.swift

@@ -89,7 +89,12 @@ open class ChatChannelVC: _ViewController,
             return isLastMessageFullyVisible && isFirstPageLoaded
         }
 
-        return isLastMessageVisibleOrSeen && hasSeenFirstUnreadMessage && isFirstPageLoaded && !hasMarkedMessageAsUnread
+        let unreadMessageCount = channelController.channel?.unreadCount.messages ?? 0
+        return isLastMessageVisibleOrSeen
+            && hasSeenFirstUnreadMessage
+            && isFirstPageLoaded
+            && !hasMarkedMessageAsUnread
+            && unreadMessageCount > 0
     }
 
     private var isLastMessageVisibleOrSeen: Bool {

Tests/StreamChatUITests/SnapshotTests/ChatChannel/ChatChannelVC_Tests.swift

@@ -904,11 +904,12 @@ final class ChatChannelVC_Tests: XCTestCase {
         XCTAssertTrue(vc.shouldMarkChannelRead)
     }
 
-    func test_shouldMarkChannelRead_jumpToUnreadEnabled_viewIsVisible_remoteDataFetched_lastMessageVisible_hasLoadedAllNextMessages_hasNotMarkedMessageAsUnread_shouldReturnTrue() {
+    func test_shouldMarkChannelRead_jumpToUnreadEnabled_viewIsVisible_remoteDataFetched_lastMessageVisible_hasLoadedAllNextMessages_hasNotMarkedMessageAsUnread_withUnreads_shouldReturnTrue() {
         let mockedListView = makeMockMessageListView()
         vc.mockIsViewVisible(true)
         vc.components.isJumpToUnreadEnabled = true
         channelControllerMock.state_mock = .remoteDataFetched
+        channelControllerMock.channel_mock = .mock(cid: .unique, unreadCount: ChannelUnreadCount(messages: 1, mentions: 0))
         mockedListView.mockIsLastCellFullyVisible = true
         channelControllerMock.hasLoadedAllNextMessages_mock = true
         channelControllerMock.markedAsUnread_mock = false
@@ -919,6 +920,23 @@ final class ChatChannelVC_Tests: XCTestCase {
 
         XCTAssertTrue(vc.shouldMarkChannelRead)
     }
+    
+    func test_shouldMarkChannelRead_jumpToUnreadEnabled_viewIsVisible_remoteDataFetched_lastMessageVisible_hasLoadedAllNextMessages_hasNotMarkedMessageAsUnread_withoutUnreads_shouldReturnFalse() {
+        let mockedListView = makeMockMessageListView()
+        vc.mockIsViewVisible(true)
+        vc.components.isJumpToUnreadEnabled = true
+        channelControllerMock.state_mock = .remoteDataFetched
+        channelControllerMock.channel_mock = .mock(cid: .unique, unreadCount: ChannelUnreadCount(messages: 0, mentions: 0))
+        mockedListView.mockIsLastCellFullyVisible = true
+        channelControllerMock.hasLoadedAllNextMessages_mock = true
+        channelControllerMock.markedAsUnread_mock = false
+
+        // Simulate display to update hasSeenLastMessage && hasSeenFirstUnreadMessage
+        vc.chatMessageListVC(ChatMessageListVC_Mock(), willDisplayMessageAt: IndexPath(item: 0, section: 0))
+        vc.chatMessageListVC(ChatMessageListVC_Mock(), scrollViewDidScroll: UIScrollView())
+
+        XCTAssertFalse(vc.shouldMarkChannelRead)
+    }
 
     func test_shouldMarkChannelRead_jumpToUnreadEnabled_whenNotSeenLastMessage_whenNotSeenFirstUnreadMessage_shouldReturnFalse() {
         let mockedListView = makeMockMessageListView()