@furious.luke/argus-js 0.5.0 → 0.5.1

package/dist/index.cjs

@@ -180,10 +180,11 @@ var Publisher = class {
   // may later reuse any compatible inactive transceiver, not necessarily the one
   // that previously carried the same logical type.
   typeSenders = /* @__PURE__ */ new Map();
-  // intentionalTrackEnds holds physical track ids removed by publish/unpublish
-  // whose server-side `media_track_ended` has not yet arrived. Correlating by id
-  // keeps a delayed end for an old screen track from being mistaken for failure
-  // of a newly-published screen track.
+  // intentionalTrackEnds holds the browser track ids removed by publish/unpublish
+  // whose server-side `media_track_ended` has not yet arrived. Correlating by the
+  // track id (the generation identity the server echoes in track_id) keeps a
+  // delayed end for an old screen track from being mistaken for failure of a
+  // newly-published screen track.
   intentionalTrackEnds = /* @__PURE__ */ new Map();
   // negotiationChain serializes every offer/answer exchange, including recovery.
   // User operations do not resolve until their answer is applied, so no caller
@@ -1124,7 +1125,7 @@ var Publisher = class {
         sdp: local.sdp,
         sdp_type: "offer",
         negotiation_id: id,
-        tracks: change?.labels ?? this.buildTrackLabels(),
+        tracks: change?.labels?.() ?? this.buildTrackLabels(),
         speech_enabled: this.speechEnabled || this.speechPending || void 0
       });
       this.releaseLocalCandidateBatch();
@@ -1533,7 +1534,7 @@ var Publisher = class {
       this.typeSenders.set(type, addedSender);
     }
     return {
-      labels: this.labelsReplacingType(type, track),
+      labels: () => this.labelsReplacingType(type, track),
       commit: () => {
         for (const { track: oldTrack } of previous) {
           this.unwatchTrack(oldTrack);
@@ -1597,18 +1598,34 @@ var Publisher = class {
   buildTrackLabels() {
     const labels = [];
     for (const [track, type] of this.published) {
-      labels.push({ id: track.id, type });
+      const mid = this.midForTrack(track);
+      if (mid !== null) labels.push({ mid, id: track.id, type });
     }
     return labels;
   }
   labelsReplacingType(type, replacement) {
     const labels = [];
     for (const [track, publishedType] of this.published) {
-      if (publishedType !== type) labels.push({ id: track.id, type: publishedType });
+      if (publishedType === type) continue;
+      const mid = this.midForTrack(track);
+      if (mid !== null) labels.push({ mid, id: track.id, type: publishedType });
+    }
+    if (replacement) {
+      const mid = this.midForTrack(replacement);
+      if (mid !== null) labels.push({ mid, id: replacement.id, type });
     }
-    if (replacement) labels.push({ id: replacement.id, type });
     return labels;
   }
+  /**
+   * The negotiated mid of the transceiver currently sending `track`, or null if
+   * none is found or it has not been negotiated yet. The mid is the identifier
+   * both peers agree on; it is assigned once setLocalDescription runs, which the
+   * publisher always does before sending an offer's labels.
+   */
+  midForTrack(track) {
+    const transceiver = this.pc?.getTransceivers().find((candidate) => candidate.sender.track === track);
+    return transceiver?.mid ?? null;
+  }
   async stagePublish(track, stream, type) {
     const pc = this.pc;
     if (!pc) throw new Error("publisher not started");
@@ -1639,7 +1656,7 @@ var Publisher = class {
       this.typeSenders.set(type, addedSender);
     }
     return {
-      labels: this.labelsReplacingType(type, track),
+      labels: () => this.labelsReplacingType(type, track),
       commit: () => {
         this.cancelMediaRecovery(type);
         this.recoveryState(type).required = false;
@@ -1687,7 +1704,7 @@ var Publisher = class {
     for (const { track } of previous) this.expectIntentionalTrackEnd(track.id, type);
     pc.removeTrack(typeSender);
     return {
-      labels: this.labelsReplacingType(type),
+      labels: () => this.labelsReplacingType(type),
       commit: () => {
         this.cancelMediaRecovery(type);
         this.recoveryState(type).required = false;