@homebridge-plugins/homebridge-govee 11.32.2-beta.16 → 11.32.2-beta.17

package/CHANGELOG.md

@@ -35,6 +35,7 @@ All notable changes to `@homebridge-plugins/homebridge-govee` will be documented
 - feat: give a diffuser model that has not been worked out yet the same starting handler the other appliances have, so it works for on and off
 - refactor: choose a device handler from a table, so adding a device type no longer means editing the platform
 - refactor: move the settings parsing into its own file, away from the code that talks to devices
+- refactor: move the work of describing a device to its handler into its own file
 
 ## v11.32.1 (2026-08-02)
 

package/lib/platform.js

@@ -17,7 +17,7 @@ import { buildCommand } from './utils/command-builder.js'
 import { applyUserConfig } from './utils/config-parser.js'
 import platformConsts from './utils/constants.js'
 import platformChars from './utils/custom-chars.js'
-import { getDeviceCapabilities } from './utils/device-capabilities.js'
+import { applyDeviceContext } from './utils/device-context.js'
 import { resolveDeviceType } from './utils/dispatch.js'
 import eveChars from './utils/eve-chars.js'
 import {
@@ -860,7 +860,6 @@ export default class {
       }
 
       const devInstance = route.handler
-      const doAWSPolling = route.awsPolling
 
       let accessory
       if (route.externalCategory) {
@@ -882,161 +881,12 @@ export default class {
       // Set the logging level for this device
       this.applyAccessoryLogging(accessory)
 
-      // Add the temperatureSource config to the context if exists
-      if (deviceConf.temperatureSource) {
-        accessory.context.temperatureSource = deviceConf.temperatureSource
-      }
-
-      // Add the openApiTempUnit config to the context if set to something other than auto
-      if (deviceConf.openApiTempUnit && deviceConf.openApiTempUnit !== 'auto') {
-        accessory.context.openApiTempUnit = deviceConf.openApiTempUnit
-      } else {
-        delete accessory.context.openApiTempUnit
-      }
-
-      // Get a supported command list if provided, with their options
-      if (device.supportCmds && Array.isArray(device.supportCmds)) {
-        accessory.context.supportedCmds = device.supportCmds
-        accessory.context.supportedCmdsOpts = {}
-
-        device.supportCmds.forEach((cmd) => {
-          if (device?.properties?.[cmd]) {
-            accessory.context.supportedCmdsOpts[cmd] = device.properties[cmd]
-          }
-        })
-      }
-
-      // Add some initial context information which is changed later
-      accessory.context.hasAwsControl = false
-      accessory.context.useAwsControl = false
-      accessory.context.hasOpenApiControl = false
-      accessory.context.useOpenApiControl = false
-      accessory.context.hasBleControl = false
-      accessory.context.useBleControl = false
-      accessory.context.hasLanControl = device.isLanDevice
-      accessory.context.useLanControl = accessory.context.hasLanControl
-      accessory.context.firmware = false
-      accessory.context.hardware = false
-      accessory.context.image = false
-
-      // Overrides for when a custom IP is provided, for a light which is not BLE only
-      if (
-        deviceConf.customIPAddress
-        && accessory.context.hasLanControl
-        && accessory.context.hasAwsControl
-        && platformConsts.models.rgb.includes(device.model)
-      ) {
-        accessory.context.hasLanControl = true
-        accessory.context.useLanControl = true
-      }
-
-      // If the device is LAN-only, then sync the display name with the label in the configuration
-      if (device.isLanOnly) {
-        accessory.displayName = device.deviceName
-      }
-
-      // See if we have extra HTTP client info for this device
-      if (device.httpInfo) {
-        // Save the hardware and firmware versions
-        accessory.context.firmware = device.httpInfo.versionSoft
-        accessory.context.hardware = device.httpInfo.versionHard
-
-        // It's possible to show a nice little icon of the device in the Homebridge UI
-        if (device.httpInfo.deviceExt && device.httpInfo.deviceExt.extResources) {
-          try {
-            const parsed = JSON.parse(device.httpInfo.deviceExt.extResources)
-            if (parsed && parsed.skuUrl) {
-              accessory.context.image = parsed.skuUrl
-            }
-          } catch {
-            // Ignore malformed extResources
-          }
-        }
-
-        // HTTP info lets us see if AWS/BLE connection methods are available
-        if (device.httpInfo.deviceExt && device.httpInfo.deviceExt.deviceSettings) {
-          let parsed
-          try {
-            parsed = JSON.parse(device.httpInfo.deviceExt.deviceSettings)
-          } catch {
-            this.log.debugWarn('[%s] failed to parse deviceSettings, skipping AWS/BLE setup.', accessory.displayName)
-          }
-
-          // Check to see if AWS is possible
-          if (parsed) {
-            if (parsed.topic) {
-              accessory.context.hasAwsControl = true
-              accessory.context.awsTopic = parsed.topic
-
-              if (this.awsClient) {
-                accessory.context.useAwsControl = true
-                accessory.context.awsBrightnessNoScale = deviceConf.awsBrightnessNoScale
-                accessory.context.awsColourMode = deviceConf.awsColourMode || platformConsts.defaultValues.awsColourMode
-                awsDevices.push(device.device)
-
-                // Certain models need AWS polling
-                if (doAWSPolling) {
-                  awsDevicesToPoll.push(device.device)
-                }
-              }
-            }
-
-            // Check to see if BLE is possible
-            if (parsed.bleName) {
-              const providedBle = parsed.address ? parsed.address.toLowerCase() : device.device.substring(6).toLowerCase()
-              accessory.context.hasBleControl = !!parsed.bleName
-              accessory.context.bleAddress = deviceConf.customAddress
-                ? deviceConf.customAddress.toLowerCase()
-                : providedBle
-              accessory.context.bleName = parsed.bleName
-              accessory.context.bleWriteUuid = getDeviceCapabilities(accessory.context.gvModel).bleWriteUuid
-              if (this.bleClient) {
-                accessory.context.useBleControl = true
-              }
-            }
-
-            // Gateway subdevices (e.g. leak sensors paired to an H5044) include
-            // their gateway id and slot number, which we need to match real-time
-            // leak events relayed by the gateway over AWS (#1276)
-            if (hasProperty(parsed, 'sno') && parsed.gatewayInfo?.device) {
-              accessory.context.gatewaySno = parsed.sno
-              accessory.context.gatewayDevice = parsed.gatewayInfo.device
-            }
-
-            // Get a min and max temperature/humidity range to show in the homebridge-ui
-            if (hasProperty(parsed, 'temCali')) {
-              accessory.context.minTemp = parsed.temMin / 100
-              accessory.context.maxTemp = parsed.temMax / 100
-              accessory.context.offTemp = parsed.temCali
-            }
-            if (hasProperty(parsed, 'humCali')) {
-              accessory.context.minHumi = parsed.humMin / 100
-              accessory.context.maxHumi = parsed.humMax / 100
-              accessory.context.offHumi = parsed.humCali
-            }
-          }
-        }
-      }
-
-      if (device.openApiInfo) {
-        accessory.context.hasOpenApiControl = true
-        accessory.context.useOpenApiControl = !!this.openApiClient
-        accessory.context.openApiCapabilities = device.openApiInfo.byInstance || {}
-
-        // Log what Govee says this device can do. Working out whether a model
-        // exposes something like a night light otherwise means reverse
-        // engineering its status messages, when the answer is right here
-        const instances = Object.entries(accessory.context.openApiCapabilities)
-          .map(([instance, capability]) => `${instance} (${capability?.type ?? 'unknown type'})`)
-        this.log.debug(
-          '[%s] openapi category [%s] capabilities [%s].',
-          accessory.displayName,
-          device.openApiInfo.category || 'unknown',
-          instances.join(', ') || 'none',
-        )
-      }
+      applyDeviceContext(this, accessory, device, deviceConf, {
+        awsDevices,
+        awsDevicesToPoll,
+        doAWSPolling: route.awsPolling,
+      })
 
-      // Create the instance for this device type
       accessory.control = new devInstance(this, accessory)
 
       // Log the device initialisation

package/lib/utils/device-context.js

@@ -0,0 +1,179 @@
+import platformConsts from './constants.js'
+import { getDeviceCapabilities } from './device-capabilities.js'
+import { hasProperty } from './functions.js'
+
+/**
+ * Fills in everything a handler needs to know about a device before it is
+ * built: which connections are available, what firmware it reports, the limits
+ * it works within, and what Govee's api says it can do.
+ *
+ * Lifted out of the platform unchanged. It is data assembly rather than
+ * anything to do with Homebridge, and it was making the one method that sets a
+ * device up twice as long as it needed to be.
+ *
+ * @param {object} platform the platform, for logging and settings
+ * @param {object} accessory the accessory being set up
+ * @param {object} device the device as Govee describes it
+ * @param {object} deviceConf this device's settings
+ * @param {object} joins where to record the connections this device can use -
+ *   `awsDevices`, `awsDevicesToPoll`, and whether this kind of device needs its
+ *   status asking for over AWS
+ */
+export function applyDeviceContext(platform, accessory, device, deviceConf, joins) {
+  const { awsDevices, awsDevicesToPoll, doAWSPolling } = joins
+  // Add the temperatureSource config to the context if exists
+  if (deviceConf.temperatureSource) {
+    accessory.context.temperatureSource = deviceConf.temperatureSource
+  }
+
+  // Add the openApiTempUnit config to the context if set to something other than auto
+  if (deviceConf.openApiTempUnit && deviceConf.openApiTempUnit !== 'auto') {
+    accessory.context.openApiTempUnit = deviceConf.openApiTempUnit
+  } else {
+    delete accessory.context.openApiTempUnit
+  }
+
+  // Get a supported command list if provided, with their options
+  if (device.supportCmds && Array.isArray(device.supportCmds)) {
+    accessory.context.supportedCmds = device.supportCmds
+    accessory.context.supportedCmdsOpts = {}
+
+    device.supportCmds.forEach((cmd) => {
+      if (device?.properties?.[cmd]) {
+        accessory.context.supportedCmdsOpts[cmd] = device.properties[cmd]
+      }
+    })
+  }
+
+  // Add some initial context information which is changed later
+  accessory.context.hasAwsControl = false
+  accessory.context.useAwsControl = false
+  accessory.context.hasOpenApiControl = false
+  accessory.context.useOpenApiControl = false
+  accessory.context.hasBleControl = false
+  accessory.context.useBleControl = false
+  accessory.context.hasLanControl = device.isLanDevice
+  accessory.context.useLanControl = accessory.context.hasLanControl
+  accessory.context.firmware = false
+  accessory.context.hardware = false
+  accessory.context.image = false
+
+  // Overrides for when a custom IP is provided, for a light which is not BLE only
+  if (
+    deviceConf.customIPAddress
+    && accessory.context.hasLanControl
+    && accessory.context.hasAwsControl
+    && platformConsts.models.rgb.includes(device.model)
+  ) {
+    accessory.context.hasLanControl = true
+    accessory.context.useLanControl = true
+  }
+
+  // If the device is LAN-only, then sync the display name with the label in the configuration
+  if (device.isLanOnly) {
+    accessory.displayName = device.deviceName
+  }
+
+  // See if we have extra HTTP client info for this device
+  if (device.httpInfo) {
+    // Save the hardware and firmware versions
+    accessory.context.firmware = device.httpInfo.versionSoft
+    accessory.context.hardware = device.httpInfo.versionHard
+
+    // It's possible to show a nice little icon of the device in the Homebridge UI
+    if (device.httpInfo.deviceExt && device.httpInfo.deviceExt.extResources) {
+      try {
+        const parsed = JSON.parse(device.httpInfo.deviceExt.extResources)
+        if (parsed && parsed.skuUrl) {
+          accessory.context.image = parsed.skuUrl
+        }
+      } catch {
+        // Ignore malformed extResources
+      }
+    }
+
+    // HTTP info lets us see if AWS/BLE connection methods are available
+    if (device.httpInfo.deviceExt && device.httpInfo.deviceExt.deviceSettings) {
+      let parsed
+      try {
+        parsed = JSON.parse(device.httpInfo.deviceExt.deviceSettings)
+      } catch {
+        platform.log.debugWarn('[%s] failed to parse deviceSettings, skipping AWS/BLE setup.', accessory.displayName)
+      }
+
+      // Check to see if AWS is possible
+      if (parsed) {
+        if (parsed.topic) {
+          accessory.context.hasAwsControl = true
+          accessory.context.awsTopic = parsed.topic
+
+          if (platform.awsClient) {
+            accessory.context.useAwsControl = true
+            accessory.context.awsBrightnessNoScale = deviceConf.awsBrightnessNoScale
+            accessory.context.awsColourMode = deviceConf.awsColourMode || platformConsts.defaultValues.awsColourMode
+            awsDevices.push(device.device)
+
+            // Certain models need AWS polling
+            if (doAWSPolling) {
+              awsDevicesToPoll.push(device.device)
+            }
+          }
+        }
+
+        // Check to see if BLE is possible
+        if (parsed.bleName) {
+          const providedBle = parsed.address ? parsed.address.toLowerCase() : device.device.substring(6).toLowerCase()
+          accessory.context.hasBleControl = !!parsed.bleName
+          accessory.context.bleAddress = deviceConf.customAddress
+            ? deviceConf.customAddress.toLowerCase()
+            : providedBle
+          accessory.context.bleName = parsed.bleName
+          accessory.context.bleWriteUuid = getDeviceCapabilities(accessory.context.gvModel).bleWriteUuid
+          if (platform.bleClient) {
+            accessory.context.useBleControl = true
+          }
+        }
+
+        // Gateway subdevices (e.g. leak sensors paired to an H5044) include
+        // their gateway id and slot number, which we need to match real-time
+        // leak events relayed by the gateway over AWS (#1276)
+        if (hasProperty(parsed, 'sno') && parsed.gatewayInfo?.device) {
+          accessory.context.gatewaySno = parsed.sno
+          accessory.context.gatewayDevice = parsed.gatewayInfo.device
+        }
+
+        // Get a min and max temperature/humidity range to show in the homebridge-ui
+        if (hasProperty(parsed, 'temCali')) {
+          accessory.context.minTemp = parsed.temMin / 100
+          accessory.context.maxTemp = parsed.temMax / 100
+          accessory.context.offTemp = parsed.temCali
+        }
+        if (hasProperty(parsed, 'humCali')) {
+          accessory.context.minHumi = parsed.humMin / 100
+          accessory.context.maxHumi = parsed.humMax / 100
+          accessory.context.offHumi = parsed.humCali
+        }
+      }
+    }
+  }
+
+  if (device.openApiInfo) {
+    accessory.context.hasOpenApiControl = true
+    accessory.context.useOpenApiControl = !!platform.openApiClient
+    accessory.context.openApiCapabilities = device.openApiInfo.byInstance || {}
+
+    // Log what Govee says this device can do. Working out whether a model
+    // exposes something like a night light otherwise means reverse
+    // engineering its status messages, when the answer is right here
+    const instances = Object.entries(accessory.context.openApiCapabilities)
+      .map(([instance, capability]) => `${instance} (${capability?.type ?? 'unknown type'})`)
+    platform.log.debug(
+      '[%s] openapi category [%s] capabilities [%s].',
+      accessory.displayName,
+      device.openApiInfo.category || 'unknown',
+      instances.join(', ') || 'none',
+    )
+  }
+
+  // Create the instance for this device type
+}

package/package.json

@@ -2,7 +2,7 @@
   "name": "@homebridge-plugins/homebridge-govee",
   "alias": "Govee",
   "type": "module",
-  "version": "11.32.2-beta.16",
+  "version": "11.32.2-beta.17",
   "description": "Homebridge plugin to integrate Govee devices into HomeKit.",
   "author": {
     "name": "bwp91",