fix(miio): Only report a new token from handshake if it is actually new

lib/core/ValetudoRobotFactory.js

@@ -61,6 +61,7 @@ const Robots = {
     "RoborockS4ValetudoRobot": require("../robots/roborock/RoborockS4ValetudoRobot"),
     "RoborockS6PureValetudoRobot": require("../robots/roborock/RoborockS6PureValetudoRobot"),
     "RoborockS5MaxValetudoRobot": require("../robots/roborock/RoborockS5MaxValetudoRobot"),
+    "RoborockS6MaxVValetudoRobot": require("../robots/roborock/RoborockS6MaxVValetudoRobot"),
     "Dreame1CValetudoRobot": require("../robots/dreame/Dreame1CValetudoRobot"),
     "DreameD9ValetudoRobot": require("../robots/dreame/DreameD9ValetudoRobot"),
     "ViomiV7ValetudoRobot": require("../robots/viomi/ViomiV7ValetudoRobot"),

lib/miio/MiioSocket.js

@@ -65,7 +65,9 @@ class MiioSocket {
             const decodedResponse = this.codec.handleResponse(incomingMsg);
             const token = decodedResponse.token;
             if (token && token.toString("hex") !== "ffffffffffffffffffffffffffffffff" &&
-                token.toString("hex") !== "00000000000000000000000000000000") {
+                token.toString("hex") !== "00000000000000000000000000000000" &&
+                !(this.codec.token.equals(token))
+            ) {
                 Logger.info("Got token from handshake:", decodedResponse.token.toString("hex"));
                 this.token = token;
                 this.codec.setToken(token);
@@ -227,4 +229,4 @@ class MiioSocket {
 /** The default remote port. @const {int} */
 MiioSocket.PORT = 54321;
 
-module.exports = MiioSocket;
+module.exports = MiioSocket;

lib/robots/roborock/RoborockS6MaxVValetudoRobot.js

@@ -0,0 +1,55 @@
+const RoborockValetudoRobot = require("./RoborockValetudoRobot");
+const MiioValetudoRobot = require("../MiioValetudoRobot");
+const capabilities = require("./capabilities");
+const entities = require("../../entities");
+
+/**
+ * Do NOT buy this robot.
+ * Roborock purposefully made this hard to repair which is a consumer-hostile practice
+ * that is not supported by Valetudo
+ *
+ * This class only exists for the unfortunate souls who already own one
+ */
+class RoborockS6MaxVValetudoRobot extends RoborockValetudoRobot {
+    /**
+     *
+     * @param {object} options
+     * @param {import("../../Configuration")} options.config
+     */
+    constructor(options) {
+        super(Object.assign({}, options, {fanSpeeds: FAN_SPEEDS}));
+
+        this.registerCapability(new capabilities.RoborockMapSnapshotCapability({
+            robot: this
+        }));
+        this.registerCapability(new capabilities.RoborockCombinedVirtualRestrictionsCapability({
+            robot: this
+        }));
+        this.registerCapability(new capabilities.RoborockMultiMapPersistentMapControlCapability({
+            robot: this
+        }));
+        this.registerCapability(new capabilities.RoborockMapSegmentationCapability({
+            robot: this
+        }));
+    }
+
+    getModelName() {
+        return "S6 MaxV";
+    }
+
+    static IMPLEMENTATION_AUTO_DETECTION_HANDLER() {
+        const deviceConf = MiioValetudoRobot.READ_DEVICE_CONF(RoborockValetudoRobot.DEVICE_CONF_PATH);
+
+        return !!(deviceConf && (deviceConf.model === "roborock.vacuum.a10" || deviceConf.model === "roborock.vacuum.a09"));
+    }
+}
+
+const FAN_SPEEDS = {
+    [entities.state.attributes.IntensityStateAttribute.VALUE.LOW]: 101,
+    [entities.state.attributes.IntensityStateAttribute.VALUE.MEDIUM]: 102,
+    [entities.state.attributes.IntensityStateAttribute.VALUE.HIGH]: 103,
+    [entities.state.attributes.IntensityStateAttribute.VALUE.MAX]: 104,
+    [entities.state.attributes.IntensityStateAttribute.VALUE.OFF] : 105 //also known as mop mode
+};
+
+module.exports = RoborockS6MaxVValetudoRobot;