@reactoo/watchtogether-sdk-js 2.6.60 → 2.6.62

package/example/index.html

@@ -149,12 +149,12 @@
             Instance.room.getSessionByConstructId(constructId).toggleAudio()
         }
 
-        function setListenIntercomGroups(groups) {
-            Instance.room.getSessionByConstructId(constructId).setListenIntercomGroups(groups)
+        function setListenIntercomChannels(groups) {
+            Instance.room.getSessionByConstructId(constructId).setListenIntercomChannels(groups)
         }
 
-        function setTalkIntercomGroups(groups) {
-            Instance.room.getSessionByConstructId(constructId).setTalkIntercomGroups(groups)
+        function setTalkIntercomChannels(groups) {
+            Instance.room.getSessionByConstructId(constructId).setTalkIntercomChannels(groups)
         }
 
         let Instance = WatchTogetherSDK({debug:true, apiUrl: 'https://api.reactoo.com/stefan/swagger.json'})({instanceType:'reactooDemo'});
@@ -212,6 +212,9 @@
             }})) // pin hash is not needed if you're owner of the room
             .then(session => {
 
+                session.setTalkIntercomChannels([]);
+                session.setListenIntercomChannels([]);
+
                 session.$on('connecting', (status) => {
                     console.log('connecting', status);
                 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reactoo/watchtogether-sdk-js",
-  "version": "2.6.60",
+  "version": "2.6.62",
   "description": "Javascript SDK for Reactoo",
   "main": "src/index.js",
   "unpkg": "dist/watchtogether-sdk.min.js",

package/src/models/room-session.js

@@ -336,16 +336,16 @@ let roomSession = function ({roomId, pinHash, role}, room, wt) {
 			return room.toggleVideo(value, source);
 		},
 
-		setTalkIntercomGroups: (groups) => {
-			return room.setTalkIntercomGroups(groups);
+		setTalkIntercomChannels: (groups) => {
+			return room.setTalkIntercomChannels(groups);
 		},
 
 		getTalkIntercomGroups: () => {
 			return room._talkIntercomGroups;
 		},
 
-		setListenIntercomGroups: (groups) => {
-			return room.setListenIntercomGroups(groups);
+		setListenIntercomChannels: (groups) => {
+			return room.setListenIntercomChannels(groups);
 		},
 
 		getListenIntercomGroups: () => {

package/src/modules/wt-room.js

@@ -108,12 +108,12 @@ class RoomSession {
 			"videowall-queue-video": ['host', 'participant'],
 		},
 		talkback: {
-			"watchparty": ['participant', 'host'],
-			"studio": ['participant', 'host', 'observer'],
-			"commentary": ['host', 'participant'],
-			"videowall": ['host', 'participant'],
-			"videowall-queue": ['host', 'participant'],
-			"videowall-queue-video": ['host', 'participant'],
+			"watchparty": ['participant', 'host', 'talkback'],
+			"studio": ['participant', 'host', 'observer', 'talkback'],
+			"commentary": ['host', 'participant', 'talkback'],
+			"videowall": ['host', 'participant', 'talkback'],
+			"videowall-queue": ['host', 'participant', 'talkback'],
+			"videowall-queue-video": ['host', 'participant', 'talkback'],
 		},
 		observer: {
 			"watchparty": ['participant'],
@@ -202,8 +202,8 @@ class RoomSession {
 		this._maxRetries = 5;
 		this._keepAliveId = null;
 		this._participants = [];
-		this._talkIntercomGroups = ['participants'];
-		this._listenIntercomGroups = ['participants'];
+		this._talkIntercomChannels = ['participants'];
+		this._listenIntercomChannels = ['participants'];
 		this._roomType = 'watchparty';
 		this._isDataChannelOpen = false;
 		this._abortController = null;
@@ -232,7 +232,7 @@ class RoomSession {
 			return true;
 		}
 		const intercomGroups = JSON.parse(stream.description || "[]")?.intercomGroups || [];
-		return intercomGroups.some(g => this._listenIntercomGroups.indexOf(g) > -1);
+		return intercomGroups.some(g => this._listenIntercomChannels.indexOf(g) > -1);
 	};
 
 	_getAddParticipantEventName(handleId) {
@@ -746,12 +746,17 @@ class RoomSession {
 
 			if (event === "updated") {
 				this._log('Remote has updated tracks', msg);
-				this._updateCurrentSubscriptions(handle.handleId, msg["streams"] || []);
+				if(msg["streams"]) {
+					this._updateCurrentSubscriptions(handle.handleId, msg["streams"]);
+				}
+
 			}
 
 			if (event === "attached") {
 				this._log('Remote have successfully joined Room', msg);
-				this._updateCurrentSubscriptions(handle.handleId, msg["streams"] || []);
+				if(msg["streams"]) {
+					this._updateCurrentSubscriptions(handle.handleId, msg["streams"] || []);
+				}
 				this.emit(this._getAddParticipantEventName(handle.handleId), {
 					tid: generateUUID(),
 					id: handle.handleId,
@@ -855,18 +860,20 @@ class RoomSession {
 				try {
 					if (handle.webrtcStuff.stream) {
 						if(!this.isRestarting) {
-							handle.webrtcStuff.stream.getTracks().forEach(track => track.stop());
+							handle.webrtcStuff.stream?.getTracks().forEach(track => track.stop());
 						}
 						else {
-							handle.webrtcStuff.stream.getTracks().forEach(track => track.onended = null);
+							handle.webrtcStuff.stream?.getTracks().forEach(track => track.onended = null);
 						}
 					}
 				} catch (e) {
 					// Do nothing
 				}
 
-				handle.webrtcStuff.stream.onremovetrack = null;
-				handle.webrtcStuff.stream = null;
+				if(handle.webrtcStuff.stream) {
+					handle.webrtcStuff.stream.onremovetrack = null;
+					handle.webrtcStuff.stream = null;
+				}
 
 				if (handle.webrtcStuff.dataChannel) {
 					Object.keys(handle.webrtcStuff.dataChannel).forEach(label => {
@@ -1030,18 +1037,18 @@ class RoomSession {
 		}
 		let config = handle.webrtcStuff;
 		config.streamMap = {};
-		config.tracksMap.forEach(tItem => {
+		config.currentSubscriptions.forEach(tItem => {
 			if(tItem.type === 'data') {
 				return;
 			}
 			if(tItem.active === false) {
 				return;
 			}
-			const source = JSON.parse(tItem.description)?.source;
+			const source = JSON.parse(tItem.feed_description)?.source;
 			if(!config.streamMap[source]) {
 				config.streamMap[source] = [];
 			}
-			let trackId = config.pc.getTransceivers().find(t => t.mid === tItem.mid).receiver.track.id;
+			let trackId = config.pc.getTransceivers().find(t => t.mid === tItem.mid)?.receiver?.track?.id;
 			if(trackId) {
 				config.streamMap[source].push(trackId);
 			}
@@ -1537,13 +1544,13 @@ class RoomSession {
 							if(!this._getHandle(handle.handleId)) {
 								return;
 							}
+							this._updateRemoteParticipantStreamMap(handle.handleId);
 
 							let transceiver = config.pc?.getTransceivers()?.find(
 								t => t.receiver.track === ev.track);
 
 							let mid = transceiver?.mid || ev.track.id;
 							let source = Object.keys(config.streamMap).find(key => config.streamMap[key].includes(ev.track.id));
-							this._updateRemoteParticipantStreamMap(handle.handleId);
 							this.emit(this._getAddParticipantEventName(handle.handleId), {
 								tid: generateUUID(),
 								id: handle.handleId,
@@ -1565,9 +1572,9 @@ class RoomSession {
 
 					if (event.track) {
 						config.stream?.addTrack(event.track);
+						this._updateRemoteParticipantStreamMap(handle.handleId);
 						let mid = event.transceiver ? event.transceiver.mid : event.track.id;
 						let source = Object.keys(config.streamMap).find(key => config.streamMap[key].includes(event.track.id));
-						this._updateRemoteParticipantStreamMap(handle.handleId);
 						this.emit(this._getAddParticipantEventName(handle.handleId), {
 							tid: generateUUID(),
 							mid,
@@ -1595,11 +1602,11 @@ class RoomSession {
 							if(!this._getHandle(handle.handleId)) {
 								return;
 							}
+							this._updateRemoteParticipantStreamMap(handle.handleId);
 							let transceiver = config.pc?.getTransceivers()?.find(
 								t => t.receiver.track === ev.target);
 							let mid = transceiver?.mid || ev.target.id;
 							let source = Object.keys(config.streamMap).find(key => config.streamMap[key].includes(ev.target.id));
-							this._updateRemoteParticipantStreamMap(handle.handleId);
 							this.emit(this._getAddParticipantEventName(handle.handleId), {
 								tid: generateUUID(),
 								id: handle.handleId,
@@ -1802,7 +1809,6 @@ class RoomSession {
 	}
 
 	_setupTransceivers(handleId, [audioSend, audioRecv, videoSend, videoRecv, audioTransceiver = null, videoTransceiver = null]) {
-
 		//TODO: this should be refactored to use handle's trackMap so we dont have to pass any parameters
 
 		let handle = this._getHandle(handleId);
@@ -1811,7 +1817,7 @@ class RoomSession {
 		}
 		let config = handle.webrtcStuff;
 
-		const setTransceiver = (transceiver, send, recv) => {
+		const setTransceiver = (transceiver, send, recv, kind = 'audio') => {
 			if (!send && !recv) {
 				// disabled: have we removed it?
 				if (transceiver) {
@@ -1847,7 +1853,7 @@ class RoomSession {
 						}
 					} else {
 						// In theory, this is the only case where we might not have a transceiver yet
-						config.pc.addTransceiver("audio", {direction: "recvonly"});
+						config.pc.addTransceiver(kind, {direction: "recvonly"});
 					}
 				}
 			}
@@ -1856,10 +1862,10 @@ class RoomSession {
 		// if we're passing any transceivers, we work only on them, doesn't matter if one of them is null
 		if(audioTransceiver || videoTransceiver) {
 			if(audioTransceiver) {
-				setTransceiver(audioTransceiver, audioSend, audioRecv);
+				setTransceiver(audioTransceiver, audioSend, audioRecv, 'audio');
 			}
 			if(videoTransceiver) {
-				setTransceiver(videoTransceiver, videoSend, videoRecv);
+				setTransceiver(videoTransceiver, videoSend, videoRecv, 'video');
 			}
 		}
 		// else we work on all transceivers
@@ -1872,13 +1878,13 @@ class RoomSession {
 						(t.sender && t.sender.track && t.sender.track.kind === "audio") ||
 						(t.receiver && t.receiver.track && t.receiver.track.kind === "audio")
 					) {
-						setTransceiver(t, audioSend, audioRecv);
+						setTransceiver(t, audioSend, audioRecv, 'audio');
 					}
 					if (
 						(t.sender && t.sender.track && t.sender.track.kind === "video") ||
 						(t.receiver && t.receiver.track && t.receiver.track.kind === "video")
 					) {
-						setTransceiver(t, videoSend, videoRecv);
+						setTransceiver(t, videoSend, videoRecv, 'video');
 					}
 				}
 			}
@@ -2238,6 +2244,7 @@ class RoomSession {
 
 		let hasAudio = !!(stream && stream.getAudioTracks().length > 0);
 		let hasVideo = !!(stream && stream.getVideoTracks().length > 0);
+
 		this._setupTransceivers(this.handleId, [hasAudio, false, hasVideo, false, audioTransceiver, videoTransceiver]);
 
 		const emitEvents = () => {
@@ -2341,7 +2348,7 @@ class RoomSession {
 					config.streamMap[source].forEach(trackId => {
 						let t = transceivers.find(transceiver => transceiver.sender.track && transceiver.sender.track.id === trackId)
 						if(t) {
-							descriptions.push({mid: t.mid, description: JSON.stringify({source, intercomGroups: this._talkIntercomGroups})});
+							descriptions.push({mid: t.mid, description: JSON.stringify({source, intercomGroups: this._talkIntercomChannels})});
 						}
 					})
 				});
@@ -2496,14 +2503,14 @@ class RoomSession {
 	}
 
 
-	setTalkIntercomGroups(groups = ['participants']) {
+	setTalkIntercomChannels(groups = ['participants']) {
 
 		if(typeof groups !== 'object' || !("length" in groups)) {
-			this._log('setTalkIntercomGroups: groups must be an array');
+			this._log('setTalkIntercomChannels: groups must be an array');
 			groups = [groups];
 		}
 
-		this._talkIntercomGroups = structuredClone(groups);
+		this._talkIntercomChannels = structuredClone(groups);
 		let handle = this._getHandle(this.handleId);
 		if (!handle) {
 			return Promise.resolve();
@@ -2527,14 +2534,14 @@ class RoomSession {
 		})
 	}
 
-	setListenIntercomGroups(groups = ['participants']) {
+	setListenIntercomChannels(groups = ['participants']) {
 
 		if(typeof groups !== 'object' || !("length" in groups)) {
-			this._log('setListenIntercomGroups: groups must be an array');
+			this._log('setListenIntercomChannels: groups must be an array');
 			groups = [groups];
 		}
 
-		this._listenIntercomGroups = structuredClone(groups);
+		this._listenIntercomChannels = structuredClone(groups);
 		let handle = this._getHandle(this.handleId);
 		if (!handle) {
 			return Promise.resolve();
@@ -2564,7 +2571,7 @@ class RoomSession {
 					const description = JSON.parse(track.description);
 					const intercomGroups = description?.intercomGroups || [];
 
-					if(this._listenIntercomGroups.some(g => intercomGroups.includes(g))) {
+					if(this._listenIntercomChannels.some(g => intercomGroups.includes(g))) {
 						subscribe.push({feed: track.id, mid: track.mid})
 					}
 					else {