@decentnetwork/peer 0.1.151 → 0.1.152

package/dist/peer.d.ts

@@ -80,13 +80,31 @@ export declare class Peer {
     };
     addKnownNodes(nodes: NetworkNode[]): void;
     knownNodes(): NetworkNode[];
-    sendFriendRequest(pubkey: string, hello?: string): Promise<void>;
+    /** @param opts.force  Send even when this friend looks already-accepted.
+     *
+     *  The idempotency guard below is right for automatic retries and wrong for
+     *  a person clicking "add". If the other side REJECTED an earlier request,
+     *  their app forgets it — but a net_crypto session may already have been
+     *  established, so this side keeps acceptedAt and skips every later attempt.
+     *  Neither peer is then in the other's friend list and nothing either user
+     *  does can fix it: every add is silently dropped here. A deliberate add has
+     *  to be able to say "I mean it". */
+    sendFriendRequest(pubkey: string, hello?: string, opts?: {
+        force?: boolean;
+    }): Promise<void>;
     lastFriendRequestDispatch(): {
         transport: "onion" | "direct";
         routes: number;
         targets: number;
     } | undefined;
     acceptFriendRequest(pubkey: string): Promise<void>;
+    /** Decline an inbound request.
+     *
+     *  Also drops any friend record and session the request already created.
+     *  Clearing only the pending list left the transport half-attached: the
+     *  sender saw a session, marked us accepted, and its idempotency guard then
+     *  skipped every later attempt — so a rejected peer could never ask again,
+     *  and neither side could see why. */
     rejectFriendRequest(pubkey: string): void;
     /**
      * Drop a friend entirely: tear down any active net_crypto session, forget

package/dist/peer.js

@@ -1040,7 +1040,16 @@ export class Peer {
     knownNodes() {
         return [...this.#knownNodes];
     }
-    async sendFriendRequest(pubkey, hello) {
+    /** @param opts.force  Send even when this friend looks already-accepted.
+     *
+     *  The idempotency guard below is right for automatic retries and wrong for
+     *  a person clicking "add". If the other side REJECTED an earlier request,
+     *  their app forgets it — but a net_crypto session may already have been
+     *  established, so this side keeps acceptedAt and skips every later attempt.
+     *  Neither peer is then in the other's friend list and nothing either user
+     *  does can fix it: every add is silently dropped here. A deliberate add has
+     *  to be able to say "I mean it". */
+    async sendFriendRequest(pubkey, hello, opts) {
         if (!this.#keyPair || !this.#announceDataKey) {
             throw new Error("Peer is not started");
         }
@@ -1056,7 +1065,7 @@ export class Peer {
         const friendAddress = parseCarrierAddress(pubkey);
         const friendId = carrierIdFromPublicKey(friendAddress.publicKey);
         const existing = this.#friends.get(friendId);
-        if (existing?.acceptedAt) {
+        if (existing?.acceptedAt && !opts?.force) {
             this.#debugLog(`friend request skipped — ${friendId} already accepted at ${new Date(existing.acceptedAt).toISOString()}`);
             this.#lastFriendRequestDispatch = {
                 transport: "onion",
@@ -1237,8 +1246,17 @@ export class Peer {
             this.#debugLog(`accept friend: initiate session failed for ${pubkey}: ${error.message}`);
         });
     }
+    /** Decline an inbound request.
+     *
+     *  Also drops any friend record and session the request already created.
+     *  Clearing only the pending list left the transport half-attached: the
+     *  sender saw a session, marked us accepted, and its idempotency guard then
+     *  skipped every later attempt — so a rejected peer could never ask again,
+     *  and neither side could see why. */
     rejectFriendRequest(pubkey) {
         this.#pendingFriendRequests.delete(pubkey);
+        if (this.#friends.has(pubkey))
+            this.removeFriend(pubkey);
     }
     /**
      * Drop a friend entirely: tear down any active net_crypto session, forget

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decentnetwork/peer",
-  "version": "0.1.151",
+  "version": "0.1.152",
   "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",