@stream-io/video-client 0.4.8 → 0.4.10

package/dist/src/StreamVideoServerClient.d.ts

@@ -1,6 +1,11 @@
 import { StreamVideoClient } from './StreamVideoClient';
 import { StreamClientOptions } from './coordinator/connection/types';
 import { CreateCallTypeRequest, CreateCallTypeResponse, GetCallTypeResponse, ListCallTypeResponse, UpdateCallTypeRequest, UpdateCallTypeResponse } from './gen/coordinator';
+/**
+ * @deprecated Please use the `@stream-io/node-sdk` package instead.
+ *
+ * @see https://getstream.io/video/docs/api/
+ */
 export declare class StreamVideoServerClient extends StreamVideoClient {
     constructor(apiKey: string, options: StreamClientOptions);
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stream-io/video-client",
-  "version": "0.4.8",
+  "version": "0.4.10",
   "packageManager": "yarn@3.2.4",
   "main": "dist/index.cjs.js",
   "module": "dist/index.es.js",

package/src/StreamVideoServerClient.ts

@@ -9,6 +9,11 @@ import {
   UpdateCallTypeResponse,
 } from './gen/coordinator';
 
+/**
+ * @deprecated Please use the `@stream-io/node-sdk` package instead.
+ *
+ * @see https://getstream.io/video/docs/api/
+ */
 export class StreamVideoServerClient extends StreamVideoClient {
   constructor(apiKey: string, options: StreamClientOptions) {
     super({ apiKey, options });

package/src/__tests__/server-side/call-types.test.ts

@@ -11,7 +11,7 @@ import {
 const apiKey = process.env.STREAM_API_KEY!;
 const secret = process.env.STREAM_SECRET!;
 
-describe('call types CRUD API', () => {
+describe.skip('call types CRUD API', () => {
   let client: StreamVideoServerClient;
   const callTypeName = `calltype${generateUUIDv4()}`;
 

package/src/store/CallState.ts

@@ -1047,13 +1047,33 @@ export class CallState {
         return session;
       }
       const { participants, participants_count_by_role } = session;
-      const { user } = event.participant;
+      const { user, user_session_id } = event.participant;
+      // It could happen that the backend delivers the same participant more than once.
+      // Once with the call.session_started event and once again with the
+      // call.session_participant_joined event. In this case,
+      // we should update the existing participant and prevent duplicating it.
+      let shouldInsertParticipant = true;
+      const updatedParticipants = participants.map((p) => {
+        if (p.user_session_id === user_session_id) {
+          shouldInsertParticipant = false;
+          return event.participant;
+        }
+        return p;
+      });
+      if (shouldInsertParticipant) {
+        // this is a new array, we can safely push the new participant
+        updatedParticipants.push(event.participant);
+      }
+
+      // If we are updating an existing participant, we don't want to increment
+      // the participant_by_role count.
+      const increment = shouldInsertParticipant ? 1 : 0;
       return {
         ...session,
-        participants: [...participants, event.participant],
+        participants: updatedParticipants,
         participants_count_by_role: {
           ...participants_count_by_role,
-          [user.role]: (participants_count_by_role[user.role] || 0) + 1,
+          [user.role]: (participants_count_by_role[user.role] || 0) + increment,
         },
       };
     });

package/src/store/__tests__/CallState.test.ts

@@ -613,6 +613,56 @@ describe('CallState', () => {
           },
         });
       });
+
+      it('should update existing participant', () => {
+        const state = new CallState();
+        state.updateFromCallResponse({
+          // @ts-ignore
+          session: {
+            participants: [
+              {
+                // @ts-ignore
+                user: {
+                  id: 'user-id',
+                  role: 'user',
+                },
+                user_session_id: '123',
+              },
+            ],
+            participants_count_by_role: {
+              user: 1,
+            },
+          },
+        });
+        state.updateFromEvent({
+          type: 'call.session_participant_joined',
+          participant: {
+            // @ts-ignore
+            user: {
+              id: 'user-id',
+              role: 'user',
+              name: 'Updated user',
+            },
+            user_session_id: '123',
+          },
+        });
+        expect(state.session).toEqual({
+          participants: [
+            {
+              // @ts-ignore
+              user: {
+                id: 'user-id',
+                role: 'user',
+                name: 'Updated user',
+              },
+              user_session_id: '123',
+            },
+          ],
+          participants_count_by_role: {
+            user: 1,
+          },
+        });
+      });
     });
   });
 });