homebridge-winix-purifiers 2.2.4 → 2.2.5

package/dist/device.js

@@ -168,9 +168,18 @@ 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');
+                this.schedulePoll(this.pollIntervalMs);
+                return;
+            }
             this.consecutiveFailures++;
             const backoffMs = Math.min(this.pollIntervalMs * Math.pow(2, this.consecutiveFailures), MAX_BACKOFF_MS);
-            this.log.error(`device:poll() error: ${e.message} (retry in ${Math.round(backoffMs / 1000)}s)`);
+            const err = e;
+            this.log.error(`device:poll() error: ${err.constructor?.name ?? 'Error'}: ${err.message} ` +
+                `(retry in ${Math.round(backoffMs / 1000)}s)`);
             this.schedulePoll(backoffMs);
             return;
         }

package/dist/winix.js

@@ -10,6 +10,7 @@ const promises_1 = require("node:fs/promises");
 const node_path_1 = __importDefault(require("node:path"));
 const TOKEN_DIRECTORY_NAME = 'winix-purifiers';
 const TOKEN_FILE_NAME = 'token.json';
+const MIN_RELOGIN_INTERVAL_MS = 5 * 60 * 1000;
 class NotConfiguredError extends Error {
     constructor() {
         super('Account not configured');
@@ -25,6 +26,7 @@ exports.UnauthenticatedError = UnauthenticatedError;
 class WinixHandler {
     constructor(storagePath, encryptionKey) {
         this.encryptionKey = encryptionKey;
+        this.lastReloginAt = 0;
         this.refreshTokenPath = node_path_1.default.join(storagePath, TOKEN_DIRECTORY_NAME, TOKEN_FILE_NAME);
     }
     /**
@@ -110,10 +112,19 @@ class WinixHandler {
             devices = await this.winix.getDevices();
         }
         catch (e) {
-            if (!(e instanceof winix_api_1.RefreshTokenExpiredError)) {
+            if (e instanceof winix_api_1.RefreshTokenExpiredError) {
+                await this.login(this.auth.username, this.auth.password);
+                return await this.winix.getDevices();
+            }
+            if (!(e instanceof winix_api_1.MobileSessionInvalidError)) {
+                throw e;
+            }
+            // Mobile session invalidated (MULTI LOGIN / "user is not valid"). Re-login
+            // is throttled to prevent ping-pong with another active session.
+            if (Date.now() - this.lastReloginAt < MIN_RELOGIN_INTERVAL_MS) {
                 throw e;
             }
-            // if we get a refresh token expiry, we need to re-login and get devices again
+            this.lastReloginAt = Date.now();
             await this.login(this.auth.username, this.auth.password);
             return await this.winix.getDevices();
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Winix Air Purifiers",
   "name": "homebridge-winix-purifiers",
-  "version": "2.2.4",
+  "version": "2.2.5",
   "description": "Homebridge plugin for Winix air purifiers",
   "license": "Apache-2.0",
   "repository": {
@@ -46,6 +46,7 @@
     "test": "vitest run",
     "test:integration": "vitest run --config vitest.integration.config.ts tests/integration/integration.test.ts --reporter=verbose",
     "test:integration:rate-limit": "vitest run --config vitest.integration.config.ts tests/integration/integration-rate-limit.test.ts --reporter=verbose",
+    "test:integration:session-recovery": "node --env-file-if-exists=.env ./node_modules/vitest/vitest.mjs run --config vitest.integration.config.ts tests/integration/integration-session-recovery.test.ts --reporter=verbose",
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage",
     "validate": "yarn lint && yarn build && yarn test",
@@ -65,7 +66,7 @@
   ],
   "dependencies": {
     "@homebridge/plugin-ui-utils": "1.0.3",
-    "winix-api": "2.0.0"
+    "winix-api": "2.0.1"
   },
   "devDependencies": {
     "@types/node": "20.11.0",