@rtcstats/rtcstats-js 2.2.3 → 2.3.0

package/README.md

@@ -3,7 +3,8 @@ This is part of the [monorepo](https://github.com/rtcstats/rtcstats) for rtcstat
 dump-importer (supporting rtcstats and webrtc-internals formats).
 
 It is part of a bigger offering that includes [rtcstats.com](https://rtcstats.com),
-an online service for debugging and troubleshooting WebRTC statistics.
+the hosted WebRTC observability service that reads collected dumps and returns Observations,
+an Experience Score, and a plain-English AI summary of what went wrong on the call.
 
 For a complete reference of every getStats() metric rtcstats collects and analyzes, see
 the [WebRTC Stats Reference](https://rtcstats.com/kb/webrtc-stats-reference-page).

package/media.js

@@ -10,9 +10,12 @@ import {compressMethod, dumpTrackWithStreams} from '@rtcstats/rtcstats-shared';
  * @param {string} property - the track whose property (e.g. `enabled`) should be wrapped.
  * @param {function} trace - RTCStats trace callback.
  */
-function wrapTrackProperty(track, property, trace) {
+function wrapTrackProperty(track, property, trace, MediaStreamTrack) {
     const prop = Object.getOwnPropertyDescriptor(
         MediaStreamTrack.prototype, property);
+    if (!prop || !prop.get || !prop.set) {
+        return;
+    }
 
     // Replace the property with a custom one
     Object.defineProperty(track, property, {
@@ -69,8 +72,8 @@ export function wrapGetUserMedia(trace, {navigator, MediaStreamTrack}) {
                         track.addEventListener('ended', () => {
                             trace('MediaStreamTrack.onended', null, track.id, track.__rtcStatsId);
                         });
-                        wrapTrackProperty(track, 'enabled', trace);
-                        wrapTrackProperty(track, 'contentHint', trace);
+                        wrapTrackProperty(track, 'enabled', trace, MediaStreamTrack);
+                        wrapTrackProperty(track, 'contentHint', trace, MediaStreamTrack);
                     });
                     return stream;
                 }, (err) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rtcstats/rtcstats-js",
-  "version": "2.2.3",
+  "version": "2.3.0",
   "description": "gather WebRTC API traces and statistics",
   "main": "rtcstats.js",
   "type": "module",

package/peerconnection.js

@@ -91,7 +91,7 @@ export function wrapRTCPeerConnection(trace, window, {getStatsInterval}) {
     }
     let lastComputePressureRecord;
     if (window.PressureObserver && getStatsInterval) {
-        const observer = new PressureObserver((records) => {
+        const observer = new window.PressureObserver((records) => {
             const lastRecord = records[records.length - 1]; // Usually only one record.
             lastComputePressureRecord = [Date.now(), lastRecord];
         });
@@ -153,9 +153,9 @@ export function wrapRTCPeerConnection(trace, window, {getStatsInterval}) {
                 e.transceiver.sender.__rtcStatsId = pcId;
                 e.transceiver.sender.__rtcStatsSenderId = e.track.id;
             }
-            if (e.track.kind === 'video') {
+            if (e.track.kind === 'video' && window.document?.querySelectorAll) {
                 setTimeout(() => {
-                    document.querySelectorAll('video').forEach(el => {
+                    window.document.querySelectorAll('video').forEach(el => {
                         if (!el.srcObject) return;
                         if (el.srcObject.getTracks().indexOf(e.track) === -1) return;
                         el.addEventListener('resize', () => {
@@ -188,7 +188,7 @@ export function wrapRTCPeerConnection(trace, window, {getStatsInterval}) {
         let statsInterval;
         const statsIdMap = {};
         const getStats = async (reason) => {
-            if (pc.signalingState === 'closed') {
+            if (pc.signalingState === 'closed' || pc.connectionState === 'closed') {
                 if (statsInterval) {
                     window.clearInterval(statsInterval);
                 }

package/rtcstats.js

@@ -7,15 +7,15 @@ import {WebSocketTrace} from './trace-websocket.js';
  *
  * @returns {function} RTCStats trace function.
  */
-export function wrapRTCStatsWithDefaultOptions(config = {getStatsInterval: 1000}) {
+export function wrapRTCStatsWithDefaultOptions(config = {getStatsInterval: 1000}, target = globalThis) {
     const trace = new WebSocketTrace(config);
 
     // Wrap RTCPeerConnection-related APIs and events
-    wrapRTCPeerConnection(trace, window, config);
+    wrapRTCPeerConnection(trace, target, config);
     // Wrap getUserMedia, getDisplayMedia and related events.
-    wrapGetUserMedia(trace, window);
+    wrapGetUserMedia(trace, target);
     // Wrap enumerateDevices.
-    wrapEnumerateDevices(trace, window);
+    wrapEnumerateDevices(trace, target);
 
     return trace;
 }

package/trace-websocket.js

@@ -68,8 +68,8 @@ export function WebSocketTrace(config = {}) {
             userAgentData: navigator.userAgentData,
             deviceMemory: navigator.deviceMemory,
             screen: {
-                width: window.screen.availWidth,
-                height: window.screen.availHeight,
+                width: window.screen?.availWidth,
+                height: window.screen?.availHeight,
                 devicePixelRatio: window.devicePixelRatio,
             },
             window: {