homebridge-winix-purifiers 2.2.3 → 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/platform.js

@@ -22,7 +22,6 @@ class WinixPurifierPlatform {
         this.handlers = new Map();
         this.deviceOverrides = (this.config.deviceOverrides ?? [])
             .reduce((m, o) => m.set(o.deviceId, o), new Map());
-        this.client = new winix_api_1.WinixClient();
         this.winix = new winix_1.WinixHandler(api.user.storagePath(), settings_1.ENCRYPTION_KEY);
         this.api.on("didFinishLaunching" /* APIEvent.DID_FINISH_LAUNCHING */, this.onFinishLaunching.bind(this));
     }
@@ -52,6 +51,7 @@ class WinixPurifierPlatform {
             this.log.error('error getting devices:', e.message);
             return;
         }
+        this.client = new winix_api_1.WinixClient(this.winix.getIdentityId());
         await this.discoverDevices();
         // refresh devices on the configured interval
         const refreshIntervalMs = (this.config.deviceRefreshIntervalMinutes ?? DEFAULT_DEVICE_REFRESH_INTERVAL_MINUTES) * 60 * 1000;

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);
     }
     /**
@@ -95,6 +97,12 @@ class WinixHandler {
             return '';
         }
     }
+    getIdentityId() {
+        if (!this.winix) {
+            throw new UnauthenticatedError();
+        }
+        return this.winix.getIdentityId();
+    }
     async getDevices() {
         if (!this.winix || !this.auth) {
             throw new UnauthenticatedError();
@@ -104,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.3",
+  "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": "1.9.0"
+    "winix-api": "2.0.1"
   },
   "devDependencies": {
     "@types/node": "20.11.0",