rx-player 4.2.0-dev.2024090300 → 4.2.0-dev.2024090500

package/src/compat/__tests__/should_wait_for_data_before_loaded.test.ts

@@ -1,10 +1,5 @@
 import { describe, beforeEach, it, expect, vi } from "vitest";
-
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-unsafe-return */
-/* eslint-disable @typescript-eslint/no-explicit-any */
+import type IShouldWaitForDataBeforeLoaded from "../should_wait_for_data_before_loaded";
 
 describe("compat - shouldWaitForDataBeforeLoaded", () => {
   beforeEach(() => {
@@ -17,10 +12,10 @@ describe("compat - shouldWaitForDataBeforeLoaded", () => {
         isSafariMobile: false,
       };
     });
-    const shouldWaitForDataBeforeLoaded = (await vi.importActual(
-      "../should_wait_for_data_before_loaded",
-    )) as any;
-    expect(shouldWaitForDataBeforeLoaded.default(false)).toBe(true);
+    const shouldWaitForDataBeforeLoaded = (
+      await vi.importActual("../should_wait_for_data_before_loaded")
+    ).default as typeof IShouldWaitForDataBeforeLoaded;
+    expect(shouldWaitForDataBeforeLoaded(false)).toBe(true);
   });
 
   it("should return true if we are not on Safari browser but in directfile mode", async () => {
@@ -29,10 +24,10 @@ describe("compat - shouldWaitForDataBeforeLoaded", () => {
         isSafariMobile: false,
       };
     });
-    const shouldWaitForDataBeforeLoaded = (await vi.importActual(
-      "../should_wait_for_data_before_loaded",
-    )) as any;
-    expect(shouldWaitForDataBeforeLoaded.default(true)).toBe(true);
+    const shouldWaitForDataBeforeLoaded = (
+      await vi.importActual("../should_wait_for_data_before_loaded")
+    ).default as typeof IShouldWaitForDataBeforeLoaded;
+    expect(shouldWaitForDataBeforeLoaded(true)).toBe(true);
   });
 
   it("should return true if we are on the Safari browser but not in directfile mode", async () => {
@@ -41,10 +36,10 @@ describe("compat - shouldWaitForDataBeforeLoaded", () => {
         isSafariMobile: true,
       };
     });
-    const shouldWaitForDataBeforeLoaded = (await vi.importActual(
-      "../should_wait_for_data_before_loaded",
-    )) as any;
-    expect(shouldWaitForDataBeforeLoaded.default(false)).toBe(true);
+    const shouldWaitForDataBeforeLoaded = (
+      await vi.importActual("../should_wait_for_data_before_loaded")
+    ).default as typeof IShouldWaitForDataBeforeLoaded;
+    expect(shouldWaitForDataBeforeLoaded(false)).toBe(true);
   });
 
   // eslint-disable-next-line max-len
@@ -54,13 +49,9 @@ describe("compat - shouldWaitForDataBeforeLoaded", () => {
         isSafariMobile: true,
       };
     });
-    const shouldWaitForDataBeforeLoaded = (await vi.importActual(
-      "../should_wait_for_data_before_loaded",
-    )) as any;
-    expect(shouldWaitForDataBeforeLoaded.default(true)).toBe(false);
-  });
-
-  beforeEach(() => {
-    vi.resetModules();
+    const shouldWaitForDataBeforeLoaded = (
+      await vi.importActual("../should_wait_for_data_before_loaded")
+    ).default as typeof IShouldWaitForDataBeforeLoaded;
+    expect(shouldWaitForDataBeforeLoaded(true)).toBe(false);
   });
 });

package/src/compat/__tests__/should_wait_for_have_enough_data.test.ts

@@ -1,10 +1,5 @@
 import { describe, beforeEach, it, expect, vi } from "vitest";
-
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-unsafe-return */
-/* eslint-disable @typescript-eslint/no-explicit-any */
+import type IShouldWaitForHaveEnoughData from "../should_wait_for_have_enough_data";
 
 describe("compat - shouldWaitForHaveEnoughData", () => {
   beforeEach(() => {
@@ -17,10 +12,10 @@ describe("compat - shouldWaitForHaveEnoughData", () => {
         isPlayStation5: false,
       };
     });
-    const shouldWaitForHaveEnoughData = (await vi.importActual(
-      "../should_wait_for_have_enough_data",
-    )) as any;
-    expect(shouldWaitForHaveEnoughData.default()).toBe(false);
+    const shouldWaitForHaveEnoughData = (
+      await vi.importActual("../should_wait_for_have_enough_data")
+    ).default as typeof IShouldWaitForHaveEnoughData;
+    expect(shouldWaitForHaveEnoughData()).toBe(false);
   });
 
   it("should return true if we are on the Playstation 5", async () => {
@@ -28,10 +23,10 @@ describe("compat - shouldWaitForHaveEnoughData", () => {
       return {
         isPlayStation5: true,
       };
-    }) as any;
-    const shouldWaitForHaveEnoughData = (await vi.importActual(
-      "../should_wait_for_have_enough_data",
-    )) as any;
-    expect(shouldWaitForHaveEnoughData.default()).toBe(true);
+    });
+    const shouldWaitForHaveEnoughData = (
+      await vi.importActual("../should_wait_for_have_enough_data")
+    ).default as typeof IShouldWaitForHaveEnoughData;
+    expect(shouldWaitForHaveEnoughData()).toBe(true);
   });
 });

package/src/compat/add_class_name.ts

@@ -26,9 +26,8 @@ export default function addClassName(elt: HTMLElement, className: string): void
   if (hasClassList === undefined) {
     hasClassList =
       elt.classList !== undefined &&
-      /* eslint-disable @typescript-eslint/unbound-method */
+      // eslint-disable-next-line @typescript-eslint/unbound-method
       typeof elt.classList.add === "function";
-    /* eslint-enable @typescript-eslint/unbound-method */
   }
 
   if (hasClassList) {

package/src/compat/browser_compatibility_types.ts

@@ -299,8 +299,8 @@ export interface IMediaElement extends IEventTarget<IMediaElementEventMap> {
   videoTracks?: ICompatVideoTrackList;
 }
 
-/* eslint-disable @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types */
 // @ts-expect-error unused function, just used for compile-time typechecking
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types
 function testMediaElement(x: HTMLMediaElement) {
   assertCompatibleIMediaElement(x);
 }
@@ -308,6 +308,7 @@ function assertCompatibleIMediaElement(_x: IMediaElement) {
   // Noop
 }
 // @ts-expect-error unused function, just used for compile-time typechecking
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types
 function testMediaSource(x: MediaSource) {
   assertCompatibleIMediaSource(x);
 }
@@ -315,6 +316,7 @@ function assertCompatibleIMediaSource(_x: IMediaSource) {
   // Noop
 }
 // @ts-expect-error unused function, just used for compile-time typechecking
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types
 function testSourceBuffer(x: SourceBuffer) {
   assertCompatibleISourceBuffer(x);
 }
@@ -322,13 +324,13 @@ function assertCompatibleISourceBuffer(_x: ISourceBuffer) {
   // Noop
 }
 // @ts-expect-error unused function, just used for compile-time typechecking
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types
 function testSourceBufferList(x: SourceBufferList) {
   assertCompatibleISourceBufferList(x);
 }
 function assertCompatibleISourceBufferList(_x: ISourceBufferList) {
   // Noop
 }
-/* eslint-enable @typescript-eslint/no-unused-vars, @typescript-eslint/no-restricted-types */
 
 /**
  * AudioTrackList implementation (that TS forgot).

package/src/compat/clear_element_src.ts

@@ -40,7 +40,7 @@ export default function clearElementSrc(element: IMediaElement): void {
         if (childNodes[j].nodeName === "track") {
           try {
             element.removeChild(childNodes[j]);
-          } catch (err) {
+          } catch (_err) {
             log.warn("Compat: Could not remove text track child from element.");
           }
         }

package/src/compat/eme/custom_media_keys/moz_media_keys_constructor.ts

@@ -32,9 +32,8 @@ if (
   MozMediaKeys !== undefined &&
   MozMediaKeys.prototype !== undefined &&
   typeof MozMediaKeys.isTypeSupported === "function" &&
-  /* eslint-disable @typescript-eslint/no-unsafe-member-access */
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   typeof MozMediaKeys.prototype.createSession === "function"
-  /* eslint-enable @typescript-eslint/no-unsafe-member-access */
 ) {
   MozMediaKeysConstructor = MozMediaKeys;
 }

package/src/compat/eme/custom_media_keys/ms_media_keys_constructor.ts

@@ -58,9 +58,8 @@ if (
   MSMediaKeys !== undefined &&
   MSMediaKeys.prototype !== undefined &&
   typeof MSMediaKeys.isTypeSupported === "function" &&
-  /* eslint-disable @typescript-eslint/no-unsafe-member-access */
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   typeof MSMediaKeys.prototype.createSession === "function"
-  /* eslint-enable @typescript-eslint/no-unsafe-member-access */
 ) {
   MSMediaKeysConstructor = MSMediaKeys;
 }

package/src/compat/eme/custom_media_keys/old_webkit_media_keys.ts

@@ -110,15 +110,12 @@ class OldWebkitMediaKeySession
         if (this._key.indexOf("clearkey") >= 0) {
           const licenseTypedArray =
             license instanceof ArrayBuffer ? new Uint8Array(license) : license;
-          /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-          /* eslint-disable @typescript-eslint/no-unsafe-argument */
-          /* eslint-disable @typescript-eslint/no-unsafe-assignment */
+          // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           const json = JSON.parse(utf8ToStr(licenseTypedArray));
+          // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-argument
           const key = base64ToBytes(json.keys[0].k);
+          // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-argument
           const kid = base64ToBytes(json.keys[0].kid);
-          /* eslint-enable @typescript-eslint/no-unsafe-member-access */
-          /* eslint-enable @typescript-eslint/no-unsafe-argument */
-          /* eslint-enable @typescript-eslint/no-unsafe-assignment */
           resolve(this._vid.webkitAddKey(this._key, key, kid, /* sessionId */ ""));
         } else {
           resolve(this._vid.webkitAddKey(this._key, license, null, /* sessionId */ ""));

package/src/compat/eme/custom_media_keys/webkit_media_keys.ts

@@ -110,7 +110,6 @@ class WebkitMediaKeySession
 
   public update(license: BufferSource): Promise<void> {
     return new Promise((resolve, reject) => {
-      /* eslint-disable @typescript-eslint/no-unsafe-member-access */
       if (
         this._nativeSession === undefined ||
         this._nativeSession.update === undefined ||
@@ -127,14 +126,11 @@ class WebkitMediaKeySession
         } else {
           uInt8Arraylicense = new Uint8Array(license.buffer);
         }
-        /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-call
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
         resolve(this._nativeSession.update(uInt8Arraylicense));
-        /* eslint-enable @typescript-eslint/no-unsafe-member-access */
       } catch (err) {
         reject(err);
       }
-      /* eslint-enable @typescript-eslint/no-unsafe-member-access */
     });
   }
 
@@ -176,9 +172,8 @@ class WebkitMediaKeySession
         reject("No session to close.");
         return;
       }
-      /* eslint-disable @typescript-eslint/no-floating-promises */
+      // eslint-disable-next-line @typescript-eslint/no-floating-promises
       this._nativeSession.close();
-      /* eslint-enable @typescript-eslint/no-floating-promises */
       resolve();
     });
   }
@@ -192,11 +187,7 @@ class WebkitMediaKeySession
   }
 
   get sessionId(): string {
-    /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-    /* eslint-disable @typescript-eslint/no-unsafe-return */
     return this._nativeSession?.sessionId ?? "";
-    /* eslint-enable @typescript-eslint/no-unsafe-member-access */
-    /* eslint-enable @typescript-eslint/no-unsafe-return */
   }
 
   private _listenEvent(session: MediaKeySession): void {
@@ -206,9 +197,6 @@ class WebkitMediaKeySession
       this.trigger(evt.type, evt);
     };
 
-    /* eslint-disable @typescript-eslint/no-unsafe-call */
-    /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-    /* eslint-disable @typescript-eslint/no-unsafe-return */
     ["keymessage", "message", "keyadded", "ready", "keyerror", "error"].forEach((evt) => {
       session.addEventListener(evt, onEvent);
       session.addEventListener(`webkit${evt}`, onEvent);
@@ -222,9 +210,6 @@ class WebkitMediaKeySession
         },
       );
     };
-    /* eslint-disable @typescript-eslint/no-unsafe-return */
-    /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-    /* eslint-enable @typescript-eslint/no-unsafe-call */
   }
 }
 

package/src/compat/eme/custom_media_keys/webkit_media_keys_constructor.ts

@@ -26,8 +26,6 @@ interface IWebKitMediaKeysConstructor {
 
 let WebKitMediaKeysConstructor: undefined | IWebKitMediaKeysConstructor;
 
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
 const { WebKitMediaKeys } = globalScope as typeof globalThis & {
   WebKitMediaKeys?: IWebKitMediaKeysConstructor;
 };
@@ -35,13 +33,12 @@ const { WebKitMediaKeys } = globalScope as typeof globalThis & {
 if (
   WebKitMediaKeys !== undefined &&
   typeof WebKitMediaKeys.isTypeSupported === "function" &&
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   typeof WebKitMediaKeys.prototype.createSession === "function" &&
   typeof (HTMLMediaElement.prototype as IMediaElement).webkitSetMediaKeys === "function"
 ) {
   WebKitMediaKeysConstructor = WebKitMediaKeys;
 }
-/* eslint-enable @typescript-eslint/no-unsafe-assignment */
-/* eslint-enable @typescript-eslint/no-unsafe-member-access */
 
 export type { IWebKitMediaKeys };
 export { WebKitMediaKeysConstructor };

package/src/compat/eme/eme-api-implementation.ts

@@ -319,11 +319,9 @@ function defaultSetMediaKeys(
 ): Promise<unknown> {
   try {
     let ret: unknown;
-    /* eslint-disable @typescript-eslint/unbound-method */
     if (typeof elt.setMediaKeys === "function") {
       ret = elt.setMediaKeys(mediaKeys as MediaKeys);
     }
-    /* eslint-enable @typescript-eslint/unbound-method */
 
     // If we get in the following code, it means that no compat case has been
     // found and no standard setMediaKeys API exists. This case is particulary

package/src/compat/event_listeners.ts

@@ -176,20 +176,16 @@ function createCompatibleEventListener(
         (element as IEventEmitterLike).addEventListener(eventName, listener);
       } else {
         hasSetOnFn = true;
-        /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-        /* eslint-disable-next-line @typescript-eslint/no-explicit-any */
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
         (element as any)["on" + eventName] = listener;
-        /* eslint-enable @typescript-eslint/no-unsafe-member-access */
       }
       cancelSignal.register(() => {
         if (typeof element.removeEventListener === "function") {
           (element as IEventEmitterLike).removeEventListener(eventName, listener);
         }
         if (hasSetOnFn) {
-          /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-          /* eslint-disable-next-line @typescript-eslint/no-explicit-any */
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
           delete (element as any)["on" + eventName];
-          /* eslint-enable @typescript-eslint/no-unsafe-member-access */
         }
       });
     });

package/src/compat/has_mse_in_worker.ts

@@ -1,6 +1,6 @@
 const hasMseInWorker =
   typeof MediaSource === "function" &&
-  /* eslint-disable-next-line */
+  // eslint-disable-next-line
   (MediaSource as any).canConstructInDedicatedWorker === true;
 
 export default hasMseInWorker;

package/src/compat/is_codec_supported.ts

@@ -47,9 +47,7 @@ export default function isCodecSupported(mimeType: string): boolean {
     return false;
   }
 
-  /* eslint-disable @typescript-eslint/unbound-method */
   if (typeof MediaSource_.isTypeSupported === "function") {
-    /* eslint-enable @typescript-eslint/unbound-method */
     const cachedSupport = supportMap.get(mimeType);
     if (cachedSupport !== undefined) {
       return cachedSupport;

package/src/compat/is_debug_mode_enabled.ts

@@ -1,4 +1,4 @@
-/* eslint-disable-next-line @typescript-eslint/naming-convention */
+// eslint-disable-next-line @typescript-eslint/naming-convention
 declare const __RX_PLAYER_DEBUG_MODE__: boolean | undefined;
 
 /**

package/src/compat/is_vtt_cue.ts

@@ -25,7 +25,6 @@ import type { ICompatVTTCue } from "./browser_compatibility_types";
 export default function isVTTCue(
   cue: ICompatVTTCue | TextTrackCue,
 ): cue is ICompatVTTCue {
-  /* eslint-disable @typescript-eslint/no-unsafe-member-access */
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
   return typeof globalScope.VTTCue === "function" && cue instanceof globalScope.VTTCue;
-  /* eslint-enable @typescript-eslint/no-unsafe-member-access */
 }

package/src/compat/on_height_width_change.ts

@@ -26,9 +26,8 @@ export interface IResolution {
 }
 
 interface IResizeObserverConstructor {
-  /* eslint-disable @typescript-eslint/prefer-function-type */
+  // eslint-disable-next-line @typescript-eslint/prefer-function-type
   new (callback: IResizeObserverCallback): IResizeObserver;
-  /* eslint-enable @typescript-eslint/prefer-function-type */
 }
 
 interface IResizeObserver {

package/src/compat/patch_webkit_source_buffer.ts

@@ -20,29 +20,45 @@ import isNode from "../utils/is_node";
 import isNullOrUndefined from "../utils/is_null_or_undefined";
 import queueMicrotask from "../utils/queue_microtask";
 
-type IWebKitSourceBufferConstructor = new () => IWebKitSourceBuffer;
+interface IWebKitSourceBufferConstructor {
+  new (): IWebKitSourceBuffer;
+  prototype: IWebKitSourceBuffer;
+}
 
-interface IWebKitSourceBuffer {
-  append(data: ArrayBuffer): void;
+interface IWebKitSourceBuffer
+  extends EventEmitter<{
+    updatestart: Event;
+    update: Event;
+    updateend: Event;
+    error: Event | Error;
+  }> {
+  _emitUpdate?: (eventName: "error" | "update", val: unknown) => void;
+  appendBuffer?: (data: BufferSource) => void;
+  updating: boolean;
+  append(data: BufferSource): void;
   remove(from: number, to: number): void;
 }
 
 // TODO This is the last ugly side-effect here.
 // Either remove it or find the best way to implement that
 export default function patchWebkitSourceBuffer(): void {
-  /* eslint-disable @typescript-eslint/no-unsafe-member-access */
-  /* eslint-disable @typescript-eslint/no-unsafe-call */
-  /* eslint-disable @typescript-eslint/no-explicit-any */
   // old WebKit SourceBuffer implementation,
   // where a synchronous append is used instead of appendBuffer
   if (
     !isNode &&
-    !isNullOrUndefined((globalScope as any).WebKitSourceBuffer) &&
-    (globalScope as any).WebKitSourceBuffer.prototype.addEventListener === undefined
+    !isNullOrUndefined(
+      (
+        globalScope as typeof globalScope & {
+          WebKitSourceBuffer?: IWebKitSourceBufferConstructor;
+        }
+      ).WebKitSourceBuffer,
+    ) &&
+    (
+      globalScope as typeof globalScope & {
+        WebKitSourceBuffer: IWebKitSourceBufferConstructor;
+      }
+    ).WebKitSourceBuffer.prototype.addEventListener === undefined
   ) {
-    /* eslint-enable @typescript-eslint/no-explicit-any */
-
-    /* eslint-disable @typescript-eslint/no-unsafe-assignment */
     const sourceBufferWebkitRef: IWebKitSourceBufferConstructor = (
       globalScope as unknown as {
         WebKitSourceBuffer: IWebKitSourceBufferConstructor;
@@ -52,42 +68,45 @@ export default function patchWebkitSourceBuffer(): void {
 
     for (const fnName in EventEmitter.prototype) {
       if (EventEmitter.prototype.hasOwnProperty(fnName)) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        sourceBufferWebkitProto[fnName] = (EventEmitter.prototype as any)[fnName];
+        (sourceBufferWebkitProto as unknown as Record<string, unknown>)[fnName] = (
+          EventEmitter.prototype as unknown as Record<string, unknown>
+        )[fnName];
       }
     }
-    /* eslint-enable @typescript-eslint/no-unsafe-assignment */
 
-    sourceBufferWebkitProto._listeners = [];
+    (
+      sourceBufferWebkitProto as unknown as {
+        _listeners: Array<() => void>;
+      }
+    )._listeners = [];
 
-    sourceBufferWebkitProto._emitUpdate = function (eventName: string, val: unknown) {
+    sourceBufferWebkitProto._emitUpdate = function (
+      eventName: "error" | "update",
+      val: unknown,
+    ) {
       queueMicrotask(() => {
-        /* eslint-disable no-invalid-this */
-        this.trigger(eventName, val);
+        // @ts-expect-error: trigger is normally protected
+        this.trigger(eventName, val as Event);
         this.updating = false;
-        this.trigger("updateend");
-        /* eslint-enable no-invalid-this */
+        // @ts-expect-error: trigger is normally protected
+        this.trigger("updateend", new Event("updateend"));
       });
     };
 
-    sourceBufferWebkitProto.appendBuffer = function (data: unknown) {
-      /* eslint-disable no-invalid-this */
-      // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
+    sourceBufferWebkitProto.appendBuffer = function (data: BufferSource) {
       if (this.updating) {
         throw new Error("updating");
       }
-      this.trigger("updatestart");
+      // @ts-expect-error: trigger is normally protected
+      this.trigger("updatestart", new Event("updatestart"));
       this.updating = true;
       try {
         this.append(data);
       } catch (error) {
-        this._emitUpdate("error", error);
+        this._emitUpdate?.("error", error);
         return;
       }
-      this._emitUpdate("update");
-      /* eslint-enable no-invalid-this */
+      this._emitUpdate?.("update", new Event("update"));
     };
   }
-  /* eslint-enable @typescript-eslint/no-unsafe-member-access */
-  /* eslint-enable @typescript-eslint/no-unsafe-call */
 }

package/src/compat/remove_cue.ts

@@ -50,7 +50,7 @@ export default function removeCue(track: TextTrack, cue: TextTrackCue): void {
     track.mode = "hidden";
     try {
       track.removeCue(cue);
-    } catch (err) {
+    } catch (_err) {
       log.warn("Compat: Could not remove cue from text track.");
     }
     track.mode = trackMode;
@@ -58,7 +58,7 @@ export default function removeCue(track: TextTrack, cue: TextTrackCue): void {
   }
   try {
     track.removeCue(cue);
-  } catch (err) {
+  } catch (_err) {
     log.warn("Compat: Could not remove cue from text track.");
   }
 }