Fix send sync message bugs

ts/background.ts

@@ -3212,7 +3212,7 @@ export async function startApp(): Promise<void> {
         return {
           ...result,
           [conversationId]: {
-            status: SendStatus.Pending,
+            status: SendStatus.Sent,
             updatedAt: timestamp,
           },
         };

ts/messageModifiers/MessageReceipts.ts

@@ -13,7 +13,11 @@ import { isOutgoing } from '../state/selectors/message';
 import { isDirectConversation } from '../util/whatTypeOfConversation';
 import { getOwn } from '../util/getOwn';
 import { missingCaseError } from '../util/missingCaseError';
-import { SendActionType, sendStateReducer } from '../messages/MessageSendState';
+import {
+  SendActionType,
+  SendStatus,
+  sendStateReducer,
+} from '../messages/MessageSendState';
 import dataInterface from '../sql/Client';
 
 const { deleteSentProtoRecipient } = dataInterface;
@@ -107,7 +111,7 @@ export class MessageReceipts extends Collection<MessageReceiptModel> {
         ids.includes(receipt.get('sourceConversationId'))
     );
     if (receipts.length) {
-      window.log.info('Found early read receipts for message');
+      window.log.info('Found early receipts for message');
       this.remove(receipts);
     }
     return receipts;
@@ -142,8 +146,8 @@ export class MessageReceipts extends Collection<MessageReceiptModel> {
       const oldSendState = getOwn(
         oldSendStateByConversationId,
         sourceConversationId
-      );
+      ) ?? { status: SendStatus.Sent, updatedAt: undefined };
-      if (oldSendState) {
+
       let sendActionType: SendActionType;
       switch (type) {
         case MessageReceiptType.Delivery:
@@ -185,15 +189,6 @@ export class MessageReceipts extends Collection<MessageReceiptModel> {
           updateLeftPane();
         }
       }
-      } else {
-        window.log.warn(
-          `Got a receipt from someone (${sourceConversationId}), but the message (sent at ${message.get(
-            'sent_at'
-          )}) wasn't sent to them. It was sent to ${
-            Object.keys(oldSendStateByConversationId).length
-          } recipients`
-        );
-      }
 
       if (
         (type === MessageReceiptType.Delivery &&

ts/models/messages.ts

@@ -2579,20 +2579,25 @@ export class MessageModel extends window.Backbone.Model<MessageAttributesType> {
                 return;
               }
 
+              const updatedAt: number = isNormalNumber(data.timestamp)
+                ? data.timestamp
+                : Date.now();
+
               const previousSendState = getOwn(
                 sendStateByConversationId,
                 destinationConversationId
               );
-              if (previousSendState) {
               sendStateByConversationId[
                 destinationConversationId
-                ] = sendStateReducer(previousSendState, {
+              ] = previousSendState
+                ? sendStateReducer(previousSendState, {
                     type: SendActionType.Sent,
-                  updatedAt: isNormalNumber(data.timestamp)
+                    updatedAt,
-                    ? data.timestamp
+                  })
-                    : Date.now(),
+                : {
-                });
+                    status: SendStatus.Sent,
-              }
+                    updatedAt,
+                  };
 
               if (unidentified) {
                 unidentifiedDeliveriesSet.add(identifier);