@unboundcx/video-sdk-client 2.0.8 → 2.0.9

package/managers/StatsCollector.js

@@ -11,6 +11,11 @@ export class StatsCollector {
     this.tracks = new Map(); // trackId -> { participantId, kind }
     this.onStatsCallback = null; // Host-app callback (UI display)
     this._internalCallbacks = []; // SDK-internal subscribers (QualityMonitor)
+    // Previous cumulative byte counts + timestamp per transport, so we can
+    // turn the monotonic bytesSent/bytesReceived counters into an actual
+    // throughput RATE (kbps) instead of a meaningless running total. Keyed by
+    // transportType ('send' | 'recv'). See calculateBandwidth().
+    this._prevBytes = { send: null, recv: null };
   }
 
   /**
@@ -214,37 +219,60 @@ export class StatsCollector {
   }
 
   /**
-   * Calculate bandwidth from stats
+   * Calculate actual throughput from stats.
+   *
+   * bytesSent / bytesReceived are MONOTONIC cumulative counters (total since
+   * the connection started), so summing them yields a meaningless ever-growing
+   * number. To get a real RATE we diff the current total against the previous
+   * sample and divide by the elapsed time. Returns both kbps (for display) and
+   * Mbps (back-compat with existing consumers). The first sample per transport
+   * has no baseline, so its rate is 0 until the next cycle establishes a delta.
+   *
+   * `upload`/`download` (Mbps) are kept for back-compat; `uploadKbps`/
+   * `downloadKbps` are the values the quality panel shows.
    */
   calculateBandwidth(stats, transportType) {
     const bandwidth = {
-      upload: 0,
-      download: 0,
+      upload: 0, // Mbps (back-compat)
+      download: 0, // Mbps (back-compat)
+      uploadKbps: 0,
+      downloadKbps: 0,
       unit: 'Mbps',
     };
 
+    // Sum cumulative bytes across all RTP streams for this transport, and take
+    // the freshest report timestamp as "now" (RTCStats timestamps are in ms).
+    let totalBytes = 0;
+    let ts = 0;
+    const wantType = transportType === 'send' ? 'outbound-rtp' : 'inbound-rtp';
+    const byteField = transportType === 'send' ? 'bytesSent' : 'bytesReceived';
     for (const report of stats.values()) {
+      if (report.type !== wantType) continue;
+      totalBytes += Number(report[byteField]) || 0;
+      if (report.timestamp && report.timestamp > ts) ts = report.timestamp;
+    }
+    if (!ts) ts = Date.now();
+
+    const prev = this._prevBytes[transportType];
+    this._prevBytes[transportType] = { bytes: totalBytes, ts };
+
+    // Need a prior sample to compute a rate. Guard against counter resets
+    // (transport recreated → bytes drop) and non-positive elapsed time.
+    if (prev && totalBytes >= prev.bytes && ts > prev.ts) {
+      const deltaBits = (totalBytes - prev.bytes) * 8;
+      const deltaSec = (ts - prev.ts) / 1000;
+      const bps = deltaBits / deltaSec;
+      const kbps = parseFloat((bps / 1000).toFixed(1));
+      const mbps = parseFloat((bps / 1_000_000).toFixed(2));
       if (transportType === 'send') {
-        if (report.type === 'outbound-rtp') {
-          // Calculate upload bandwidth
-          if (report.bytesSent) {
-            bandwidth.upload += report.bytesSent * 8; // Convert to bits
-          }
-        }
+        bandwidth.uploadKbps = kbps;
+        bandwidth.upload = mbps;
       } else {
-        if (report.type === 'inbound-rtp') {
-          // Calculate download bandwidth
-          if (report.bytesReceived) {
-            bandwidth.download += report.bytesReceived * 8; // Convert to bits
-          }
-        }
+        bandwidth.downloadKbps = kbps;
+        bandwidth.download = mbps;
       }
     }
 
-    // Convert to Mbps
-    bandwidth.upload = parseFloat((bandwidth.upload / 1000000).toFixed(2));
-    bandwidth.download = parseFloat((bandwidth.download / 1000000).toFixed(2));
-
     return bandwidth;
   }
 

package/package.json

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