Self review changes

Author: nuno-vieira

Sources/StreamChat/APIClient/ChatRemoteNotificationHandler.swift

@@ -160,9 +160,9 @@ public class ChatRemoteNotificationHandler {
         guard let currentUser = client.currentUserController().currentUser else {
             return log.debug("No current user to mark messages as delivered")
         }
-        /// Make sure if the message was already delivered, do not mark it as delivered.
-        /// If the app is active, the middleware will mark it as delivered so the push
-        /// does not need to do it.
+        // Make sure if the message was already delivered, do not mark it as delivered.
+        // If the app is active, the middleware will mark it as delivered so the push
+        // does not need to do it.
         guard deliveryCriteriaValidator.canMarkMessageAsDelivered(message, for: currentUser, in: channel) else {
             log.debug("No message to be marked as delivered for messageId:\(message.id))")
             return

Sources/StreamChat/Models/ChannelRead.swift

@@ -9,7 +9,7 @@ public struct ChatChannelRead: Equatable {
     /// The last time the user has read the channel.
     public let lastReadAt: Date
 
-    /// Id for the last message the user has read. Nil means the user has never read this channel
+    /// Id for the last message the user has read. Nil means the user has never read this channel.
     public let lastReadMessageId: MessageId?
 
     /// Number of unread messages the user has in this channel.
@@ -18,10 +18,10 @@ public struct ChatChannelRead: Equatable {
     /// The user who read the channel.
     public let user: ChatUser
 
-    /// The last time the user has delivered the channel.
+    /// The last time a message has been delivered to this user.
     public let lastDeliveredAt: Date?
 
-    /// Id for the last message the user has delivered. Nil means the user has never delivered this channel
+    /// The last message ID that has been delivered to this user.
     public let lastDeliveredMessageId: MessageId?
 
     init(

Sources/StreamChat/Models/MessageDelivery/MessageDeliveryCriteriaValidator.swift

@@ -23,7 +23,7 @@ protocol MessageDeliveryCriteriaValidating {
 
 /// Default implementation of message delivery criteria validation.
 struct MessageDeliveryCriteriaValidator: MessageDeliveryCriteriaValidating {
-    public init() {}
+    init() {}
 
     /// Determines if a message can be marked as delivered for a specific user in a channel.
     ///
@@ -43,7 +43,7 @@ struct MessageDeliveryCriteriaValidator: MessageDeliveryCriteriaValidating {
     ///   - currentUser: The current user who would mark the message as delivered.
     ///   - channel: The channel containing the message.
     /// - Returns: `true` if the message can be marked as delivered, `false` otherwise.
-    public func canMarkMessageAsDelivered(
+    func canMarkMessageAsDelivered(
         _ message: ChatMessage,
         for currentUser: CurrentChatUser,
         in channel: ChatChannel