Skip to content

BACKLOG: mark getStats remote-outbound-rtp / inbound fields as already done#49

Merged
nus merged 1 commit into
mainfrom
backlog-getstats-already-done
Jun 10, 2026
Merged

BACKLOG: mark getStats remote-outbound-rtp / inbound fields as already done#49
nus merged 1 commit into
mainfrom
backlog-getstats-already-done

Conversation

@nus

@nus nus commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Doc-only. The 2026-05-29 RFC/W3C divergence audit listed remote-outbound-rtp and several InboundRtpStats fields as missing (marked Unverified). Verification shows both are already implemented and tested:

  • remote-outbound-rtp is emitted from received Sender Reports (_onRtcpReceived captures srPacketCount/srOctetCount/srNtp*/reportsReceived → remote-outbound-rtp-<ssrc>, paired with the inbound entry).
  • InboundRtpStats already carries kind, packetsReceived, bytesReceived, packetsLost, jitter, codecId, all populated with real values in getStats().
  • test/api/stats_test.dart already asserts these on a loopback flow.

This updates the BACKLOG entry to record the divergence (audit was stale) instead of leaving a misleading open item. No code change.

🤖 Generated with Claude Code

…y done

The 2026-05-29 audit flagged these as missing, but verification shows both
remote-outbound-rtp emission (from received SRs) and the InboundRtpStats
jitter/packetsLost/kind/codecId fields are already implemented and asserted in
stats_test.dart. Update the entry to record the divergence rather than leave a
stale Unverified item.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nus nus merged commit 2a89650 into main Jun 10, 2026
15 checks passed
@nus nus deleted the backlog-getstats-already-done branch June 10, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant