node-red-contrib-dmx-for-ha 0.3.9 → 0.4.1

package/nodes/ha-mqtt-dmx-group.js

@@ -236,6 +236,15 @@ module.exports = function (RED) {
             });
         }
 
+
+        function groupStatus(state, label) {
+            if (state === 'ON') {
+                setStatus('green', 'dot', label || `${groupId} ON`);
+            } else {
+                setStatus('grey', 'ring', label || `${groupId} OFF`);
+            }
+        }
+
         // ── Device add ────────────────────────────────────────────
         function handleDeviceAdd(incomingTrace) {
             if (!ctxGet('state') && !ctxGet('state', 'disk_values')) {
@@ -284,7 +293,7 @@ module.exports = function (RED) {
                 pubState({ state: payload.state, color_mode: S.colorMode, brightness: payload.brightness });
                 forwardToChildren(payload, null);
                 const _lbl = payload.effect ? `effect:${payload.effect}` : `${payload.state}`;
-                setStatus('green', 'dot', `${groupId} → ${_lbl}`);
+                groupStatus(payload.state, `${groupId} → ${_lbl}`);
             }, node.id);
 
             setStatus('green', 'ring', `${groupId} discovery sent`);
@@ -297,8 +306,10 @@ module.exports = function (RED) {
                 const brightness = recall('brightness', 'brightness_disk', 255);
                 const color      = recall('color',      'color_disk',      null);
                 pubState({ state, color_mode: S.colorMode, brightness, color });
-                forwardToChildren({ state, brightness, color }, null);
-                setStatus('yellow', 'ring', `${groupId} ready`);
+                // NOTE: do NOT forward to children on recovery — each child node
+                // recovers its own state independently from its own disk store.
+                // Forwarding here would overwrite children with the group's state.
+                groupStatus(state, `${groupId} → ${state}`);
                 node.log(`${groupId} recovery — state:${state}`);
             }, 2000);
         }
@@ -306,6 +317,7 @@ module.exports = function (RED) {
         // ── Device remove ─────────────────────────────────────────
         function handleDeviceRemove(incomingTrace) {
             if (diskTimer) { clearTimeout(diskTimer); diskTimer = null; }
+            // No flush needed on remove — state is cleared below
             ctxSet('state', null); ctxSet('state', null, 'disk_values');
             pub(cfgTopic, '', true);
             broker.unsubscribe(cmdTopic, node.id);
@@ -327,7 +339,7 @@ module.exports = function (RED) {
                     saveGroupState(msg.payload);
                     pubState({ state: msg.payload.state, color_mode: S.colorMode, brightness: msg.payload.brightness });
                     forwardToChildren(msg.payload, msg.dmx_trace);
-                    setStatus('green', 'dot', `${groupId} → cascade:${msg.payload.state}`);
+                    groupStatus(msg.payload.state, `${groupId} → cascade:${msg.payload.state}`);
                 }
             } else {
                 const devReq = typeof msg.device === 'string'
@@ -344,7 +356,7 @@ module.exports = function (RED) {
                     saveGroupState(msg.payload);
                     pubState({ state: msg.payload.state, color_mode: S.colorMode, brightness: msg.payload.brightness });
                     forwardToChildren(msg.payload, null);
-                    setStatus('green', 'dot', `${groupId} → ${msg.payload.state}`);
+                    groupStatus(msg.payload.state, `${groupId} → ${msg.payload.state}`);
                 } else {
                     node.warn(`${groupId} — unrecognised message received and dropped. See node documentation.`);
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-red-contrib-dmx-for-ha",
-  "version": "0.3.9",
+  "version": "0.4.1",
   "description": "DMX lighting control for Home Assistant via Node-RED and MQTT. Place a node, fill in the settings, deploy. Full HA device registry integration with RGBW/RGBWW/CCT/brightness colour modes, transitions, effects, and group control.",
   "keywords": [
     "node-red",