From 234fafa1c324239bbccaadc47ee96507e35bf93a Mon Sep 17 00:00:00 2001 From: fkwp Date: Thu, 26 Mar 2026 16:26:53 +0100 Subject: [PATCH] linting --- src/UrlParams.ts | 2 +- src/livekit/NoiseSuppressionTransformer.ts | 13 +++++-------- src/livekit/audioTrackNoiseSuppressionSync.ts | 2 +- src/livekit/useNoiseSuppressionTransformer.ts | 4 ++-- 4 files changed, 9 insertions(+), 12 deletions(-) diff --git a/src/UrlParams.ts b/src/UrlParams.ts index 8cd72fa5..752be948 100644 --- a/src/UrlParams.ts +++ b/src/UrlParams.ts @@ -518,7 +518,7 @@ export const computeUrlParams = (search = "", hash = ""): UrlParams => { noiseSuppression: parser.getFlagParam("noiseSuppression", true), echoCancellation: parser.getFlagParam("echoCancellation", true), noiseSuppressionEnabled: parser.getFlagParam("noiseSuppressionEnabled"), - noiseSuppressionLevel: (() => { + noiseSuppressionLevel: ((): number | undefined => { const val = parseInt(parser.getParam("noiseSuppressionLevel") ?? "", 10); return isNaN(val) ? undefined : val / 100; })(), diff --git a/src/livekit/NoiseSuppressionTransformer.ts b/src/livekit/NoiseSuppressionTransformer.ts index cbae0bb7..2ff1d291 100644 --- a/src/livekit/NoiseSuppressionTransformer.ts +++ b/src/livekit/NoiseSuppressionTransformer.ts @@ -22,10 +22,7 @@ export class NoiseSuppressionTransformer { * @param level - Noise reduction level (0-1) * @param enabled - Whether noise suppression is enabled */ - public async initialize( - level: number = 0.75, - enabled: boolean = true, - ): Promise { + public initialize(level: number = 0.75, enabled: boolean = true): void { if (this.initialized) { return; } @@ -106,12 +103,12 @@ export class NoiseSuppressionTransformer { } try { - this.processor.setEnabled(enabled); + void this.processor.setEnabled(enabled); logger.log( `[NoiseSuppressionTransformer] Noise suppression ${enabled ? "enabled" : "disabled"}`, ); // Log processor state for debugging - const processorState = (this.processor as any).enabled; + const processorState = (this.processor as { enabled?: boolean }).enabled; logger.debug( `[NoiseSuppressionTransformer] Processor internal state: enabled=${processorState}`, ); @@ -131,8 +128,8 @@ export class NoiseSuppressionTransformer { try { // Note: DeepFilterNoiseFilterProcessor may have a destroy method // Call it if available - if (typeof (this.processor as any).destroy === "function") { - (this.processor as any).destroy(); + if (typeof (this.processor as { destroy?: () => void }).destroy === "function") { + (this.processor as { destroy: () => void }).destroy(); } } catch (error) { logger.error("[NoiseSuppressionTransformer] Cleanup failed:", error); diff --git a/src/livekit/audioTrackNoiseSuppressionSync.ts b/src/livekit/audioTrackNoiseSuppressionSync.ts index 1665efdd..c00423b7 100644 --- a/src/livekit/audioTrackNoiseSuppressionSync.ts +++ b/src/livekit/audioTrackNoiseSuppressionSync.ts @@ -5,10 +5,10 @@ SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-Element-Commercial Please see LICENSE in the repository root for full details. */ -import type { LocalAudioTrack } from "livekit-client"; import { combineLatest } from "rxjs"; import { logger } from "matrix-js-sdk/lib/logger"; +import type { LocalAudioTrack } from "livekit-client"; import { noiseSuppressionEnabled, noiseSuppressionLevel, diff --git a/src/livekit/useNoiseSuppressionTransformer.ts b/src/livekit/useNoiseSuppressionTransformer.ts index 15e7fa68..d9b92dd1 100644 --- a/src/livekit/useNoiseSuppressionTransformer.ts +++ b/src/livekit/useNoiseSuppressionTransformer.ts @@ -32,7 +32,7 @@ export const useNoiseSuppressionTransformer = // Initialize with current settings void transformerRef.current.initialize(levelValue, enabledValue); } - }, []); + }, [enabledValue, levelValue]); // Sync enabled state when setting changes useEffect(() => { @@ -50,7 +50,7 @@ export const useNoiseSuppressionTransformer = // Cleanup on unmount useEffect(() => { - return () => { + return (): void => { if (transformerRef.current) { transformerRef.current.destroy(); }