fix(discovery): correctly handle up/down commands for covers (#3916)

api/hass/configurations.ts

@@ -145,6 +145,7 @@ const configurations: Record<HassDeviceKey, HassDevice> = {
 			position_closed: 0,
 			payload_open: '99',
 			payload_close: '0',
+			payload_stop: 'stop',
 		},
 	},
 	// Barrier operator support for zwave-js (numeric commands/states)

api/lib/Gateway.ts

@@ -431,6 +431,26 @@ export default class Gateway {
 						// for other command classes use the mode_map
 						payload = hassDevice.mode_map[payload]
 					}
+				} else if (
+					hassDevice.type === 'cover' &&
+					valueId.property === 'targetValue'
+				) {
+					// ref issue https://github.com/zwave-js/zwave-js-ui/issues/3862
+					if (
+						payload ===
+						(hassDevice.discovery_payload.payload_stop ?? 'STOP')
+					) {
+						this._zwave
+							.writeValue(
+								{
+									...valueId,
+									property: 'Up',
+								},
+								false,
+							)
+							.catch(() => {})
+						return null
+					}
 				}
 			}
 
@@ -2271,7 +2291,7 @@ export default class Gateway {
 	}
 
 	/**
-	 * Handle broadcast request reeived from Mqtt client
+	 * Handle broadcast request received from Mqtt client
 	 */
 	private async _onBroadRequest(
 		parts: string[],
@@ -2290,6 +2310,11 @@ export default class Gateway {
 					this.topicValues[values[0]],
 					this.topicValues[values[0]].conf,
 				)
+
+				if (payload === null) {
+					return
+				}
+
 				for (let i = 0; i < values.length; i++) {
 					await this._zwave.writeValue(
 						this.topicValues[values[i]],
@@ -2332,6 +2357,11 @@ export default class Gateway {
 
 		if (valueId) {
 			const value = this.parsePayload(payload, valueId, valueId.conf)
+
+			if (value === null) {
+				return
+			}
+
 			await this._zwave.writeValue(valueId, value, payload?.options)
 		} else {
 			logger.debug(`No writeable valueId found for ${valueTopic}`)