homebridge-winix-purifiers 2.2.5 → 2.2.6

package/dist/characteristic.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CharacteristicWrapper = exports.CharacteristicManager = void 0;
+const winix_api_1 = require("winix-api");
 const errors_1 = require("./errors");
 /**
  * Manager to provide a consistent way to get and set characteristics
@@ -62,7 +63,15 @@ class CharacteristicWrapper {
         }
         catch (e) {
             (0, errors_1.assertError)(e);
-            this.log.error('error calling Winix API:', e.message);
+            // Known transient upstream conditions (rate limit, 5xx, DNS, malformed body)
+            // still fail the HAP call so HomeKit knows the action didn't land, but log at
+            // warn without the implication of a plugin bug.
+            if (e instanceof winix_api_1.RateLimitError || e instanceof winix_api_1.UpstreamUnavailableError) {
+                this.log.warn(`Winix API transient: ${e.constructor.name}: ${e.message}`);
+            }
+            else {
+                this.log.error('error calling Winix API:', e.message);
+            }
             throw new this.platform.HapStatusError(-70402 /* HAPStatus.SERVICE_COMMUNICATION_FAILURE */);
         }
     }

package/dist/device.js

@@ -5,6 +5,9 @@ const winix_api_1 = require("winix-api");
 const MAX_BACKOFF_MS = 5 * 60 * 1000;
 const COMMAND_DELAY_MS = 1500;
 const UNREACHABLE_THRESHOLD = 3;
+// 90s clears WinixClient's internal 60s rate-limit cooldown plus a small buffer,
+// so the first poll after a rate-limited initialFetch actually has a chance to succeed.
+const FAILED_FETCH_RETRY_MS = 90 * 1000;
 class Device {
     constructor(deviceId, pollIntervalMs, log, client) {
         this.deviceId = deviceId;
@@ -41,8 +44,8 @@ class Device {
             this.log.debug('device:initialFetch()', JSON.stringify(this.state));
         }
         catch (e) {
-            if (e instanceof winix_api_1.RateLimitError) {
-                this.log.warn('device:initialFetch() rate limited, using defaults');
+            if (e instanceof winix_api_1.RateLimitError || e instanceof winix_api_1.UpstreamUnavailableError) {
+                this.log.warn(`device:initialFetch() ${e.constructor.name}, using defaults (will retry soon)`);
                 return;
             }
             this.log.warn('device:initialFetch() failed, using defaults:', e.message);
@@ -50,10 +53,14 @@ class Device {
     }
     startPolling(onUpdate) {
         this.onUpdate = onUpdate;
-        // Stagger the first poll with a random delay to avoid all devices
-        // hitting the API at the same time
-        const jitter = Math.floor(Math.random() * this.pollIntervalMs);
-        this.schedulePoll(jitter);
+        // If initialFetch failed (rate limited, upstream unavailable, etc.), the device
+        // is unreachable in HomeKit until the first successful poll. Skip the full-interval
+        // jitter and retry soon to close that window. Otherwise stagger the first poll
+        // with a random delay to avoid all devices hitting the API at the same time.
+        const delay = this.hasReceivedData
+            ? Math.floor(Math.random() * this.pollIntervalMs)
+            : Math.min(FAILED_FETCH_RETRY_MS, this.pollIntervalMs);
+        this.schedulePoll(delay);
     }
     resetPollTimer(delayMs = 3000) {
         this.schedulePoll(delayMs);
@@ -168,10 +175,10 @@ class Device {
                 this.schedulePoll(this.pollIntervalMs);
                 return;
             }
-            // Winix occasionally returns "no data" for a healthy device. Treat as
-            // transient: keep last-known state and don't count toward unreachable.
-            if (e instanceof winix_api_1.NoDataError) {
-                this.log.debug('device:poll() got NoDataError, retaining last-known state');
+            // Transient upstream conditions (Winix "no data", 5xx, DNS/connect failures,
+            // malformed bodies). Keep last-known state and don't count toward unreachable.
+            if (e instanceof winix_api_1.NoDataError || e instanceof winix_api_1.UpstreamUnavailableError) {
+                this.log.debug(`device:poll() ${e.constructor.name}, retaining last-known state`);
                 this.schedulePoll(this.pollIntervalMs);
                 return;
             }

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Winix Air Purifiers",
   "name": "homebridge-winix-purifiers",
-  "version": "2.2.5",
+  "version": "2.2.6",
   "description": "Homebridge plugin for Winix air purifiers",
   "license": "Apache-2.0",
   "repository": {
@@ -66,7 +66,7 @@
   ],
   "dependencies": {
     "@homebridge/plugin-ui-utils": "1.0.3",
-    "winix-api": "2.0.1"
+    "winix-api": "2.0.2"
   },
   "devDependencies": {
     "@types/node": "20.11.0",