@switchbot/homebridge-switchbot 5.0.0-beta.133 → 5.0.0-beta.135

package/src/deviceFactory.ts

@@ -139,16 +139,10 @@ function classForType(type: string) {
   return DEVICE_CLASS_MAP[key] ?? GenericDevice
 }
 
-export async function createDevice(opts: DeviceOptions, cfg: SwitchBotPluginConfig, useMatter: boolean) {
+export async function createDevice(opts: DeviceOptions, cfg: SwitchBotPluginConfig, useMatter: boolean, log?: { info: (...args: any[]) => void }) {
   // Debug: Log the options passed to the device constructor
-  if (opts && opts.name) {
-    if (typeof opts.matterType !== 'undefined') {
-      // eslint-disable-next-line no-console
-      console.info(`[Matter/Debug] createDevice: Passing opts for ${opts.name}:`, JSON.stringify(opts, null, 2))
-    } else {
-      // eslint-disable-next-line no-console
-      console.info(`[Matter/Debug] createDevice: Passing opts for ${opts.name} (no matterType):`, JSON.stringify(opts, null, 2))
-    }
+  if (opts && opts.name && log && typeof log.info === 'function') {
+    log.info(`[Matter/Debug] createDevice: Passing opts for ${opts.name}:`, JSON.stringify(opts, null, 2))
   }
   // Reuse existing client when provided via config to avoid creating
   // a new SwitchBotClient per device (which can be expensive).
@@ -169,15 +163,6 @@ export async function createDevice(opts: DeviceOptions, cfg: SwitchBotPluginConf
     deviceOpts.keyId = opts.keyId
   }
 
-  // Ensure matterType override is set for curtain devices
-  const typeKey = (opts.type || '').toLowerCase()
-  if ((typeKey === 'curtain' || typeKey === 'curtain3' || typeKey === 'rollershade' || typeKey === 'blindtilt') && cfg && cfg.devices) {
-    const configDevice = cfg.devices.find((d: any) => d.id === opts.id)
-    if (configDevice && typeof configDevice.matterType !== 'undefined') {
-      deviceOpts.matterType = configDevice.matterType
-    }
-  }
-
   const DeviceCtor = classForType(opts.type)
   const device = new DeviceCtor(deviceOpts, mergedCfg)
   await device.init()

package/src/devices/genericDevice.ts

@@ -7,9 +7,9 @@ import { Buffer } from 'node:buffer'
 import { MATTER_ATTRIBUTE_IDS, MATTER_CLUSTER_IDS } from '../utils.js'
 import { DeviceBase } from './deviceBase.js'
 
-function validateBLEResponseLength(buf: Buffer | Uint8Array | any[], expected: number, context = ''): boolean {
+function validateBLEResponseLength(buf: Buffer | Uint8Array | any[], expected: number, context = '', log: import('homebridge').Logger): boolean {
   if (!buf || typeof buf.length !== 'number' || buf.length !== expected) {
-    console.warn(`[BLE] Invalid response length${context ? ` for ${context}` : ''}: expected ${expected}, got ${buf?.length}`)
+    log.warn(`[BLE] Invalid response length${context ? ` for ${context}` : ''}: expected ${expected}, got ${buf?.length}`)
     return false
   }
   return true
@@ -22,12 +22,15 @@ const BLE_NOTIFICATION_HANDLERS = new Map<string, (payload: Buffer) => void>()
 const HEX_COLOR_REGEX = /^#?[0-9A-F]{6}$/i
 
 export class GenericDevice extends DeviceBase {
-  protected log: any
+  protected log: import('homebridge').Logger
 
   constructor(opts: any, cfg: SwitchBotPluginConfig) {
     super(opts, cfg)
-    // Prefer logger from opts, then cfg, fallback to console
-    this.log = opts?.log || cfg?.log || console
+    // Require logger from opts or cfg
+    this.log = opts?.log || cfg?.log
+    if (!this.log) {
+      throw new Error('Device requires a logger (Homebridge logger) in opts or cfg')
+    }
     // If BLE encryptionKey/keyId provided, set on node-switchbot device instance if possible
     if (opts.encryptionKey && this.client && typeof this.client.devices?.get === 'function') {
       try {
@@ -42,10 +45,7 @@ export class GenericDevice extends DeviceBase {
         // ignore if device not found or setKey not available
       }
     }
-    // Ensure log is always set
-    if (!this.log) {
-      this.log = console
-    }
+    // ...existing code...
     // Subscribe to BLE notifications if supported (node-switchbot v4+)
     this._subscribeBLENotifications()
   }
@@ -115,7 +115,7 @@ export class GenericDevice extends DeviceBase {
         // If this is a BLE buffer/array, validate length (common BLE status: 12 bytes, but may vary by device)
         if (raw && (raw instanceof Buffer || Array.isArray(raw) || raw instanceof Uint8Array)) {
           // Default to 12, override per device if needed
-          if (!validateBLEResponseLength(raw, 12, this.opts.type)) {
+          if (!validateBLEResponseLength(raw, 12, this.opts.type, this.log)) {
             return { id: this.opts.id, type: this.opts.type, error: 'invalid_ble_response_length', raw }
           }
         }
@@ -431,32 +431,80 @@ export class CurtainDevice extends GenericDevice {
     }
   }
 
-  // Matter-specific descriptor for Curtain (Shade cluster) with conformance type
+  // Matter-specific descriptor for Curtain (WindowCovering cluster) with new attributes
   async createMatterAccessory(api: any): Promise<any> {
-    // Always use the platform-provided Matter type (e.g., 5 for roller shade)
-    const matterType = 5
+    // Get current state for dynamic attributes
+    const state = await this.getState()
+    // Compose attributes for Matter WindowCovering cluster
     const attributes: Record<string, any> = {
-      type: {
-        read: async () => matterType,
+      currentPositionLiftPercent100ths: {
+        read: async () => {
+          const s = await this.getState()
+          return typeof s.position === 'number' ? Math.round(s.position * 100) : 0
+        },
+        write: undefined,
+      },
+      targetPositionLiftPercent100ths: {
+        read: async () => {
+          const s = await this.getState()
+          return typeof s.position === 'number' ? Math.round(s.position * 100) : 0
+        },
+        write: async (v: any) => this.setState({ position: Math.round(Number(v) / 100) }),
+      },
+      operationalStatus: {
+        read: async () => state.operationalStatus ?? { global: 0, lift: 0, tilt: 0 },
+        write: undefined,
+      },
+      endProductType: {
+        read: async () => state.endProductType ?? 0,
+        write: undefined,
+      },
+      configStatus: {
+        read: async () => state.configStatus ?? {
+          operational: true,
+          onlineReserved: true,
+          liftMovementReversed: false,
+          liftPositionAware: true,
+          tiltPositionAware: false,
+          liftEncoderControlled: true,
+          tiltEncoderControlled: false,
+        },
         write: undefined,
       },
-      currentPosition: { read: async () => { const s = await this.getState(); return typeof s.position === 'number' ? s.position : 0 }, write: undefined },
-      targetPosition: { read: async () => { const s = await this.getState(); return typeof s.position === 'number' ? s.position : 0 }, write: async (v: any) => this.setState({ position: Number(v) }) },
+    }
+    // If tilt is supported, add tilt attributes
+    if (typeof state.tilt === 'number') {
+      attributes.currentPositionTiltPercent100ths = {
+        read: async () => {
+          const s = await this.getState()
+          return typeof s.tilt === 'number' ? Math.round(s.tilt * 100) : 0
+        },
+        write: undefined,
+      }
+      attributes.targetPositionTiltPercent100ths = {
+        read: async () => {
+          const s = await this.getState()
+          return typeof s.tilt === 'number' ? Math.round(s.tilt * 100) : 0
+        },
+        write: async (v: any) => this.setState({ tilt: Math.round(Number(v) / 100) }),
+      }
     }
     const windowCoveringCluster = {
       type: 'WindowCovering',
-      matterType,
       clusterId: MATTER_CLUSTER_IDS.WindowCovering,
       attributes,
     }
-    const clustersArr: any = [windowCoveringCluster]
+    // Provide both array and named property for clusters for compatibility with test expectations
+    const clustersArr: any[] = [windowCoveringCluster]
+    const clusters: any = [...clustersArr]
+    // Always set clusters.windowCovering to the WindowCovering cluster by clusterId
     const foundWC = clustersArr.find((c: any) => c && c.clusterId === MATTER_CLUSTER_IDS.WindowCovering)
-    clustersArr.windowCovering = foundWC || null
+    clusters.windowCovering = foundWC || null
     return {
       id: this.opts.id,
       name: this.opts.name ?? this.opts.type,
       protocol: 'matter',
-      clusters: clustersArr,
+      clusters,
     }
   }
 }

package/src/homebridge-ui/public/index.html

@@ -1,145 +1,128 @@
 <!doctype html>
 <html>
-  <head>
-    <meta charset="utf-8" />
-    <meta name="viewport" content="width=device-width,initial-scale=1" />
-    <title>SwitchBot Plugin UI</title>
-    <link rel="stylesheet" href="css/styles.css" />
-  </head>
-  <body>
-    <h1>🤖 SwitchBot Configuration</h1>
 
-    <div class="card compact">
-      <h2>API Credentials</h2>
-      <div class="credentials-row">
-        <div class="form-group">
-          <label for="token">Token:</label>
-          <input type="password" id="token" placeholder="Enter token" />
-          <div class="status" id="tokenStatus"></div>
-        </div>
-        <div class="form-group">
-          <label for="secret">Secret:</label>
-          <input type="password" id="secret" placeholder="Enter secret" />
-          <div class="status" id="secretStatus"></div>
-        </div>
-      </div>
-      <button id="saveBtn" onclick="saveCredentials()">Save</button>
-      <div id="saveStatus"></div>
-    </div>
+<head>
+  <meta charset="utf-8" />
+  <meta name="viewport" content="width=device-width,initial-scale=1" />
+  <title>SwitchBot Plugin UI</title>
+  <link rel="stylesheet" href="css/styles.css" />
+</head>
 
-    <div class="card compact">
-      <h2>Discover Devices</h2>
-      <div style="margin-bottom: 6px">
-        <label style="display: flex; align-items: center; gap: 4px; cursor: pointer; font-size: 11px">
-          <input type="checkbox" id="autoAddAllCheckbox" style="width: auto; margin: 0" />
-          <span>Auto-add all</span>
-          <span
-            title="When enabled, Discover immediately adds all found devices to your config. Existing devices are skipped automatically."
-            style="font-size: 11px; opacity: 0.7"
-          >ⓘ</span>
-        </label>
-        <div style="margin-top: 2px; margin-left: 22px; font-size: 10px; opacity: 0.75; line-height: 1.25">
-          Adds all discovered devices automatically; already configured devices are skipped.
-        </div>
-      </div>
-      <div class="discovery-settings-row" style="margin-bottom: 2px">
-        <label id="bleScanSetting" style="display: inline-flex; align-items: center; gap: 4px; font-size: 11px">
-          <span>Scan:</span>
-          <select id="bleScanDurationSelect" style="font-size: 11px; padding: 2px 6px">
-            <option value="3">3s</option>
-            <option value="5" selected>5s</option>
-            <option value="10">10s</option>
-            <option value="15">15s</option>
-          </select>
-        </label>
-        <label id="bleTimeoutSetting" style="display: inline-flex; align-items: center; gap: 4px; font-size: 11px">
-          <span>Timeout:</span>
-          <input
-            id="bleTimeoutInput"
-            type="number"
-            min="3"
-            max="30"
-            step="1"
-            value="8"
-            style="width: 56px; font-size: 11px; padding: 2px 4px"
-          />
-          <span>s</span>
-        </label>
-        <label style="display: flex; align-items: center; gap: 4px; cursor: pointer; font-size: 11px">
-          <input type="checkbox" id="disableBleScanCheckbox" style="width: auto; margin: 0" />
-          <span>Disable BLE scan</span>
-        </label>
+<body>
+  <h1>🤖 SwitchBot Configuration</h1>
+
+  <div class="card compact">
+    <h2>API Credentials</h2>
+    <div class="credentials-row">
+      <div class="form-group">
+        <label for="token">Token:</label>
+        <input type="password" id="token" placeholder="Enter token" />
+        <div class="status" id="tokenStatus"></div>
       </div>
-      <div id="bluetoothStatus" style="margin-bottom: 4px; font-size: 10px; opacity: 0.8">Bluetooth: checking...</div>
-      <div class="discovery-settings-row" style="margin-bottom: 8px">
-        <div id="lastScannedStatus" style="font-size: 10px; opacity: 0.8; flex: 1 1 auto">Last scanned: never</div>
-        <label style="font-size: 11px">
-          Auto-refresh:
-          <select id="autoRefreshIntervalSelect" style="margin-left: 4px; font-size: 11px; padding: 2px 6px">
-            <option value="0" selected>Off</option>
-            <option value="30">30s</option>
-            <option value="60">1m</option>
-            <option value="300">5m</option>
-          </select>
-        </label>
-        <button id="refreshDiscoverBtn" class="secondary" style="padding: 4px 9px; font-size: 11px">Refresh</button>
+      <div class="form-group">
+        <label for="secret">Secret:</label>
+        <input type="password" id="secret" placeholder="Enter secret" />
+        <div class="status" id="secretStatus"></div>
       </div>
-      <button id="discoverBtn" onclick="discoverDevices()" style="margin-bottom: 8px">
-        🔍 Discover
-      </button>
-      <button
-        id="cancelDiscoverBtn"
-        style="margin-left: 6px; margin-bottom: 8px; display: none; padding: 6px 10px; font-size: 11px"
-        class="secondary"
-      >
-        Cancel
-      </button>
-      <div id="discoverStatus" style="margin-bottom: 6px; font-size: 11px"></div>
-      <div id="discoverDevicesFound" style="margin-bottom: 6px; font-size: 11px; display: none"></div>
-      <div id="discoverPhaseProgress" class="discover-phase-progress" style="display: none">
-        <div class="discover-phase-track">
-          <div id="discoverPhaseFill" class="discover-phase-fill"></div>
-        </div>
-        <div id="discoverPhaseLabel" class="discover-phase-label"></div>
+    </div>
+    <button id="saveBtn" onclick="saveCredentials()">Save</button>
+    <div id="saveStatus"></div>
+  </div>
+
+  <div class="card compact">
+    <h2>Discover Devices</h2>
+    <div style="margin-bottom: 6px">
+      <label style="display: flex; align-items: center; gap: 4px; cursor: pointer; font-size: 11px">
+        <input type="checkbox" id="autoAddAllCheckbox" style="width: auto; margin: 0" />
+        <span>Auto-add all</span>
+        <span
+          title="When enabled, Discover immediately adds all found devices to your config. Existing devices are skipped automatically."
+          style="font-size: 11px; opacity: 0.7">ⓘ</span>
+      </label>
+      <div style="margin-top: 2px; margin-left: 22px; font-size: 10px; opacity: 0.75; line-height: 1.25">
+        Adds all discovered devices automatically; already configured devices are skipped.
       </div>
-      <div id="discoveredList" style="display: none">
-        <h3 style="margin-top: 0; font-size: 12px; margin-bottom: 6px; font-weight: 600">Available Devices</h3>
-        <ul
-          id="discoveredDevices"
-          style="max-height: 300px; overflow-y: auto"
-        ></ul>
+    </div>
+    <div class="discovery-settings-row" style="margin-bottom: 2px">
+      <label id="bleScanSetting" style="display: inline-flex; align-items: center; gap: 4px; font-size: 11px">
+        <span>Scan:</span>
+        <select id="bleScanDurationSelect" style="font-size: 11px; padding: 2px 6px">
+          <option value="3">3s</option>
+          <option value="5" selected>5s</option>
+          <option value="10">10s</option>
+          <option value="15">15s</option>
+        </select>
+      </label>
+      <label id="bleTimeoutSetting" style="display: inline-flex; align-items: center; gap: 4px; font-size: 11px">
+        <span>Timeout:</span>
+        <input id="bleTimeoutInput" type="number" min="3" max="30" step="1" value="8"
+          style="width: 56px; font-size: 11px; padding: 2px 4px" />
+        <span>s</span>
+      </label>
+      <label style="display: flex; align-items: center; gap: 4px; cursor: pointer; font-size: 11px">
+        <input type="checkbox" id="disableBleScanCheckbox" style="width: auto; margin: 0" />
+        <span>Disable BLE scan</span>
+      </label>
+    </div>
+    <div id="bluetoothStatus" style="margin-bottom: 4px; font-size: 10px; opacity: 0.8">Bluetooth: checking...</div>
+    <div class="discovery-settings-row" style="margin-bottom: 8px">
+      <div id="lastScannedStatus" style="font-size: 10px; opacity: 0.8; flex: 1 1 auto">Last scanned: never</div>
+      <label style="font-size: 11px">
+        Auto-refresh:
+        <select id="autoRefreshIntervalSelect" style="margin-left: 4px; font-size: 11px; padding: 2px 6px">
+          <option value="0" selected>Off</option>
+          <option value="30">30s</option>
+          <option value="60">1m</option>
+          <option value="300">5m</option>
+        </select>
+      </label>
+      <button id="refreshDiscoverBtn" class="secondary" style="padding: 4px 9px; font-size: 11px">Refresh</button>
+    </div>
+    <button id="discoverBtn" onclick="discoverDevices()" style="margin-bottom: 8px">
+      🔍 Discover
+    </button>
+    <button id="cancelDiscoverBtn"
+      style="margin-left: 6px; margin-bottom: 8px; display: none; padding: 6px 10px; font-size: 11px" class="secondary">
+      Cancel
+    </button>
+    <div id="discoverStatus" style="margin-bottom: 6px; font-size: 11px"></div>
+    <div id="discoverDevicesFound" style="margin-bottom: 6px; font-size: 11px; display: none"></div>
+    <div id="discoverPhaseProgress" class="discover-phase-progress" style="display: none">
+      <div class="discover-phase-track">
+        <div id="discoverPhaseFill" class="discover-phase-fill"></div>
       </div>
+      <div id="discoverPhaseLabel" class="discover-phase-label"></div>
+    </div>
+    <div id="discoveredList" style="display: none">
+      <h3 style="margin-top: 0; font-size: 12px; margin-bottom: 6px; font-weight: 600">Available Devices</h3>
+      <ul id="discoveredDevices" style="max-height: 300px; overflow-y: auto"></ul>
     </div>
+  </div>
 
-    <div class="card compact">
-      <h2>Configured Devices</h2>
-      <div id="status">Loading…</div>
-      <ul id="devices"></ul>
-      <!-- BLE encryption key fields will be rendered dynamically per device in the UI (see app.ts/devices.js) -->
-      <div
-        id="removeAllContainer"
-        style="
+  <div class="card compact">
+    <h2>Configured Devices</h2>
+    <div id="status">Loading…</div>
+    <ul id="devices"></ul>
+    <!-- BLE encryption key fields will be rendered dynamically per device in the UI (see app.ts/devices.js) -->
+    <div id="removeAllContainer" style="
           display: none;
           margin-top: 12px;
           padding-top: 10px;
           border-top: 1px solid rgba(0, 0, 0, 0.1);
-        "
-      >
-        <button
-          id="removeAllBtn"
-          style="
+        ">
+      <button id="removeAllBtn" style="
             background: #dc2626;
             width: 100%;
             padding: 7px 12px;
             font-size: 12px;
             font-weight: 500;
-          "
-        >
-          🗑️ Remove All Devices
-        </button>
-      </div>
+          ">
+        🗑️ Remove All Devices
+      </button>
     </div>
+  </div>
+
+  <script type="module" src="js/app.js"></script>
+</body>
 
-    <script type="module" src="js/app.js"></script>
-  </body>
-</html>
+</html>

package/src/homebridge-ui/public/js/api.ts

@@ -192,7 +192,7 @@ export async function addDevice(
   deviceId: string,
   name: string,
   type: string,
-  options?: { address?: string, model?: string, rssi?: number, encryptionKey?: string, keyId?: string, matterType?: number },
+  options?: { address?: string, model?: string, rssi?: number, encryptionKey?: string, keyId?: string },
 ): Promise<any> {
   const payload: any = { deviceId, name, type }
   if (options?.address) {
@@ -210,9 +210,6 @@ export async function addDevice(
   if (options?.keyId) {
     payload.keyId = options.keyId
   }
-  if (options?.matterType !== undefined) {
-    payload.matterType = options.matterType
-  }
   uiLog.info('Adding device to config:', payload)
   const resp = await homebridge.request('/add-device', payload)
   uiLog.info('Add device response:', resp)
@@ -237,7 +234,6 @@ export async function updateDevice(
   deviceId: string,
   configDeviceName?: string,
   configDeviceType?: string,
-  options?: { matterType?: number },
 ): Promise<any> {
   const params: any = { deviceId }
   if (configDeviceName) {
@@ -246,9 +242,6 @@ export async function updateDevice(
   if (configDeviceType) {
     params.configDeviceType = configDeviceType
   }
-  if (options?.matterType !== undefined) {
-    params.matterType = options.matterType
-  }
   uiLog.info('[Update Device] Sending update request with params:', params)
   const resp = await homebridge.request('/update-device', params)
   uiLog.info('[Update Device] Update response:', resp)

package/src/homebridge-ui/public/js/app.ts

@@ -1,17 +1,16 @@
 import { loadCredentialStatus, saveCredentials } from './credentials.js'
-import { loadConfiguredDevices, initRemoveAllButton } from './devices.js'
-// Wire up global event handlers
+import { initRemoveAllButton, loadConfiguredDevices } from './devices.js'
 import { discoverDevices, initializeDiscoverySettings } from './discovery.js';
 
-(window as any).loadCredentialStatus = loadCredentialStatus
-;(window as any).saveCredentials = saveCredentials
-;(window as any).discoverDevices = discoverDevices
+(window as any).loadCredentialStatus = loadCredentialStatus;
+(window as any).saveCredentials = saveCredentials;
+(window as any).discoverDevices = discoverDevices
 
 // Initialize on page load
 async function init(): Promise<void> {
   await loadCredentialStatus()
   await initializeDiscoverySettings()
-  const list = await loadConfiguredDevices()
+  await loadConfiguredDevices()
   await initRemoveAllButton()
 }
 

package/src/homebridge-ui/public/js/credentials.ts

@@ -15,7 +15,9 @@ export async function loadCredentialStatus(): Promise<void> {
     const tokenStatus = document.getElementById('tokenStatus')
     const secretStatus = document.getElementById('secretStatus')
 
-    if (!tokenStatus || !secretStatus) { return }
+    if (!tokenStatus || !secretStatus) {
+      return
+    }
 
     if (creds.hasToken) {
       tokenStatus.textContent = `✓ Configured (${creds.tokenLength} characters)`
@@ -43,7 +45,9 @@ export async function saveCredentials(): Promise<void> {
   const saveStatus = document.getElementById('saveStatus')
   const saveBtn = document.getElementById('saveBtn') as HTMLButtonElement
 
-  if (!saveStatus || !saveBtn) { return }
+  if (!saveStatus || !saveBtn) {
+    return
+  }
 
   if (!token || !secret) {
     saveStatus.textContent = 'Please enter both token and secret'
@@ -80,7 +84,7 @@ export async function saveCredentials(): Promise<void> {
     ;(document.getElementById('secret') as HTMLInputElement).value = ''
 
     // Reload status to verify save
-    setTimeout(() => loadCredentialStatus(), 500)
+    setTimeout(loadCredentialStatus, 500)
 
     // Clear status message
     setTimeout(() => {

package/src/homebridge-ui/public/js/devices-delete.ts

@@ -1,4 +1,4 @@
-import { deleteDevice as apiDeleteDevice, deleteAllDevices as apiDeleteAllDevices, syncParentPluginConfigFromDisk, fetchDevices } from './api.js'
+import { deleteAllDevices as apiDeleteAllDevices, deleteDevice as apiDeleteDevice, fetchDevices, syncParentPluginConfigFromDisk } from './api.js'
 import { uiLog } from './logger.js'
 import { hideBusyUi, showBusyUi } from './modal.js'
 import { renderDeviceList } from './render.js'

package/src/homebridge-ui/public/js/devices.ts

@@ -10,7 +10,6 @@ type ImportDiscoveredDeviceResult = {
   address?: string
   encryptionKey?: string
   keyId?: string
-  matterType?: number
 } | null
 
 export async function addDeviceToConfig(device: any, options: { refresh?: boolean, showStatus?: boolean } = {}): Promise<{ added: boolean }> {
@@ -32,7 +31,6 @@ export async function addDeviceToConfig(device: any, options: { refresh?: boolea
       rssi: device.rssi,
       encryptionKey: importValues.encryptionKey,
       keyId: importValues.keyId,
-      matterType: importValues.matterType,
     })
     uiLog.info('Add device response:', resp)
 

package/src/homebridge-ui/public/js/discovery.ts

@@ -1301,9 +1301,15 @@ async function updateDiscoveryView(
     const enableSelectedBtn = document.querySelector('button')?.parentElement?.querySelector('button[aria-label="Enable Selected"]') as HTMLButtonElement | null
     const disableSelectedBtn = document.querySelector('button')?.parentElement?.querySelector('button[aria-label="Disable Selected"]') as HTMLButtonElement | null
     const hasSelection = selectedIds.size > 0
-    if (addSelectedBtn) addSelectedBtn.disabled = !hasSelection
-    if (enableSelectedBtn) enableSelectedBtn.disabled = !hasSelection
-    if (disableSelectedBtn) disableSelectedBtn.disabled = !hasSelection
+    if (addSelectedBtn) {
+      addSelectedBtn.disabled = !hasSelection
+    }
+    if (enableSelectedBtn) {
+      enableSelectedBtn.disabled = !hasSelection
+    }
+    if (disableSelectedBtn) {
+      disableSelectedBtn.disabled = !hasSelection
+    }
   }
   window.removeEventListener('discovery-selection-changed', updateBatchButtonStates)
   window.addEventListener('discovery-selection-changed', updateBatchButtonStates)

package/src/homebridge-ui/public/js/logger.ts

@@ -1,11 +1,17 @@
+/* eslint-disable no-console */
+const PREFIX = '[SwitchBot UI/html]'
+
 export const uiLog = {
-  info: (message: string, ...parameters: any[]): void => {
-    console.warn('[SwitchBot UI]', message, ...parameters)
+  info: (message: string, ...parameters: any[]) => {
+    console.log(PREFIX, message, ...parameters)
   },
-  warn: (message: string, ...parameters: any[]): void => {
-    console.warn('[SwitchBot UI]', message, ...parameters)
+  warn: (message: string, ...parameters: any[]) => {
+    console.warn(PREFIX, message, ...parameters)
   },
-  error: (message: string, ...parameters: any[]): void => {
-    console.error('[SwitchBot UI]', message, ...parameters)
+  error: (message: string, ...parameters: any[]) => {
+    console.error(PREFIX, message, ...parameters)
+  },
+  debug: (message: string, ...parameters: any[]) => {
+    console.debug(PREFIX, message, ...parameters)
   },
 }