@unboundcx/video-sdk-client 2.0.11 → 2.0.12

package/VideoMeetingClient.js

@@ -829,6 +829,10 @@ export class VideoMeetingClient extends EventEmitter {
         profile.maxSpatialLayer,
         profile.maxTemporalLayer,
       );
+      // Keep the send-cap flag in sync on every level change so QualityMonitor
+      // discounts the BWE floor while we're intentionally throttled — even on
+      // the mid-call path that doesn't re-produce.
+      this.mediasoup.setSendBitrateCap(profile.sendMaxBitrate || 0);
     }
     if (this.remoteMedia) {
       this.remoteMedia.setLayerCeiling(

package/managers/MediasoupManager.js

@@ -38,6 +38,12 @@ export class MediasoupManager extends EventEmitter {
     // no cap. See videoQualityProfiles.js + RemoteMediaManager.setLayerCeiling.
     this.layerCeiling = { maxSpatialLayer: null, maxTemporalLayer: null };
 
+    // Active send bitrate cap (bps) from the data-saver profile, or 0 when
+    // uncapped. QualityMonitor reads this so it can ignore Chrome's BWE
+    // "available upload" floor while we're INTENTIONALLY sending low — a low
+    // estimate then means "no demand to probe higher", not "bad network".
+    this.sendBitrateCap = 0;
+
     // Initialize stats collector
     this.statsCollector = new StatsCollector(this.logger);
     this.virtualBackgroundStore = null; // Will be set externally
@@ -684,6 +690,9 @@ export class MediasoupManager extends EventEmitter {
         // gets the lion's share; lower layers get proportionally less.
         // (Screen share never reaches here.)
         const maxBitrate = Number(options.maxBitrate) || 0;
+        // Remember the active send cap so QualityMonitor can discount the BWE
+        // floor while we're intentionally throttled (data saver).
+        this.sendBitrateCap = maxBitrate;
         if (maxBitrate > 0 && produceOptions.encodings?.length) {
           // Relative weights per rid; we normalize over the layers actually
           // present so the allocated total always equals the budget whether
@@ -1101,6 +1110,21 @@ export class MediasoupManager extends EventEmitter {
     return this.producers.get("video") || this.producers.get("camera") || null;
   }
 
+  /**
+   * Set/clear the active send bitrate cap (bps; 0 = uncapped). Lets the
+   * mid-call data-saver path (setParameters, no re-produce) keep
+   * sendBitrateCap in sync so QualityMonitor discounts the BWE floor.
+   * @param {number} bps
+   */
+  setSendBitrateCap(bps) {
+    this.sendBitrateCap = Number(bps) || 0;
+  }
+
+  /** @returns {number} active send bitrate cap in bps (0 = uncapped) */
+  getSendBitrateCap() {
+    return this.sendBitrateCap || 0;
+  }
+
   /**
    * Ask the SFU to forward a lower simulcast layer of the given peer's
    * video producer to us. spatialLayer 0=low, 1=mid, 2=high.

package/managers/QualityMonitor.js

@@ -423,15 +423,27 @@ export class QualityMonitor {
     // counting that as a bad network wrongly marks audio-only participants
     // "poor" and triggers a disable-camera suggestion for a camera that's
     // already off. RTT + loss remain valid either way.
-    // Only count availMbps once we've seen a real reading (>0); some
-    // browsers report 0 transiently at session start.
+    //
+    // The SAME artifact happens under data saver: when we deliberately cap the
+    // send to e.g. 600kbps and a low-motion scene only uses ~30kbps, the BWE
+    // collapses toward what we're actually sending (no demand to probe higher),
+    // so availMbps reads ~0.1 even on a perfectly healthy link. So when a send
+    // cap is active we IGNORE the availMbps floor entirely and rely only on the
+    // encoder-bandwidth-limited ratio, which genuinely means "the encoder wants
+    // more bits than the link will give" — the real signal that survives a cap.
+    const sendCapped =
+      typeof this.mediasoupManager?.getSendBitrateCap === "function" &&
+      this.mediasoupManager.getSendBitrateCap() > 0;
+    const useAvailFloor = this._isSendingVideo() && !sendCapped;
     const bandwidthSeverity = !this._isSendingVideo()
       ? 0
-      : availMbps > 0 && availMbps <= this.thresholds.availMbpsCrit
+      : useAvailFloor && availMbps > 0 && availMbps <= this.thresholds.availMbpsCrit
         ? 2
         : bwLimitedRatio >= this.thresholds.encoderBwLimitedCritRatio
           ? 2
-          : availMbps > 0 && availMbps <= this.thresholds.availMbpsWarn
+          : useAvailFloor &&
+              availMbps > 0 &&
+              availMbps <= this.thresholds.availMbpsWarn
             ? 1
             : bwLimitedRatio >= this.thresholds.encoderBwLimitedWarnRatio
               ? 1

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@unboundcx/video-sdk-client",
-	"version": "2.0.11",
+	"version": "2.0.12",
 	"description": "Framework-agnostic WebRTC video meeting SDK powered by mediasoup",
 	"type": "module",
 	"main": "index.js",