@fishjam-cloud/ts-client 0.5.5 → 0.7.0

package/dist/src/FishjamClient.d.ts

@@ -12,6 +12,11 @@ export type Component<PeerMetadata, TrackMetadata> = Omit<Endpoint<PeerMetadata,
  * Events emitted by the client with their arguments.
  */
 export interface MessageEvents<PeerMetadata, TrackMetadata> {
+    /**
+     * Emitted when connect method invoked
+     *
+     */
+    connectionStarted: () => void;
     /**
      * Emitted when the websocket connection is closed
      *
@@ -210,7 +215,7 @@ export declare class FishjamClient<PeerMetadata, TrackMetadata> extends FishjamC
      *
      * @param {ConnectConfig} config - Configuration object for the client
      */
-    connect(config: ConnectConfig<PeerMetadata>): void;
+    connect(config: ConnectConfig<PeerMetadata>): Promise<void>;
     private initConnection;
     private getUrl;
     private initWebsocket;

package/dist/src/FishjamClient.js

@@ -3,6 +3,7 @@ import { EventEmitter } from 'events';
 import { PeerMessage } from './protos';
 import { ReconnectManager } from './reconnection';
 import { isAuthError } from './auth';
+import { connectEventsHandler } from './connectEventsHandler';
 const STATISTICS_INTERVAL = 10_000;
 const isPeer = (endpoint) => endpoint.type === 'webrtc';
 const isComponent = (endpoint) => endpoint.type === 'recording' ||
@@ -77,10 +78,13 @@ export class FishjamClient extends EventEmitter {
      *
      * @param {ConnectConfig} config - Configuration object for the client
      */
-    connect(config) {
+    async connect(config) {
+        this.emit('connectionStarted');
+        const result = connectEventsHandler(this);
         this.reconnectManager.reset(config.peerMetadata);
         this.connectConfig = config;
         this.initConnection(config.peerMetadata);
+        return result;
     }
     async initConnection(peerMetadata) {
         if (this.status === 'initialized') {

package/dist/src/connectEventsHandler.d.ts

@@ -0,0 +1,2 @@
+import type { FishjamClient } from './FishjamClient';
+export declare function connectEventsHandler<P, T>(fishjamClient: FishjamClient<P, T>): Promise<void>;

package/dist/src/connectEventsHandler.js

@@ -0,0 +1,22 @@
+export function connectEventsHandler(fishjamClient) {
+    return new Promise((resolve, reject) => {
+        const onSuccess = () => {
+            clearCallbacks();
+            resolve();
+        };
+        const onError = () => {
+            clearCallbacks();
+            reject();
+        };
+        fishjamClient.on('joined', onSuccess);
+        fishjamClient.on('joinError', onError);
+        fishjamClient.on('authError', onError);
+        fishjamClient.on('socketError', onError);
+        const clearCallbacks = () => {
+            fishjamClient.removeListener('joined', onSuccess);
+            fishjamClient.removeListener('joinError', onError);
+            fishjamClient.removeListener('authError', onError);
+            fishjamClient.removeListener('socketError', onError);
+        };
+    });
+}

package/dist/src/webrtc/ConnectionManager.d.ts

@@ -15,14 +15,12 @@ export declare class ConnectionManager {
      */
     private getIceServers;
     getConnection: () => RTCPeerConnection;
-    setTransceiversToReadOnly: () => void;
     addTransceiversIfNeeded: (serverTracks: Map<string, number>) => void;
     private getNeededTransceiversTypes;
     addTransceiver: (track: MediaStreamTrack, transceiverConfig: RTCRtpTransceiverInit) => void;
     setOnTrackReady: (onTrackReady: (event: RTCTrackEvent) => void) => void;
     setRemoteDescription: (data: RTCSessionDescriptionInit) => Promise<void>;
     isTrackInUse: (track: MediaStreamTrack) => boolean;
-    setTransceiverDirection: () => void;
     removeTrack: (sender: RTCRtpSender) => void;
     findSender: (mediaStreamTrackId: MediaStreamTrackId) => RTCRtpSender;
     addIceCandidate: (iceCandidate: RTCIceCandidate) => Promise<void>;

package/dist/src/webrtc/ConnectionManager.js

@@ -34,9 +34,6 @@ export class ConnectionManager {
     getConnection = () => {
         return this.connection;
     };
-    setTransceiversToReadOnly = () => {
-        this.connection.getTransceivers().forEach((transceiver) => (transceiver.direction = 'sendonly'));
-    };
     addTransceiversIfNeeded = (serverTracks) => {
         const recvTransceivers = this.connection.getTransceivers().filter((elem) => elem.direction === 'recvonly');
         ['audio', 'video']
@@ -58,14 +55,6 @@ export class ConnectionManager {
         await this.connection.setRemoteDescription(data);
     };
     isTrackInUse = (track) => this.connection.getSenders().some((val) => val.track === track);
-    setTransceiverDirection = () => {
-        this.connection
-            .getTransceivers()
-            .filter((transceiver) => transceiver.direction === 'sendrecv')
-            .forEach((transceiver) => {
-            transceiver.direction = 'sendonly';
-        });
-    };
     removeTrack = (sender) => {
         this.connection.removeTrack(sender);
     };

package/dist/src/webrtc/tracks/Local.d.ts

@@ -28,7 +28,6 @@ export declare class Local<EndpointMetadata, TrackMetadata> {
     updateConnection: (connection: ConnectionManager) => void;
     createSdpOfferEvent: (offer: RTCSessionDescriptionInit) => MediaEvent;
     addTrack: (connection: ConnectionManager | undefined, trackId: string, track: MediaStreamTrack, stream: MediaStream, trackMetadata: TrackMetadata | undefined, simulcastConfig: SimulcastConfig, maxBandwidth: TrackBandwidthLimit) => LocalTrack<EndpointMetadata, TrackMetadata>;
-    disableAllLocalTrackEncodings: () => Promise<void>;
     getTrackByMidOrNull: (mid: string) => LocalTrack<EndpointMetadata, TrackMetadata> | null;
     removeTrack: (trackId: TrackId) => void;
     replaceTrack: (webrtc: WebRTCEndpoint<EndpointMetadata, TrackMetadata>, trackId: TrackId, newTrack: MediaStreamTrack | null) => Promise<void>;

package/dist/src/webrtc/tracks/Local.js

@@ -74,16 +74,6 @@ export class Local {
         this.localTracks[trackId] = trackManager;
         return trackManager;
     };
-    disableAllLocalTrackEncodings = async () => {
-        // TODO: Why are we disabling already disabled encodings?
-        //  I think this part of the code is invoked to mutate RTCRtpSender.getParameters().encodings.active.
-        //  We should probably implement a separate method for that, because disabling already disabled tracks seems weird.
-        for (const [trackId, trackManager] of Object.entries(this.localTracks)) {
-            for (const encoding of trackManager.getDisabledEncodings()) {
-                await this.disableLocalTrackEncoding(trackId, encoding);
-            }
-        }
-    };
     getTrackByMidOrNull = (mid) => {
         return Object.values(this.localTracks).find((track) => track.mLineId === mid) ?? null;
     };

package/dist/src/webrtc/tracks/LocalTrack.d.ts

@@ -58,5 +58,5 @@ export declare class LocalTrack<EndpointMetadata, TrackMetadata> implements Trac
     setMLineId: (mLineId: MLineId) => void;
     private isNotSimulcastTrack;
     getTrackBitrates: () => Bitrates;
-    createTrackVariantBitratesEvent: () => import("../mediaEvent").MediaEvent;
+    createTrackVariantBitratesEvent: () => import("..").MediaEvent;
 }

package/dist/src/webrtc/tracks/LocalTrackManager.js

@@ -48,7 +48,6 @@ export class LocalTrackManager {
         const trackManager = this.local.addTrack(this.connection, trackId, track, stream, trackMetadata, simulcastConfig, maxBandwidth);
         if (this.connection) {
             trackManager.addTrackToConnection();
-            this.connection.setTransceiverDirection();
         }
         const mediaEvent = generateCustomEvent({ type: 'renegotiateTracks' });
         this.sendMediaEvent(mediaEvent);

package/dist/src/webrtc/webRTCEndpoint.js

@@ -265,7 +265,6 @@ export class WebRTCEndpoint extends EventEmitter {
         });
         try {
             await this.connectionManager.setRemoteDescription(data);
-            await this.local.disableAllLocalTrackEncodings();
         }
         catch (err) {
             console.error(err);
@@ -626,7 +625,6 @@ export class WebRTCEndpoint extends EventEmitter {
             connection.getConnection().addEventListener('iceconnectionstatechange', onIceConnectionStateChange);
             this.commandsQueue.initConnection(connection);
             this.local.addAllTracksToConnection();
-            connection.setTransceiversToReadOnly();
         }
         this.localTrackManager.updateSenders();
         const tracks = new Map(Object.entries(offerData.data.tracksTypes));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fishjam-cloud/ts-client",
-  "version": "0.5.5",
+  "version": "0.7.0",
   "description": "Typescript client library for Fishjam Cloud",
   "license": "Apache-2.0",
   "author": "Fishjam Cloud Team",
@@ -41,7 +41,6 @@
     "test": "vitest run tests/**",
     "test:e2e": "NODE_OPTIONS=--dns-result-order=ipv4first playwright test",
     "test:coverage": "vitest run tests/** --coverage",
-    "typecheck": "tsc",
     "prepare": "tsc",
     "prepack": "yarn build"
   },
@@ -52,22 +51,22 @@
     "uuid": "^10.0.0"
   },
   "devDependencies": {
-    "@playwright/test": "^1.46.1",
+    "@playwright/test": "^1.47.1",
     "@types/events": "^3.0.3",
-    "@types/node": "^22.4.1",
+    "@types/node": "^22.5.5",
     "@types/uuid": "^10.0.0",
-    "@vitest/coverage-v8": "^2.0.5",
+    "@vitest/coverage-v8": "^2.1.1",
     "fake-mediastreamtrack": "^1.2.0",
-    "husky": "^9.1.4",
-    "lint-staged": "^15.2.9",
+    "husky": "^9.1.6",
+    "lint-staged": "^15.2.10",
     "react": "^18.2.0",
-    "ts-proto": "^2.0.2",
+    "ts-proto": "^2.2.0",
     "typed-emitter": "^2.1.0",
-    "typedoc": "^0.26.6",
+    "typedoc": "^0.26.7",
     "typedoc-plugin-external-resolver": "^1.0.3",
-    "typedoc-plugin-mdn-links": "^3.2.10",
-    "typescript": "^5.5.4",
-    "vitest": "^2.0.5",
+    "typedoc-plugin-mdn-links": "^3.2.12",
+    "typescript": "^5.6.2",
+    "vitest": "^2.1.1",
     "zod": "^3.23.6"
   },
   "lint-staged": {