@coventia/softphone 0.1.0 → 0.2.0

package/dist/zadarma/index.cjs

@@ -421,6 +421,14 @@ var SipCallEngine = class {
   #uaInFlight = null;
   #domain = "";
   #activeCall = null;
+  /**
+   * The consult leg from `startConsult` (SPEC §5.6), if one is live on this UA right now.
+   *
+   * Tracked separately from `#activeCall` so `#maybeTeardownUA` can tell "both legs are actually gone"
+   * apart from "the main leg ended, but a consult is still up" — see that method's own comment for the
+   * bug this exists to prevent.
+   */
+  #consultCall = null;
   /** The chosen microphone, or `null` for the browser default. Read fresh by every new leg's
    *  `mediaConstraints`, and pushed live to `#activeCall` by `setInputDevice`. */
   #inputDeviceId = null;
@@ -539,9 +547,7 @@ var SipCallEngine = class {
     const ms = this.provider.quirks?.noInviteTeardownMs;
     if (!ms) return;
     this.#clearNoInviteTimer();
-    this.#noInviteTimer = setTimeout(() => {
-      if (!this.#activeCall) this.#teardownUA();
-    }, ms);
+    this.#noInviteTimer = setTimeout(() => this.#maybeTeardownUA(), ms);
   }
   /**
    * Rebuild the incoming-signal channel with fresh credentials, after a registration failure.
@@ -663,6 +669,7 @@ var SipCallEngine = class {
     }
     this.#ua = null;
     this.#activeCall = null;
+    this.#consultCall = null;
   }
   /** Place a call once the UA is registered (its 200 OK for REGISTER has arrived). */
   /**
@@ -705,7 +712,7 @@ var SipCallEngine = class {
         const code = missed ? "call/missed" : CALL_END_CODES[event.reason];
         if (code) this.#reportError(code, event.cause);
         if (this.#activeCall === call) this.#activeCall = null;
-        if (this.provider.registrationMode === "on-demand") this.#teardownUA();
+        this.#maybeTeardownUA();
       }
     });
     return call;
@@ -716,12 +723,38 @@ var SipCallEngine = class {
       mediaConstraints: buildAudioConstraints(this.#inputDeviceId),
       pcConfig: PC_CONFIG
     });
-    return new SipEngineCall(consultSession, {
+    const consultCall = new SipEngineCall(consultSession, {
       id: nextCallId(),
       buildTarget: (x) => this.#buildTarget(x),
       placeConsult: () => Promise.reject(new Error("cannot consult from a consult leg")),
       iceGatheringCapMs: this.provider.quirks?.iceGatheringCapMs ?? DEFAULT_ICE_CAP_MS
     });
+    this.#consultCall = consultCall;
+    consultCall.onEvent((event) => {
+      if (event.type === "ended") {
+        if (this.#consultCall === consultCall) this.#consultCall = null;
+        this.#maybeTeardownUA();
+      }
+    });
+    return consultCall;
+  }
+  /**
+   * On-demand registration (SPEC §3.3): the UA is disposable once NOTHING is left on it — but "the
+   * main leg ended" (or "no INVITE followed this push") is not the same fact as "nothing is left."
+   * `completeTransfer()` (SPEC §5.6) ends the main leg via a REFER…Replaces BYE *while the consult leg
+   * is still what the PBX needs to bridge the caller onto* — tearing the UA down (and with it, EVERY
+   * session on it, via `terminateSessions`) the instant the main leg's `ended` fires killed that
+   * consult leg out from under the transfer before the bridge could use it, and the caller was simply
+   * gone. Two real call sites hit this independently (found live, 2026-08-04): the main leg's own
+   * `ended` handler, and `#armNoInviteTeardown`'s 5s timer — the latter re-arms on EVERY push,
+   * including unrelated "answered" pushes Zadarma sends around other account activity, so it can fire
+   * minutes after a transfer with no `#activeCall` and no idea a consult leg is mid-handoff. Both now
+   * route through here rather than checking `#activeCall` alone.
+   */
+  #maybeTeardownUA() {
+    if (this.provider.registrationMode !== "on-demand") return;
+    if (this.#activeCall || this.#consultCall) return;
+    this.#teardownUA();
   }
   #startRejectGuard() {
     const ms = this.provider.quirks?.rejectGuardMs;