livekit-client 1.15.10 → 2.0.0

package/src/room/defaults.ts

@@ -10,10 +10,6 @@ import { AudioPresets, ScreenSharePresets, VideoPresets } from './track/options'
 export const defaultVideoCodec = 'vp8';
 
 export const publishDefaults: TrackPublishDefaults = {
-  /**
-   * @deprecated
-   */
-  audioBitrate: AudioPresets.music.maxBitrate,
   audioPreset: AudioPresets.music,
   dtx: true,
   red: true,
@@ -41,7 +37,7 @@ export const roomOptionDefaults: InternalRoomOptions = {
   stopLocalTrackOnUnpublish: true,
   reconnectPolicy: new DefaultReconnectPolicy(),
   disconnectOnPageLeave: true,
-  expWebAudioMix: false,
+  webAudioMix: true,
 } as const;
 
 export const roomConnectOptionDefaults: InternalRoomConnectOptions = {

package/src/room/events.ts

@@ -38,11 +38,6 @@ export enum RoomEvent {
    */
   ConnectionStateChanged = 'connectionStateChanged',
 
-  /**
-   * @deprecated StateChanged has been renamed to ConnectionStateChanged
-   */
-  StateChanged = 'connectionStateChanged',
-
   /**
    * When input or output devices on the machine have changed.
    */

package/src/room/participant/LocalParticipant.ts

@@ -53,7 +53,6 @@ import {
 import Participant from './Participant';
 import type { ParticipantTrackPermission } from './ParticipantTrackPermission';
 import { trackPermissionToProto } from './ParticipantTrackPermission';
-import RemoteParticipant from './RemoteParticipant';
 import {
   computeTrackBackupEncodings,
   computeVideoEncodings,
@@ -61,12 +60,12 @@ import {
 } from './publishUtils';
 
 export default class LocalParticipant extends Participant {
-  audioTracks: Map<string, LocalTrackPublication>;
+  audioTrackPublications: Map<string, LocalTrackPublication>;
 
-  videoTracks: Map<string, LocalTrackPublication>;
+  videoTrackPublications: Map<string, LocalTrackPublication>;
 
   /** map of track sid => all published tracks */
-  tracks: Map<string, LocalTrackPublication>;
+  trackPublications: Map<string, LocalTrackPublication>;
 
   /** @internal */
   engine: RTCEngine;
@@ -99,9 +98,9 @@ export default class LocalParticipant extends Participant {
       loggerName: options.loggerName,
       loggerContextCb: () => this.engine.logContext,
     });
-    this.audioTracks = new Map();
-    this.videoTracks = new Map();
-    this.tracks = new Map();
+    this.audioTrackPublications = new Map();
+    this.videoTrackPublications = new Map();
+    this.trackPublications = new Map();
     this.engine = engine;
     this.roomOptions = options;
     this.setupEngine(engine);
@@ -120,15 +119,15 @@ export default class LocalParticipant extends Participant {
     return this.encryptionType !== Encryption_Type.NONE;
   }
 
-  getTrack(source: Track.Source): LocalTrackPublication | undefined {
-    const track = super.getTrack(source);
+  getTrackPublication(source: Track.Source): LocalTrackPublication | undefined {
+    const track = super.getTrackPublication(source);
     if (track) {
       return track as LocalTrackPublication;
     }
   }
 
-  getTrackByName(name: string): LocalTrackPublication | undefined {
-    const track = super.getTrackByName(name);
+  getTrackPublicationByName(name: string): LocalTrackPublication | undefined {
+    const track = super.getTrackPublicationByName(name);
     if (track) {
       return track as LocalTrackPublication;
     }
@@ -140,7 +139,7 @@ export default class LocalParticipant extends Participant {
   setupEngine(engine: RTCEngine) {
     this.engine = engine;
     this.engine.on(EngineEvent.RemoteMute, (trackSid: string, muted: boolean) => {
-      const pub = this.tracks.get(trackSid);
+      const pub = this.trackPublications.get(trackSid);
       if (!pub || !pub.track) {
         return;
       }
@@ -290,7 +289,7 @@ export default class LocalParticipant extends Participant {
     publishOptions?: TrackPublishOptions,
   ) {
     this.log.debug('setTrackEnabled', { ...this.logContext, source, enabled });
-    let track = this.getTrack(source);
+    let track = this.getTrackPublication(source);
     if (enabled) {
       if (track) {
         await track.unmute();
@@ -351,7 +350,7 @@ export default class LocalParticipant extends Participant {
       // screenshare cannot be muted, unpublish instead
       if (source === Track.Source.ScreenShare) {
         track = await this.unpublishTrack(track.track);
-        const screenAudioTrack = this.getTrack(Track.Source.ScreenShareAudio);
+        const screenAudioTrack = this.getTrackPublication(Track.Source.ScreenShareAudio);
         if (screenAudioTrack && screenAudioTrack.track) {
           this.unpublishTrack(screenAudioTrack.track);
         }
@@ -573,7 +572,7 @@ export default class LocalParticipant extends Participant {
 
     // is it already published? if so skip
     let existingPublication: LocalTrackPublication | undefined;
-    this.tracks.forEach((publication) => {
+    this.trackPublications.forEach((publication) => {
       if (!publication.track) {
         return;
       }
@@ -651,7 +650,7 @@ export default class LocalParticipant extends Participant {
   }
 
   private async publish(track: LocalTrack, opts: TrackPublishOptions, isStereo: boolean) {
-    const existingTrackOfSource = Array.from(this.tracks.values()).find(
+    const existingTrackOfSource = Array.from(this.trackPublications.values()).find(
       (publishedTrack) => track instanceof LocalTrack && publishedTrack.source === track.source,
     );
     if (existingTrackOfSource && track.source !== Track.Source.Unknown) {
@@ -788,7 +787,7 @@ export default class LocalParticipant extends Participant {
     } else if (track.kind === Track.Kind.Audio) {
       encodings = [
         {
-          maxBitrate: opts.audioPreset?.maxBitrate ?? opts.audioBitrate,
+          maxBitrate: opts.audioPreset?.maxBitrate,
           priority: opts.audioPreset?.priority ?? 'high',
           networkPriority: opts.audioPreset?.priority ?? 'high',
         },
@@ -914,7 +913,7 @@ export default class LocalParticipant extends Participant {
 
     // is it not published? if so skip
     let existingPublication: LocalTrackPublication | undefined;
-    this.tracks.forEach((publication) => {
+    this.trackPublications.forEach((publication) => {
       if (!publication.track) {
         return;
       }
@@ -947,6 +946,9 @@ export default class LocalParticipant extends Participant {
       return;
     }
     const simulcastTrack = track.addSimulcastTrack(videoCodec, encodings);
+    if (!simulcastTrack) {
+      return;
+    }
     const req = new AddTrackRequest({
       cid: simulcastTrack.mediaStreamTrack.id,
       type: Track.kindToProto(track.kind),
@@ -1061,13 +1063,13 @@ export default class LocalParticipant extends Participant {
     }
 
     // remove from our maps
-    this.tracks.delete(publication.trackSid);
+    this.trackPublications.delete(publication.trackSid);
     switch (publication.kind) {
       case Track.Kind.Audio:
-        this.audioTracks.delete(publication.trackSid);
+        this.audioTrackPublications.delete(publication.trackSid);
         break;
       case Track.Kind.Video:
-        this.videoTracks.delete(publication.trackSid);
+        this.videoTrackPublications.delete(publication.trackSid);
         break;
       default:
         break;
@@ -1093,7 +1095,7 @@ export default class LocalParticipant extends Participant {
 
   async republishAllTracks(options?: TrackPublishOptions, restartTracks: boolean = true) {
     const localPubs: LocalTrackPublication[] = [];
-    this.tracks.forEach((pub) => {
+    this.trackPublications.forEach((pub) => {
       if (pub.track) {
         if (options) {
           pub.options = { ...pub.options, ...options };
@@ -1132,64 +1134,21 @@ export default class LocalParticipant extends Participant {
    * participant in the room if the destination field in publishOptions is empty
    *
    * @param data Uint8Array of the payload. To send string data, use TextEncoder.encode
-   * @param kind whether to send this as reliable or lossy.
-   * For data that you need delivery guarantee (such as chat messages), use Reliable.
-   * For data that should arrive as quickly as possible, but you are ok with dropped
-   * packets, use Lossy.
-   * @param publishOptions optionally specify a `topic` and `destination`
+   * @param options optionally specify a `reliable`, `topic` and `destination`
    */
-  async publishData(
-    data: Uint8Array,
-    kind: DataPacket_Kind,
-    publishOptions?: DataPublishOptions,
-  ): Promise<void>;
-  /**
-   * Publish a new data payload to the room. Data will be forwarded to each
-   * participant in the room if the destination argument is empty
-   *
-   * @param data Uint8Array of the payload. To send string data, use TextEncoder.encode
-   * @param kind whether to send this as reliable or lossy.
-   * For data that you need delivery guarantee (such as chat messages), use Reliable.
-   * For data that should arrive as quickly as possible, but you are ok with dropped
-   * packets, use Lossy.
-   * @param destination the participants who will receive the message
-   */
-  async publishData(
-    data: Uint8Array,
-    kind: DataPacket_Kind,
-    destination?: RemoteParticipant[] | string[],
-  ): Promise<void>;
-
-  async publishData(
-    data: Uint8Array,
-    kind: DataPacket_Kind,
-    publishOptions: DataPublishOptions | RemoteParticipant[] | string[] = {},
-  ) {
-    const destination = Array.isArray(publishOptions)
-      ? publishOptions
-      : publishOptions?.destination;
-    const destinationSids: string[] = [];
-
-    const topic = !Array.isArray(publishOptions) ? publishOptions.topic : undefined;
-
-    if (destination !== undefined) {
-      destination.forEach((val: any) => {
-        if (val instanceof RemoteParticipant) {
-          destinationSids.push(val.sid);
-        } else {
-          destinationSids.push(val);
-        }
-      });
-    }
+  async publishData(data: Uint8Array, options: DataPublishOptions = {}): Promise<void> {
+    const kind = options.reliable ? DataPacket_Kind.RELIABLE : DataPacket_Kind.LOSSY;
+    const destinationIdentities = options.destinationIdentities;
+    const topic = options.topic;
 
     const packet = new DataPacket({
-      kind,
+      kind: kind,
       value: {
         case: 'user',
         value: new UserPacket({
-          participantSid: this.sid,
+          participantIdentity: this.identity,
           payload: data,
-          destinationSids: destinationSids,
+          destinationIdentities,
           topic,
         }),
       },
@@ -1241,7 +1200,7 @@ export default class LocalParticipant extends Participant {
     // if server's track mute status doesn't match actual, we'll have to update
     // the server's copy
     info.tracks.forEach((ti) => {
-      const pub = this.tracks.get(ti.sid);
+      const pub = this.trackPublications.get(ti.sid);
 
       if (pub) {
         const mutedOnServer = pub.isMuted || (pub.track?.isUpstreamPaused ?? false);
@@ -1313,7 +1272,7 @@ export default class LocalParticipant extends Participant {
     if (!this.roomOptions?.dynacast) {
       return;
     }
-    const pub = this.videoTracks.get(update.trackSid);
+    const pub = this.videoTrackPublications.get(update.trackSid);
     if (!pub) {
       this.log.warn('received subscribed quality update for unknown track', {
         ...this.logContext,
@@ -1341,7 +1300,7 @@ export default class LocalParticipant extends Participant {
   };
 
   private handleLocalTrackUnpublished = (unpublished: TrackUnpublishedResponse) => {
-    const track = this.tracks.get(unpublished.trackSid);
+    const track = this.trackPublications.get(unpublished.trackSid);
     if (!track) {
       this.log.warn('received unpublished event for unknown track', {
         ...this.logContext,
@@ -1415,7 +1374,7 @@ export default class LocalParticipant extends Participant {
     track: LocalTrack | MediaStreamTrack,
   ): LocalTrackPublication | undefined {
     let publication: LocalTrackPublication | undefined;
-    this.tracks.forEach((pub) => {
+    this.trackPublications.forEach((pub) => {
       const localTrack = pub.track;
       if (!localTrack) {
         return;

package/src/room/participant/Participant.ts

@@ -48,12 +48,12 @@ function qualityFromProto(q: ProtoQuality): ConnectionQuality {
 export default class Participant extends (EventEmitter as new () => TypedEmitter<ParticipantEventCallbacks>) {
   protected participantInfo?: ParticipantInfo;
 
-  audioTracks: Map<string, TrackPublication>;
+  audioTrackPublications: Map<string, TrackPublication>;
 
-  videoTracks: Map<string, TrackPublication>;
+  videoTrackPublications: Map<string, TrackPublication>;
 
   /** map of track sid => all published tracks */
-  tracks: Map<string, TrackPublication>;
+  trackPublications: Map<string, TrackPublication>;
 
   /** audio level between 0-1.0, 1 being loudest, 0 being softest */
   audioLevel: number = 0;
@@ -94,7 +94,10 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
   }
 
   get isEncrypted() {
-    return this.tracks.size > 0 && Array.from(this.tracks.values()).every((tr) => tr.isEncrypted);
+    return (
+      this.trackPublications.size > 0 &&
+      Array.from(this.trackPublications.values()).every((tr) => tr.isEncrypted)
+    );
   }
 
   get isAgent() {
@@ -119,23 +122,21 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
     this.identity = identity;
     this.name = name;
     this.metadata = metadata;
-    this.audioTracks = new Map();
-    this.videoTracks = new Map();
-    this.tracks = new Map();
+    this.audioTrackPublications = new Map();
+    this.videoTrackPublications = new Map();
+    this.trackPublications = new Map();
   }
 
-  getTracks(): TrackPublication[] {
-    return Array.from(this.tracks.values());
+  getTrackPublications(): TrackPublication[] {
+    return Array.from(this.trackPublications.values());
   }
 
   /**
    * Finds the first track that matches the source filter, for example, getting
    * the user's camera track with getTrackBySource(Track.Source.Camera).
-   * @param source
-   * @returns
    */
-  getTrack(source: Track.Source): TrackPublication | undefined {
-    for (const [, pub] of this.tracks) {
+  getTrackPublication(source: Track.Source): TrackPublication | undefined {
+    for (const [, pub] of this.trackPublications) {
       if (pub.source === source) {
         return pub;
       }
@@ -144,11 +145,9 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
 
   /**
    * Finds the first track that matches the track's name.
-   * @param name
-   * @returns
    */
-  getTrackByName(name: string): TrackPublication | undefined {
-    for (const [, pub] of this.tracks) {
+  getTrackPublicationByName(name: string): TrackPublication | undefined {
+    for (const [, pub] of this.trackPublications) {
       if (pub.trackName === name) {
         return pub;
       }
@@ -160,17 +159,17 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
   }
 
   get isCameraEnabled(): boolean {
-    const track = this.getTrack(Track.Source.Camera);
+    const track = this.getTrackPublication(Track.Source.Camera);
     return !(track?.isMuted ?? true);
   }
 
   get isMicrophoneEnabled(): boolean {
-    const track = this.getTrack(Track.Source.Microphone);
+    const track = this.getTrackPublication(Track.Source.Microphone);
     return !(track?.isMuted ?? true);
   }
 
   get isScreenShareEnabled(): boolean {
-    const track = this.getTrack(Track.Source.ScreenShare);
+    const track = this.getTrackPublication(Track.Source.ScreenShare);
     return !!track;
   }
 
@@ -283,7 +282,7 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
    */
   setAudioContext(ctx: AudioContext | undefined) {
     this.audioContext = ctx;
-    this.audioTracks.forEach(
+    this.audioTrackPublications.forEach(
       (track) =>
         (track.track instanceof RemoteAudioTrack || track.track instanceof LocalAudioTrack) &&
         track.track.setAudioContext(ctx),
@@ -305,13 +304,13 @@ export default class Participant extends (EventEmitter as new () => TypedEmitter
       pub.track.sid = publication.trackSid;
     }
 
-    this.tracks.set(publication.trackSid, publication);
+    this.trackPublications.set(publication.trackSid, publication);
     switch (publication.kind) {
       case Track.Kind.Audio:
-        this.audioTracks.set(publication.trackSid, publication);
+        this.audioTrackPublications.set(publication.trackSid, publication);
         break;
       case Track.Kind.Video:
-        this.videoTracks.set(publication.trackSid, publication);
+        this.videoTrackPublications.set(publication.trackSid, publication);
         break;
       default:
         break;

package/src/room/participant/RemoteParticipant.ts

@@ -16,11 +16,11 @@ import Participant from './Participant';
 import type { ParticipantEventCallbacks } from './Participant';
 
 export default class RemoteParticipant extends Participant {
-  audioTracks: Map<string, RemoteTrackPublication>;
+  audioTrackPublications: Map<string, RemoteTrackPublication>;
 
-  videoTracks: Map<string, RemoteTrackPublication>;
+  videoTrackPublications: Map<string, RemoteTrackPublication>;
 
-  tracks: Map<string, RemoteTrackPublication>;
+  trackPublications: Map<string, RemoteTrackPublication>;
 
   signalClient: SignalClient;
 
@@ -44,9 +44,9 @@ export default class RemoteParticipant extends Participant {
   ) {
     super(sid, identity || '', name, metadata, loggerOptions);
     this.signalClient = signalClient;
-    this.tracks = new Map();
-    this.audioTracks = new Map();
-    this.videoTracks = new Map();
+    this.trackPublications = new Map();
+    this.audioTrackPublications = new Map();
+    this.videoTrackPublications = new Map();
     this.volumeMap = new Map();
   }
 
@@ -90,15 +90,15 @@ export default class RemoteParticipant extends Participant {
     });
   }
 
-  getTrack(source: Track.Source): RemoteTrackPublication | undefined {
-    const track = super.getTrack(source);
+  getTrackPublication(source: Track.Source): RemoteTrackPublication | undefined {
+    const track = super.getTrackPublication(source);
     if (track) {
       return track as RemoteTrackPublication;
     }
   }
 
-  getTrackByName(name: string): RemoteTrackPublication | undefined {
-    const track = super.getTrackByName(name);
+  getTrackPublicationByName(name: string): RemoteTrackPublication | undefined {
+    const track = super.getTrackPublicationByName(name);
     if (track) {
       return track as RemoteTrackPublication;
     }
@@ -115,7 +115,7 @@ export default class RemoteParticipant extends Participant {
     source: Track.Source.Microphone | Track.Source.ScreenShareAudio = Track.Source.Microphone,
   ) {
     this.volumeMap.set(source, volume);
-    const audioPublication = this.getTrack(source);
+    const audioPublication = this.getTrackPublication(source);
     if (audioPublication && audioPublication.track) {
       (audioPublication.track as RemoteAudioTrack).setVolume(volume);
     }
@@ -127,7 +127,7 @@ export default class RemoteParticipant extends Participant {
   getVolume(
     source: Track.Source.Microphone | Track.Source.ScreenShareAudio = Track.Source.Microphone,
   ) {
-    const audioPublication = this.getTrack(source);
+    const audioPublication = this.getTrackPublication(source);
     if (audioPublication && audioPublication.track) {
       return (audioPublication.track as RemoteAudioTrack).getVolume();
     }
@@ -145,14 +145,14 @@ export default class RemoteParticipant extends Participant {
   ) {
     // find the track publication
     // it's possible for the media track to arrive before participant info
-    let publication = this.getTrackPublication(sid);
+    let publication = this.getTrackPublicationBySid(sid);
 
     // it's also possible that the browser didn't honor our original track id
     // FireFox would use its own local uuid instead of server track id
     if (!publication) {
       if (!sid.startsWith('TR')) {
         // find the first track that matches type
-        this.tracks.forEach((p) => {
+        this.trackPublications.forEach((p) => {
           if (!publication && mediaTrack.kind === p.kind.toString()) {
             publication = p;
           }
@@ -224,8 +224,11 @@ export default class RemoteParticipant extends Participant {
     return !!this.participantInfo;
   }
 
-  getTrackPublication(sid: Track.SID): RemoteTrackPublication | undefined {
-    return this.tracks.get(sid);
+  /**
+   * @internal
+   */
+  getTrackPublicationBySid(sid: Track.SID): RemoteTrackPublication | undefined {
+    return this.trackPublications.get(sid);
   }
 
   /** @internal */
@@ -243,7 +246,7 @@ export default class RemoteParticipant extends Participant {
     const newTracks = new Map<string, RemoteTrackPublication>();
 
     info.tracks.forEach((ti) => {
-      let publication = this.getTrackPublication(ti.sid);
+      let publication = this.getTrackPublicationBySid(ti.sid);
       if (!publication) {
         // new publication
         const kind = Track.kindFromProto(ti.type);
@@ -258,7 +261,7 @@ export default class RemoteParticipant extends Participant {
         );
         publication.updateInfo(ti);
         newTracks.set(ti.sid, publication);
-        const existingTrackOfSource = Array.from(this.tracks.values()).find(
+        const existingTrackOfSource = Array.from(this.trackPublications.values()).find(
           (publishedTrack) => publishedTrack.source === publication?.source,
         );
         if (existingTrackOfSource && publication.source !== Track.Source.Unknown) {
@@ -279,7 +282,7 @@ export default class RemoteParticipant extends Participant {
     });
 
     // detect removed tracks
-    this.tracks.forEach((publication) => {
+    this.trackPublications.forEach((publication) => {
       if (!validTracks.has(publication.trackSid)) {
         this.log.trace('detected removed track on remote participant, unpublishing', {
           ...this.logContext,
@@ -298,7 +301,7 @@ export default class RemoteParticipant extends Participant {
 
   /** @internal */
   unpublishTrack(sid: Track.SID, sendUnpublish?: boolean) {
-    const publication = <RemoteTrackPublication>this.tracks.get(sid);
+    const publication = <RemoteTrackPublication>this.trackPublications.get(sid);
     if (!publication) {
       return;
     }
@@ -311,15 +314,15 @@ export default class RemoteParticipant extends Participant {
     }
 
     // remove track from maps only after unsubscribed has been fired
-    this.tracks.delete(sid);
+    this.trackPublications.delete(sid);
 
     // remove from the right type map
     switch (publication.kind) {
       case Track.Kind.Audio:
-        this.audioTracks.delete(sid);
+        this.audioTrackPublications.delete(sid);
         break;
       case Track.Kind.Video:
-        this.videoTracks.delete(sid);
+        this.videoTrackPublications.delete(sid);
         break;
       default:
         break;
@@ -336,7 +339,7 @@ export default class RemoteParticipant extends Participant {
   async setAudioOutput(output: AudioOutputOptions) {
     this.audioOutput = output;
     const promises: Promise<void>[] = [];
-    this.audioTracks.forEach((pub) => {
+    this.audioTrackPublications.forEach((pub) => {
       if (pub.track instanceof RemoteAudioTrack) {
         promises.push(pub.track.setSinkId(output.deviceId ?? 'default'));
       }

package/src/room/track/LocalVideoTrack.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from 'vitest';
-import { VideoQuality } from '../../proto/livekit_models_pb';
 import { videoLayersFromEncodings } from './LocalVideoTrack';
+import { VideoQuality } from './Track';
 
 describe('videoLayersFromEncodings', () => {
   it('returns single layer for no encoding', () => {

package/src/room/track/LocalVideoTrack.ts

@@ -1,6 +1,6 @@
 import type { SignalClient } from '../../api/SignalClient';
 import type { StructuredLogger } from '../../logger';
-import { VideoLayer, VideoQuality } from '../../proto/livekit_models_pb';
+import { VideoQuality as ProtoVideoQuality, VideoLayer } from '../../proto/livekit_models_pb';
 import { SubscribedCodec, SubscribedQuality } from '../../proto/livekit_rtc_pb';
 import { ScalabilityMode } from '../participant/publishUtils';
 import type { VideoSenderStats } from '../stats';
@@ -8,7 +8,7 @@ import { computeBitrate, monitorFrequency } from '../stats';
 import type { LoggerOptions } from '../types';
 import { Mutex, isFireFox, isMobile, isWeb, unwrapConstraint } from '../utils';
 import LocalTrack from './LocalTrack';
-import { Track } from './Track';
+import { Track, VideoQuality } from './Track';
 import type { VideoCaptureOptions, VideoCodec } from './options';
 import type { TrackProcessor } from './processor/types';
 import { constraintsForOptions } from './utils';
@@ -254,9 +254,13 @@ export default class LocalVideoTrack extends LocalTrack {
     }
   }
 
-  addSimulcastTrack(codec: VideoCodec, encodings?: RTCRtpEncodingParameters[]): SimulcastTrackInfo {
+  addSimulcastTrack(
+    codec: VideoCodec,
+    encodings?: RTCRtpEncodingParameters[],
+  ): SimulcastTrackInfo | undefined {
     if (this.simulcastCodecs.has(codec)) {
-      throw new Error(`${codec} already added`);
+      this.log.error(`${codec} already added, skipping adding simulcast codec`, this.logContext);
+      return;
     }
     const simulcastCodecInfo: SimulcastTrackInfo = {
       codec,
@@ -427,14 +431,14 @@ async function setPublishingLayersForSender(
       const encoding = encodings[0];
       /* @ts-ignore */
       // const mode = new ScalabilityMode(encoding.scalabilityMode);
-      let maxQuality = VideoQuality.OFF;
+      let maxQuality = ProtoVideoQuality.OFF;
       qualities.forEach((q) => {
-        if (q.enabled && (maxQuality === VideoQuality.OFF || q.quality > maxQuality)) {
+        if (q.enabled && (maxQuality === ProtoVideoQuality.OFF || q.quality > maxQuality)) {
           maxQuality = q.quality;
         }
       });
 
-      if (maxQuality === VideoQuality.OFF) {
+      if (maxQuality === ProtoVideoQuality.OFF) {
         if (encoding.active) {
           encoding.active = false;
           hasChanged = true;

package/src/room/track/RemoteTrackPublication.ts

@@ -1,15 +1,10 @@
-import {
-  ParticipantTracks,
-  SubscriptionError,
-  TrackInfo,
-  VideoQuality,
-} from '../../proto/livekit_models_pb';
+import { ParticipantTracks, SubscriptionError, TrackInfo } from '../../proto/livekit_models_pb';
 import { UpdateSubscription, UpdateTrackSettings } from '../../proto/livekit_rtc_pb';
 import { TrackEvent } from '../events';
 import type { LoggerOptions } from '../types';
 import type RemoteTrack from './RemoteTrack';
 import RemoteVideoTrack from './RemoteVideoTrack';
-import { Track } from './Track';
+import { Track, VideoQuality } from './Track';
 import { TrackPublication } from './TrackPublication';
 
 export default class RemoteTrackPublication extends TrackPublication {

package/src/room/track/Track.ts

@@ -2,7 +2,11 @@ import { EventEmitter } from 'events';
 import type TypedEventEmitter from 'typed-emitter';
 import type { SignalClient } from '../../api/SignalClient';
 import log, { LoggerNames, StructuredLogger, getLogger } from '../../logger';
-import { TrackSource, TrackType } from '../../proto/livekit_models_pb';
+import {
+  VideoQuality as ProtoQuality,
+  TrackSource,
+  TrackType,
+} from '../../proto/livekit_models_pb';
 import { StreamState as ProtoStreamState } from '../../proto/livekit_rtc_pb';
 import { TrackEvent } from '../events';
 import type { LoggerOptions } from '../types';
@@ -15,6 +19,11 @@ const BACKGROUND_REACTION_DELAY = 5000;
 // Safari tracks which audio elements have been "blessed" by the user.
 const recycledElements: Array<HTMLAudioElement> = [];
 
+export enum VideoQuality {
+  LOW = ProtoQuality.LOW,
+  MEDIUM = ProtoQuality.MEDIUM,
+  HIGH = ProtoQuality.HIGH,
+}
 export abstract class Track extends (EventEmitter as new () => TypedEventEmitter<TrackEventCallbacks>) {
   kind: Track.Kind;