node-red-contrib-homekit-bridged 1.7.0-dev.1 → 1.7.0-dev.2

package/build/lib/HAPServiceNode.js

@@ -161,6 +161,7 @@ module.exports = (RED) => {
             config: self.config,
         }, self.parentService);
         self.characteristicProperties = CharacteristicUtils.load(self.service, self.config);
+        ServiceUtils.configureAdaptiveLightning();
         if (self.config.isParent) {
             BridgeUtils.delayedPublish(self);
         }

package/build/lib/HAPServiceNode2.js

@@ -161,6 +161,7 @@ module.exports = (RED) => {
             config: self.config,
         }, self.parentService);
         self.characteristicProperties = CharacteristicUtils.load(self.service, self.config);
+        ServiceUtils.configureAdaptiveLightning();
         if (self.config.isParent) {
             BridgeUtils.delayedPublish(self);
         }

package/build/lib/utils/ServiceUtils.js

@@ -120,7 +120,8 @@ module.exports = function (node) {
         }
     };
     const onInput = function (msg) {
-        if (msg.hasOwnProperty('payload')) {
+        var _a, _b;
+        if (msg.payload) {
             const type = typeof msg.payload;
             if (type !== 'object') {
                 log.error(`Invalid payload type: ${type}`);
@@ -131,17 +132,17 @@ module.exports = function (node) {
             log.error('Invalid message (payload missing)');
             return;
         }
-        const topic = node.config.topic ? node.config.topic : node.name;
+        const topic = (_a = node.config.topic) !== null && _a !== void 0 ? _a : node.name;
         if (node.config.filter && msg.topic !== topic) {
             log.debug("msg.topic doesn't match configured value and filter is enabled. Dropping message.");
             return;
         }
         let context = null;
-        if (msg.payload.hasOwnProperty('Context')) {
+        if (msg.payload.Context) {
             context = msg.payload.Context;
             delete msg.payload.Context;
         }
-        node.topic_in = msg.topic ? msg.topic : '';
+        node.topic_in = (_b = msg.topic) !== null && _b !== void 0 ? _b : '';
         Object.keys(msg.payload).map((key) => {
             var _a, _b;
             if (node.supported.indexOf(key) < 0) {
@@ -153,7 +154,7 @@ module.exports = function (node) {
                 parentNode.reachable = value !== NO_RESPONSE_MSG;
                 const characteristic = node.service.getCharacteristic(Characteristic[key]);
                 if (context !== null) {
-                    characteristic.setValue(value, context);
+                    characteristic.setValue(value, undefined, context);
                 }
                 else {
                     characteristic.setValue(value);
@@ -184,7 +185,6 @@ module.exports = function (node) {
         done();
     };
     const getOrCreate = function (accessory, serviceInformation, parentService) {
-        var _a;
         const newService = new Service[serviceInformation.serviceName](serviceInformation.name, serviceInformation.UUID);
         log.debug(`Looking for service with UUID ${serviceInformation.UUID} ...`);
         let service = accessory.services.find((service) => {
@@ -203,21 +203,6 @@ module.exports = function (node) {
             }
             else {
                 service = accessory.addService(newService);
-                if (serviceInformation.serviceName === 'Lightbulb' &&
-                    serviceInformation.config.adaptiveLightingOptionsEnable) {
-                    try {
-                        const adaptiveLightingController = new hap_nodejs_1.AdaptiveLightingController(service, {
-                            controllerMode: (_a = serviceInformation.config
-                                .adaptiveLightingOptionsMode) !== null && _a !== void 0 ? _a : 1,
-                            customTemperatureAdjustment: serviceInformation.config
-                                .adaptiveLightingOptionsCustomTemperatureAdjustment,
-                        });
-                        accessory.configureController(adaptiveLightingController);
-                    }
-                    catch (error) {
-                        log.error(`Failed to configure adaptive lightning due to ${error}`);
-                    }
-                }
             }
         }
         else {
@@ -290,6 +275,23 @@ module.exports = function (node) {
             log.error('Invalid message (required {"payload":{"nrchkb":{"setup":{}}}})');
         }
     };
+    const configureAdaptiveLightning = () => {
+        var _a;
+        if (node.service.name === 'Lightbulb' &&
+            node.config.adaptiveLightingOptionsEnable) {
+            try {
+                const adaptiveLightingController = new hap_nodejs_1.AdaptiveLightingController(node.service, {
+                    controllerMode: (_a = node.config.adaptiveLightingOptionsMode) !== null && _a !== void 0 ? _a : 1,
+                    customTemperatureAdjustment: node.config
+                        .adaptiveLightingOptionsCustomTemperatureAdjustment,
+                });
+                node.accessory.configureController(adaptiveLightingController);
+            }
+            catch (error) {
+                log.error(`Failed to configure adaptive lightning due to ${error}`);
+            }
+        }
+    };
     return {
         getOrCreate,
         onCharacteristicGet,
@@ -299,5 +301,6 @@ module.exports = function (node) {
         onClose,
         waitForParent,
         handleWaitForSetup,
+        configureAdaptiveLightning,
     };
 };

package/build/lib/utils/ServiceUtils2.js

@@ -9,6 +9,7 @@ const NRCHKBError_1 = __importDefault(require("../NRCHKBError"));
 const Storage_1 = require("../Storage");
 module.exports = function (node) {
     const log = (0, logger_1.logger)('NRCHKB', 'ServiceUtils2', node.config.name, node);
+    const ServiceUtilsLegacy = require('./ServiceUtils')(node);
     const HapNodeJS = require('hap-nodejs');
     const Service = HapNodeJS.Service;
     const Characteristic = HapNodeJS.Characteristic;
@@ -127,7 +128,7 @@ module.exports = function (node) {
         }
     };
     const onInput = function (msg) {
-        var _a;
+        var _a, _b;
         if (msg.payload) {
             const type = typeof msg.payload;
             if (type !== 'object') {
@@ -139,7 +140,7 @@ module.exports = function (node) {
             log.error('Invalid message (payload missing)');
             return;
         }
-        const topic = node.config.topic ? node.config.topic : node.name;
+        const topic = (_a = node.config.topic) !== null && _a !== void 0 ? _a : node.name;
         if (node.config.filter && msg.topic !== topic) {
             log.debug("msg.topic doesn't match configured value and filter is enabled. Dropping message.");
             return;
@@ -149,7 +150,7 @@ module.exports = function (node) {
             context = msg.payload.Context;
             delete msg.payload.Context;
         }
-        node.topic_in = (_a = msg.topic) !== null && _a !== void 0 ? _a : '';
+        node.topic_in = (_b = msg.topic) !== null && _b !== void 0 ? _b : '';
         Object.keys(msg.payload).map((key) => {
             var _a, _b, _c;
             if (node.supported.indexOf(key) < 0) {
@@ -176,7 +177,7 @@ module.exports = function (node) {
                 parentNode.reachable = value !== NO_RESPONSE_MSG;
                 const characteristic = node.service.getCharacteristic(Characteristic[key]);
                 if (context !== null) {
-                    characteristic.setValue(value, context);
+                    characteristic.setValue(value, undefined, context);
                 }
                 else {
                     characteristic.setValue(value);
@@ -207,7 +208,6 @@ module.exports = function (node) {
         done();
     };
     const getOrCreate = function (accessory, serviceInformation, parentService) {
-        var _a;
         const newService = new Service[serviceInformation.serviceName](serviceInformation.name, serviceInformation.UUID);
         log.debug(`Looking for service with UUID ${serviceInformation.UUID} ...`);
         let service = accessory.services.find((service) => {
@@ -226,21 +226,6 @@ module.exports = function (node) {
             }
             else {
                 service = accessory.addService(newService);
-                if (serviceInformation.serviceName === 'Lightbulb' &&
-                    serviceInformation.config.adaptiveLightingOptionsEnable) {
-                    try {
-                        const adaptiveLightingController = new hap_nodejs_1.AdaptiveLightingController(service, {
-                            controllerMode: (_a = serviceInformation.config
-                                .adaptiveLightingOptionsMode) !== null && _a !== void 0 ? _a : 1,
-                            customTemperatureAdjustment: serviceInformation.config
-                                .adaptiveLightingOptionsCustomTemperatureAdjustment,
-                        });
-                        accessory.configureController(adaptiveLightingController);
-                    }
-                    catch (error) {
-                        log.error(`Failed to configure adaptive lightning due to ${error}`);
-                    }
-                }
             }
         }
         else {
@@ -322,5 +307,6 @@ module.exports = function (node) {
         onClose,
         waitForParent,
         handleWaitForSetup,
+        configureAdaptiveLightning: ServiceUtilsLegacy.configureAdaptiveLightning,
     };
 };

package/build/nodes/service.html

@@ -204,7 +204,7 @@
             <label for="node-input-adaptiveLightingOptionsMode"><i class="fa fa-hand-o-up"></i> Mode</label>
             <select id="node-input-adaptiveLightingOptionsMode">
                 <option value="0" selected="selected">AUTOMATIC</option>
-                <option value="1">MANUAL</option>
+                <option value="1" disabled>MANUAL</option>
             </select>
         </div>
         <div class="form-row">

package/build/nodes/service2.html

@@ -204,7 +204,7 @@
             <label for="node-input-adaptiveLightingOptionsMode"><i class="fa fa-hand-o-up"></i> Mode</label>
             <select id="node-input-adaptiveLightingOptionsMode">
                 <option value="0" selected="selected">AUTOMATIC</option>
-                <option value="1">MANUAL</option>
+                <option value="1" disabled>MANUAL</option>
             </select>
         </div>
         <div class="form-row">

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "node-red-contrib-homekit-bridged",
-    "version": "1.7.0-dev.1",
+    "version": "1.7.0-dev.2",
     "description": "Node-RED nodes to simulate Apple HomeKit devices.",
     "main": "build/nodes/nrchkb.js",
     "scripts": {
@@ -50,7 +50,7 @@
     },
     "devDependencies": {
         "@homebridge/ciao": "^1.2.0",
-        "@node-red/registry": "^3.1.10",
+        "@node-red/registry": "^3.1.11",
         "@types/mocha": "^10.0.6",
         "@types/node": "^18",
         "@types/node-persist": "^3.1.8",
@@ -58,8 +58,8 @@
         "@types/node-red-node-test-helper": "^0.3.4",
         "@types/semver": "^7.5.8",
         "@types/uuid": "^9.0.8",
-        "@typescript-eslint/eslint-plugin": "^7.13.0",
-        "@typescript-eslint/parser": "^7.13.0",
+        "@typescript-eslint/eslint-plugin": "^7.13.1",
+        "@typescript-eslint/parser": "^7.13.1",
         "babel-eslint": "^10.1.0",
         "del-cli": "^5.1.0",
         "eslint": "^8",
@@ -68,7 +68,7 @@
         "eslint-plugin-simple-import-sort": "^12.1.0",
         "husky": "^9.0.11",
         "mocha": "^10.4.0",
-        "node-red": "^3.1.10",
+        "node-red": "^3.1.11",
         "node-red-node-test-helper": "^0.3.4",
         "prettier": "^3.3.2",
         "ts-node": "^10.9.2",