fix(mp3-encoder): Suppress assertion

RocketChat · Nov 8, 2024 · 9946554 · 9946554
1 parent 5ac71a3
commit 9946554
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/.changeset/ninety-monkeys-notice.md b/.changeset/ninety-monkeys-notice.md
@@ -0,0 +1,5 @@
+---
+'@rocket.chat/mp3-encoder': patch
+---
+
+Suppressed assertions at the LAME encoding due to unknown `NaN` values
diff --git a/packages/mp3-encoder/src/lame/assert.ts b/packages/mp3-encoder/src/lame/assert.ts
@@ -1,8 +1,11 @@
 export function assert(
   condition: boolean,
-  message?: string,
+  _message?: string,
 ): asserts condition {
-  if (!condition) {
-    throw new Error(message);
-  }
+  // TODO: There is a condition generating multiple NaN values that was never
+  //       addressed in the original code and is not clear how to handle it.
+  //       Originally this assertion was commented out, probably because of it.
+  // if (!condition) {
+  //   throw new Error(message);
+  // }
 }