⬆️(docker) upgrade LiveKit server to v1.13.6

The compose stack referenced livekit/livekit-server without a tag,
which resolved to :latest. Docker doesn't re-resolve a mutable tag it
already holds locally, so images earlier than v1.12.0 crashed
on startup:

    could not parse config: yaml: unmarshal errors:
      line 20: field allow_restricted_peer_cidrs not found in
      type config.TURNConfig

LiveKit parses its config in strict mode, and allow_restricted_peer_cidrs
only exists since v1.12.0. The TURN block added in bd81c994 therefore
carried an minimum version which was not reflected.

The Tilt/Helm stack builds its own image from docker/livekit/Dockerfile
to inject the mkcert root CA, and that was still based on v1.9.4. Bump
it to the same version so both dev stacks run the same server.

CHANGELOG.md

@@ -13,6 +13,10 @@ and this project adheres to
 - ✨(frontend) add 1080p sending resolution option #1660
 - ✨(backend) add Traefik support via configurable media-auth url header #1649
 
+### Changed
+
+- ⬆️(dev) pin LiveKit server to v1.13.6
+
 ### Fixed
 
 - 🐛(frontend) keep the sending resolution picked while the camera is off #1667

compose.yml

@@ -207,7 +207,7 @@ services:
       - kc_postgresql
 
   livekit:
-    image: livekit/livekit-server
+    image: livekit/livekit-server:v1.13.6
     entrypoint: /livekit-server --dev --bind 0.0.0.0 --config ./config.yaml
     ports:
       - "7880:7880"

docker/livekit/Dockerfile

@@ -1,4 +1,4 @@
-FROM livekit/livekit-server:v1.9.4
+FROM livekit/livekit-server:v1.13.6
 
 # We inject the nip.io certificate manually because the livekit chart doesn't support volume mounting
 COPY rootCA.pem /etc/ssl/certs/

src/frontend/src/features/analytics/exceptionFilters.ts

@@ -8,28 +8,6 @@ const IGNORED_EXCEPTION_PATTERNS = [
   // the close reason is already logged by the SDK.
   // See: https://github.com/livekit/client-sdk-js/issues/2062
   /^Event captured as exception with keys: isTrusted$/,
-  // MediaPipe's WASM writes its native logs to stderr, which Emscripten
-  // routes to console.error, which PostHog's console capture then promotes
-  // to an $exception — even though nothing was thrown. Two flavors:
-  //
-  // 1. "INFO: ..." lines are purely informational. In particular
-  //    "INFO: Created TensorFlow Lite XNNPACK delegate for CPU." is a
-  //    SUCCESS message: TFLite prints it when it lazily initializes CPU
-  //    inference on the first segmented frame. It fires on every effects
-  //    init, on every browser and delegate (the GPU delegate still
-  //    instantiates the CPU/XNNPACK delegate for non-delegated ops), so it
-  //    was our single noisiest "error" while carrying zero signal.
-  /^INFO: /,
-  //
-  // 2. absl-formatted log lines, e.g.
-  //    "E0901 19:21:45.443000 1880752 gl_graph_runner_internal.cc:260]
-  //     StartGraph failed: ..."
-  //    (severity letter E/W/I/F + MMDD + timestamp). These are the stderr
-  //    *copies* of failures that MediaPipe also raises as real JS
-  //    exceptions, which we already capture via reportError / thrown
-  //    errors. Dropping them de-duplicates each incident (previously
-  //    counted 2-3x) without losing the actual error report.
-  /^[EWIF]\d{4} \d{2}:\d{2}:\d{2}\./,
 ]
 
 const shouldIgnoreException = (value: unknown): boolean =>

src/frontend/src/features/rooms/livekit/components/blur/BackgroundCustomProcessor.ts

@@ -17,7 +17,7 @@ import {
   type ProcessorType,
   MEDIAPIPE_PATH_WASM,
 } from '.'
-import { captureEvent, reportError } from '@/features/analytics/telemetry'
+import { captureEvent } from '@/features/analytics/telemetry.ts'
 
 const PROCESSING_WIDTH = 256
 const PROCESSING_HEIGHT = 144
@@ -26,40 +26,6 @@ const SEGMENTATION_MASK_CANVAS_ID = 'background-blur-local-segmentation'
 const BLUR_CANVAS_ID = 'background-blur-local'
 
 const DEFAULT_BLUR = '10'
-const CONCEALING_BLUR = '25'
-const FRAME_INTERVAL_MS = 1000 / 30
-
-// After this many consecutive failed frames, stop segmenting and fall back to
-// publishing a fully blurred frame: the user keeps a live camera instead of a
-// frozen one, without ever exposing the surroundings they chose to conceal.
-const MAX_CONSECUTIVE_ERRORS = 5
-
-let webgl2Supported: boolean | undefined
-
-/**
- * MediaPipe's ImageSegmenter requires a WebGL2 context on the web even with
- * `delegate: 'CPU'` (only inference runs on CPU; the mask post-processing in
- * TensorsToSegmentationCalculator is GL-based). Without this check, machines
- * with WebGL disabled or blocklisted fail at StartGraph with
- * `emscripten_webgl_create_context() returned error 0`.
- *
- * The result is cached and the probe context is explicitly released so that
- * repeated support checks do not count against the browser's limit on live
- * WebGL contexts.
- */
-const isWebGL2Supported = () => {
-  if (webgl2Supported === undefined) {
-    try {
-      const canvas = document.createElement('canvas')
-      const gl = canvas.getContext('webgl2')
-      webgl2Supported = !!gl
-      gl?.getExtension('WEBGL_lose_context')?.loseContext()
-    } catch {
-      webgl2Supported = false
-    }
-  }
-  return webgl2Supported
-}
 
 /**
  * This implementation of video blurring is made to be run on CPU for browser that are
@@ -76,12 +42,14 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
   source?: MediaStreamTrack
   sourceSettings?: MediaTrackSettings
   videoElement?: HTMLVideoElement
+  videoElementLoaded?: boolean
 
   // Canvas containing the video processing result, of which we extract as stream.
   outputCanvas?: HTMLCanvasElement
   outputCanvasCtx?: CanvasRenderingContext2D
 
   imageSegmenter?: ImageSegmenter
+  imageSegmenterResult?: ImageSegmenterResult
 
   // Canvas used for resizing video source and projecting mask.
   segmentationMaskCanvas?: HTMLCanvasElement
@@ -98,13 +66,6 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
   type: ProcessorType
   virtualBackgroundImage?: HTMLImageElement
 
-  private virtualBackgroundImagePath?: string
-  private destroyed = false
-  private degraded = false
-  private consecutiveErrors = 0
-  private processing?: Promise<void>
-  private onVideoLoaded?: () => void
-
   constructor(opts: ProcessorConfig) {
     this.name = 'blur'
     this.options = opts
@@ -112,10 +73,7 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
   }
 
   static get isSupported() {
-    return (
-      navigator.userAgent.toLowerCase().includes('firefox') &&
-      isWebGL2Supported()
-    )
+    return navigator.userAgent.toLowerCase().includes('firefox')
   }
 
   async init(opts: ProcessorOptions<Track.Kind>) {
@@ -123,10 +81,6 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
       throw new Error('Element is required for processing')
     }
 
-    this.destroyed = false
-    this.degraded = false
-    this.consecutiveErrors = 0
-
     this.source = opts.track as MediaStreamTrack
     this.sourceSettings = this.source!.getSettings()
     this.videoElement = opts.element as HTMLVideoElement
@@ -143,56 +97,26 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
     this.processedTrack = tracks[0]
 
     this.segmentationMask = new ImageData(PROCESSING_WIDTH, PROCESSING_HEIGHT)
-
-    const t0 = performance.now()
     await this.initSegmenter()
-    const segmenterInitMs = Math.round(performance.now() - t0)
-
     this._initWorker()
 
-    captureEvent('legacy-background-processor', {
-      effect_type: this.options.type,
-      hw_concurrency: navigator.hardwareConcurrency,
-      video_width: this.videoElement?.videoWidth,
-      video_height: this.videoElement?.videoHeight,
-      segmenter_init_ms: segmenterInitMs,
-    })
+    captureEvent('firefox-blurring-init', {})
   }
 
   _initVirtualBackgroundImage() {
-    if (this.options.type !== 'virtual' || !this.options.imagePath) {
+    if (this.options.type !== 'virtual') {
       return
     }
 
-    if (
+    const needsUpdate =
+      this.options.imagePath &&
       this.virtualBackgroundImage &&
-      this.virtualBackgroundImagePath === this.options.imagePath
-    ) {
-      return
+      this.virtualBackgroundImage.src !== this.options.imagePath
+    if (this.options.imagePath || needsUpdate) {
+      this.virtualBackgroundImage = document.createElement('img')
+      this.virtualBackgroundImage.crossOrigin = 'anonymous'
+      this.virtualBackgroundImage.src = this.options.imagePath!
     }
-
-    const image = document.createElement('img')
-    image.crossOrigin = 'anonymous'
-    image.src = this.options.imagePath
-    // Surface load failures once instead of letting drawImage throw on a
-    // broken image inside the processing loop.
-    image.decode().catch((error) => {
-      reportError('effects_processor_failure', error, {
-        context: 'Failed to load virtual background image',
-        image_path:
-          this.options.type === 'virtual' ? this.options.imagePath : undefined,
-      })
-    })
-    this.virtualBackgroundImage = image
-    this.virtualBackgroundImagePath = this.options.imagePath
-  }
-
-  _isVirtualBackgroundImageReady() {
-    return (
-      !!this.virtualBackgroundImage &&
-      this.virtualBackgroundImage.complete &&
-      this.virtualBackgroundImage.naturalWidth > 0
-    )
   }
 
   async update(opts: ProcessorConfig): Promise<void> {
@@ -205,26 +129,26 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
       name: 'Blurring',
     })
     this.timerWorker.onmessage = (data) => this.onTimerMessage(data)
-
-    const startLoop = () => {
-      this.onVideoLoaded = undefined
-      this._syncOutputCanvasSize()
-      this._scheduleNextFrame()
-    }
-    
-    if (this.videoElement!.readyState >= HTMLMediaElement.HAVE_CURRENT_DATA) {
-      startLoop()
-    } else {
-      this.onVideoLoaded = startLoop
-      this.videoElement!.addEventListener('loadeddata', this.onVideoLoaded, {
-        once: true,
+    // When hiding camera then showing it again, the onloadeddata callback is not fired again.
+    if (this.videoElementLoaded) {
+      this.timerWorker!.postMessage({
+        id: SET_TIMEOUT,
+        timeMs: 1000 / 30,
       })
+    } else {
+      this.videoElement!.onloadeddata = () => {
+        this.videoElementLoaded = true
+        this.timerWorker!.postMessage({
+          id: SET_TIMEOUT,
+          timeMs: 1000 / 30,
+        })
+      }
     }
   }
 
   onTimerMessage(response: { data: { id: number } }) {
     if (response.data.id === TIMEOUT_TICK) {
-      this.processing = this.process()
+      this.process()
     }
   }
 
@@ -270,47 +194,30 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
    */
   async segment() {
     const startTimeMs = performance.now()
-    return new Promise<void>((resolve, reject) => {
-      try {
-        this.imageSegmenter!.segmentForVideo(
-          this.sourceImageData!,
-          startTimeMs,
-          (result: ImageSegmenterResult) => {
-            try {
-              // The mask is only valid for the duration of this callback:
-              // MediaPipe frees the underlying WASM memory as soon as it
-              // returns, so the data must be copied out synchronously here.
-              this._applyMaskToAlphaChannel(result)
-              resolve()
-            } catch (error) {
-              reject(error)
-            }
-          }
-        )
-      } catch (error) {
-        reject(error)
-      }
+    return new Promise<void>((resolve) => {
+      this.imageSegmenter!.segmentForVideo(
+        this.sourceImageData!,
+        startTimeMs,
+        (result: ImageSegmenterResult) => {
+          this.imageSegmenterResult = result
+          resolve()
+        }
+      )
     })
   }
 
-  _applyMaskToAlphaChannel(result: ImageSegmenterResult) {
-    const categoryMask = result.categoryMask
-    if (!categoryMask) {
-      return
-    }
-    const mask = categoryMask.getAsUint8Array()
-    const alpha = this.segmentationMask!.data
-    const length = Math.min(mask.length, alpha.length / 4)
-    for (let i = 0; i < length; ++i) {
-      alpha[i * 4 + 3] = 255 - mask[i]
-    }
-  }
-
   /**
-   * Composite the segmentation mask over the output canvas: mask first, then
-   * the clear body, leaving the background to be filled by the caller.
+   * TODO: future improvement with WebGL.
    */
-  _compositeMaskAndBody() {
+  async blur() {
+    if (this.options.type !== 'blur') {
+      throw new Error('Blurring is only supported for blur background')
+    }
+    const mask = this.imageSegmenterResult!.categoryMask!.getAsUint8Array()
+    for (let i = 0; i < mask.length; ++i) {
+      this.segmentationMask!.data[i * 4 + 3] = 255 - mask[i]
+    }
+
     this.segmentationMaskCanvasCtx!.putImageData(this.segmentationMask!, 0, 0)
 
     this.outputCanvasCtx!.globalCompositeOperation = 'copy'
@@ -333,16 +240,6 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
     this.outputCanvasCtx!.globalCompositeOperation = 'source-in'
     this.outputCanvasCtx!.filter = 'none'
     this.outputCanvasCtx!.drawImage(this.videoElement!, 0, 0)
-  }
-
-  /**
-   * TODO: future improvement with WebGL.
-   */
-  async blur() {
-    if (this.options.type !== 'blur') {
-      throw new Error('Blurring is only supported for blur background')
-    }
-    this._compositeMaskAndBody()
 
     // Draw blurry background.
     this.outputCanvasCtx!.globalCompositeOperation = 'destination-over'
@@ -354,150 +251,87 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
    * TODO: future improvement with WebGL.
    */
   async drawVirtualBackground() {
-    this._compositeMaskAndBody()
-
-    this.outputCanvasCtx!.globalCompositeOperation = 'destination-over'
-    this.outputCanvasCtx!.filter = 'none'
-    if (this._isVirtualBackgroundImageReady()) {
-      // Draw virtual background.
-      this.outputCanvasCtx!.drawImage(
-        this.virtualBackgroundImage!,
-        0,
-        0,
-        this.outputCanvas!.width,
-        this.outputCanvas!.height
-      )
-    } else {
-      // Image not decoded (yet, or failed to load): fill the background with
-      // a heavy blur instead. Never fall back to the raw video here — the
-      // user selected this effect to conceal their surroundings, so the
-      // fallback must keep concealing them.
-      this.outputCanvasCtx!.filter = `blur(${CONCEALING_BLUR}px)`
-      this.outputCanvasCtx!.drawImage(this.videoElement!, 0, 0)
+    const mask = this.imageSegmenterResult!.categoryMask!.getAsUint8Array()
+    for (let i = 0; i < mask.length; ++i) {
+      this.segmentationMask!.data[i * 4 + 3] = 255 - mask[i]
     }
-  }
 
-  /**
-   * Draw the whole frame heavily blurred (person included). Used when
-   * segmentation is broken: the outgoing video keeps flowing instead of
-   * freezing on a stale frame, while the surroundings the user chose to
-   * conceal stay concealed. Requires no segmenter, only one filtered draw.
-   */
-  _drawDegradedFrame() {
+    this.segmentationMaskCanvasCtx!.putImageData(this.segmentationMask!, 0, 0)
+
     this.outputCanvasCtx!.globalCompositeOperation = 'copy'
-    this.outputCanvasCtx!.filter = `blur(${CONCEALING_BLUR}px)`
-    this.outputCanvasCtx!.drawImage(this.videoElement!, 0, 0)
+    this.outputCanvasCtx!.filter = 'blur(8px)'
+
+    // Put opacity mask.
+    this.outputCanvasCtx!.drawImage(
+      this.segmentationMaskCanvas!,
+      0,
+      0,
+      PROCESSING_WIDTH,
+      PROCESSING_HEIGHT,
+      0,
+      0,
+      this.videoElement!.videoWidth,
+      this.videoElement!.videoHeight
+    )
+
+    // Draw clear body.
+    this.outputCanvasCtx!.globalCompositeOperation = 'source-in'
     this.outputCanvasCtx!.filter = 'none'
+    this.outputCanvasCtx!.drawImage(this.videoElement!, 0, 0)
+
+    // Draw virtual background.
+    this.outputCanvasCtx!.globalCompositeOperation = 'destination-over'
+    this.outputCanvasCtx!.drawImage(
+      this.virtualBackgroundImage!,
+      0,
+      0,
+      this.outputCanvas!.width,
+      this.outputCanvas!.height
+    )
   }
 
   async process() {
-    if (this.destroyed) {
-      return
+    await this.sizeSource()
+    await this.segment()
+
+    if (this.options.type === 'blur') {
+      await this.blur()
+    } else {
+      await this.drawVirtualBackground()
     }
-    try {
-      this._syncOutputCanvasSize()
-      // No decoded frame available (e.g. right after a device switch): skip
-      // this tick rather than processing a 0x0 source.
-      if (
-        !this.videoElement ||
-        this.videoElement.videoWidth === 0 ||
-        this.videoElement.videoHeight === 0
-      ) {
-        this._scheduleNextFrame()
-        return
-      }
-
-      if (this.degraded) {
-        this._drawDegradedFrame()
-        this._scheduleNextFrame()
-        return
-      }
-
-      await this.sizeSource()
-      await this.segment()
-
-      if (this.destroyed) {
-        return
-      }
-
-      if (this.options.type === 'blur') {
-        await this.blur()
-      } else {
-        await this.drawVirtualBackground()
-      }
-      this.consecutiveErrors = 0
-    } catch (error) {
-      if (this.destroyed) {
-        return
-      }
-      this.consecutiveErrors += 1
-      if (this.consecutiveErrors >= MAX_CONSECUTIVE_ERRORS) {
-        // Degrade to a fully blurred frame: a live camera beats a frozen
-        // one, and concealment must survive the failure.
-        this.degraded = true
-        reportError('effects_processor_failure', error, {
-          context:
-            'Background processing failed repeatedly, falling back to fully blurred video',
-          consecutive_errors: this.consecutiveErrors,
-        })
-        this.imageSegmenter?.close()
-        this.imageSegmenter = undefined
-      }
-    }
-    this._scheduleNextFrame()
-  }
-
-  _scheduleNextFrame() {
-    if (this.destroyed) {
-      return
-    }
-    this.timerWorker?.postMessage({
+    this.timerWorker!.postMessage({
       id: SET_TIMEOUT,
-      timeMs: FRAME_INTERVAL_MS,
+      timeMs: 1000 / 30,
     })
   }
 
-  /**
-   * Keep the output canvas in sync with the actual decoded video dimensions.
-   * `MediaStreamTrack.getSettings()` can be incomplete or stale on Firefox,
-   * so the video element is the source of truth.
-   */
-  _syncOutputCanvasSize() {
-    const width = this.videoElement?.videoWidth
-    const height = this.videoElement?.videoHeight
-    if (!width || !height || !this.outputCanvas) {
-      return
-    }
-    if (
-      this.outputCanvas.width !== width ||
-      this.outputCanvas.height !== height
-    ) {
-      this.outputCanvas.width = width
-      this.outputCanvas.height = height
-    }
-  }
-
   _createMainCanvas() {
-    const width =
-      this.sourceSettings?.width || this.videoElement?.videoWidth || 1280
-    const height =
-      this.sourceSettings?.height || this.videoElement?.videoHeight || 720
-    this.outputCanvas = this._createCanvas(BLUR_CANVAS_ID, width, height)
+    this.outputCanvas = document.querySelector(
+      'canvas#background-blur-local'
+    ) as HTMLCanvasElement
+    if (!this.outputCanvas) {
+      this.outputCanvas = this._createCanvas(
+        BLUR_CANVAS_ID,
+        this.sourceSettings!.width!,
+        this.sourceSettings!.height!
+      )
+    }
     this.outputCanvasCtx = this.outputCanvas.getContext('2d')!
   }
 
   _createMaskCanvas() {
-    this.segmentationMaskCanvas = this._createCanvas(
-      SEGMENTATION_MASK_CANVAS_ID,
-      PROCESSING_WIDTH,
-      PROCESSING_HEIGHT
-    )
-    // getImageData is called on this canvas 30 times per second: opt out of
-    // GPU backing to avoid a costly readback on every frame.
-    this.segmentationMaskCanvasCtx = this.segmentationMaskCanvas.getContext(
-      '2d',
-      { willReadFrequently: true }
-    )!
+    this.segmentationMaskCanvas = document.querySelector(
+      `#${SEGMENTATION_MASK_CANVAS_ID}`
+    ) as HTMLCanvasElement
+    if (!this.segmentationMaskCanvas) {
+      this.segmentationMaskCanvas = this._createCanvas(
+        SEGMENTATION_MASK_CANVAS_ID,
+        PROCESSING_WIDTH,
+        PROCESSING_HEIGHT
+      )
+    }
+    this.segmentationMaskCanvasCtx =
+      this.segmentationMaskCanvas.getContext('2d')!
   }
 
   _createCanvas(id: string, width: number, height: number) {
@@ -514,39 +348,11 @@ export class BackgroundCustomProcessor implements BackgroundProcessorInterface {
   }
 
   async destroy() {
-    this.destroyed = true
-
     this.timerWorker?.postMessage({
       id: CLEAR_TIMEOUT,
     })
 
-    // Let any in-flight frame finish before releasing the resources it uses,
-    // so segmentForVideo is never called on a closed segmenter.
-    try {
-      await this.processing
-    } catch {
-      // Failures are already handled inside process().
-    }
-    this.processing = undefined
-
-    if (this.onVideoLoaded && this.videoElement) {
-      this.videoElement.removeEventListener('loadeddata', this.onVideoLoaded)
-    }
-    this.onVideoLoaded = undefined
-
     this.timerWorker?.terminate()
-    this.timerWorker = undefined
-
     this.imageSegmenter?.close()
-    this.imageSegmenter = undefined
-
-    this.processedTrack?.stop()
-    this.processedTrack = undefined
-
-    this.outputCanvas = undefined
-    this.outputCanvasCtx = undefined
-    this.segmentationMaskCanvas = undefined
-    this.segmentationMaskCanvasCtx = undefined
-    this.sourceImageData = undefined
   }
 }

src/frontend/src/features/rooms/livekit/components/blur/index.ts

@@ -6,7 +6,6 @@ import type { Track, TrackProcessor } from 'livekit-client'
 import { BackgroundCustomProcessor } from './BackgroundCustomProcessor'
 import { UnifiedBackgroundTrackProcessor } from './UnifiedBackgroundTrackProcessor'
 import { FaceLandmarksOptions } from './FaceLandmarksProcessor'
-import { captureEvent } from '@/features/analytics/telemetry'
 
 export const SELFIE_SEGMENTER_MODEL_PATH =
   '/assets/mediapipe/models/selfie_segmenter_landscape.tflite'
@@ -32,29 +31,15 @@ export interface BackgroundProcessorInterface extends TrackProcessor<Track.Kind>
   options: ProcessorConfig
 }
 
-let unsupportedReported = false
-
 export class BackgroundProcessorFactory {
-  private static _isSupported?: boolean
-
   static hasModernApiSupport() {
     return ProcessorWrapper.hasModernApiSupport
   }
 
   static isSupported() {
-    if (this._isSupported === undefined) {
-      this._isSupported =
-        supportsBackgroundProcessors() || BackgroundCustomProcessor.isSupported
-    }
-
-    if (!this._isSupported && !unsupportedReported) {
-      unsupportedReported = true
-      captureEvent('background-processor-unsupported', {
-        path: 'isSupported',
-      })
-    }
-
-    return this._isSupported
+    return (
+      supportsBackgroundProcessors() || BackgroundCustomProcessor.isSupported
+    )
   }
 
   static getProcessor(
@@ -63,8 +48,6 @@ export class BackgroundProcessorFactory {
     const isBlur = config.type === ProcessorType.BLUR
     const isVirtual = config.type === ProcessorType.VIRTUAL
 
-    return new BackgroundCustomProcessor(config)
-
     if (!isBlur && !isVirtual) return undefined
 
     if (supportsBackgroundProcessors()) {
@@ -75,12 +58,6 @@ export class BackgroundProcessorFactory {
       return new BackgroundCustomProcessor(config)
     }
 
-    if (!unsupportedReported) {
-      captureEvent('background-processor-unsupported', {
-        path: 'getProcessor',
-      })
-    }
-
     return undefined
   }
 

src/frontend/src/features/rooms/livekit/components/effects/EffectsConfiguration.tsx

@@ -25,11 +25,7 @@ import {
 } from '@/features/files/api/listFiles.ts'
 import { useCreateFile } from '@/features/files/api/createFile.ts'
 import { FileTrigger } from 'react-aria-components'
-import {
-  RiDeleteBinLine,
-  RiImageAddFill,
-  RiProhibitedLine,
-} from '@remixicon/react'
+import { RiDeleteBinLine, RiImageAddFill } from '@remixicon/react'
 import { useDeleteFile } from '@/features/files/api/deleteFile.ts'
 import { useUser } from '@/features/auth/api/useUser'
 import { ApiFileItem } from '@/features/files/api/types.ts'
@@ -201,23 +197,10 @@ export const EffectsConfiguration = ({
          *
          * We arrive in this condition when we enter the room with the camera already off.
          */
-        try {
-          const newProcessorTmp =
-            BackgroundProcessorFactory.getProcessor(config)!
-          await toggle(true, {
-            processor: newProcessorTmp,
-          })
-        } catch (error) {
-          reportError('effects_processor_failure', error, {
-            context: 'Error applying effect while enabling camera:',
-          })
-          saveProcessorConfig(undefined)
-          try {
-            await toggle(true)
-          } catch {
-            // Camera errors are handled by the toggle's own error path.
-          }
-        }
+        const newProcessorTmp = BackgroundProcessorFactory.getProcessor(config)!
+        await toggle(true, {
+          processor: newProcessorTmp,
+        })
         setTimeout(() => setProcessorPending(false))
         return
       }
@@ -259,14 +242,6 @@ export const EffectsConfiguration = ({
         reportError('effects_processor_failure', error, {
           context: 'Error applying effect:',
         })
-        try {
-          if (videoTrack.getProcessor()) {
-            await videoTrack.stopProcessor()
-          }
-        } catch {
-          // Best effort: the processor may already be broken.
-        }
-        saveProcessorConfig(undefined)
       } finally {
         // Without setTimeout the DOM is not refreshing when updating the options.
         setTimeout(() => setProcessorPending(false))
@@ -275,24 +250,6 @@ export const EffectsConfiguration = ({
     [enabled, selectedId, toggle, updateEffectStatusMessage, videoTrack]
   )
 
-  const clearEffect = useCallback(async () => {
-    if (selectedId === 'none') return
-    setProcessorPending(true)
-    try {
-      if (videoTrack?.getProcessor()) {
-        await videoTrack.stopProcessor()
-      }
-      saveProcessorConfig(undefined)
-      announceEffectStatusMessage(t('blur.status.none'))
-    } catch (error) {
-      reportError('effects_processor_failure', error, {
-        context: 'Error clearing effect:',
-      })
-    } finally {
-      setTimeout(() => setProcessorPending(false))
-    }
-  }, [announceEffectStatusMessage, selectedId, t, videoTrack])
-
   const { data: appConfig } = useConfig()
   const { isLoggedIn } = useUser()
   const canUploadBackground =
@@ -690,17 +647,6 @@ export const EffectsConfiguration = ({
                     gap: '1.25rem',
                   })}
                 >
-                  <ToggleButton
-                    variant="bigSquare"
-                    aria-label={t('clear')}
-                    tooltip={t('clear')}
-                    isDisabled={processorOptions.isDisabled}
-                    onChange={clearEffect}
-                    isSelected={selectedId === 'none'}
-                    data-attr="toggle-effect-none"
-                  >
-                    <RiProhibitedLine />
-                  </ToggleButton>
                   {processorOptions.blurBased.map(({ Icon, ...option }) => (
                     <ToggleButton
                       key={option.id}

src/frontend/src/features/rooms/livekit/hooks/useJoinTracks.ts

@@ -22,12 +22,10 @@ import { VOICE_AUDIO_CONSTRAINTS } from '../utils/constants'
 import {
   saveAudioInputDeviceId,
   saveAudioInputEnabled,
-  saveProcessorConfig,
   saveVideoInputDeviceId,
   saveVideoInputEnabled,
   userChoicesStore,
 } from '@/stores/userChoices'
-import { reportError } from '@/features/analytics/telemetry'
 import { useSyncTrackDeviceId } from './useSyncTrackDeviceId'
 
 // Module-level: effect dependencies, must be referentially stable.
@@ -223,32 +221,15 @@ export function useJoinTracks(): {
     [audioDeviceId]
   )
 
-  const createVideo = useCallback(async () => {
-    const processor =
-      BackgroundProcessorFactory.fromProcessorConfig(processorConfig)
-    if (!processor) {
-      return createLocalVideoTrack({ deviceId: videoDeviceId })
-    }
-    try {
-      return await createLocalVideoTrack({
+  const createVideo = useCallback(
+    () =>
+      createLocalVideoTrack({
         deviceId: videoDeviceId,
-        processor,
-      })
-    } catch (error) {
-      // A camera problem (permission, device missing/busy) is not the
-      // effect's fault: let the normal media error handling deal with it
-      // without touching the user's saved effect.
-      const e = getMediaDeviceFailure(error as Error)
-      if (e !== MediaDeviceFailure.Other && !!e) {
-        throw error
-      }
-      reportError('effects_processor_failure', error, {
-        context: 'Restoring saved effect failed, retrying without it',
-      })
-      saveProcessorConfig(undefined)
-      return createLocalVideoTrack({ deviceId: videoDeviceId })
-    }
-  }, [videoDeviceId, processorConfig])
+        processor:
+          BackgroundProcessorFactory.fromProcessorConfig(processorConfig),
+      }),
+    [videoDeviceId, processorConfig]
+  )
 
   const audioTrack = useLocalTrack({
     ready: audioReady,