whalibmob 5.14.14 → 5.14.15

package/lib/Client.js

@@ -407,7 +407,11 @@ class WhalibmobClient extends EventEmitter {
     // groupJid → parsed metadata.  Mirrors DeviceManager's device cache: a TTL
     // so membership changes we never saw a notification for still heal, plus
     // in-flight dedup so a burst of sends to the same group issues one IQ.
-    this._groupMetaCache    = new NodeCache({ ttl: GROUP_META_CACHE_TTL });
+    // `stdTTL`, not `ttl` — see the note in DeviceManager. Written the other
+    // way this held a group's participant list for the life of the connection,
+    // so anyone who joined or left after it was first read stayed wrong until
+    // a reconnect or an explicit { force: true }.
+    this._groupMetaCache    = new NodeCache({ stdTTL: GROUP_META_CACHE_TTL });
     this._groupMetaInflight = new Map();  // groupJid → Promise<metadata>
     this._groupMembers     = new Map();   // groupJid → Set<memberJid>
     this._lidToPn          = new Map();   // LID user → phone number

package/lib/DeviceManager.js

@@ -153,7 +153,16 @@ class DeviceManager {
   constructor(client) {
     this._client        = client;
     // phone → deviceId[]  (TTL-based: expires after 5 min, triggers fresh usync)
-    this._deviceCache   = new NodeCache({ ttl: DEVICE_CACHE_TTL });
+    //
+    // The option is `stdTTL`. It was `ttl`, which is the name of the per-entry
+    // argument to set(), not of the default in the constructor — an unknown
+    // option is simply ignored, so the cache was built with no expiry at all
+    // and a contact's device list, once learned, was never asked about again
+    // for the life of the connection. Someone who linked WhatsApp Web or a
+    // desktop while the client was up stayed invisible: their new device was
+    // left out of every stanza, and a stanza that does not cover a recipient's
+    // current devices is refused by the server.
+    this._deviceCache   = new NodeCache({ stdTTL: DEVICE_CACHE_TTL });
     // phone → Promise<jids[]>  (in-flight usync dedup)
     this._usyncInflight = new Map();
     // own device list cache + expiry timestamp (refreshed every 5 min)
@@ -259,7 +268,20 @@ class DeviceManager {
   _saveToDisk() {
     if (!this._sessionDir || !this._cacheFile) return;
     try {
-      const entries = Object.assign({}, this._cacheSnapshot);
+      // The snapshot mirrors the cache on every write, but nothing tells it
+      // when an entry expires — so once entries actually start expiring it
+      // keeps them, and writing it out unfiltered would put a device list on
+      // disk after the cache had already decided it was too old to trust. The
+      // next start reads that file back in, and the expiry never happens.
+      //
+      // Ask the cache what it still holds. `keys()` is no use here: it lists
+      // expired entries too, until the periodic sweep gets to them. `has()`
+      // answers for the entry itself.
+      const entries = {};
+      for (const key of Object.keys(this._cacheSnapshot)) {
+        if (this._deviceCache.has(key)) entries[key] = this._cacheSnapshot[key];
+        else delete this._cacheSnapshot[key];
+      }
 
       // ── Combined cache file ─────────────────────────────────────────────────
       fs.writeFileSync(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "whalibmob",
-  "version": "5.14.14",
+  "version": "5.14.15",
   "description": "WhatsApp library for interaction with WhatsApp Mobile API and web ",
   "author": "Kunboruto20",
   "main": "index.js",