homebridge-melcloud-control 4.6.4 → 4.6.6-beta.0

package/config.schema.json

@@ -10,6 +10,11 @@
   "schema": {
     "type": "object",
     "properties": {
+      "name": {
+        "title": "Platform",
+        "type": "string",
+        "default": "MELCLoud Control"
+      },
       "accounts": {
         "type": "array",
         "title": "Accounts",
@@ -2343,8 +2348,8 @@
                 "host": {
                   "title": "IP/Hostname",
                   "type": "string",
-                  "placeholder": "ip or hostname",
-                  "format": "hostname",
+                  "placeholder": "192.168.1.20 or mqtt.local",
+                  "pattern": "^((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)(\\.(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)){3}|([a-zA-Z0-9]([a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?)(\\.([a-zA-Z0-9]([a-zA-Z0-9-]{0,61}[a-zA-Z0-9])?))*)$",
                   "description": "Here set the IP/Hostname of MQTT Broker.",
                   "condition": {
                     "functionBody": "return model.devices[arrayIndices].mqtt.enable === true;"
@@ -2430,7 +2435,26 @@
                     }
                   ]
                 }
-              }
+              },
+              "allOf": [
+                {
+                  "if": {
+                    "required": [
+                      "enable"
+                    ],
+                    "properties": {
+                      "enable": {
+                        "const": true
+                      }
+                    }
+                  },
+                  "then": {
+                    "required": [
+                      "host"
+                    ]
+                  }
+                }
+              ]
             }
           },
           "required": [

package/homebridge-ui/public/index.html

@@ -257,8 +257,17 @@
           return;
         }
 
+        // Ensure device arrays exist before handling
+        account.ataDevices = account.ataDevices ?? [];
+        account.atwDevices = account.atwDevices ?? [];
+        account.ervDevices = account.ervDevices ?? [];
+
         // Prepare MELCloud data
-        const newInMelCloud = { ata: [], ataPresets: [], ataSchedules: [], ataScenes: [], atw: [], atwPresets: [], atwSchedules: [], atwScenes: [], erv: [], ervPresets: [], ervSchedules: [], ervScenes: [] };
+        const newInMelCloud = {
+          ata: [], ataPresets: [], ataSchedules: [], ataScenes: [],
+          atw: [], atwPresets: [], atwSchedules: [], atwScenes: [],
+          erv: [], ervPresets: [], ervSchedules: [], ervScenes: []
+        };
         const devicesInMelCloudByType = { ata: [], atw: [], erv: [] };
         const scenesInMelCloud = response.Scenes ?? [];
 
@@ -279,7 +288,7 @@
         // Map UnitId → Scenes
         const unitIdToScenes = mapUnitIdToScenes(scenesInMelCloud);
 
-        // Generic device handler (obsługuje urządzenia, presety, harmonogramy i sceny)
+        // Generic device handler (handles devices, presets, schedules, and scenes)
         const handleDevices = (devicesInMelCloud, devicesInConfig, newDevices, newPresets, newSchedules, newScenes) => {
           const configDevicesMap = new Map(devicesInConfig.map(dev => [String(dev.id), dev]));
 
@@ -362,10 +371,33 @@
           return devicesInConfig;
         };
 
-        // Execute device handlers
-        account.ataDevices = handleDevices(devicesInMelCloudByType.ata, account.ataDevices, newInMelCloud.ata, newInMelCloud.ataPresets, newInMelCloud.ataSchedules, newInMelCloud.ataScenes);
-        account.atwDevices = handleDevices(devicesInMelCloudByType.atw, account.atwDevices, newInMelCloud.atw, newInMelCloud.atwPresets, newInMelCloud.atwSchedules, newInMelCloud.atwScenes);
-        account.ervDevices = handleDevices(devicesInMelCloudByType.erv, account.ervDevices, newInMelCloud.erv, newInMelCloud.ervPresets, newInMelCloud.ervSchedules, newInMelCloud.ervScenes);
+        // Execute device handlers with safe initialization
+        account.ataDevices = handleDevices(
+          devicesInMelCloudByType.ata,
+          account.ataDevices,
+          newInMelCloud.ata,
+          newInMelCloud.ataPresets,
+          newInMelCloud.ataSchedules,
+          newInMelCloud.ataScenes
+        );
+
+        account.atwDevices = handleDevices(
+          devicesInMelCloudByType.atw,
+          account.atwDevices,
+          newInMelCloud.atw,
+          newInMelCloud.atwPresets,
+          newInMelCloud.atwSchedules,
+          newInMelCloud.atwScenes
+        );
+
+        account.ervDevices = handleDevices(
+          devicesInMelCloudByType.erv,
+          account.ervDevices,
+          newInMelCloud.erv,
+          newInMelCloud.ervPresets,
+          newInMelCloud.ervSchedules,
+          newInMelCloud.ervScenes
+        );
 
         // Summary counts
         const newDevicesCount = newInMelCloud.ata.length + newInMelCloud.atw.length + newInMelCloud.erv.length;
@@ -389,6 +421,7 @@
 
         await homebridge.updatePluginConfig(pluginConfig);
         await homebridge.savePluginConfig();
+
       } catch (error) {
         updateInfo('info', `Prepare config error`, "red");
         updateInfo('info1', `Error: ${JSON.stringify(error)}`, "red");
@@ -397,6 +430,5 @@
         homebridge.hideSpinner();
       }
     });
-
   })();
 </script>

package/package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "MELCloud Control",
   "name": "homebridge-melcloud-control",
-  "version": "4.6.4",
+  "version": "4.6.6-beta.0",
   "description": "Homebridge plugin to control Mitsubishi Air Conditioner, Heat Pump and Energy Recovery Ventilation.",
   "license": "MIT",
   "author": "grzegorz914",
@@ -39,7 +39,7 @@
     "mqtt": "^5.14.1",
     "axios": "^1.13.2",
     "express": "^5.2.1",
-    "puppeteer": "^24.34.0",
+    "puppeteer": "^24.36.0",
     "ws": "^8.19.0"
   },
   "keywords": [