homebridge-tuya-plus 3.14.0-dev.43 → 3.14.0-dev.44

package/lib/TuyaCloudDevice.js

@@ -215,7 +215,22 @@ class TuyaCloudDevice extends EventEmitter {
         // first 'online' is caught. On any (re)connect we re-read the full state
         // to recover anything missed while the stream was down.
         this.messaging.on('online', () => this._refreshState());
-        this.messaging.subscribeDevice(this.context.id, status => this._applyStatus(status));
+        this.messaging.subscribeDevice(this.context.id, status => this._onRealtime(status));
+    }
+
+    // A realtime (MQTT) update arrived for this device, so it reports its changes
+    // over the stream and the post-write catch-up read is redundant — drop any
+    // pending one. A device whose reports never arrive over MQTT (some custom
+    // controllers — the stream carries nothing for them) gets none of these, so its
+    // catch-up stays armed and remains the only way its state reaches HomeKit. This
+    // makes the catch-up self-cancelling: it costs an HTTP read only on devices that
+    // actually need it.
+    _onRealtime(status) {
+        if (this._catchupTimers) {
+            this._catchupTimers.forEach(t => clearTimeout(t));
+            this._catchupTimers = null;
+        }
+        this._applyStatus(status);
     }
 
     async _refreshState() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homebridge-tuya-plus",
-  "version": "3.14.0-dev.43",
+  "version": "3.14.0-dev.44",
   "description": "A community-maintained Homebridge plugin for controlling Tuya devices in HomeKit. LAN-first, with an optional Tuya Cloud fallback for any device the LAN can't reach.",
   "main": "index.js",
   "scripts": {

package/test/TuyaCloudDevice.test.js

@@ -342,6 +342,19 @@ describe('TuyaCloudDevice', () => {
             dev.stop();
         });
 
+        test('a realtime update cancels the pending catch-up (no redundant read on MQTT-covered devices)', async () => {
+            const api = makeApi();
+            api.getShadowProperties = jest.fn().mockResolvedValue([{code: 'switch_1', dp_id: 1, value: false}]);
+            const dev = makeDevice(api, null, {key: 'abc'});
+            await dev._connect();
+            await dev.update({'1': true});
+            expect(Array.isArray(dev._catchupTimers)).toBe(true);
+
+            dev._onRealtime([{code: 'switch_1', value: true}]); // MQTT delivers → catch-up not needed
+            expect(dev._catchupTimers).toBeNull();
+            dev.stop();
+        });
+
         test('a refresh reads via the shadow, so a thing-model device whose /status is empty still updates', async () => {
             const api = makeApi();
             let rs = 13;