homebridge-bluos 1.1.4 → 1.1.5

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.1.5](https://github.com/tbaur/homebridge-bluos/compare/v1.1.4...v1.1.5) (2026-08-31)
+
+
+### Bug Fixes
+
+* keep discover progress in view and leave volume opt-in ([#33](https://github.com/tbaur/homebridge-bluos/issues/33)) ([1dec836](https://github.com/tbaur/homebridge-bluos/commit/1dec8361f2a82d2a1c355f74c4258078bcb945d2))
+
 ## [1.1.4](https://github.com/tbaur/homebridge-bluos/compare/v1.1.3...v1.1.4) (2026-08-31)
 
 

package/README.md

@@ -56,7 +56,7 @@ Nothing needs enabling on the player, because the BluOS LAN API is always on. A
 
 ### 3. Configure
 
-**Homebridge UI** (recommended): open the plugin settings and press **Discover Players**. Every zone that answers is listed with a suggested set of accessories. Tick what you want, then press the Homebridge Save button.
+**Homebridge UI** (recommended): open the plugin settings and press **Discover Players**. Every zone that answers is listed. Tick the players and accessories you want, then press the Homebridge Save button.
 
 Or in `config.json`:
 

package/homebridge-ui/public/index.html

@@ -10,7 +10,39 @@
 -->
 <style>
   .bluos-intro { margin-bottom: 1rem; }
-  .bluos-toolbar { display: flex; gap: .5rem; align-items: flex-end; flex-wrap: wrap; margin-bottom: 1rem; }
+  .bluos-toolbar {
+    display: flex;
+    gap: .5rem;
+    align-items: flex-end;
+    flex-wrap: wrap;
+    position: sticky;
+    top: 0;
+    z-index: 5;
+    margin: 0 0 1rem;
+    padding: .55rem 0;
+    /* Frosted so cards do not show through while this bar stays on screen. */
+    background: rgba(128, 128, 128, .12);
+    backdrop-filter: blur(10px);
+  }
+  .bluos-busy {
+    display: inline-flex;
+    align-items: center;
+    align-self: center;
+    gap: .4rem;
+    font-size: .85rem;
+    white-space: nowrap;
+  }
+  .bluos-busy[hidden] { display: none !important; }
+  .bluos-busy::before {
+    content: '';
+    width: .85rem;
+    height: .85rem;
+    border: 2px solid currentColor;
+    border-right-color: transparent;
+    border-radius: 50%;
+    animation: bluos-spin .7s linear infinite;
+  }
+  @keyframes bluos-spin { to { transform: rotate(360deg); } }
   .bluos-toolbar .form-group { margin-bottom: 0; }
   .bluos-card { border: 1px solid rgba(128, 128, 128, .3); border-radius: .5rem; padding: .85rem 1rem; margin-bottom: .75rem; }
   .bluos-card.is-selected { border-color: #007bff; box-shadow: 0 0 0 1px rgba(0, 123, 255, .35); }
@@ -46,6 +78,7 @@
 
 <div class="bluos-toolbar">
   <button class="btn btn-primary" id="discover">Discover Players</button>
+  <span id="discover-status" class="bluos-busy text-muted" hidden></span>
   <div class="form-group">
     <label for="timeout" class="mb-0" style="font-size: .8rem;">Listen for</label>
     <select class="form-control form-control-sm" id="timeout">

package/homebridge-ui/public/index.js

@@ -157,7 +157,10 @@
         derivedIdentity: player.derivedIdentity,
         online: true,
         selected: false,
-        volumeSlider: player.suggested.volumeSlider,
+        // Discover listing a zone is not a request for a HomeKit accessory.
+        // Volume is a fan tile; leaving it ticked on every new player would
+        // mint one per zone the moment the user pressed Save.
+        volumeSlider: false,
         mute: player.suggested.mute,
         battery: player.suggested.battery,
         reboot: player.suggested.reboot,
@@ -454,9 +457,30 @@
 
   // --- Actions ------------------------------------------------------------
 
+  let requestInFlight = false
+
+  /**
+   * Show progress next to Discover, which is where the user just pressed.
+   *
+   * Homebridge's spinner is centred on the whole settings page. With a long
+   * list of players that centre sits below the fold, so a user who pressed
+   * Discover would see nothing happen for several seconds.
+   */
+  function setBusy(isBusy, message) {
+    requestInFlight = isBusy
+    byId('discover').disabled = isBusy
+    byId('manual-probe').disabled = isBusy
+    const status = byId('discover-status')
+    status.hidden = !isBusy
+    status.textContent = isBusy ? message : ''
+  }
+
   async function discover() {
+    if (requestInFlight) {
+      return
+    }
     const timeoutSec = Number(byId('timeout').value) || DEFAULT_TIMEOUT_SEC
-    homebridge.showSpinner()
+    setBusy(true, 'Listening for players…')
     try {
       const response = await homebridge.request('/discover', { timeoutSec })
       const found = Array.isArray(response && response.players) ? response.players : []
@@ -481,11 +505,14 @@
     } catch (error) {
       homebridge.toast.error(describeError(error), 'Discovery failed')
     } finally {
-      homebridge.hideSpinner()
+      setBusy(false)
     }
   }
 
   async function probe() {
+    if (requestInFlight) {
+      return
+    }
     const host = byId('manual-host').value.trim()
     const portValue = byId('manual-port').value.trim()
     if (host.length === 0) {
@@ -496,7 +523,7 @@
     if (portValue.length > 0) {
       payload.port = Number(portValue)
     }
-    homebridge.showSpinner()
+    setBusy(true, 'Probing…')
     try {
       const response = await homebridge.request('/probe', payload)
       const found = Array.isArray(response && response.players) ? response.players : []
@@ -512,7 +539,7 @@
     } catch (error) {
       homebridge.toast.error(describeError(error), 'Probe failed')
     } finally {
-      homebridge.hideSpinner()
+      setBusy(false)
     }
   }
 

package/homebridge-ui/server.js

@@ -155,11 +155,14 @@ class BluOSUiServer extends HomebridgePluginUiServer {
   }
 
   /**
-   * Shape a discovered player for the page, including sensible defaults.
+   * Shape a discovered player for the page, including opt-in accessory defaults.
    *
-   * A player that reports a fixed output level gets no slider suggested, because
-   * writing a level to it does nothing. A player with no usable MAC is given a
-   * generated identity here, once, so that it stays stable from then on.
+   * Volume, mute and reboot stay off until the user asks for them: Discover
+   * listing a zone is not a request for a HomeKit accessory. A player with a
+   * battery pack is offered that sensor, because it is a reading rather than a
+   * control. A player that reports a fixed output cannot take a slider at all,
+   * which the page learns from `fixedVolume`. A player with no usable MAC is
+   * given a generated identity here, once, so that it stays stable from then on.
    */
   describe(player) {
     const api = this.loadApi()
@@ -176,7 +179,7 @@ class BluOSUiServer extends HomebridgePluginUiServer {
       hasBattery: player.hasBattery === true,
       derivedIdentity: !(player.mac && player.mac.length > 0),
       suggested: {
-        volumeSlider: player.fixedVolume !== true,
+        volumeSlider: false,
         mute: false,
         battery: player.hasBattery === true,
         // Never suggested. Restarting a player interrupts whatever it is doing,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-bluos",
   "displayName": "Homebridge BluOS",
-  "version": "1.1.4",
+  "version": "1.1.5",
   "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",