fix: manually add missing values on updates (#1056)

* fix: manually add missing values on updates

* fix: dedupe updates

lib/ZwaveClient.js

@@ -1027,7 +1027,7 @@ function updateValueMetadata (zwaveNode, zwaveValue, zwaveValueMeta) {
     description: zwaveValueMeta.description,
     label: zwaveValueMeta.label || zwaveValue.propertyName + ' (property)', // when label is missing, re use propertyName. Usefull for webinterface
     default: zwaveValueMeta.default,
-    stateless: false // used for notifications to specify that this should not be persisted (retained)
+    stateless: zwaveValue.stateless || false // used for notifications to specify that this should not be persisted (retained)
   }
 
   if (zwaveValueMeta.ccSpecific) {
@@ -1150,6 +1150,15 @@ function updateValue (zwaveNode, args) {
   if (!node) {
     logger.info(`valueChanged: no such node: ${zwaveNode.id} error`)
   } else {
+    let skipUpdate = false
+
+    // notifications events are not defined as values, manually create them once we get the first update
+    if (!node.values[getValueID(args)]) {
+      addValue.call(this, zwaveNode, args)
+      // addValue call already trigger valueChanged event
+      skipUpdate = true
+    }
+
     const valueId = node.values[getValueID(args)]
 
     if (valueId) {
@@ -1173,10 +1182,9 @@ function updateValue (zwaveNode, args) {
         valueId.value = new Duration(undefined, 'seconds')
       }
 
-      this.emit('valueChanged', valueId, node, prevValue !== newValue)
-    } else {
-      // notifications don't hava a defined valueId, manually create a value
-      addValue.call(this, zwaveNode, args)
+      if (!skipUpdate) {
+        this.emit('valueChanged', valueId, node, prevValue !== newValue)
+      }
     }
 
     // if valueId is stateless, automatically reset the value after 1 sec