fix: allow the composition of unchanged edited message (#1578)

Author: MartinCupela

src/messageComposer/messageComposer.ts

@@ -306,7 +306,7 @@ export class MessageComposer extends WithSubscriptions {
     const initiatedWithoutDraft = this.lastChange.draftUpdate === null;
     const composingMessageFromScratch = initiatedWithoutDraft && !this.editedMessage;
 
-    // does not mean that the original editted message is different from the current state
+    // does not mean that the original edited message is different from the current state
     const editedMessageWasUpdated =
       !!this.editedMessage?.updated_at &&
       new Date(this.editedMessage.updated_at).getTime() < this.lastChange.stateUpdate;

src/messageComposer/middleware/messageComposer/compositionValidation.ts

@@ -28,10 +28,7 @@ export const createCompositionValidationMiddleware = (
       const hasExceededMaxLength =
         typeof maxLengthOnSend === 'number' && inputText.length > maxLengthOnSend;
 
-      const editedMessageIsUnchanged =
-        composer.editedMessage && !composer.lastChangeOriginIsLocal;
-
-      if (isEmptyMessage || editedMessageIsUnchanged || hasExceededMaxLength) {
+      if (isEmptyMessage || hasExceededMaxLength) {
         return await discard();
       }
 

test/unit/MessageComposer/middleware/messageComposer/compositionValidation.test.ts

@@ -322,7 +322,7 @@ describe('stream-io/message-composer-middleware/data-validation', () => {
     expect(result.status).toBeUndefined;
   });
 
-  it('should discard composition for edited message without any local change', async () => {
+  it('should not discard composition for edited message without any local change', async () => {
     const { messageComposer, validationMiddleware } = setupMiddleware();
     const localMessage: LocalMessage = {
       attachments: [],
@@ -355,7 +355,7 @@ describe('stream-io/message-composer-middleware/data-validation', () => {
       }),
     );
 
-    expect(result.status).toBe('discard');
+    expect(result.status).toBeUndefined();
   });
 
   it('should not discard composition for newly composed message initiated with draft', async () => {