mediasoup-client 3.7.14 → 3.7.15

package/lib/test/test.js

@@ -69,14 +69,14 @@ beforeEach(async () => {
         setTimeout(callback);
     });
     ctx.connectedSendTransport.on('produce', 
-    // eslint-disable-next-line no-shadow, @typescript-eslint/no-unused-vars
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     ({ kind, rtpParameters, appData }, callback /* errback */) => {
         // eslint-disable-next-line no-shadow
         const id = fakeParameters.generateProducerRemoteParameters().id;
         setTimeout(() => callback({ id }));
     });
     ctx.connectedSendTransport.on('producedata', (
-    // eslint-disable-next-line no-shadow, @typescript-eslint/no-unused-vars
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     { sctpStreamParameters, label, protocol, appData }, callback /* errback */) => {
         // eslint-disable-next-line no-shadow
         const id = fakeParameters.generateDataProducerRemoteParameters().id;
@@ -149,7 +149,7 @@ test('create a Device in Node without custom handlerName/handlerFactory throws U
     expect(() => new Device()).toThrow(errors_1.UnsupportedError);
 });
 test('create a Device with an unknown handlerName string throws TypeError', () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     expect(() => new Device({ handlerName: 'FooBrowser666' })).toThrow(TypeError);
 });
 test('create a Device in Node with a valid handlerFactory succeeds', () => {
@@ -180,7 +180,7 @@ test('device.createSendTransport() throws InvalidStateError if not loaded', () =
     })).toThrow(errors_1.InvalidStateError);
 });
 test('device.load() without routerRtpCapabilities rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.device.load({})).rejects.toThrow(TypeError);
     expect(ctx.device.loaded).toBe(false);
 }, 500);
@@ -188,7 +188,7 @@ test('device.load() with invalid routerRtpCapabilities rejects with TypeError',
     // Clone fake router RTP capabilities to make them invalid.
     const routerRtpCapabilities = utils.clone(fakeParameters.generateRouterRtpCapabilities());
     for (const codec of routerRtpCapabilities.codecs) {
-        // @ts-ignore
+        // @ts-expect-error --- On purpose.
         delete codec.mimeType;
     }
     await expect(ctx.device.load({ routerRtpCapabilities })).rejects.toThrow(TypeError);
@@ -216,7 +216,7 @@ test('device.canProduce() with "audio"/"video" kind returns true', () => {
     expect(ctx.loadedDevice.canProduce('video')).toBe(true);
 });
 test('device.canProduce() with invalid kind throws TypeError', () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     expect(() => ctx.loadedDevice.canProduce('chicken')).toThrow(TypeError);
 });
 test('device.createSendTransport() for sending media succeeds', () => {
@@ -259,14 +259,14 @@ test('device.createRecvTransport() for receiving media succeeds', () => {
     expect(recvTransport.appData).toEqual({});
 });
 test('device.createSendTransport() with missing remote Transport parameters throws TypeError', () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     expect(() => ctx.loadedDevice.createSendTransport({ id: '1234' })).toThrow(TypeError);
     expect(() => 
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.loadedDevice.createSendTransport({ id: '1234', iceParameters: {} })).toThrow(TypeError);
     expect(() => ctx.loadedDevice.createSendTransport({
         id: '1234',
-        // @ts-ignore
+        // @ts-expect-error --- On purpose.
         iceParameters: {},
         iceCandidates: [],
     })).toThrow(TypeError);
@@ -279,7 +279,7 @@ test('device.createRecvTransport() with a non object appData throws TypeError',
         iceCandidates,
         dtlsParameters,
         sctpParameters,
-        // @ts-ignore
+        // @ts-expect-error --- On purpose.
         appData: 1234,
     })).toThrow(TypeError);
 });
@@ -472,7 +472,7 @@ test('transport.produce() with an ended track rejects with InvalidStateError', a
 test('transport.produce() with a non object appData rejects with TypeError', async () => {
     const track = new fake_mediastreamtrack_1.FakeMediaStreamTrack({ kind: 'audio' });
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.produce({ track, appData: true })).rejects.toThrow(TypeError);
 }, 500);
 test('transport.consume() succeeds', async () => {
@@ -647,7 +647,7 @@ test('transport.consume() batches consumers created in same macrotask into the s
     const videoConsumerRemoteParameters2 = fakeParameters.generateConsumerRemoteParameters({
         codecMimeType: 'video/VP8',
     });
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     const pushSpy = jest.spyOn(ctx.connectedRecvTransport._awaitQueue, 'push');
     const waitForConsumer = (id) => {
         return new Promise(resolve => {
@@ -677,27 +677,27 @@ test('transport.consume() batches consumers created in same macrotask into the s
         }),
     ]);
     await allConsumersCreated;
-    expect(pushSpy).toBeCalledTimes(1);
+    expect(pushSpy).toHaveBeenCalledTimes(1);
 }, 500);
 test('transport.consume() without remote Consumer parameters rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.recvTransport.consume({})).rejects.toThrow(TypeError);
 }, 500);
 test('transport.consume() with missing remote Consumer parameters rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.recvTransport.consume({ id: '1234' })).rejects.toThrow(TypeError);
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.recvTransport.consume({ id: '1234', producerId: '4444' })).rejects.toThrow(TypeError);
     await expect(ctx.recvTransport.consume(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     {
         id: '1234',
         producerId: '4444',
         kind: 'audio',
     })).rejects.toThrow(TypeError);
     await expect(ctx.recvTransport.consume(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     {
         id: '1234',
         producerId: '4444',
@@ -731,7 +731,7 @@ test('transport.consume() with a non object appData rejects with TypeError', asy
         codecMimeType: 'audio/opus',
     });
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.recvTransport.consume({ consumerRemoteParameters, appData: true })).rejects.toThrow(TypeError);
 }, 500);
 test('transport.produceData() succeeds', async () => {
@@ -763,7 +763,7 @@ test('transport.produceData() in a receiving Transport rejects with UnsupportedE
 }, 500);
 test('transport.produceData() with a non object appData rejects with TypeError', async () => {
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.produceData({ appData: true })).rejects.toThrow(TypeError);
 }, 500);
 test('transport.consumeData() succeeds', async () => {
@@ -786,14 +786,14 @@ test('transport.consumeData() succeeds', async () => {
     expect(dataConsumer.protocol).toBe('BAR');
 }, 500);
 test('transport.consumeData() without remote DataConsumer parameters rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.recvTransport.consumeData({})).rejects.toThrow(TypeError);
 }, 500);
 test('transport.consumeData() with missing remote DataConsumer parameters rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.recvTransport.consumeData({ id: '1234' })).rejects.toThrow(TypeError);
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.recvTransport.consumeData({ id: '1234', dataProducerId: '4444' })).rejects.toThrow(TypeError);
 }, 500);
 test('transport.consumeData() in a sending Transport rejects with UnsupportedError', async () => {
@@ -808,7 +808,7 @@ test('transport.consumeData() with a non object appData rejects with TypeError',
     const dataConsumerRemoteParameters = fakeParameters.generateDataConsumerRemoteParameters();
     await expect(ctx.recvTransport.consumeData({
         dataConsumerRemoteParameters,
-        // @ts-ignore
+        // @ts-expect-error --- On purpose.
         appData: true,
     })).rejects.toThrow(TypeError);
 }, 500);
@@ -825,7 +825,7 @@ test('transport.restartIce() succeeds', async () => {
     })).resolves.toBe(undefined);
 }, 500);
 test('transport.restartIce() without remote iceParameters rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.sendTransport.restartIce({})).rejects.toThrow(TypeError);
 }, 500);
 test('transport.updateIceServers() succeeds', async () => {
@@ -839,9 +839,9 @@ test('ICE gathering state change fires "icegatheringstatechange" in live Transpo
     // emits '@connectionstatechange' with value 'connecting' as soon as its
     // private setupTransport() method is called (which has happens many times in
     // tests above already). So here we have to reset it manually to test things.
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.handler.setIceGatheringState('new');
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.handler.setConnectionState('new');
     let iceGatheringStateChangeEventNumTimesCalled = 0;
     let connectionStateChangeEventNumTimesCalled = 0;
@@ -855,7 +855,7 @@ test('ICE gathering state change fires "icegatheringstatechange" in live Transpo
     ctx.sendTransport.on('connectionstatechange', connectionState => {
         connectionStateChangeEventNumTimesCalled++;
     });
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.handler.setIceGatheringState('complete');
     expect(iceGatheringStateChangeEventNumTimesCalled).toBe(1);
     expect(connectionStateChangeEventNumTimesCalled).toBe(0);
@@ -868,7 +868,7 @@ test('connection state change fires "connectionstatechange" in live Transport',
         connectionStateChangeEventNumTimesCalled++;
         expect(connectionState).toBe('completed');
     });
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.sendTransport.handler.setConnectionState('completed');
     expect(connectionStateChangeEventNumTimesCalled).toBe(1);
     expect(ctx.sendTransport.connectionState).toBe('completed');
@@ -952,11 +952,11 @@ test('producer.setMaxSpatialLayer() in an audio Producer rejects with Unsupporte
 }, 500);
 test('producer.setMaxSpatialLayer() with invalid spatialLayer rejects with TypeError', async () => {
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.videoProducer.setMaxSpatialLayer('chicken')).rejects.toThrow(TypeError);
 }, 500);
 test('producer.setMaxSpatialLayer() without spatialLayer rejects with TypeError', async () => {
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.videoProducer.setMaxSpatialLayer()).rejects.toThrow(TypeError);
 }, 500);
 test('producer.setRtpEncodingParameters() succeeds', async () => {
@@ -1028,18 +1028,18 @@ test('remotetely stopped track fires "trackended" in live Producers/Consumers',
     ctx.videoConsumer.on('trackended', () => {
         videoConsumerTrackendedEventCalled = true;
     });
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.audioProducer.track.remoteStop();
     expect(audioProducerTrackendedEventCalled).toBe(true);
     // Let's close the video producer.
     ctx.videoProducer.close();
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.videoProducer.track.remoteStop();
     expect(videoProducerTrackendedEventCalled).toBe(false);
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.audioConsumer.track.remoteStop();
     expect(audiosConsumerTrackendedEventCalled).toBe(true);
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.videoConsumer.track.remoteStop();
     expect(videoConsumerTrackendedEventCalled).toBe(true);
 });
@@ -1082,7 +1082,7 @@ test('transport.produce() rejects with InvalidStateError if closed', async () =>
 }, 500);
 test('transport.consume() rejects with InvalidStateError if closed', async () => {
     ctx.connectedRecvTransport.close();
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.connectedRecvTransport.consume({})).rejects.toThrow(errors_1.InvalidStateError);
 }, 500);
 test('transport.produceData() rejects with InvalidStateError if closed', async () => {
@@ -1091,7 +1091,7 @@ test('transport.produceData() rejects with InvalidStateError if closed', async (
 }, 500);
 test('transport.consumeData() rejects with InvalidStateError if closed', async () => {
     ctx.connectedRecvTransport.close();
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     await expect(ctx.connectedRecvTransport.consumeData({})).rejects.toThrow(errors_1.InvalidStateError);
 }, 500);
 test('transport.getStats() rejects with InvalidStateError if closed', async () => {
@@ -1101,7 +1101,7 @@ test('transport.getStats() rejects with InvalidStateError if closed', async () =
 test('transport.restartIce() rejects with InvalidStateError if closed', async () => {
     ctx.connectedSendTransport.close();
     await expect(
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.connectedSendTransport.restartIce({ ieParameters: {} })).rejects.toThrow(errors_1.InvalidStateError);
 }, 500);
 test('transport.updateIceServers() rejects with InvalidStateError if closed', async () => {
@@ -1114,16 +1114,16 @@ test('connection state change does not fire "connectionstatechange" in closed Tr
         connectionStateChangeEventNumTimesCalled++;
     });
     ctx.connectedSendTransport.close();
-    // @ts-ignore
+    // @ts-expect-error --- On purpose.
     ctx.connectedSendTransport.handler.setConnectionState('disconnected');
     expect(connectionStateChangeEventNumTimesCalled).toBe(0);
     expect(ctx.connectedSendTransport.connectionState).toBe('disconnected');
 });
-test('RemoteSdp properly handles multiple streams of the same type in planB', async () => {
+test('RemoteSdp properly handles multiple streams of the same type in planB', () => {
     let sdp = undefined;
     let sdpObject = undefined;
     const remoteSdp = new RemoteSdp_1.RemoteSdp({ planB: true });
-    await remoteSdp.receive({
+    remoteSdp.receive({
         mid: 'video',
         kind: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({
@@ -1142,7 +1142,7 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[1].payload).toBe(102);
     expect(sdpObject.media[0].rtp[1].codec).toBe('rtx');
     expect(sdpObject.media[0].ssrcs?.length).toBe(4);
-    await remoteSdp.receive({
+    remoteSdp.receive({
         mid: 'video',
         kind: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({
@@ -1165,7 +1165,7 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[3].payload).toBe(104);
     expect(sdpObject.media[0].rtp[3].codec).toBe('rtx');
     expect(sdpObject.media[0].ssrcs?.length).toBe(8);
-    await remoteSdp.planBStopReceiving({
+    remoteSdp.planBStopReceiving({
         mid: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({
             codecMimeType: 'video/H264',
@@ -1186,11 +1186,11 @@ test('RemoteSdp properly handles multiple streams of the same type in planB', as
     expect(sdpObject.media[0].rtp[3].codec).toBe('rtx');
     expect(sdpObject.media[0].ssrcs?.length).toBe(4);
 }, 500);
-test('RemoteSdp does not duplicate codec descriptions', async () => {
+test('RemoteSdp does not duplicate codec descriptions', () => {
     let sdp = undefined;
     let sdpObject = undefined;
     const remoteSdp = new RemoteSdp_1.RemoteSdp({ planB: true });
-    await remoteSdp.receive({
+    remoteSdp.receive({
         mid: 'video',
         kind: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({
@@ -1209,7 +1209,7 @@ test('RemoteSdp does not duplicate codec descriptions', async () => {
     expect(sdpObject.media[0].rtp[1].payload).toBe(102);
     expect(sdpObject.media[0].rtp[1].codec).toBe('rtx');
     expect(sdpObject.media[0].ssrcs?.length).toBe(4);
-    await remoteSdp.receive({
+    remoteSdp.receive({
         mid: 'video',
         kind: 'video',
         offerRtpParameters: fakeParameters.generateConsumerRemoteParameters({
@@ -1270,8 +1270,10 @@ test('parseScalabilityMode() works', () => {
 describe('detectDevice() assigns proper handler based on UserAgent', () => {
     const originalNavigator = global.navigator;
     for (const uaTestCase of uaTestCases_1.uaTestCases) {
-        test(uaTestCase.desc, () => {
-            // @ts-ignore
+        test(
+        // eslint-disable-next-line jest/valid-title --- Jest is not that smart.
+        uaTestCase.desc, () => {
+            // @ts-expect-error --- On purpose.
             global.navigator = {
                 userAgent: uaTestCase.ua,
             };

package/npm-scripts.mjs

@@ -7,7 +7,7 @@ const PKG = JSON.parse(fs.readFileSync('./package.json').toString());
 const MAYOR_VERSION = PKG.version.split('.')[0];
 
 // Paths for ESLint to check. Converted to string for convenience.
-const ESLINT_PATHS = ['src', 'npm-scripts.mjs'].join(' ');
+const ESLINT_PATHS = ['eslint.config.mjs', 'src', 'npm-scripts.mjs'].join(' ');
 // Paths for ESLint to ignore. Converted to string argument for convenience.
 const ESLINT_IGNORE_PATTERN_ARGS = []
 	.map(entry => `--ignore-pattern ${entry}`)
@@ -17,6 +17,7 @@ const ESLINT_IGNORE_PATTERN_ARGS = []
 // node/src/fbs.
 const PRETTIER_PATHS = [
 	'README.md',
+	'eslint.config.mjs',
 	'src',
 	'npm-scripts.mjs',
 	'package.json',
@@ -165,10 +166,10 @@ function lint() {
 
 	// Ensure there are no rules that are unnecessary or conflict with Prettier
 	// rules.
-	executeCmd('eslint-config-prettier .eslintrc.js');
+	executeCmd('eslint-config-prettier eslint.config.mjs');
 
 	executeCmd(
-		`eslint -c .eslintrc.js --ext=ts,js,mjs --max-warnings 0 ${ESLINT_IGNORE_PATTERN_ARGS} ${ESLINT_PATHS}`
+		`eslint -c eslint.config.mjs --max-warnings 0 ${ESLINT_IGNORE_PATTERN_ARGS} ${ESLINT_PATHS}`
 	);
 
 	executeCmd(`prettier --check ${PRETTIER_PATHS}`);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "mediasoup-client",
-	"version": "3.7.14",
+	"version": "3.7.15",
 	"description": "mediasoup client side TypeScript library",
 	"contributors": [
 		"Iñaki Baz Castillo <ibc@aliax.net> (https://inakibaz.me)",
@@ -23,7 +23,7 @@
 		"lib"
 	],
 	"engines": {
-		"node": ">=16"
+		"node": ">=18"
 	},
 	"keywords": [
 		"webrtc",
@@ -67,7 +67,7 @@
 		"@types/debug": "^4.1.12",
 		"@types/npm-events-package": "npm:@types/events@^3.0.3",
 		"awaitqueue": "^3.0.2",
-		"debug": "^4.3.5",
+		"debug": "^4.3.6",
 		"npm-events-package": "npm:events@^3.3.0",
 		"fake-mediastreamtrack": "^1.2.0",
 		"h264-profile-level-id": "^2.0.0",
@@ -77,20 +77,24 @@
 		"ua-parser-js": "^1.0.38"
 	},
 	"devDependencies": {
+		"@eslint/js": "^9.9.0",
 		"@types/jest": "^29.5.12",
-		"@types/node": "20",
+		"@types/node": "22",
 		"@types/sdp-transform": "^2.4.9",
 		"@types/ua-parser-js": "^0.7.39",
-		"@typescript-eslint/eslint-plugin": "^7.17.0",
-		"@typescript-eslint/parser": "^7.17.0",
+		"@typescript-eslint/eslint-plugin": "^8.0.1",
+		"@typescript-eslint/parser": "^8.0.1",
 		"eslint": "^8.57.0",
+		"eslint": "^9.9.0",
 		"eslint-config-prettier": "^9.1.0",
-		"eslint-plugin-jest": "^28.6.0",
+		"eslint-plugin-jest": "^28.8.0",
 		"eslint-plugin-prettier": "^5.2.1",
+		"globals": "^15.9.0",
 		"jest": "^29.7.0",
 		"open-cli": "^8.0.0",
 		"prettier": "^3.3.3",
-		"ts-jest": "^29.2.3",
-		"typescript": "^5.5.4"
+		"ts-jest": "^29.2.4",
+		"typescript": "^5.5.4",
+		"typescript-eslint": "^8.0.1"
 	}
 }