@decentnetwork/peer 0.1.145 → 0.1.147

package/dist/compat/express.d.ts

@@ -12,6 +12,7 @@ export declare class LegacyExpressClient {
         selfUserId: string;
         selfAddress: string;
         callbacks: PullCallbacks;
+        debug?: boolean;
     });
     hasNodes(): boolean;
     sendOfflineFriendRequest(address: string, hello: Uint8Array): Promise<void>;

package/dist/compat/express.js

@@ -12,8 +12,13 @@ export class LegacyExpressClient {
     #selfAddress;
     #currNode = 0;
     #callbacks;
+    // Own gate, and env-only until now — so in a browser this client was mute
+    // whatever the peer's debug setting was. That is why a stalled pull looked
+    // identical to a pull that never ran.
     #debug = process.env.DECENT_DEBUG === "1";
     constructor(opts) {
+        if (opts.debug)
+            this.#debug = true;
         this.#selfKeyPair = opts.selfKeyPair;
         this.#selfUserId = opts.selfUserId;
         this.#selfAddress = opts.selfAddress;
@@ -63,8 +68,12 @@ export class LegacyExpressClient {
             try {
                 body = await this.#http(node, "GET", encodeURIComponent(this.#selfUserId));
             }
-            catch {
-                continue; // relay unreachable right now — try the next
+            catch (error) {
+                // Say so. Swallowing this made an unreachable relay indistinguishable
+                // from an empty inbox, and offline mail piled up on the server with
+                // nothing anywhere reporting why.
+                this.#debugLog(`pull from ${node.host}:${node.port} failed: ${error?.message ?? error}`);
+                continue;
             }
             const messages = parseExpressResponseFrames(body);
             if (messages.length === 0) {

package/dist/peer.js

@@ -598,6 +598,7 @@ export class Peer {
                 selfKeyPair: this.#keyPair,
                 selfUserId: this.userid(),
                 selfAddress: this.address(),
+                debug: this.#debug,
                 callbacks: {
                     onOfflineFriendRequest: (fromUserId, packet) => {
                         this.#emitOfflineFriendRequest(fromUserId, packet);
@@ -3738,8 +3739,14 @@ export class Peer {
     }
     #ensureExpressPullLoop() {
         if (!this.#express?.hasNodes() || this.#expressPollTimer || EXPRESS_PULL_INTERVAL_MS <= 0) {
+            // Three ways to silently do nothing, and no way to tell them apart from
+            // outside — which cost a long evening of guessing why offline mail was
+            // never collected in a browser.
+            this.#debugLog(`express pull loop NOT started: hasClient=${!!this.#express} hasNodes=${!!this.#express?.hasNodes()} ` +
+                `alreadyRunning=${!!this.#expressPollTimer} interval=${EXPRESS_PULL_INTERVAL_MS}`);
             return;
         }
+        this.#debugLog(`express pull loop started, every ${EXPRESS_PULL_INTERVAL_MS}ms`);
         const pull = () => {
             void this.#express?.pullOnce().catch((error) => {
                 this.#debugLog(`express pull failed: ${error.message}`);

package/package.json

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