@decentnetwork/peer 0.1.87 → 0.1.88

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.88";
 // 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
@@ -2057,50 +2057,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.88",
   "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",