homebridge-bluos 1.1.1 → 1.1.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.1.2](https://github.com/tbaur/homebridge-bluos/compare/v1.1.1...v1.1.2) (2026-08-30)
+
+
+### Bug Fixes
+
+* do not end the reboot quiet window on a last-gasp poll ([#26](https://github.com/tbaur/homebridge-bluos/issues/26)) ([91651a4](https://github.com/tbaur/homebridge-bluos/commit/91651a4c4b114c8a8e7bfba0cc7c4d6148375b7d))
+
 ## [1.1.1](https://github.com/tbaur/homebridge-bluos/compare/v1.1.0...v1.1.1) (2026-08-30)
 
 

package/dist/platform.js

@@ -529,9 +529,10 @@ class BluOSPlatform {
         }
     }
     publish(deviceId, observation, reason) {
-        // The player answered, so the reboot is over even if the window has time
-        // left. Anything that fails after this is a real outage and is said so.
-        this.rebootGrace.clear(this.hostOf(deviceId));
+        // Ends the window only after the box has already gone quiet. A last-gasp
+        // reading while the ports are still up must not cancel it, or the real
+        // outage that follows is logged as a surprise.
+        this.rebootGrace.clearIfRecovered(this.hostOf(deviceId));
         for (const handler of this.handlers.get(deviceId) ?? []) {
             try {
                 handler.applyObservation(observation, reason);
@@ -542,7 +543,9 @@ class BluOSPlatform {
         }
     }
     reportUnavailable(deviceId, error) {
-        if (this.rebootGrace.isExpected(this.hostOf(deviceId))) {
+        const host = this.hostOf(deviceId);
+        if (this.rebootGrace.isExpected(host)) {
+            this.rebootGrace.noteSilence(host);
             return;
         }
         for (const handler of this.handlers.get(deviceId) ?? []) {

package/dist/utils/reboot-grace.d.ts

@@ -10,20 +10,40 @@
  * address would otherwise warn and later claim to have recovered. That is noise:
  * we asked it to go down. This clock is what lets the platform stay quiet until
  * the player answers again, or until the window ends and a real outage remains.
+ *
+ * A reading that arrives before the box actually drops is not recovery. The
+ * control ports often answer one more time after POST /reboot. That last gasp
+ * must not end the window, or the real outage that follows is logged as usual.
+ * Recovery is a successful poll after we have already seen the expected silence.
  */
 /** Hosts we have just asked to reboot, and when that expectation expires. */
 export declare class RebootGrace {
     private readonly graceMs;
     private readonly now;
     private readonly expiresAt;
+    /** Addresses that have gone quiet at least once inside their window. */
+    private readonly sawSilence;
     constructor(graceMs: number, now?: () => number);
     /** Start (or refresh) the quiet window for this address. */
     expect(host: string): void;
     /**
-     * End the window early, because the address answered.
+     * Record that this address failed while a reboot was still expected.
+     *
+     * The next successful poll is then treated as the player coming back, not as
+     * a last gasp from a box that has not gone down yet.
+     */
+    noteSilence(host: string | undefined): void;
+    /**
+     * End the window if this address already went quiet and has now answered.
+     *
+     * A success with no prior silence is ignored: the box has not dropped yet.
+     */
+    clearIfRecovered(host: string | undefined): void;
+    /**
+     * End the window now.
      *
-     * A box is usually back well inside the window. Without this, a genuine
-     * failure in the time that remains would be swallowed as part of the reboot.
+     * Used when the clock runs out, and by tests. Recovery goes through
+     * {@link clearIfRecovered} so a last-gasp reading cannot cancel the window.
      */
     clear(host: string | undefined): void;
     /** True while a reboot of this address is still expected to look like an outage. */

package/dist/utils/reboot-grace.js

@@ -11,6 +11,11 @@
  * address would otherwise warn and later claim to have recovered. That is noise:
  * we asked it to go down. This clock is what lets the platform stay quiet until
  * the player answers again, or until the window ends and a real outage remains.
+ *
+ * A reading that arrives before the box actually drops is not recovery. The
+ * control ports often answer one more time after POST /reboot. That last gasp
+ * must not end the window, or the real outage that follows is logged as usual.
+ * Recovery is a successful poll after we have already seen the expected silence.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.RebootGrace = void 0;
@@ -19,6 +24,8 @@ class RebootGrace {
     graceMs;
     now;
     expiresAt = new Map();
+    /** Addresses that have gone quiet at least once inside their window. */
+    sawSilence = new Set();
     constructor(graceMs, now = Date.now) {
         this.graceMs = graceMs;
         this.now = now;
@@ -26,16 +33,39 @@ class RebootGrace {
     /** Start (or refresh) the quiet window for this address. */
     expect(host) {
         this.expiresAt.set(host, this.now() + this.graceMs);
+        this.sawSilence.delete(host);
+    }
+    /**
+     * Record that this address failed while a reboot was still expected.
+     *
+     * The next successful poll is then treated as the player coming back, not as
+     * a last gasp from a box that has not gone down yet.
+     */
+    noteSilence(host) {
+        if (host !== undefined && this.isExpected(host)) {
+            this.sawSilence.add(host);
+        }
     }
     /**
-     * End the window early, because the address answered.
+     * End the window if this address already went quiet and has now answered.
      *
-     * A box is usually back well inside the window. Without this, a genuine
-     * failure in the time that remains would be swallowed as part of the reboot.
+     * A success with no prior silence is ignored: the box has not dropped yet.
+     */
+    clearIfRecovered(host) {
+        if (host !== undefined && this.sawSilence.has(host)) {
+            this.clear(host);
+        }
+    }
+    /**
+     * End the window now.
+     *
+     * Used when the clock runs out, and by tests. Recovery goes through
+     * {@link clearIfRecovered} so a last-gasp reading cannot cancel the window.
      */
     clear(host) {
         if (host !== undefined) {
             this.expiresAt.delete(host);
+            this.sawSilence.delete(host);
         }
     }
     /** True while a reboot of this address is still expected to look like an outage. */
@@ -48,7 +78,7 @@ class RebootGrace {
             return false;
         }
         if (this.now() >= expires) {
-            this.expiresAt.delete(host);
+            this.clear(host);
             return false;
         }
         return true;

package/docs/README-DETAILED.md

@@ -209,7 +209,7 @@ A per-player reboot switch on a shared chassis, at startup:
 [BluOS] Zone One Reboot: will also reboot Zone Two: they are zones of one chassis, and BluOS reboots the whole box
 ```
 
-After a reboot the other accessories stay quiet. The player is expected to stop answering, so they do not log `is not responding` or `is responding again`. The next successful poll writes whatever the player is actually doing into HomeKit. A player that is still silent after that window is logged as not responding, the usual way.
+After a reboot the other accessories stay quiet. The player is expected to stop answering, so they do not log `is not responding` or `is responding again`. A reading that arrives before the box actually drops is ignored for that purpose: the control ports often answer once more after the reboot is sent. The first successful poll after the silence writes whatever the player is actually doing into HomeKit. A player that is still silent after that window is logged as not responding, the usual way.
 
 A configuration the plugin will not act on:
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-bluos",
   "displayName": "Homebridge BluOS",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Homebridge plugin for BluOS players — per-zone volume, mute, volume-preset and battery accessories over the LAN Custom Integration API. Verified on NAD and Bluesound hardware",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",