@decentnetwork/peer 0.1.87 → 0.1.89

package/dist/peer.d.ts

@@ -131,6 +131,7 @@ export declare class Peer {
     sendInvite(pubkey: string, data: Uint8Array | string, opts?: {
         ext?: string | null;
         bundle?: string;
+        establishTimeoutMs?: number;
     }): Promise<bigint>;
     /**
      * Send an application-defined custom packet to a friend. `id` selects the

package/dist/peer.js

@@ -174,7 +174,7 @@ const AGENTNET_PROTO_VERSION = 1;
 // Peer package version, advertised as the default appVersion when the embedder
 // doesn't override it. Read lazily so a bundler that inlines this file doesn't
 // need the package.json at runtime.
-const PEER_PKG_VERSION = "0.1.87";
+const PEER_PKG_VERSION = "0.1.89";
 // Toxcore Messenger.h packet IDs (live inside encrypted 0x1b crypto data plain payload)
 const PACKET_ID_PADDING = 0;
 const PACKET_ID_REQUEST = 1; // request retransmission of unreceived packets
@@ -1217,10 +1217,14 @@ export class Peer {
         // extension the native WebRTC SDK registers under.
         const ext = opts.ext === null ? undefined : (opts.ext ?? CARRIER_EXTENSION_NAME);
         const bundle = opts.bundle;
+        // How long to wait for a session to (re)establish before giving up. A caller
+        // that wants to FAIL FAST (e.g. to fall back to an offline transport instead
+        // of blocking the UI ~5s per signal) passes establishTimeoutMs: 0.
+        const establishTimeoutMs = opts.establishTimeoutMs ?? 5000;
         let session = this.#friendSessions.get(pubkey);
-        if (!session?.established) {
+        if (!session?.established && establishTimeoutMs > 0) {
             await this.#initiateSession(pubkey).catch(() => { });
-            const established = await this.#waitForFriendConnected(pubkey, 5000).catch(() => false);
+            const established = await this.#waitForFriendConnected(pubkey, establishTimeoutMs).catch(() => false);
             if (established)
                 session = this.#friendSessions.get(pubkey);
         }
@@ -2057,50 +2061,24 @@ export class Peer {
                 //    deliver immediately (a reorder buffer would only add latency and
                 //    would wait forever for retransmits that arrive at fresh numbers);
                 //    they still advance the ack point eagerly, exactly as before.
-                //  - Everything else is the DUAL-SENT reliable message stream (chat 64,
-                //    bulkmsg, invites, profile, control): delivered STRICTLY in order
-                //    through a toxcore-style recv_array (below) so a reordered or
-                //    retransmitted fragment is neither delivered early nor dropped as a
-                //    duplicate. Advancing receiveBufferStart past a gap (the old bug)
-                //    ACKed a packet we never delivered, so the sender never retransmit —
-                //    corrupting every multi-fragment message (large inline files from
-                //    native peers surfaced as "bulkmsg totalsz 0 dropped").
+                //  - Everything else is a reliable message (chat 64, bulkmsg, invites,
+                //    profile, control): deliver immediately and advance the ack point.
+                //
+                // NOTE: a strict in-order recv_array (buffer-ahead + retransmit-request)
+                // was tried here to fix reordered multi-fragment inline files, but it
+                // STALLED the whole reliable stream against native peers whose retransmit
+                // didn't recover our gaps — breaking chat + files in BOTH directions
+                // (the stalled ack point starved the peer's send window). Reverted to the
+                // forgiving skip-the-gap behavior: a lost fragment corrupts one message
+                // (rare) instead of wedging the session. Large files should use the
+                // toxcore file-transfer channel (sendFile), which has its own reliability.
                 const consumesReliableNumber = kind !== PACKET_ID_REQUEST &&
                     kind < 192 &&
                     kind !== this.#opts.bulkDataPacketId;
-                if (!consumesReliableNumber || isSingleTransportKind) {
-                    if (consumesReliableNumber) {
-                        state.receiveBufferStart = Math.max(state.receiveBufferStart ?? 0, (opened.packetNumber + 1) >>> 0);
-                    }
-                    this.#deliverLosslessPayload(friendId, state, kind, inner, remote);
-                    return;
-                }
-                const expected = state.receiveBufferStart ?? 0;
-                const ahead = (opened.packetNumber - expected) >>> 0;
-                if (ahead >= 0x80000000) {
-                    // Below the low-water mark — already delivered. Duplicate; drop.
-                    return;
-                }
-                if (ahead === 0) {
-                    // In order: deliver, advance, then drain any now-contiguous buffered.
-                    state.receiveBufferStart = (expected + 1) >>> 0;
-                    this.#deliverLosslessPayload(friendId, state, kind, inner, remote);
-                    this.#drainRecvBufferContiguous(friendId, state, remote);
-                    return;
-                }
-                // A gap sits below this packet. Hold it and drive recovery: ask the
-                // sender to retransmit the missing number(s). If the buffer outgrows the
-                // reorder window the gap is a genuine loss that won't come back — force
-                // the stream forward rather than wedge every later packet forever.
-                const buf = (state.recvBuffer ??= new Map());
-                if (!buf.has(opened.packetNumber))
-                    buf.set(opened.packetNumber, { kind, inner });
-                if (buf.size > RECV_REORDER_WINDOW) {
-                    this.#forceAdvanceRecvBuffer(friendId, state, remote);
-                }
-                else {
-                    this.#requestMissingReliablePackets(friendId, state);
+                if (consumesReliableNumber) {
+                    state.receiveBufferStart = Math.max(state.receiveBufferStart ?? 0, (opened.packetNumber + 1) >>> 0);
                 }
+                this.#deliverLosslessPayload(friendId, state, kind, inner, remote);
                 return;
             }
             // Reverted: we used to DELETE a "desynced" session here to force a clean

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decentnetwork/peer",
-  "version": "0.1.87",
+  "version": "0.1.89",
   "description": "Pure TypeScript port of Elastos Carrier (toxcore-derived) P2P messaging. DHT, onion routing, TCP relay, FlatBuffers app payloads, Express offline relay. Wire-compatible with iOS Beagle and the Carrier C SDK.",
   "type": "module",
   "main": "./dist/index.js",