feat: require Channel instance as the first argument to doSendMessageRequest (#2171)

Avoid forcing integrators to recreate `Channel` instance from CID inside
`doSendMessageRequest` and provide it directly as the first argument.

BREAKING CHANGE: the first argument to `doSendMessageRequest` is now Channel instance instead of `Channel.cid`

Author: MartinCupela

docusaurus/docs/React/release-guides/v11/doSendMessageRequest-signature.mdx

@@ -0,0 +1,20 @@
+---
+id: minor-breaking-changes-v11
+sidebar_position: 4
+title: Minor breaking changes 11.0.0
+keywords: [migration guide, upgrade, breaking changes, v11]
+---
+
+### Channel instance as a first argument to doSendMessageRequest
+
+The `doSendMessageRequest` will from now on be passed the `Channel` instance instead of its CID to avoid forcing the developers to recreate a reference to the `Channel` instance inside the `doSendMessageRequest` function. The developers should adjust their implementation of `doSendMessageRequest` to call directly `await channel.sendMessage(messageData, options)`:
+
+```ts
+import { ChannelProps } from 'stream-chat-react';
+
+const doSendMessageRequest: ChannelProps['doSendMessageRequest'] = async (channel, messageData, options) => {
+    // optional custom logic
+   await channel.sendMessage(messageData, options);
+   // optional custom logic
+}
+```

src/components/Channel/Channel.tsx

@@ -218,7 +218,7 @@ export type ChannelProps<
     ) => Promise<MessageResponse<StreamChatGenerics>> | void;
     /** Custom action handler to override the default `channel.sendMessage` request function (advanced usage only) */
     doSendMessageRequest?: (
-      channelId: string,
+      channel: StreamChannel<StreamChatGenerics>,
       message: Message<StreamChatGenerics>,
       options?: SendMessageOptions,
     ) => ReturnType<StreamChannel<StreamChatGenerics>['sendMessage']> | void;
@@ -771,7 +771,7 @@ const ChannelInner = <
       let messageResponse: void | SendMessageAPIResponse<StreamChatGenerics>;
 
       if (doSendMessageRequest) {
-        messageResponse = await doSendMessageRequest(channel.cid, messageData, options);
+        messageResponse = await doSendMessageRequest(channel, messageData, options);
       } else {
         messageResponse = await channel.sendMessage(messageData, options);
       }

src/components/Channel/__tests__/Channel.test.js

@@ -994,7 +994,7 @@ describe('Channel', () => {
 
         await waitFor(() =>
           expect(doSendMessageRequest).toHaveBeenCalledWith(
-            channel.cid,
+            channel,
             expect.objectContaining(message),
             undefined,
           ),

src/components/MessageInput/__tests__/MessageInput.test.js

@@ -646,7 +646,7 @@ function axeNoViolations(container) {
         await act(() => submit());
 
         expect(submitMock).toHaveBeenCalledWith(
-          channel.cid,
+          channel,
           expect.objectContaining({
             text: messageText,
           }),
@@ -697,9 +697,10 @@ function axeNoViolations(container) {
         await act(() => submit());
 
         await waitFor(() => {
-          const calledMock = componentName === 'EditMessageForm' ? editMock : submitMock;
+          const isEdit = componentName === 'EditMessageForm';
+          const calledMock = isEdit ? editMock : submitMock;
           expect(calledMock).toHaveBeenCalledWith(
-            expect.stringMatching(/.+:.+/),
+            isEdit ? channel.cid : channel,
             expect.objectContaining(customMessageData),
             undefined,
           );
@@ -765,7 +766,7 @@ function axeNoViolations(container) {
         await act(() => submit());
 
         expect(submitMock).toHaveBeenCalledWith(
-          channel.cid,
+          channel,
           expect.objectContaining({
             attachments: expect.arrayContaining([
               expect.objectContaining({
@@ -799,7 +800,7 @@ function axeNoViolations(container) {
         await act(() => submit());
 
         expect(submitMock).toHaveBeenCalledWith(
-          channel.cid,
+          channel,
           expect.objectContaining({
             attachments: expect.arrayContaining([
               expect.objectContaining({
@@ -835,7 +836,7 @@ function axeNoViolations(container) {
         await act(() => submit());
 
         expect(submitMock).toHaveBeenCalledWith(
-          channel.cid,
+          channel,
           expect.objectContaining({
             attachments: expect.arrayContaining([
               expect.objectContaining({
@@ -1040,7 +1041,7 @@ function axeNoViolations(container) {
       await act(() => submit());
 
       expect(submitMock).toHaveBeenCalledWith(
-        channel.cid,
+        channel,
         expect.objectContaining({
           mentioned_users: expect.arrayContaining([mentionId]),
         }),