homebridge-gree-ac 2.1.2 → 2.1.4

package/CHANGELOG.md

@@ -1,10 +1,41 @@
 # Changelog
 
+## [2.1.4] - 2024-11-11
+
+**<ins>Reminder:</ins> New (v2) network encryption protocol supported since v2.1.0**
+
+When upgrading from v2.0.0 - v2.1.1 to v2.1.2 or later, configuration settings should be updated.
+
+### New features
+- Added support of devices with hardware version V3.x which use a mix of V1 and V2 network encryption
+- Reorganized device registration with binding error detection
+
+### Fixes
+- Fixed error on disabling device after successful registration
+- After homebridge startup, don't wait before scanning devices on the network
+- Handling conflicting port numbers
+
+## [2.1.3] - 2024-11-05
+
+**<ins>Reminder:</ins> New (v2) network encryption protocol supported since v2.1.0**
+
+When upgrading from v2.0.0 - v2.1.1 to v2.1.2 or later, configuration settings should be updated.
+
+#### Updated dependencies
+- added Node.js v22 to supported versions (18.15.0 or later, 20.7.0 or later and 22.0.0 or later are supported)
+
+### New features
+- new optional IP address parameter to support devices on different subnets
+- new optional device port parameter to support cases when automatic port assigment is not appropriate
+
+### Fixes
+- consistent xFan default setting in configuration UI and in plugin behaviour
+
 ## [2.1.2] - 2024-10-01
 
 **<ins>Reminder:</ins> New (v2) network encryption protocol supported since v2.1.0**
 
-When upgrading from v2.0.0 - v2.1.1 to v2.1.2 configuration settings should be updated.
+When upgrading from v2.0.0 - v2.1.1 to v2.1.2, configuration settings should be updated.
 
 #### Configuration update steps
 

package/README.md

@@ -31,10 +31,10 @@ This plugin is designed to be as simple and clear as possible and supports prima
 
 ## Requirements
 
-* Node.js (>= 18.15.0 || >= 20.8.0) with NPM
+* Node.js (>= 18.15.0 || >= 20.7.0 || >= 22.0.0) with NPM
 * Homebridge (>= 1.8.0 || >= 2.0.0-beta.0)
 
-Homebridge and all AC units have to be on the same subnet. The plugin finds all supported units automatically but controls only those which MAC address is added to configuration.
+The plugin finds all supported units automatically if they are located on the same subnet but controls only those which MAC address is added to the configuration. AC units on different subnets are also supported if the unit's IP address is set in the configuration. (MAC address have to be set correctly in this case also.)
 
 IPv4 address is required. GREE Air Conditioners do not support IPv6 nor other network protocols.
 
@@ -48,7 +48,7 @@ This is not plugin dependency but its good to know that Homebridge server host a
  
 > If you get _"error:1C80006B:Provider routines::wrong final block length"_ error message then your device is not supported.
 >
-> If you don't get _"Device is bound ..."_ message within a few minutes after Homebridge startup and the correct MAC address is added to the configuration and the AC unit is accessible on the same subnet then your device is not supported.
+> If you get _"Error: Device is not bound ..."_ error message then your device may not be supported. (The same error occures when device is malfunctioning but if turning the power supply off and on does not solve the problem then your device is not supported.)
 
 By default this plugin tries to auto detect the network protocol encryption version. If not the right version is selected there can get errors and the AC device will not correctly work. It is possible to force a network protocol encryption version in configuration file. If auto detection does not work then it is recommended to try all possible values to check if the device is compatible or not.
 
@@ -64,7 +64,6 @@ This plugin was designed to support the Home App's Heater Cooler functionality u
 * Not all half a degree values are supported in °C mode (GREE AC units are designed to support only integer °C and °F values). Unsupported values are automatically updated to the nearest supported values.
 * There is no way to get current heating-cooling state from the AC unit in auto mode, so displayed state in the Home App is based on temperature measurement, but internal sensor is not precise enough to always display the correct state.
 * Cooling / Heating temperature threshold limits (minimum and maximum values) can only be set in active cooling / heating mode. So the gauge in Home App may show invalid minimum and maximum values for the first use of cooling and heating modes. If so please restart Home App. Next time the correct values will be displayed.
-* Homebridge and AC unit on different subnets is a not supported configuration.
 * Devices without a built-in temperature sensor display the target temperature as current temperature not the measured one. (Some AC firmware versions do not report the measured temperature but the unit has a built-in sensor. They are handled by the plugin as devices without a sensor.)
 
 ## Installation instructions
@@ -126,15 +125,17 @@ _Only the relevant part of the configuration file is displayed:_
                 {
                     "mac": "502cc6000000",
                     "name": "Living room AC",
+                    "ip": "192.168.1.2",
+                    "port": 7003
+                    "statusUpdateInterval": 10,
+                    "encryptionVersion": 0,
                     "model": "Pulse 3.2kW GWH12AGB-K6DNA1A/I",
                     "speedSteps": 5,
-                    "encryptionVersion": 0,
-                    "statusUpdateInterval": 10,
-                    "sensorOffset": 40,
                     "minimumTargetTemperature": 16,
                     "maximumTargetTemperature": 30,
-                    "xFanEnabled": true,
+                    "sensorOffset": 40,
                     "temperatureSensor": "disabled",
+                    "xFanEnabled": true,
                     "overrideDefaultVerticalSwing": 0,
                     "defaultVerticalSwing": 0,
                     "disabled": false
@@ -143,28 +144,32 @@ _Only the relevant part of the configuration file is displayed:_
         }
     ]
 ```
-_It's not recommended to add the port parameter. The above example contains it but only for showing all optional parameters also._
+_It's not recommended to add the port and ip parameters. The above example contains them but only for showing all optional parameters also._
 
 * name - Unique name of the platform plugin
 * platform - **GREEAirConditioner** (fixed name, it identifies the plugin)
-* port - free UDP port (optional) (homebridge will use this port for network communication; select a port which is not used and the next 256 ports are also available because devices will be bound to a separate port based on the last part of the device's IPv4 address and the port specified in the configuration; valid values: 1025 - 65279); **Do not specify a port unless you have trouble with automatic port assignment!**
+* port - free UDP port (optional) (plugin will use this port for network communication; valid values: 1025 - 65535) **Do not specify a port unless you have trouble with automatic port assignment!**
 * scanInterval - time period in seconds between device query retries (defaults to 60 sec if missing)
 * devices - devices should be listed in this block (specify as many devices as you have on your network)
   * mac - MAC address (Serial Number) of the device
   * name - custom name of the device (optional) Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis.
+  * ip - device IP address (optional) Address is auto detected if this parameter is missing. **Specify only if device is located on a different subnet then homebridge!**
+  * port - free UDP port (optional) (plugin will listen on this port for data received from the device; valid values: 1025 - 65535) **Do not specify a port unless you have trouble with automatic port assignment!**
+  * statusUpdateInterval - device status will be refreshed based on this interval (in seconds, default is 10 seconds)
+  * encryptionVersion - Auto (0) is fine for most AC units. If auto does not work then you can force v1 (1) or v2 (2) encryption version to use in network communication
   * model - model name, information only (optional)
   * speedSteps - fan speed steps of the unit (valid values are: 3 and 5)
-  * encryptionVersion - Auto (0) is fine for most AC units. If auto does not work then you can force v1 (1) or v2 (2) encryption version to use in network communication
-  * statusUpdateInterval - device status will be refreshed based on this interval (in seconds)
-  * sensorOffset - device temperature sensor offset value for current temperature calibration (default is 40 °C, must be specified in °C)
   * minimumTargetTemperature - minimum target temperature accepted by the device (default is 16 °C, must be specified in °C, valid values: 16-30)
   * maximumTargetTemperature - maximum target temperature accepted by the device (default is 30 °C, must be specified in °C, valid values: 16-30)
-  * xFanEnabled - automatically turn on xFan functionality in supported device modes (xFan actual setting is not modified by the Home App if disabled)
+  * sensorOffset - device temperature sensor offset value for current temperature calibration (default is 40 °C, must be specified in °C)
   * temperatureSensor - control additional temperature sensor accessory in Home App (disabled = do not add to Home App / child = add as a child accessory / separate = add as a separate (independent) accessory)
+  * xFanEnabled - automatically turn on xFan functionality in supported device modes (xFan actual setting is not modified by the Home App if disabled)
   * overrideDefaultVerticalSwing - by default this plugin does not change the vertical swing position of the AC unit but some devices do not keep the original vertical position set by the remote control if controlled from Homebridge and return back to device default position; this setting allows to override the default position -> if AC unit is set to default vertical swing position Homebridge modifies it to a predefined position (set by defaultVerticalSwing) (Never (0) = turn off override, let device use default / After power on (1) = override default position on each power on / After power on and swing disable (2) = override default position on each power on and each time when swing is switched to disabled)
   * defaultVerticalSwing - specify the vertical swing position to be used instead of device default when overriding is enabled (Device default (0) = use device default, same position as used by device by default without overriding / one of the following 5 positions: fixed Highest (2), fixed Higher (3), fixed Middle (4), fixed Lower (5), fixed Lowest (6))
   * disabled - set to true if you do not want to control this device in the Home App _(can be used also to temporarily remove the device from Home App but not if the device is not responding any more on the network)_
 
+Recommended configuration:
+
 ![Homebridge UI](./uiconfig.jpg)
 
 ## Tips
@@ -199,6 +204,19 @@ Some settings are initialized by Home App only once (when enabling the device).
 
 All other settings are applied when starting up Homebridge. You have to restart Homebridge to apply changes in configuration settings.
 
+### IP address
+
+IP addresses of the AC units are determined automatically by the plugin. However this auto detection works only if the AC unit is on the same subnet as homebridge. There is an optional IP address parameter which can be used to specifiy the unit's IP address if it is on a different subnet. (Routing should be correctly set up to communicate with units on different subnets.)
+
+### Port
+
+Network communication uses UDP ports. There are two kind of ports:
+
+- Plugin port. This port is used by the plugin to communicate on the network.
+- Device specific port. The plugin is listening for data received from the device using this port.
+
+All ports are set up automatically by default. In some cases auto detection is not appropriate. (E.g. when firewall rules should be set up) It is possible to overwrite the default ports by optional port parameters (for the plugin and also for each devices). _Note that the ports must be unique._
+
 ### Temperature display units
 
 Home App allows to set the device temperature display units but it is independent from the temperature units shown in Home App. Home App always displays temperature values as specified by iOS/MacOS (can be changed in Preferences / Regional settings section). Display unit conversion is made by the Home App device (e.g. iPhone).
@@ -209,7 +227,7 @@ Temperature measurement is not perfect if using the built-in sensor. It is highl
 
 ### Invalid room temperature
 
-Some AC units have a built-in temperature sensor but the actual room temperature is not displayed in Home App. This is an AC firmware problem. Older firware versions do not report temperature values at all and there are some firmware versions which report a fixed value (e.g. zero) instead of the measured one. This plugin replaces the missing value and the fixed zero value by the desired target temperature. To get the correct measured temperature please try to upgrade or downgrade the AC firmware.
+Some AC units have a built-in temperature sensor but the actual room temperature is not displayed in Home App. This is an AC firmware problem. Older firmware versions do not report temperature values at all and there are some firmware versions which report a fixed value (e.g. zero) instead of the measured one. This plugin replaces the missing value and the fixed zero value by the desired target temperature. To get the correct measured temperature please try to upgrade or downgrade the AC firmware.
 
 ### Fan speed
 

package/config.schema.json

@@ -62,28 +62,28 @@
               "type": "integer",
               "required": false,
               "minimum": 1,
-              "default": 10
+              "placeholder": 10
             },
             "sensorOffset": {
               "type": "integer",
               "required": false,
-              "default": 40
+              "placeholder": 40
             },
             "minimumTargetTemperature": {
               "type": "integer",
               "required": false,
               "minimum": 16,
-              "default": 16
+              "placeholder": 16
             },
             "maximumTargetTemperature": {
               "type": "integer",
               "required": false,
               "maximum": 30,
-              "default": 30
+              "placeholder": 30
             },
             "xFanEnabled": {
               "type": "boolean",
-              "default": false
+              "default": true
             },
             "temperatureSensor": {
               "type": "string",
@@ -205,6 +205,16 @@
                   ]
                 }
               ]
+            },
+            "port": {
+              "type": "integer",
+              "required": false,
+              "minimum": 1025
+            },
+            "ip": {
+              "type": "string",
+              "required": false,
+              "format": "ipv4"
             }
           }
         }
@@ -224,7 +234,7 @@
           "key": "port",
           "flex": "1 1 50%",
           "title": "Port:",
-          "description": "Requested UDP port (auto if empty)"
+          "description": "Platform level UDP port (auto if empty)"
         },
         {
           "key": "scanInterval",
@@ -269,17 +279,19 @@
               }
             },
             {
-              "key": "devices[].model",
+              "key": "devices[].ip",
               "flex": "1 1 50%",
-              "title": "Device model:",
+              "title": "IP address:",
+              "description": "Device IP address (auto if empty)",
               "condition": {
                 "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
               }
             },
             {
-              "key": "devices[].speedSteps",
+              "key": "devices[].port",
               "flex": "1 1 50%",
-              "title": "Fan speed steps:",
+              "title": "Port:",
+              "description": "Device specific UDP port (auto if empty)",
               "condition": {
                 "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
               }
@@ -302,6 +314,22 @@
                 "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
               }
             },
+            {
+              "key": "devices[].model",
+              "flex": "1 1 50%",
+              "title": "Device model:",
+              "condition": {
+                "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
+              }
+            },
+            {
+              "key": "devices[].speedSteps",
+              "flex": "1 1 50%",
+              "title": "Fan speed steps:",
+              "condition": {
+                "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
+              }
+            },
             {
               "key": "devices[].minimumTargetTemperature",
               "flex": "1 1 50%",
@@ -329,15 +357,6 @@
                 "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
               }
             },
-            {
-              "key": "devices[].xFanEnabled",
-              "flex": "1 1 50%",
-              "title": "xFan enabled",
-              "description": "If enabled, then xFan functionality is turned on automatically on the device",
-              "condition": {
-                "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
-              }
-            },
             {
               "key": "devices[].temperatureSensor",
               "flex": "1 1 50%",
@@ -348,7 +367,10 @@
               }
             },
             {
-              "flex": "1 1 50%",
+              "key": "devices[].xFanEnabled",
+              "flex": "1 1 100%",
+              "title": "xFan enabled",
+              "description": "If enabled, then xFan functionality is turned on automatically on the device",
               "condition": {
                 "functionBody": "return (model.devices && model.devices[arrayIndices] && model.devices[arrayIndices].mac && /^[a-f0-9]{12}$/.test(model.devices[arrayIndices].mac) && model.devices[arrayIndices].disabled !== true);"
               }

package/dist/platform.d.ts

@@ -1,4 +1,8 @@
 import { API, DynamicPlatformPlugin, Logger, PlatformAccessory, PlatformConfig, Service, Characteristic } from 'homebridge';
+export interface MyPlatformAccessory extends PlatformAccessory {
+    bound?: boolean;
+    registered?: boolean;
+}
 /**
  * HomebridgePlatform
  * This class is the main constructor for your plugin, this is where you should
@@ -11,16 +15,17 @@ export declare class GreeACPlatform implements DynamicPlatformPlugin {
     readonly Service: typeof Service;
     readonly Characteristic: typeof Characteristic;
     private devices;
-    private initializedDevices;
+    private processedDevices;
     private skippedDevices;
     private socket;
     private pluginAddresses;
+    ports: number[];
     constructor(log: Logger, config: PlatformConfig, api: API);
     /**
      * This function is invoked when homebridge restores cached accessories from disk at startup.
      * It should be used to setup event handlers for characteristics and update respective values.
      */
-    configureAccessory(accessory: PlatformAccessory): void;
+    configureAccessory(accessory: MyPlatformAccessory): void;
     /**
      * Accessories must only be registered once, previously created accessories
      * must not be registered again to prevent "duplicate UUID" errors.
@@ -29,7 +34,8 @@ export declare class GreeACPlatform implements DynamicPlatformPlugin {
     discoverDevices(): void;
     handleMessage: (msg: any, rinfo: any) => void;
     registerDevice: (deviceInfo: any) => void;
-    broadcastScan(): void;
+    sendScan(): void;
     getNetworkAddresses(): {};
+    getAccessory(mac: string): MyPlatformAccessory;
 }
 //# sourceMappingURL=platform.d.ts.map

package/dist/platform.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"platform.d.ts","sourceRoot":"","sources":["../src/platform.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,GAAG,EAAE,qBAAqB,EAAE,MAAM,EAAE,iBAAiB,EAAE,cAAc,EAAE,OAAO,EAAE,cAAc,EAAc,MAAM,YAAY,CAAC;AAUxI;;;;GAIG;AACH,qBAAa,cAAe,YAAW,qBAAqB;aAYxC,GAAG,EAAE,MAAM;aACX,MAAM,EAAE,cAAc;aACtB,GAAG,EAAE,GAAG;IAb1B,SAAgB,OAAO,EAAE,OAAO,OAAO,CAAwB;IAC/D,SAAgB,cAAc,EAAE,OAAO,cAAc,CAA+B;IAGpF,OAAO,CAAC,OAAO,CAAoC;IACnD,OAAO,CAAC,kBAAkB,CAA0B;IACpD,OAAO,CAAC,cAAc,CAA0B;IAChD,OAAO,CAAC,MAAM,CAAe;IAC7B,OAAO,CAAC,eAAe,CAA8B;gBAGnC,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,cAAc,EACtB,GAAG,EAAE,GAAG;IAsC1B;;;OAGG;IACH,kBAAkB,CAAC,SAAS,EAAE,iBAAiB;IAe/C;;;OAGG;IACH,YAAY;IAQZ,eAAe;IAUf,aAAa,iCA4CX;IAEF,cAAc,4BAyIZ;IAEF,aAAa;IAYb,mBAAmB;CA8BpB"}
+{"version":3,"file":"platform.d.ts","sourceRoot":"","sources":["../src/platform.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,GAAG,EAAE,qBAAqB,EAAE,MAAM,EAAE,iBAAiB,EAAE,cAAc,EAAE,OAAO,EAAE,cAAc,EAAc,MAAM,YAAY,CAAC;AAUxI,MAAM,WAAW,mBAAoB,SAAQ,iBAAiB;IAC5D,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,UAAU,CAAC,EAAE,OAAO,CAAC;CACtB;AAED;;;;GAIG;AACH,qBAAa,cAAe,YAAW,qBAAqB;aAaxC,GAAG,EAAE,MAAM;aACX,MAAM,EAAE,cAAc;aACtB,GAAG,EAAE,GAAG;IAd1B,SAAgB,OAAO,EAAE,OAAO,OAAO,CAAwB;IAC/D,SAAgB,cAAc,EAAE,OAAO,cAAc,CAA+B;IAGpF,OAAO,CAAC,OAAO,CAAsC;IACrD,OAAO,CAAC,gBAAgB,CAA0B;IAClD,OAAO,CAAC,cAAc,CAA0B;IAChD,OAAO,CAAC,MAAM,CAAe;IAC7B,OAAO,CAAC,eAAe,CAA8B;IAC9C,KAAK,EAAE,MAAM,EAAE,CAAM;gBAGV,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,cAAc,EACtB,GAAG,EAAE,GAAG;IAsC1B;;;OAGG;IACH,kBAAkB,CAAC,SAAS,EAAE,mBAAmB;IAsBjD;;;OAGG;IACH,YAAY;IAUZ,eAAe;IAUf,aAAa,iCA6CX;IAEF,cAAc,4BA4IZ;IAEF,QAAQ;IAiBR,mBAAmB;IAqDZ,YAAY,CAAC,GAAG,EAAE,MAAM;CAGhC"}

package/dist/platform.js

@@ -9,7 +9,7 @@ const crypto_1 = __importDefault(require("./crypto"));
 const os_1 = require("os");
 const settings_1 = require("./settings");
 const platformAccessory_1 = require("./platformAccessory");
-const tsAccessory_1 = require("./tsAccessory");
+//import { GreeAirConditionerTS } from './tsAccessory';
 const commands_1 = __importDefault(require("./commands"));
 const version_1 = require("./version");
 /**
@@ -25,6 +25,7 @@ class GreeACPlatform {
         this.Service = this.api.hap.Service;
         this.Characteristic = this.api.hap.Characteristic;
         this.pluginAddresses = {};
+        this.ports = [];
         this.handleMessage = (msg, rinfo) => {
             this.log.debug('handleMessage -> %s', msg.toString());
             try {
@@ -52,8 +53,9 @@ class GreeACPlatform {
                     encryptionVersion = 2;
                 }
                 this.log.debug('handleMessage - Package -> %j', pack);
-                if (encryptionVersion === 1 && pack.t === 'dev' && pack.ver && pack.ver.toString().startsWith('V2.')) {
-                    // first V2 version responded to scan command with V1 encryption but binding requires V2 encryption
+                if (encryptionVersion === 1 && pack.t === 'dev' && pack.ver && !pack.ver.toString().startsWith('V1.')) {
+                    // some devices respond to scan command with V1 encryption but binding requires V2 encryption
+                    // we set encryption to V2 if device version is not V1.x
                     encryptionVersion = 2;
                 }
                 if (pack.t === 'dev') {
@@ -103,6 +105,12 @@ class GreeACPlatform {
                     deviceConfig[key] = value;
                 }
             });
+            if (deviceConfig.port !== undefined && (typeof deviceConfig.port !== 'number' || deviceConfig.port !== deviceConfig.port ||
+                (typeof deviceConfig.port === 'number' && (deviceConfig.port < 1025 || deviceConfig.port > 65535)))) {
+                this.log.warn('Warning: Port is misconfigured (Valid port values: 1025~65535 or leave port empty to auto select) - ' +
+                    `Accessory ${deviceInfo.mac} listening port overridden: ${deviceConfig.port} -> auto`);
+                deviceConfig.port = undefined;
+            }
             // force encryption version if set in config
             if (deviceConfig.encryptionVersion !== settings_1.ENCRYPTION_VERSION.auto) {
                 deviceInfo.encryptionVersion = deviceConfig.encryptionVersion;
@@ -119,13 +127,13 @@ class GreeACPlatform {
                     this.log.debug(`Accessory ${deviceInfo.mac}${devcfg.mac === undefined ? ' not configured -' : ''} skipped`);
                 }
                 if (accessory) {
-                    delete this.devices[accessory.context.deviceInfo.mac];
+                    delete this.devices[accessory.context.device.mac];
                     this.api.unregisterPlatformAccessories(settings_1.PLUGIN_NAME, settings_1.PLATFORM_NAME, [accessory]);
                     this.log.debug(`registerDevice - unregister (${devcfg.mac === undefined ? 'not configured' : 'disabled'}):`, accessory.displayName, accessory.UUID);
                     accessory = undefined;
                 }
                 if (accessory_ts) {
-                    delete this.devices[accessory_ts.context.deviceInfo.mac];
+                    delete this.devices[accessory_ts.context.device.mac];
                     this.api.unregisterPlatformAccessories(settings_1.PLUGIN_NAME, settings_1.PLATFORM_NAME, [accessory_ts]);
                     this.log.debug(`registerDevice - unregister (${devcfg.mac === undefined ? 'not configured' : 'disabled'}):`, accessory_ts.displayName, accessory_ts.UUID);
                     accessory_ts = undefined;
@@ -140,28 +148,29 @@ class GreeACPlatform {
             if (accessory_ts && deviceInfo.address !== accessory_ts.context.device.address) {
                 accessory_ts.context.device.address = deviceInfo.address;
             }
-            if (accessory && this.initializedDevices[accessory.UUID]) {
+            if (accessory && this.processedDevices[accessory.UUID]) {
                 // already initalized
-                this.log.debug('registerDevice - already initalized:', accessory.displayName, accessory.UUID, accessory.context.device.mac);
+                this.log.debug('registerDevice - already processed:', accessory.displayName, accessory.context.device.mac, accessory.UUID);
                 return;
             }
-            // register heatercooler accessory
+            // create heatercooler accessory if not loaded from cache
             const deviceName = (_a = deviceConfig === null || deviceConfig === void 0 ? void 0 : deviceConfig.name) !== null && _a !== void 0 ? _a : (deviceInfo.name || deviceInfo.mac);
             if (!accessory) {
-                this.log.debug(`Initializing new accessory ${deviceInfo.mac} with name ${deviceName} ...`);
+                this.log.debug(`Creating new accessory ${deviceInfo.mac} with name ${deviceName} ...`);
                 const uuid = this.api.hap.uuid.generate(deviceInfo.mac);
                 accessory = new this.api.platformAccessory(deviceName, uuid, 21 /* Categories.AIR_CONDITIONER */);
+                accessory.bound = false;
+                accessory.registered = false;
                 this.devices[deviceInfo.mac] = accessory;
-                this.api.registerPlatformAccessories(settings_1.PLUGIN_NAME, settings_1.PLATFORM_NAME, [accessory]);
             }
-            // register temperaturesensor accessory if configured as separate
+            // create temperaturesensor accessory if configured as separate and not loaded from cache
             const tsDeviceName = 'Temperature Sensor - ' + ((_b = deviceConfig === null || deviceConfig === void 0 ? void 0 : deviceConfig.name) !== null && _b !== void 0 ? _b : (deviceInfo.name || deviceInfo.mac));
             if (!accessory_ts && deviceConfig.temperatureSensor === settings_1.TS_TYPE.separate) {
-                this.log.debug(`Initializing new accessory ${deviceInfo.mac}_ts with name ${tsDeviceName} ...`);
+                this.log.debug(`Creating new accessory ${deviceInfo.mac}_ts with name ${tsDeviceName} ...`);
                 const uuid = this.api.hap.uuid.generate(deviceInfo.mac + '_ts');
                 accessory_ts = new this.api.platformAccessory(tsDeviceName, uuid, 10 /* Categories.SENSOR */);
+                accessory_ts.registered = false;
                 this.devices[deviceInfo.mac + '_ts'] = accessory_ts;
-                this.api.registerPlatformAccessories(settings_1.PLUGIN_NAME, settings_1.PLATFORM_NAME, [accessory_ts]);
             }
             // unregister temperaturesensor accessory if configuration has changed from separate to any other
             if (accessory_ts && deviceConfig.temperatureSensor !== settings_1.TS_TYPE.separate) {
@@ -170,39 +179,36 @@ class GreeACPlatform {
                 this.log.debug('registerDevice - unregister:', accessory_ts.displayName, accessory_ts.UUID);
                 accessory_ts = undefined;
             }
-            let tsService = null;
             if (accessory_ts && deviceConfig.temperatureSensor === settings_1.TS_TYPE.separate) {
                 // mark temperature sensor device as initialized
                 accessory_ts.context.device = { ...deviceInfo };
                 accessory_ts.context.device.mac = deviceInfo.mac + '_ts';
                 accessory_ts.context.deviceType = 'TemperatureSensor';
+                if (deviceConfig.model) {
+                    accessory_ts.context.device.model = deviceConfig.model;
+                }
                 accessory_ts.displayName = tsDeviceName;
-                this.initializedDevices[accessory_ts.UUID] = true;
-                tsService = new tsAccessory_1.GreeAirConditionerTS(this, accessory_ts, deviceConfig);
-                this.log.debug(`registerDevice - ${accessory_ts.context.deviceType} initialized:`, accessory_ts.displayName, accessory_ts.context.device.mac, accessory_ts.UUID);
+                this.processedDevices[accessory_ts.UUID] = true;
+                this.log.debug(`registerDevice - ${accessory_ts.context.deviceType} created:`, accessory_ts.displayName, accessory_ts.context.device.mac, accessory_ts.UUID);
+                // do not load temperature sensor accessory here (it will be loaded from heatercooler accessory)
             }
             if (accessory) {
                 // mark heatercooler device as initialized
                 accessory.context.device = deviceInfo;
                 accessory.context.deviceType = 'HeaterCooler';
                 accessory.displayName = deviceName;
-                this.initializedDevices[accessory.UUID] = true;
-                this.log.debug(`registerDevice - ${accessory.context.deviceType} initialized:`, accessory.displayName, accessory.context.device.mac, accessory.UUID);
-                new platformAccessory_1.GreeAirConditioner(this, accessory, deviceConfig, this.config.port, tsService);
-            }
-            // update registered accessories at the end of initialization
-            const accessories = [accessory];
-            if (accessory_ts) {
-                accessories.push(accessory_ts);
+                this.processedDevices[accessory.UUID] = true;
+                this.log.debug(`registerDevice - ${accessory.context.deviceType} created:`, accessory.displayName, accessory.context.device.mac, accessory.UUID);
+                // load heatercooler accessory
+                new platformAccessory_1.GreeAirConditioner(this, accessory, deviceConfig, accessory_ts === null || accessory_ts === void 0 ? void 0 : accessory_ts.context.device.mac);
             }
-            this.api.updatePlatformAccessories(accessories);
         };
         this.devices = {};
-        this.initializedDevices = {};
+        this.processedDevices = {};
         this.skippedDevices = {};
         this.pluginAddresses = this.getNetworkAddresses();
         if (Object.entries(this.pluginAddresses).length > 0) {
-            this.log.debug('Homebridge host addresses: ', this.pluginAddresses);
+            this.log.debug('Device detection address list {(address : netmask) pairs}:', this.pluginAddresses);
         }
         else {
             this.log.error('Error: Homebridge host has no IPv4 address');
@@ -238,11 +244,14 @@ class GreeACPlatform {
      * It should be used to setup event handlers for characteristics and update respective values.
      */
     configureAccessory(accessory) {
-        var _a;
+        var _a, _b, _c;
         this.log.debug('Loading accessory from cache:', accessory.displayName, JSON.stringify(accessory.context.device));
         // add the restored accessory to the accessories cache so we can track if it has already been registered
-        if ((_a = accessory.context.device) === null || _a === void 0 ? void 0 : _a.mac) {
-            accessory.context.bound = false;
+        if ((_b = (_a = accessory.context) === null || _a === void 0 ? void 0 : _a.device) === null || _b === void 0 ? void 0 : _b.mac) {
+            if (!accessory.context.deviceType || accessory.context.deviceType === 'HeaterCooler') {
+                accessory.bound = false;
+            }
+            accessory.registered = true;
             this.devices[accessory.context.device.mac] = accessory;
         }
         // clean all invalid accessories found in cache
@@ -250,6 +259,10 @@ class GreeACPlatform {
             this.log.debug('Invalid accessory found in cache - deleting:', accessory.displayName, accessory.UUID);
             this.api.unregisterPlatformAccessories(settings_1.PLUGIN_NAME, settings_1.PLATFORM_NAME, [accessory]);
         }
+        // remove deprecated properties from cached accessory
+        if (((_c = accessory.context) === null || _c === void 0 ? void 0 : _c.bound) !== undefined) {
+            delete accessory.context.bound;
+        }
     }
     /**
      * Accessories must only be registered once, previously created accessories
@@ -257,28 +270,36 @@ class GreeACPlatform {
      */
     bindCallback() {
         this.log.info(`${settings_1.PLATFORM_NAME} (${settings_1.PLUGIN_NAME}) v%s is running on UDP port %d`, version_1.version, this.socket.address().port);
+        this.ports.push(this.socket.address().port);
         this.socket.setBroadcast(true);
+        this.sendScan();
         setInterval(() => {
-            this.broadcastScan();
+            this.sendScan();
         }, (this.config.scanInterval || settings_1.DEF_SCAN_INTERVAL) * 1000); // scanInterval in seconds (default = 60 sec)
     }
     discoverDevices() {
         if (this.config.port === undefined || (this.config.port !== undefined && typeof this.config.port === 'number' &&
-            this.config.port === this.config.port && this.config.port >= 0 && this.config.port <= 65279)) {
+            this.config.port === this.config.port && this.config.port >= 1025 && this.config.port <= 65535)) {
             this.socket.bind(this.config.port, undefined, () => this.bindCallback());
         }
         else {
-            this.log.error('Error: Port is misconfigured (Valid port values: 1025~65279 or leave port empty to auto select)');
+            this.log.error('Error: Port is misconfigured (Valid port values: 1025~65535 or leave port empty to auto select)');
             this.socket.close();
         }
     }
-    broadcastScan() {
+    sendScan() {
         const message = Buffer.from(JSON.stringify({ t: 'scan' }));
         Object.entries(this.pluginAddresses).forEach((value) => {
-            this.socket.send(message, 0, message.length, settings_1.UDP_SCAN_PORT, value[0], (error) => {
-                this.log.debug(`Broadcast '${message}' ${value[0]}:${settings_1.UDP_SCAN_PORT}`);
+            const addr = value[0];
+            this.socket.send(message, 0, message.length, settings_1.UDP_SCAN_PORT, addr, (error) => {
+                if (this.pluginAddresses[addr] === '255.255.255.255') {
+                    this.log.debug(`Scanning for device (unicast) '${message}' ${addr}:${settings_1.UDP_SCAN_PORT}`);
+                }
+                else {
+                    this.log.debug(`Scanning for devices (broadcast) '${message}' ${addr}:${settings_1.UDP_SCAN_PORT}`);
+                }
                 if (error) {
-                    this.log.error('broadcastScan - Error:', error.message);
+                    this.log.error('Device scan - Error:', error.message);
                 }
             });
         });
@@ -298,21 +319,48 @@ class GreeACPlatform {
                         const netmaskParts = iface.netmask.split('.');
                         const broadcast = addrParts.map((e, i) => ((~Number(netmaskParts[i]) & 0xFF) | Number(e)).toString()).join('.');
                         this.log.debug('Interface: \'%s\' Address: %s Netmask: %s Broadcast: %s', name, iface.address, iface.netmask, broadcast);
-                        pluginAddresses[broadcast] = pluginAddresses[broadcast] === undefined ? iface.address :
-                            pluginAddresses[broadcast] + ';' + iface.address;
+                        if (pluginAddresses[broadcast] === undefined) {
+                            pluginAddresses[broadcast] = iface.netmask;
+                        }
                     }
                 }
             }
         }
-        // Sort IP addresses for consistent comparison
-        for (const bcast of Object.keys(pluginAddresses)) {
-            // Keep only unique IPs
-            const ips = Array.from(new Set(pluginAddresses[bcast].split(';')));
-            ips.sort((a, b) => (a < b ? -1 : 1));
-            pluginAddresses[bcast] = ips.join(';');
+        // Add IPs from configuration but only if at least one host address found
+        if (Object.keys(pluginAddresses).length > 0) {
+            const devcfgs = this.config.devices.filter((item) => item.ip && !item.disabled) || [];
+            devcfgs.forEach((value) => {
+                const ip = value['ip'];
+                const ipv4Pattern = /^(25[0-5]|2[0-4]\d|1\d{2}|\d{1,2})(\.(25[0-5]|2[0-4]\d|1\d{2}|\d{1,2})){3}$/;
+                if (ipv4Pattern.test(ip)) {
+                    this.log.debug('Found AC Unit address in configuration:', ip);
+                    const addrParts = ip.split('.');
+                    const addresses = {};
+                    Object.keys(pluginAddresses).forEach((addr) => {
+                        const netmaskParts = pluginAddresses[addr].split('.');
+                        const broadcast = addrParts.map((e, i) => ((~Number(netmaskParts[i]) & 0xFF) | Number(e)).toString()).join('.');
+                        if (addr === broadcast) {
+                            addresses[ip] = true;
+                        }
+                    });
+                    const skipAddress = Object.keys(addresses).find((addr) => addr === ip);
+                    if (skipAddress === undefined) {
+                        pluginAddresses[ip] = '255.255.255.255';
+                    }
+                    else {
+                        this.log.debug('AC Unit (%s) is already on broadcast list - skipping', skipAddress);
+                    }
+                }
+                else {
+                    this.log.warn('Warning: Invalid IP address found in configuration: %s - skipping', ip);
+                }
+            });
         }
         return pluginAddresses;
     }
+    getAccessory(mac) {
+        return this.devices[mac];
+    }
 }
 exports.GreeACPlatform = GreeACPlatform;
 //# sourceMappingURL=platform.js.map