@observertc/observer-js 1.0.0-beta.16 → 1.0.0-beta.17

package/dist/index.js

@@ -35,6 +35,7 @@ __export(src_exports, {
   CallConcurrentIssueDetector: () => CallConcurrentIssueDetector,
   CallConcurrentIssueTypes: () => CallConcurrentIssueTypes,
   CallHealthAggregator: () => CallHealthAggregator,
+  CallSummaryCollector: () => CallSummaryCollector,
   ClientEventTypes: () => ClientEventTypes,
   ClientMetaTypes: () => ClientMetaTypes,
   ClientPopulationIssueDetector: () => ClientPopulationIssueDetector,
@@ -93,16 +94,17 @@ __export(src_exports, {
   concludeObserverIssue: () => concludeObserverIssue,
   correlation: () => correlation,
   counterDelta: () => counterDelta,
+  createCallSummary: () => createCallSummary,
   createDefaultMediasoupRemoteTrackResolverFactory: () => createDefaultMediasoupRemoteTrackResolverFactory,
   createInMemorySink: () => createInMemorySink,
   createJsonlFileSink: () => createJsonlFileSink,
   createJsonlFileSinkFactory: () => createJsonlFileSinkFactory,
   createLogger: () => createLogger,
   createP2pRemoteTrackResolverFactory: () => createP2pRemoteTrackResolverFactory,
+  defaultCallSummaryConfig: () => defaultCallSummaryConfig,
   defaultClientHealthThresholds: () => defaultClientHealthThresholds,
   isClientIssueResolutionEntry: () => isClientIssueResolutionEntry,
   issuePayloadAsString: () => issuePayloadAsString,
-  issuePayloadOf: () => issuePayloadOf,
   mannKendall: () => mannKendall,
   mannKendallVerdict: () => mannKendallVerdict,
   median: () => median,
@@ -155,8 +157,8 @@ function createLogger(moduleName) {
     }
   }();
 }
-function setObserverLogger(logger9) {
-  mainLogger = logger9;
+function setObserverLogger(logger10) {
+  mainLogger = logger10;
 }
 
 // src/ObservedCall.ts
@@ -3574,7 +3576,6 @@ var UnconsumedTrackDetector = class _UnconsumedTrackDetector {
         type: UnconsumedTrackTypes.unconsumedPublishedTrack,
         timestamp: now,
         payload: {
-          type: UnconsumedTrackTypes.unconsumedPublishedTrack,
           trackId: outboundTrack.id,
           kind: outboundTrack.kind,
           publisherClientId: peerConnection?.client.clientId,
@@ -3720,7 +3721,6 @@ var TrackDeliveryMismatchDetector = class _TrackDeliveryMismatchDetector {
         type,
         timestamp: now,
         payload: {
-          type,
           trackId: outboundTrackId,
           publisherClientId: delivery.publisherClientId,
           publisherSending: delivery.publisherSending,
@@ -3973,12 +3973,9 @@ var CallConcurrentIssueDetector = class _CallConcurrentIssueDetector {
       this._call.addIssue({
         type: issueType,
         timestamp: now,
+        conclusion,
         payload: {
-          type: issueType,
           issueType: type,
-          scope: "call",
-          conclusion,
-          callId: this._call.callId,
           clients: group.totalClients,
           affectedClients: group.clientIds.length,
           affectedRatio: group.affectedRatio,
@@ -4102,10 +4099,9 @@ var IssueFanOutDetector = class _IssueFanOutDetector {
         this._call.addIssue({
           type,
           timestamp: now,
+          conclusion,
           payload: {
-            type,
             issueType,
-            conclusion,
             trackId: publisher.id,
             kind: publisher.kind,
             publisherClientId: publisher.getPeerConnection().client.clientId,
@@ -4252,18 +4248,14 @@ var PublisherFaultCorroborationDetector = class _PublisherFaultCorroborationDete
       this._call.addIssue({
         type: PublisherFaultTypes.corroboratedPublisherFault,
         timestamp: now,
-        payload: {
-          type: PublisherFaultTypes.corroboratedPublisherFault,
-          callId: this._call.callId,
-          ...fault,
-          conclusion: {
-            faultDomain: "published-track",
-            summary: `${fault.publisherClientId} reports ${fault.publisherIssueTypes.join(", ")} on track ${fault.trackId} while ${fault.affectedReceivers} of ${fault.receivers} subscribers report ${fault.receiverIssueTypes.join(", ")} \u2014 both ends agree`,
-            recommendation: "the source is implicated, not inferred: check that publisher's capture, encoder and uplink before looking at the SFU or the receivers",
-            // Higher than any single-ended finding: two independent parties, one conclusion.
-            confidence: 0.9
-          }
-        }
+        conclusion: {
+          faultDomain: "published-track",
+          summary: `${fault.publisherClientId} reports ${fault.publisherIssueTypes.join(", ")} on track ${fault.trackId} while ${fault.affectedReceivers} of ${fault.receivers} subscribers report ${fault.receiverIssueTypes.join(", ")} \u2014 both ends agree`,
+          recommendation: "the source is implicated, not inferred: check that publisher's capture, encoder and uplink before looking at the SFU or the receivers",
+          // Higher than any single-ended finding: two independent parties, one conclusion.
+          confidence: 0.9
+        },
+        payload: { ...fault }
       });
     }
   }
@@ -4319,6 +4311,14 @@ var ObservedCall = class extends import_events3.EventEmitter {
     value: void 0
   };
   remoteTrackResolver;
+  /**
+   * The accumulating record of this call's life, or `undefined` when no summary was configured.
+   *
+   * Live — read it at any point during the call. It is also delivered once on `call-summary` when
+   * the call closes. See `CallSummary`: an absent section means "not collected", never "nothing
+   * happened".
+   */
+  summary;
   /**
    * Published tracks that currently have **no** subscriber linked to them.
    *
@@ -4396,6 +4396,21 @@ var ObservedCall = class extends import_events3.EventEmitter {
     this.detectors.add(detector);
     return this;
   }
+  /**
+   * Start accumulating this call's summary, if the observer was configured for summaries.
+   *
+   * Called by `createObservedCall`; you should not need it. It takes no configuration of its own on
+   * purpose: the collector subscribes to exactly the events the observer's `include` requires, so a
+   * per-call section outside that set would be created and then never written to — an empty section
+   * that reads as "nothing happened". One shape per observer is the only shape that can be filled.
+   *
+   * The collector builds it rather than this method, so the resolved configuration never has to
+   * leave the one object that owns it. Returns `undefined` when summaries are off, and is
+   * idempotent: an existing summary is kept, not restarted.
+   */
+  enableSummary() {
+    return this.summary ??= this.observer.callSummaryCollector?.createSummary(this.callId);
+  }
   /**
    * Remove a detector from **this call** by name, returning how many were removed.
    *
@@ -4420,12 +4435,13 @@ var ObservedCall = class extends import_events3.EventEmitter {
   /**
    * Raise a call-level (server-side) finding; surfaced on the Observer bus as `call-issue`.
    *
-   * `payload` takes an **object** — it is delivered to in-process handlers, so there is nothing to
-   * serialise for. Pass a string only if you already have one.
+   * `payload` is an **object** and holds evidence only — it is delivered to an in-process handler,
+   * so there is nothing to serialise for. `scope` is stamped here, and the `callId` is already on
+   * the event, so neither belongs in the payload. Put the interpretation in `conclusion`.
    */
   addIssue(issue) {
     if (this.closed) return;
-    this._notify("call-issue", { ...this.eventScope, issue });
+    this._notify("call-issue", { ...this.eventScope, issue: { ...issue, scope: "call" } });
   }
   close() {
     if (this.closed) return;
@@ -4446,6 +4462,10 @@ var ObservedCall = class extends import_events3.EventEmitter {
     if (this.startedAt === void 0) this.startedAt = minSampleTimestamps;
     if (this.endedAt === void 0) this.endedAt = maxSampleTimestamps;
     this.closedAt = Date.now();
+    if (this.summary) {
+      this.observer.callSummaryCollector?.finalise(this);
+      this._notify("call-summary", { ...this.eventScope, summary: this.summary });
+    }
     this.detectors.clear();
     this.activeIssuesRegistry.clear();
     this.emit("close");
@@ -5206,6 +5226,166 @@ var ActiveIssuesRegistry = class {
   }
 };
 
+// src/summaries/CallSummary.ts
+var defaultCallSummaryConfig = {
+  include: [],
+  maxIssues: 500,
+  maxClientIds: 1e4
+};
+function createCallSummary(callId, config) {
+  const summary = { callId, attachments: {} };
+  if (config.include.includes("clients")) {
+    summary.clients = { clientIds: [], peak: 0, joined: 0, left: 0 };
+  }
+  if (config.include.includes("issues")) {
+    summary.issues = [];
+  }
+  if (config.include.includes("turnServers")) {
+    summary.turnServers = { serverUrls: [], clientsRelayed: 0 };
+  }
+  if (config.include.includes("scores")) {
+    summary.scores = { samples: 0 };
+  }
+  return summary;
+}
+
+// src/summaries/CallSummaryCollector.ts
+var logger8 = createLogger("CallSummaryCollector");
+var CallSummaryCollector = class {
+  constructor(_observer, _config) {
+    this._observer = _observer;
+    this._config = _config;
+    this._subscribeBuiltIns();
+    this._subscribeEnrichers();
+  }
+  _observer;
+  _config;
+  _scratch = /* @__PURE__ */ new WeakMap();
+  _listeners = [];
+  _closed = false;
+  /**
+   * Build a summary for `callId` and start tracking it.
+   *
+   * Creating it here, rather than letting the call create one and hand it over, keeps the resolved
+   * configuration inside the single object that owns it — and makes it impossible to end up with a
+   * summary whose sections nobody subscribed to fill.
+   */
+  createSummary(callId) {
+    const summary = createCallSummary(callId, this._config);
+    this._scratch.set(summary, { scores: [], turnClientIds: /* @__PURE__ */ new Set() });
+    return summary;
+  }
+  /**
+   * Finalise `call`'s summary: fold in what only makes sense once, and stamp the closing times.
+   *
+   * Percentiles are computed here rather than on every update — a median recomputed per tick over a
+   * growing array is quadratic work to produce a number nobody reads until the end.
+   */
+  finalise(call) {
+    const summary = call.summary;
+    if (!summary) return;
+    const scratch = this._scratch.get(summary);
+    summary.startedAt = call.startedAt;
+    summary.endedAt = call.endedAt;
+    summary.durationInMs = summary.startedAt !== void 0 && summary.endedAt !== void 0 ? Math.max(0, summary.endedAt - summary.startedAt) : void 0;
+    summary.closedAt = Date.now();
+    if (summary.scores && scratch) {
+      summary.scores.samples = scratch.scores.length;
+      if (0 < scratch.scores.length) {
+        summary.scores.min = Math.min(...scratch.scores);
+        summary.scores.max = Math.max(...scratch.scores);
+        summary.scores.median = percentile(scratch.scores, 0.5);
+      }
+    }
+    if (summary.turnServers && scratch) {
+      summary.turnServers.clientsRelayed = scratch.turnClientIds.size;
+    }
+  }
+  /** Drop every bus subscription. Called when the observer closes. */
+  close() {
+    if (this._closed) return;
+    this._closed = true;
+    for (const { event, listener } of this._listeners) {
+      this._observer.off(event, listener);
+    }
+    this._listeners.length = 0;
+  }
+  /**
+   * Subscribe `listener` to `event`, routed to the summary of the call the event names.
+   *
+   * The `observedCall` is read off the payload rather than closed over, which is what lets one
+   * subscription serve every call.
+   */
+  _on(event, handler) {
+    const listener = (...args) => {
+      const summary = args[0].observedCall.summary;
+      if (!summary) return;
+      const scratch = this._scratch.get(summary);
+      if (!scratch) return;
+      try {
+        handler(summary, scratch, ...args);
+      } catch (err) {
+        logger8.warn("A call-summary handler for %s threw; continuing. %o", event, err);
+      }
+    };
+    this._observer.on(event, listener);
+    this._listeners.push({ event, listener });
+  }
+  _subscribeBuiltIns() {
+    const include = this._config.include;
+    if (include.includes("clients")) {
+      this._on("client-added", (summary, _scratch, { observedCall, observedClient }) => {
+        const clients = summary.clients;
+        if (!clients) return;
+        clients.joined += 1;
+        clients.peak = Math.max(clients.peak, observedCall.observedClients.size);
+        if (clients.clientIds.length < this._config.maxClientIds) {
+          clients.clientIds.push(observedClient.clientId);
+        } else {
+          summary.truncated = { ...summary.truncated, clientIds: (summary.truncated?.clientIds ?? 0) + 1 };
+        }
+      });
+      this._on("client-closed", (summary) => {
+        if (summary.clients) summary.clients.left += 1;
+      });
+    }
+    if (include.includes("issues")) {
+      this._on("call-issue", (summary, _scratch, { issue }) => {
+        const issues = summary.issues;
+        if (!issues) return;
+        if (issues.length < this._config.maxIssues) issues.push(issue);
+        else summary.truncated = { ...summary.truncated, issues: (summary.truncated?.issues ?? 0) + 1 };
+      });
+    }
+    if (include.includes("scores") || include.includes("turnServers")) {
+      this._on("call-updated", (summary, scratch, { observedCall }) => {
+        if (summary.scores && observedCall.score !== void 0) scratch.scores.push(observedCall.score);
+        if (!summary.turnServers) return;
+        for (const clientId of observedCall.clientsUsedTurn) scratch.turnClientIds.add(clientId);
+        for (const client of observedCall.observedClients.values()) {
+          for (const peerConnection of client.observedPeerConnections.values()) {
+            for (const pair of peerConnection.selectedIceCandiadtePairForTurn) {
+              const url = pair.getLocalCandidate()?.url;
+              if (url && !summary.turnServers.serverUrls.includes(url)) {
+                summary.turnServers.serverUrls.push(url);
+              }
+            }
+          }
+        }
+      });
+    }
+  }
+  _subscribeEnrichers() {
+    const enrich = this._config.enrich;
+    if (!enrich) return;
+    for (const name of Object.keys(enrich)) {
+      const enricher = enrich[name];
+      if (!enricher) continue;
+      this._on(name, (summary, _scratch, ...args) => enricher(summary, ...args));
+    }
+  }
+};
+
 // src/detectors/SfuCongestionDetector.ts
 var SfuCongestionDetector = class _SfuCongestionDetector {
   constructor(_observer, config = {}) {
@@ -5334,13 +5514,10 @@ var SfuCongestionDetector = class _SfuCongestionDetector {
       absoluteIncrease: evaluation.absoluteIncrease,
       relativeIncrease: evaluation.relativeIncrease
     };
-    this._observer.emit("observer-issue", {
-      issue: {
-        type: this._config.emittedObserverIssueType,
-        timestamp: Date.now(),
-        payload
-      },
-      observer: this._observer
+    this._observer.addIssue({
+      type: this._config.emittedObserverIssueType,
+      timestamp: Date.now(),
+      payload: { ...payload }
     });
   }
   /**
@@ -5454,11 +5631,9 @@ var ObserverConcurrentIssueDetector = class _ObserverConcurrentIssueDetector {
       this._observer.addIssue({
         type: issueType,
         timestamp: now,
+        conclusion,
         payload: {
-          type: issueType,
           issueType: type,
-          scope: "observer",
-          conclusion,
           clients: group.totalClients,
           affectedClients: group.clientIds.length,
           affectedRatio: group.affectedRatio,
@@ -5631,16 +5806,13 @@ var ClientPopulationIssueDetector = class _ClientPopulationIssueDetector {
         this._observer.addIssue({
           type: ClientPopulationIssueTypes.clientPopulationIssue,
           timestamp: now,
-          payload: {
-            type: ClientPopulationIssueTypes.clientPopulationIssue,
-            ...rollup,
-            conclusion: {
-              faultDomain: "client-population",
-              summary: `'${issueType}' is ${this._riskText(rollup.relativeRisk)} more likely on ${population} than on the rest of the fleet (${rollup.affectedClients}/${rollup.clients} vs ${rollup.controlAffectedClients}/${rollup.controlClients})`,
-              recommendation: "this is not an SFU symptom \u2014 look at what those clients share: a recent release, a browser version, or shared/virtualised hardware",
-              confidence: this._confidenceOf(rollup)
-            }
-          }
+          conclusion: {
+            faultDomain: "client-population",
+            summary: `'${issueType}' is ${this._riskText(rollup.relativeRisk)} more likely on ${population} than on the rest of the fleet (${rollup.affectedClients}/${rollup.clients} vs ${rollup.controlAffectedClients}/${rollup.controlClients})`,
+            recommendation: "this is not an SFU symptom \u2014 look at what those clients share: a recent release, a browser version, or shared/virtualised hardware",
+            confidence: this._confidenceOf(rollup)
+          },
+          payload: { ...rollup }
         });
       }
     }
@@ -5742,7 +5914,7 @@ var TurnServerHealthDetector = class _TurnServerHealthDetector {
       this._observer.addIssue({
         type: TurnServerHealthTypes.turnServerDegraded,
         timestamp: now,
-        payload: { type: TurnServerHealthTypes.turnServerDegraded, ...health, otherServers }
+        payload: { ...health, otherServers }
       });
     }
     for (const serverUrl of [...this._streaks.keys()]) {
@@ -5850,7 +6022,6 @@ var TurnServerOutageDetector = class _TurnServerOutageDetector {
         type: TurnServerOutageTypes.turnServerOutage,
         timestamp: now,
         payload: {
-          type: TurnServerOutageTypes.turnServerOutage,
           serverUrl,
           peakClients: peak,
           currentClients: live,
@@ -6124,17 +6295,13 @@ var SimulcastReceiverValidator = class _SimulcastReceiverValidator {
       this._observer.addIssue({
         type: LOWEST_COMMON_DENOMINATOR_ISSUE,
         timestamp: decidedAt,
-        payload: {
-          type: LOWEST_COMMON_DENOMINATOR_ISSUE,
-          checks: this._checks,
-          ...outcome.evidence,
-          conclusion: {
-            faultDomain: "infrastructure",
-            summary: "one bad receiver is dragging a publisher's bitrate down for everyone \u2014 the SFU is not selecting layers per consumer",
-            recommendation: "check that simulcast/SVC is enabled and layers are chosen per consumer, and that the SFU terminates receiver reports instead of forwarding them; this is a build/config property, not a transient",
-            confidence: 0.8
-          }
-        }
+        conclusion: {
+          faultDomain: "infrastructure",
+          summary: "one bad receiver is dragging a publisher's bitrate down for everyone \u2014 the SFU is not selecting layers per consumer",
+          recommendation: "check that simulcast/SVC is enabled and layers are chosen per consumer, and that the SFU terminates receiver reports instead of forwarding them; this is a build/config property, not a transient",
+          confidence: 0.8
+        },
+        payload: { checks: this._checks, ...outcome.evidence }
       });
     }
     this.onDone(this.report);
@@ -6237,17 +6404,13 @@ var RemoteTrackResolverValidator = class _RemoteTrackResolverValidator {
       this._observer.addIssue({
         type: UNRESOLVED_TRACK_LINKS_ISSUE,
         timestamp: decidedAt,
-        payload: {
-          type: UNRESOLVED_TRACK_LINKS_ISSUE,
-          checks: this._checks,
-          ...outcome.evidence,
-          conclusion: {
-            faultDomain: "infrastructure",
-            summary: "a RemoteTrackResolver is configured but has never linked a subscriber to a publisher",
-            recommendation: "check the id the resolver joins on (mediasoup producerId, or your own convention) \u2014 until it links, IssueFanOutDetector, TrackDeliveryMismatchDetector, UnconsumedTrackDetector and SimulcastReceiverValidator all silently do nothing",
-            confidence: 0.9
-          }
-        }
+        conclusion: {
+          faultDomain: "infrastructure",
+          summary: "a RemoteTrackResolver is configured but has never linked a subscriber to a publisher",
+          recommendation: "check the id the resolver joins on (mediasoup producerId, or your own convention) \u2014 until it links, IssueFanOutDetector, TrackDeliveryMismatchDetector, UnconsumedTrackDetector and SimulcastReceiverValidator all silently do nothing",
+          confidence: 0.9
+        },
+        payload: { checks: this._checks, ...outcome.evidence }
       });
     }
     this.onDone(this.report);
@@ -6358,17 +6521,16 @@ var CodecConsistencyValidator = class _CodecConsistencyValidator {
       this._observer.addIssue({
         type: CODEC_MISMATCH_ISSUE,
         timestamp: decidedAt,
+        conclusion: {
+          faultDomain: "infrastructure",
+          summary: outcome.verdict === "codec-split" ? "participants of one call are using different codecs \u2014 an SFU that forwards without transcoding cannot serve all of them" : "the deployment is consistently negotiating a codec other than the expected one",
+          recommendation: outcome.verdict === "codec-split" ? "check codec preferences and any SDP munging; a split usually means one endpoint could not negotiate the preferred codec and the others were not renegotiated with it" : "check codec preferences and endpoint support \u2014 a silent fallback keeps working, at a higher bitrate than you budgeted for",
+          confidence: 0.85
+        },
         payload: {
-          type: CODEC_MISMATCH_ISSUE,
           verdict: outcome.verdict,
           checks: this._checks,
-          evidence: outcome.evidence,
-          conclusion: {
-            faultDomain: "infrastructure",
-            summary: outcome.verdict === "codec-split" ? "participants of one call are using different codecs \u2014 an SFU that forwards without transcoding cannot serve all of them" : "the deployment is consistently negotiating a codec other than the expected one",
-            recommendation: outcome.verdict === "codec-split" ? "check codec preferences and any SDP munging; a split usually means one endpoint could not negotiate the preferred codec and the others were not renegotiated with it" : "check codec preferences and endpoint support \u2014 a silent fallback keeps working, at a higher bitrate than you budgeted for",
-            confidence: 0.85
-          }
+          evidence: outcome.evidence
         }
       });
     }
@@ -6382,7 +6544,7 @@ function mediaKindOf(mimeType) {
 }
 
 // src/Observer.ts
-var logger8 = createLogger("Observer");
+var logger9 = createLogger("Observer");
 var Observer = class extends import_events6.EventEmitter {
   observedTURN = new ObservedTURN();
   observedCalls = /* @__PURE__ */ new Map();
@@ -6438,6 +6600,14 @@ var Observer = class extends import_events6.EventEmitter {
    * ```
    */
   callDetectorConfigs = /* @__PURE__ */ new Map();
+  /**
+   * Owns every call's summary: the resolved `config.callSummary`, the bus subscriptions that keep
+   * the summaries current (one per event type, not one per call), and the summaries themselves.
+   *
+   * `undefined` when `config.callSummary` was absent or `null` — so its presence *is* the answer to
+   * "are summaries on", and nothing is subscribed to anything.
+   */
+  callSummaryCollector;
   constructor(config = {}) {
     super();
     this.setMaxListeners(Infinity);
@@ -6449,6 +6619,12 @@ var Observer = class extends import_events6.EventEmitter {
       closeClientIfIdleForMs: 6e4,
       ...config
     };
+    if (this.config.callSummary) {
+      this.callSummaryCollector = new CallSummaryCollector(this, {
+        ...defaultCallSummaryConfig,
+        ...this.config.callSummary
+      });
+    }
   }
   get numberOfCalls() {
     return this.observedCalls.size;
@@ -6487,7 +6663,7 @@ var Observer = class extends import_events6.EventEmitter {
         break;
       }
       default: {
-        logger8.warn("Unknown detector name %s; skipping", name);
+        logger9.warn("Unknown detector name %s; skipping", name);
         return this;
       }
     }
@@ -6572,7 +6748,7 @@ var Observer = class extends import_events6.EventEmitter {
         break;
     }
     if (!validator) {
-      logger8.warn("Unknown validator name %s; skipping", name);
+      logger9.warn("Unknown validator name %s; skipping", name);
       return this;
     }
     this.validators.add(validator);
@@ -6603,7 +6779,7 @@ var Observer = class extends import_events6.EventEmitter {
       try {
         validator.cancel(reason);
       } catch (err) {
-        logger8.warn("Error cancelling validator %s: %o", validator.name, err);
+        logger9.warn("Error cancelling validator %s: %o", validator.name, err);
       }
     }
     return matching.length;
@@ -6614,11 +6790,11 @@ var Observer = class extends import_events6.EventEmitter {
   }
   createObservedCall(settings) {
     if (this.closed) {
-      logger8.warn("Attempted to create a call (callId: %s) on a closed observer", settings.callId);
+      logger9.warn("Attempted to create a call (callId: %s) on a closed observer", settings.callId);
       return void 0;
     }
     if (this.observedCalls.has(settings.callId)) {
-      logger8.warn("Observed Call with id %s already exists; returning the existing instance", settings.callId);
+      logger9.warn("Observed Call with id %s already exists; returning the existing instance", settings.callId);
       return this.observedCalls.get(settings.callId);
     }
     const callSettings = {
@@ -6633,6 +6809,7 @@ var Observer = class extends import_events6.EventEmitter {
       callActiveIssuesRegistry
     );
     observedCall.remoteTrackResolver = this.config.createRemoteTrackResolver?.(observedCall);
+    observedCall.enableSummary();
     for (const [name, detectorConfig] of this.callDetectorConfigs) {
       observedCall.addDetector(name, detectorConfig);
     }
@@ -6655,11 +6832,11 @@ var Observer = class extends import_events6.EventEmitter {
   }
   createObservedMediasoupRouter(settings) {
     if (this.closed) {
-      logger8.warn("Attempted to create mediasoup router (id: %d) on a closed observer", settings.router.id);
+      logger9.warn("Attempted to create mediasoup router (id: %d) on a closed observer", settings.router.id);
       return void 0;
     }
     if (this.observedMediasoupRouters.has(settings.router.id)) {
-      logger8.warn("Observed Mediasoup Router (id %s) already exists; returning the existing instance", settings.router.id);
+      logger9.warn("Observed Mediasoup Router (id %s) already exists; returning the existing instance", settings.router.id);
       return this.observedMediasoupRouters.get(settings.router.id);
     }
     const observedMediasoupRouter = new ObservedMediasoupRouter(settings);
@@ -6690,13 +6867,14 @@ var Observer = class extends import_events6.EventEmitter {
   }
   close() {
     if (this.closed) {
-      return logger8.debug("Attempted to close twice");
+      return logger9.debug("Attempted to close twice");
     }
     this.closed = true;
     for (const call of [...this.observedCalls.values()]) call.close();
     this.detectors.clear();
     for (const validator of [...this.validators]) validator.cancel("observer closed");
     this.validators.clear();
+    this.callSummaryCollector?.close();
     this.activeIssuesRegistry.clear();
     this._notify("observer-closed", { ...this.eventScope });
   }
@@ -6707,7 +6885,7 @@ var Observer = class extends import_events6.EventEmitter {
     try {
       this.acceptMiddlewares.process({ sample, context });
     } catch (err) {
-      logger8.warn("An accept middleware threw; dropping the sample. %o", err);
+      logger9.warn("An accept middleware threw; dropping the sample. %o", err);
     }
     if (!sample.callId) {
       this._notify("sample-rejected", { ...this.eventScope, reason: "missing-callId", sample });
@@ -6759,7 +6937,7 @@ var Observer = class extends import_events6.EventEmitter {
       try {
         validator.update();
       } catch (err) {
-        logger8.warn("Error running validator %s: %o", validator.name, err);
+        logger9.warn("Error running validator %s: %o", validator.name, err);
       }
     }
     this._notify("observer-updated", { ...this.eventScope });
@@ -6772,7 +6950,7 @@ var Observer = class extends import_events6.EventEmitter {
    */
   addIssue(issue) {
     if (this.closed) return;
-    this._notify("observer-issue", { ...this.eventScope, issue });
+    this._notify("observer-issue", { ...this.eventScope, issue: { ...issue, scope: "observer" } });
   }
   /** Emit an Observer-bus event. */
   _notify(type, ...args) {
@@ -6890,18 +7068,11 @@ var CallHealthAggregator = class {
   }
 };
 
-// src/common/ObserverIssue.ts
-function issuePayloadOf(issue) {
-  const payload = issue.payload;
-  if (payload === void 0) return void 0;
-  return typeof payload === "string" ? parseJsonObject(payload) : payload;
-}
+// src/common/Issue.ts
 function issuePayloadAsString(issue) {
-  const payload = issue.payload;
-  if (payload === void 0) return void 0;
-  if (typeof payload === "string") return payload;
+  if (issue.payload === void 0) return void 0;
   try {
-    return JSON.stringify(payload);
+    return JSON.stringify(issue.payload);
   } catch {
     return void 0;
   }
@@ -7196,6 +7367,7 @@ function createP2pRemoteTrackResolverFactory() {
   CallConcurrentIssueDetector,
   CallConcurrentIssueTypes,
   CallHealthAggregator,
+  CallSummaryCollector,
   ClientEventTypes,
   ClientMetaTypes,
   ClientPopulationIssueDetector,
@@ -7254,16 +7426,17 @@ function createP2pRemoteTrackResolverFactory() {
   concludeObserverIssue,
   correlation,
   counterDelta,
+  createCallSummary,
   createDefaultMediasoupRemoteTrackResolverFactory,
   createInMemorySink,
   createJsonlFileSink,
   createJsonlFileSinkFactory,
   createLogger,
   createP2pRemoteTrackResolverFactory,
+  defaultCallSummaryConfig,
   defaultClientHealthThresholds,
   isClientIssueResolutionEntry,
   issuePayloadAsString,
-  issuePayloadOf,
   mannKendall,
   mannKendallVerdict,
   median,