@serfab/cadre-core 1.10.0 → 1.12.0

package/src/diagnostics/connection-path.ts

@@ -13,7 +13,6 @@ export type ConnectionPathKind = 'relayed' | 'direct';
 export type ConnectionTransport =
   | 'circuit-relay'
   | 'webrtc'
-  | 'webrtc-turn'   // WebRTC session whose ICE selected a TURN relay candidate
   | 'webrtc-direct'
   | 'websocket'
   | 'tcp'
@@ -56,13 +55,6 @@ export interface ConnectionLike {
   remoteAddr?: { toString(): string };
   direction?: 'inbound' | 'outbound';
   timeline?: { open?: number };
-  /**
-   * Set externally (by {@link CadreNode}, from a {@link TurnRelayTracker}
-   * observation) when ICE selected a TURN relay candidate for this `/webrtc`
-   * session. `undefined` = unknown (the multiaddr alone cannot reveal it). Only
-   * honoured for the `webrtc` transport — see {@link classifyConnectionPath}.
-   */
-  turnRelayed?: boolean;
 }
 
 export const DEFAULT_SETTLE_WINDOW_MS = 10_000;
@@ -70,7 +62,6 @@ export const DEFAULT_SETTLE_WINDOW_MS = 10_000;
 const ALL_TRANSPORTS: readonly ConnectionTransport[] = [
   'circuit-relay',
   'webrtc',
-  'webrtc-turn',
   'webrtc-direct',
   'websocket',
   'tcp',
@@ -122,23 +113,6 @@ function addrString(conn: ConnectionLike): string {
   }
 }
 
-/**
- * Per-connection classifier. Starts from the pure multiaddr classification, then
- * applies the TURN override: a `/webrtc` session whose ICE selected a TURN relay
- * candidate (`conn.turnRelayed === true`, set externally — the multiaddr alone
- * cannot reveal it) is reclassified `relayed`/`webrtc-turn`. The
- * `transport === 'webrtc'` guard means a `webrtc-direct` connection is never
- * promoted (it uses no ICE and so can never be TURN-relayed), and `undefined`
- * (unknown) degrades safely to the multiaddr result — not relayed.
- */
-export function classifyConnectionPath(conn: ConnectionLike): TransportClass {
-  const { kind, transport } = classifyTransport(addrString(conn));
-  if (transport === 'webrtc' && conn.turnRelayed === true) {
-    return { kind: 'relayed', transport: 'webrtc-turn' };
-  }
-  return { kind, transport };
-}
-
 /**
  * Best-effort per-connection byte counter. The stock libp2p `Connection`
  * interface carries no byte counter, so unless a metrics implementation is
@@ -200,9 +174,7 @@ export function summarizeConnectionPaths(
   // least one direct connection.
   const classified = list.map((conn) => {
     const remoteAddr = addrString(conn);
-    // classifyConnectionPath (not classifyTransport) so the externally-set
-    // turnRelayed hint promotes webrtc → webrtc-turn (relayed).
-    const { kind, transport } = classifyConnectionPath(conn);
+    const { kind, transport } = classifyTransport(remoteAddr);
     const openedAtMs = typeof conn.timeline?.open === 'number' ? conn.timeline.open : null;
     const ageMs = openedAtMs != null ? Math.max(0, now - openedAtMs) : null;
     const peerId = safePeerId(conn);