From 4679eba30c23254922fe4ebf39cca76f4e021942 Mon Sep 17 00:00:00 2001
From: David Zhao <dz@livekit.io>
Date: Sat, 23 Mar 2024 12:36:36 -0700
Subject: [PATCH] Improve VideoSenderStats with FPS and targetBitrate (#1090)

---
 .changeset/nervous-rivers-invite.md |  5 +++++
 src/room/stats.ts                   | 12 +++++++++---
 src/room/track/LocalVideoTrack.ts   | 13 +++++++++----
 3 files changed, 23 insertions(+), 7 deletions(-)
 create mode 100644 .changeset/nervous-rivers-invite.md

diff --git a/.changeset/nervous-rivers-invite.md b/.changeset/nervous-rivers-invite.md
new file mode 100644
index 0000000000..52e7b0ce0e
--- /dev/null
+++ b/.changeset/nervous-rivers-invite.md
@@ -0,0 +1,5 @@
+---
+"livekit-client": patch
+---
+
+Improve VideoSenderStats with FPS and targetBitrate
diff --git a/src/room/stats.ts b/src/room/stats.ts
index 3ceb658419..794cac142d 100644
--- a/src/room/stats.ts
+++ b/src/room/stats.ts
@@ -42,14 +42,20 @@ export interface VideoSenderStats extends SenderStats {
 
   frameHeight: number;
 
+  framesPerSecond: number;
+
   framesSent: number;
 
   // bandwidth, cpu, other, none
-  qualityLimitationReason: string;
+  qualityLimitationReason?: string;
+
+  qualityLimitationDurations?: Record<string, number>;
+
+  qualityLimitationResolutionChanges?: number;
 
-  qualityLimitationResolutionChanges: number;
+  retransmittedPacketsSent?: number;
 
-  retransmittedPacketsSent: number;
+  targetBitrate: number;
 }
 
 interface ReceiverStats {
diff --git a/src/room/track/LocalVideoTrack.ts b/src/room/track/LocalVideoTrack.ts
index 68a06641ed..68b3833184 100644
--- a/src/room/track/LocalVideoTrack.ts
+++ b/src/room/track/LocalVideoTrack.ts
@@ -180,17 +180,20 @@ export default class LocalVideoTrack extends LocalTrack<Track.Kind.Video> {
           streamId: v.id,
           frameHeight: v.frameHeight,
           frameWidth: v.frameWidth,
+          framesPerSecond: v.framesPerSecond,
+          framesSent: v.framesSent,
           firCount: v.firCount,
           pliCount: v.pliCount,
           nackCount: v.nackCount,
           packetsSent: v.packetsSent,
           bytesSent: v.bytesSent,
-          framesSent: v.framesSent,
-          timestamp: v.timestamp,
-          rid: v.rid ?? v.id,
-          retransmittedPacketsSent: v.retransmittedPacketsSent,
           qualityLimitationReason: v.qualityLimitationReason,
+          qualityLimitationDurations: v.qualityLimitationDurations,
           qualityLimitationResolutionChanges: v.qualityLimitationResolutionChanges,
+          rid: v.rid ?? v.id,
+          retransmittedPacketsSent: v.retransmittedPacketsSent,
+          targetBitrate: v.targetBitrate,
+          timestamp: v.timestamp,
         };
 
         // locate the appropriate remote-inbound-rtp item
@@ -205,6 +208,8 @@ export default class LocalVideoTrack extends LocalTrack<Track.Kind.Video> {
       }
     });
 
+    // make sure highest res layer is always first
+    items.sort((a, b) => (b.frameWidth ?? 0) - (a.frameWidth ?? 0));
     return items;
   }