zas-agent 0.6.1 → 0.6.3

package/CHANGELOG.md

@@ -7,6 +7,55 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.6.3] - 2026-09-04
+
+### Fixed
+
+- The selected candidate pair is read from the transport, which names its
+  own pair on every engine. A transfer that connected and carried its whole
+  file could report `pair=none`, because the browser at the other end does
+  not mark a pair `nominated` on the controlled side. The empty field was
+  read as a missing pair and sent two rounds of diagnosis after a
+  connection that had been there all along.
+- A peer connection state of `failed` has to hold before the engine acts on
+  it. Chrome reports `failed` while it is still gathering and returns to
+  `connecting` in the same millisecond; acting on the first one killed a
+  receiver at 0.88 s with pairs still waiting to be tried. The settle window
+  is shorter than the ICE grace, so a connection that really is dead still
+  gives up sooner than it did before.
+
+### Added
+
+- The trace counts candidate pairs by check state just before the verdict:
+  `checks failed=24 in-progress=4`, or `checks none`. No pair selected could
+  not tell a run that formed no pairs from one that formed pairs and lost
+  every check, and those are different faults.
+- A candidate the peer refuses is traced as `ice apply rejected <type>
+  <ErrorName>`. The rejection used to be swallowed, so a refused candidate
+  looked exactly like one the network lost.
+
+## [0.6.2] - 2026-09-04
+
+### Added
+
+- The trace records the peer connection state, not only the ICE state.
+  ICE `connected` means a candidate pair answered; the peer connection
+  reaches `connected` only once DTLS completes. A session that holds an ICE
+  path and sends nothing over it used to look the same as one that never
+  found a path at all.
+- A failure that had a working ICE pair is now `channel_never_opened`, not
+  `connect_timeout`. Same clock, different fault: one is the candidate hunt,
+  the other is the handshake above it.
+- The verdict line carries the peer connection state and the data channel's
+  own state, so a stalled DTLS handshake and a stalled SCTP association can
+  be told apart from one line.
+- The selected candidate pair is read when ICE connects, not only when the
+  data channel opens. A run that connected and then went quiet reported
+  `pair=none` and looked as if it had never selected one.
+- `onicecandidateerror` is traced, by URL shape and error code only. A
+  gather that ends with no relay candidate now says whether the allocation
+  was refused or never answered.
+
 ## [0.6.1] - 2026-09-04
 
 ### Added
@@ -183,7 +232,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   two separate identities that cannot read each other's keys.
 - Install snippets for Claude Code and Codex, printed by `zas-agent pair`.
 
-[Unreleased]: https://github.com/soke1556/zas-agent/compare/v0.6.1...HEAD
+[Unreleased]: https://github.com/soke1556/zas-agent/compare/v0.6.3...HEAD
+[0.6.3]: https://github.com/soke1556/zas-agent/compare/v0.6.2...v0.6.3
+[0.6.2]: https://github.com/soke1556/zas-agent/compare/v0.6.1...v0.6.2
 [0.6.1]: https://github.com/soke1556/zas-agent/compare/v0.6.0...v0.6.1
 [0.6.0]: https://github.com/soke1556/zas-agent/compare/v0.5.1...v0.6.0
 [0.5.1]: https://github.com/soke1556/zas-agent/compare/v0.5.0...v0.5.1

package/dist/cli.js

@@ -854,6 +854,8 @@ function directFallbackCipherSize(size) {
 }
 var DIRECT_CONNECT_TIMEOUT_MS = 30 * 1e3;
 var DIRECT_STALL_GRACE_MS = 5 * 1e3;
+var DIRECT_CENSUS_WAIT_MS = 250;
+var DIRECT_CONN_FAILED_SETTLE_MS = 1500;
 var DIRECT_SIGNAL_WAIT_MS = 90 * 1e3;
 
 // src/shared/direct-engine.ts
@@ -969,14 +971,17 @@ function traceAdd(diag, line) {
   const at = t0 === void 0 ? 0 : (Date.now() - t0) / 1e3;
   diag.trace.push(`${at.toFixed(2)} ${line}`);
 }
+function urlShape(url) {
+  const scheme = /^(stuns?|turns?):/i.exec(url)?.[1]?.toLowerCase() ?? "other";
+  const transport = /[?&]transport=(\w+)/i.exec(url)?.[1]?.toLowerCase();
+  return transport ? `${scheme}/${transport}` : scheme;
+}
 function iceShapes(servers) {
   const seen = /* @__PURE__ */ new Map();
   for (const server of servers ?? []) {
     const urls = typeof server.urls === "string" ? [server.urls] : server.urls;
     for (const url of urls ?? []) {
-      const scheme = /^(stuns?|turns?):/i.exec(url)?.[1]?.toLowerCase() ?? "other";
-      const transport = /[?&]transport=(\w+)/i.exec(url)?.[1]?.toLowerCase();
-      const shape = transport ? `${scheme}/${transport}` : scheme;
+      const shape = urlShape(url);
       seen.set(shape, (seen.get(shape) ?? 0) + 1);
     }
   }
@@ -1019,17 +1024,36 @@ function countTurnUrls(servers) {
 }
 function pairFromStats(stats) {
   let pair;
+  let selectedId;
   stats.forEach((s) => {
+    if (s.type === "transport" && typeof s.selectedCandidatePairId === "string") {
+      selectedId = s.selectedCandidatePairId;
+    }
     if (s.type === "candidate-pair" && s.state === "succeeded" && (s.nominated || s.selected)) {
       pair = s;
     }
   });
+  if (selectedId) {
+    const named = stats.get(selectedId);
+    if (named?.localCandidateId && named.remoteCandidateId) pair = named;
+  }
   if (!pair?.localCandidateId || !pair.remoteCandidateId) return void 0;
   const local = stats.get(pair.localCandidateId);
   const remote = stats.get(pair.remoteCandidateId);
   if (!local?.candidateType || !remote?.candidateType) return void 0;
   return { local: local.candidateType, remote: remote.candidateType };
 }
+function checkCensus(stats) {
+  const counts = /* @__PURE__ */ new Map();
+  stats.forEach((s) => {
+    if (s.type !== "candidate-pair") return;
+    const state = String(s.state ?? "?");
+    counts.set(state, (counts.get(state) ?? 0) + 1);
+  });
+  if (counts.size === 0) return "checks none";
+  const parts = [...counts].map(([state, n]) => `${state}=${n}`);
+  return `checks ${parts.join(" ")}`;
+}
 function pathOf(pair) {
   if (pair.local === "relay" || pair.remote === "relay") return "relay";
   return pair.local === "host" && pair.remote === "host" ? "lan" : "wan";
@@ -1056,6 +1080,7 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
   let phase = "connecting";
   let disconnectTimer;
   let restartTimer;
+  let connFailTimer;
   let stallState = "idle";
   const startedAt = Date.now();
   const diag = {
@@ -1081,14 +1106,39 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
   pc.onicegatheringstatechange = () => {
     traceAdd(diag, `gather ${String(pc.iceGatheringState ?? "")}`);
   };
+  const connState = () => String(pc.connectionState ?? "");
+  pc.onconnectionstatechange = () => {
+    const state = connState();
+    traceAdd(diag, `conn ${state}`);
+    if (state !== "failed") {
+      clearTimeout(connFailTimer);
+      connFailTimer = void 0;
+      return;
+    }
+    if (connFailTimer !== void 0) return;
+    connFailTimer = setTimeout(() => {
+      connFailTimer = void 0;
+      if (connState() !== "failed") return;
+      if (diag.msConnect !== void 0 && onStall) stalled();
+      else failWithCensus("ice_failed");
+    }, DIRECT_CONN_FAILED_SETTLE_MS);
+  };
+  let channel;
+  pc.onicecandidateerror = (e) => {
+    const err = e;
+    traceAdd(diag, `ice error ${err.url ? urlShape(err.url) : "?"} code=${err.errorCode ?? "?"}`);
+  };
   const cleanup = () => {
     clearTimeout(waitTimer);
+    clearTimeout(connFailTimer);
     clearTimeout(connectTimer);
     clearTimeout(disconnectTimer);
     clearTimeout(restartTimer);
     pc.onicecandidate = null;
     pc.oniceconnectionstatechange = null;
     pc.onicegatheringstatechange = null;
+    pc.onicecandidateerror = null;
+    pc.onconnectionstatechange = null;
     pc.close();
   };
   const setPhase = (p) => {
@@ -1100,7 +1150,10 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
       diag.gatherState = String(pc.iceGatheringState ?? "");
       diag.hadRemoteDesc = hasRemoteDescription(pc);
       diag.turnUrlsConfigured = countTurnUrls(pc.getConfiguration().iceServers);
-      traceAdd(diag, `${p}${diag.reason ? ` ${diag.reason}` : ""} ice=${diag.iceState} gather=${diag.gatherState}`);
+      traceAdd(
+        diag,
+        `${p}${diag.reason ? ` ${diag.reason}` : ""} ice=${diag.iceState} gather=${diag.gatherState} conn=${connState()} chan=${channel === void 0 ? "-" : String(channel.readyState ?? "")}`
+      );
       onDiag?.(diag);
       onPhase(p);
       cleanup();
@@ -1117,6 +1170,20 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
     }
     setPhase("failed");
   };
+  const failWithCensus = (reason) => {
+    if (phase === "done" || phase === "failed") return;
+    let reported = false;
+    const report2 = () => {
+      if (reported) return;
+      reported = true;
+      fail(reason);
+    };
+    const guard = setTimeout(report2, DIRECT_CENSUS_WAIT_MS);
+    void pc.getStats().then((stats) => traceAdd(diag, checkCensus(stats))).catch(() => void 0).then(() => {
+      clearTimeout(guard);
+      report2();
+    });
+  };
   const waitTimer = setTimeout(() => fail("peer_silent"), DIRECT_SIGNAL_WAIT_MS);
   let connectTimer;
   let heard = false;
@@ -1124,7 +1191,10 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
     if (heard) return;
     heard = true;
     clearTimeout(waitTimer);
-    connectTimer = setTimeout(() => fail("connect_timeout"), DIRECT_CONNECT_TIMEOUT_MS);
+    connectTimer = setTimeout(() => {
+      const iceUp = pc.iceConnectionState === "connected" || pc.iceConnectionState === "completed";
+      failWithCensus(iceUp ? "channel_never_opened" : "connect_timeout");
+    }, DIRECT_CONNECT_TIMEOUT_MS);
   };
   let onStall;
   let stallWindowMs = DIRECT_CONNECT_TIMEOUT_MS;
@@ -1133,7 +1203,7 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
     clearTimeout(disconnectTimer);
     if (stallState === "restarting") return;
     if (diag.msConnect === void 0 || !onStall) {
-      fail("disconnected");
+      failWithCensus(diag.msConnect === void 0 ? "ice_failed" : "disconnected");
       return;
     }
     stallState = "restarting";
@@ -1148,7 +1218,7 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
     traceAdd(diag, `ice ${String(s ?? "")}`);
     if (s === "failed") {
       if (diag.msConnect !== void 0 && onStall) stalled();
-      else fail("ice_failed");
+      else failWithCensus("ice_failed");
     }
     if (s === "disconnected" && stallState === "idle") {
       stallState = "grace";
@@ -1158,6 +1228,13 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
       clearTimeout(disconnectTimer);
       clearTimeout(restartTimer);
       stallState = "idle";
+      void pc.getStats().then((stats) => {
+        const pair = pairFromStats(stats);
+        if (!pair || diag.pairLocal) return;
+        diag.pairLocal = pair.local;
+        diag.pairRemote = pair.remote;
+        traceAdd(diag, `pair ${pair.local}/${pair.remote}`);
+      }).catch(() => void 0);
     }
   };
   return {
@@ -1176,6 +1253,7 @@ function session(ice, onPhase, onDiag, iceTransportPolicy = "all") {
       }
     },
     isTerminal: () => phase === "done" || phase === "failed",
+    setChannel: (dc) => void (channel = dc),
     setStall: (fn, windowMs) => {
       onStall = fn;
       stallWindowMs = windowMs;
@@ -1203,8 +1281,14 @@ function signalPlumbing(pc, send, onSendError, diag) {
       generation: localGeneration
     }).catch(onSendError);
   };
+  let applyRejects = 0;
   const apply = (candidate) => {
-    void pc.addIceCandidate(candidate ?? void 0).catch(() => void 0);
+    void pc.addIceCandidate(candidate ?? void 0).catch((e) => {
+      if (applyRejects >= 3) return;
+      applyRejects++;
+      const name = e?.name || "Error";
+      traceAdd(diag, `ice apply rejected ${candidate ? candType(candidate) : "end"} ${name}`);
+    });
   };
   const applyIce = (candidate, generation) => {
     countCand(diag, "remote", candidate);
@@ -1267,6 +1351,7 @@ function startSender(opts) {
     })().catch((e) => s.fail("signaling", e));
   }, DIRECT_CONNECT_TIMEOUT_MS);
   const dc = pc.createDataChannel("zas-direct", { ordered: true });
+  s.setChannel(dc);
   dc.binaryType = "arraybuffer";
   dc.bufferedAmountLowThreshold = DIRECT_BUFFERED_HIGH / 8;
   const waitLow = () => new Promise((resolve2) => {
@@ -1415,6 +1500,7 @@ function startReceiver(opts) {
   let finished = false;
   pc.ondatachannel = (e) => {
     dc = e.channel;
+    s.setChannel(dc);
     dc.binaryType = "arraybuffer";
     dc.onopen = () => {
       s.connected();
@@ -3898,7 +3984,7 @@ async function report(client, properties) {
 
 // src/server.ts
 function agentVersion() {
-  return true ? "0.6.1" : "0.0.0-dev";
+  return true ? "0.6.3" : "0.0.0-dev";
 }
 var AGENT_CUE = " The owner sees every item this agent sends with the >_ agent mark and this agent's name, on every device.";
 var PAIR_ANNOUNCE_MS = 15e3;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zas-agent",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "MCP server that lets a coding agent send files and notes into your Zas channels, and send or receive files live through Directo.",
   "type": "module",
   "license": "MIT",