@decentnetwork/lan 0.1.9 → 0.1.11

package/dist/daemon/server.js

@@ -90,21 +90,26 @@ export class DaemonServer {
             // optional dora registration can decide our IP.
             const keyFile = resolve(this.config.carrier.dataDir, "keypair.json");
             this.peerManager = new PeerManager();
-            // Use express nodes. Previously the daemon passed an empty array
-            // here on the theory that "express is for offline messages, not
-            // live packet forwarding". But asymmetric Carrier sessions (peer
-            // A thinks B is online, B thinks A is offline — common across
-            // China-WAN paths) make dora's sendText reply fail direct and
-            // fall back to express; if we don't subscribe to express we
-            // never see the response. dora -> cn for register-ok / list-ok
-            // hits this exact case. The cost of enabling express here is
-            // that idle text messages may take an extra hop through the
-            // HTTPS relay — packet-router doesn't use sendText, so its
-            // hot path is unaffected.
+            // Daemon does NOT use express nodes. decentlan is a virtual
+            // LAN — peers must be ONLINE for IP packets to flow. Express
+            // is an offline-message store-and-forward relay for chat-style
+            // apps; if we enable it, sendText silently falls back to a
+            // queue when a friend is offline. That creates the illusion of
+            // connectivity ("the daemon says X is reachable") while
+            // packets actually pile up in HTTPS storage and never get
+            // forwarded in real time. For a VPN-like data plane, that's
+            // wrong — better to fail fast and let the operator see the
+            // peer as offline.
+            //
+            // Friend-request bootstrap is the only thing that benefits
+            // from express; for that case use the standalone CLI
+            // `agentnet friend-request` (which DOES use express via its
+            // own short-lived Peer instance) and accept the request on
+            // both ends before bringing the daemon up.
             await this.peerManager.create({
                 keyFile,
                 bootstrapNodes: this.config.carrier.bootstrapNodes,
-                expressNodes: this.config.carrier.expressNodes ?? [],
+                expressNodes: [],
             });
             await this.peerManager.start();
             this.logger.info(`Identity: ${this.peerManager.getAddress()}`);
@@ -279,10 +284,21 @@ export class DaemonServer {
             const autoAcceptFriends = this.config.friends?.autoAccept ?? true;
             const pendingPath = resolve(this.config.carrier.dataDir, "..", "pending-friends.json");
             this.pendingFriends = new PendingFriendsStore(pendingPath);
-            const pubkeyHexToUserid = async (pubkeyHex) => {
-                const { carrierIdFromPublicKey } = await import("@decentnetwork/peer");
-                const bytes = Buffer.from(pubkeyHex, "hex");
-                return carrierIdFromPublicKey(new Uint8Array(bytes));
+            const pubkeyHexToUserid = async (input) => {
+                // The SDK's friend-request event hands us EITHER a 64-char
+                // hex pubkey OR a base58 userid directly, depending on
+                // version. Detect by character set: hex is exactly 64 chars
+                // of [0-9a-f]; otherwise treat input as already-base58 userid
+                // and pass through. This caught us when the SDK switched
+                // formats and decentlan crashed the daemon (unhandled
+                // throw from carrierIdFromPublicKey because the base58
+                // bytes-from-hex was the wrong length).
+                if (/^[0-9a-fA-F]{64}$/.test(input)) {
+                    const { carrierIdFromPublicKey } = await import("@decentnetwork/peer");
+                    const bytes = Buffer.from(input, "hex");
+                    return carrierIdFromPublicKey(new Uint8Array(bytes));
+                }
+                return input;
             };
             this.peerManager.on("friend-request", (req) => {
                 void pubkeyHexToUserid(req.pubkey).then((userid) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decentnetwork/lan",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "Private virtual LAN for self-hosted services and AI agents, built on Elastos Carrier. NAT-traversal, name service, ACL, all over a peer-to-peer mesh — no public IP required.",
   "type": "module",
   "main": "./dist/index.js",