Address PR comments

Make new code in `proxyStreamingClientEvents` safer and use existing helper functions.

Author: patrickfeeney03

react-demo-app/package-lock.json

@@ -4,6 +4,7 @@ import { GenesysCloudWebrtcSdk } from './client';
 import { SessionManager } from './sessions/session-manager';
 import { SubscriptionEvent } from './types/interfaces';
 import { ConversationUpdate } from './conversations/conversation-update';
+import { isAgentVideoJid } from "./utils";
 
 /**
  * Establish the connection with the streaming client.
@@ -93,11 +94,11 @@ export async function proxyStreamingClientEvents (this: GenesysCloudWebrtcSdk):
 
   if (this._personDetails) {
     if (this.isJwtAuth) {
-      const id = this._customerData.conversation?.id;
-      const userId = this._personDetails.id;
+      const convId = this._customerData.conversation?.id;
+      const isGuest = !this._personDetails?.id;
       const roomJid = this._personDetails.chat?.jabberId;
-      if (id && !userId && roomJid?.startsWith('agent-')) {
-        this._streamingConnection.on(`notify:v2.guest.conversations.${id}`, (conversationEvent) => {
+      if (convId && isGuest && isAgentVideoJid(roomJid)) {
+        this._streamingConnection.on(`notify:v2.guest.conversations.${convId}`, (conversationEvent) => {
           handleConversationUpdate.call(this, conversationEvent);
         });
       }

src/client-private.ts

@@ -266,7 +266,7 @@ describe('proxyStreamingClientEvents', () => {
     expect(mockSdk._customerData).toBe(undefined);
   });
 
-  async function createEventDataAndCallProxyFunction(options = {} as any) {
+  async function createEventDataAndCallProxyFunction(options = {} as { userId?: string, jabberId?: string, conversationId?: string, isJwtAuth?: boolean }) {
     const userId = 'userId' in options ? options.userId : undefined;
     const jabberId = 'jabberId' in options ? options.jabberId : 'agent-123@conference.com';
     const conversationId = 'conversationId' in options ? options.conversationId : 'conv123';