incyclist-devices 3.0.33 → 3.0.34

package/lib/cjs/ble/characteristics/csc/measurement.js

@@ -111,6 +111,8 @@ class CscMeasurement {
         this.data = {};
         this.prevCrankData = undefined;
         this.currentCrankData = undefined;
+        this.prevWheelData = undefined;
+        this.currentWheelData = undefined;
         this.timeOffset = 0;
     }
 }

package/lib/cjs/ble/csc/sensor.js

@@ -58,6 +58,11 @@ class BleCyclingSpeedCadenceDevice extends sensor_js_1.TBleSensor {
     }
     reset() {
         this.data = {};
+        this.measurement?.reset();
+    }
+    onDisconnect() {
+        super.onDisconnect();
+        this.measurement.reset();
     }
     async setWheelCircumference(wheelCircumference) {
         this.measurement.setWheelCircumference(wheelCircumference);

package/lib/cjs/ble/fm/sensor.js

@@ -25,6 +25,7 @@ class BleFitnessMachineDevice extends sensor_js_1.TBleSensor {
     ftmsServiceDataAttempts = 0;
     rowerDataTS;
     rowerMaxPower;
+    indoorBikeDataTruncationLogged = false;
     constructor(peripheral, props) {
         super(peripheral, props);
         this.reset();
@@ -176,6 +177,8 @@ class BleFitnessMachineDevice extends sensor_js_1.TBleSensor {
     }
     parseHrm(_data) {
         const data = Buffer.from(_data);
+        if (data.length == 0)
+            return { ...this.data, raw: '<empty>' };
         try {
             const flags = data.readUInt8(0);
             if (flags % 1 === 0) {
@@ -186,7 +189,7 @@ class BleFitnessMachineDevice extends sensor_js_1.TBleSensor {
             }
         }
         catch (err) {
-            this.logEvent({ message: 'error', fn: 'parseHrm()', error: err.message | err, stack: err.stack });
+            this.logEvent({ message: 'error', fn: 'parseHrm()', error: err.message, stack: err.stack, raw: `2a37:${data.toString('hex')}` });
         }
         return { ...this.data, raw: `2a37:${data.toString('hex')}` };
     }
@@ -343,48 +346,61 @@ class BleFitnessMachineDevice extends sensor_js_1.TBleSensor {
         }
         return { ...this.data, raw: `2ad1:${data.toString('hex')}` };
     }
+    logIndoorBikeDataTruncated(data, flags, offset, needed) {
+        if (this.indoorBikeDataTruncationLogged)
+            return;
+        this.indoorBikeDataTruncationLogged = true;
+        this.logEvent({ message: 'error', fn: 'parseIndoorBikeData()', device: this.getName(), data: data.toString('hex'), flags, offset, needed, error: 'Indoor Bike Data flags declare more fields than the packet contains' });
+    }
     parseIndoorBikeData(_data) {
         const data = Buffer.from(_data);
         let offset = 2;
         if (data.length > 2) {
             try {
                 const flags = data.readUInt16LE(0);
-                if ((flags & consts_js_2.IndoorBikeDataFlag.MoreData) === 0) {
+                const need = (size) => {
+                    if (offset + size > data.length) {
+                        this.logIndoorBikeDataTruncated(data, flags, offset, size);
+                        return false;
+                    }
+                    return true;
+                };
+                if ((flags & consts_js_2.IndoorBikeDataFlag.MoreData) === 0 && need(2)) {
                     this.data.speed = data.readUInt16LE(offset) / 100;
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.AverageSpeedPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.AverageSpeedPresent) && need(2)) {
                     this.data.averageSpeed = data.readUInt16LE(offset) / 100;
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.InstantaneousCadence) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.InstantaneousCadence) && need(2)) {
                     this.data.cadence = data.readUInt16LE(offset) / 2;
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.AverageCadencePresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.AverageCadencePresent) && need(2)) {
                     this.data.averageCadence = data.readUInt16LE(offset) / 2;
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.TotalDistancePresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.TotalDistancePresent) && need(3)) {
                     const dvLow = data.readUInt8(offset);
                     offset += 1;
                     const dvHigh = data.readUInt16LE(offset);
                     offset += 2;
                     this.data.totalDistance = (dvHigh << 8) + dvLow;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.ResistanceLevelPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.ResistanceLevelPresent) && need(2)) {
                     this.data.resistanceLevel = data.readInt16LE(offset);
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.InstantaneousPowerPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.InstantaneousPowerPresent) && need(2)) {
                     this.data.instantaneousPower = data.readInt16LE(offset);
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.AveragePowerPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.AveragePowerPresent) && need(2)) {
                     this.data.averagePower = data.readInt16LE(offset);
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.ExpendedEnergyPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.ExpendedEnergyPresent) && need(5)) {
                     this.data.totalEnergy = data.readUInt16LE(offset);
                     offset += 2;
                     this.data.energyPerHour = data.readUInt16LE(offset);
@@ -392,19 +408,19 @@ class BleFitnessMachineDevice extends sensor_js_1.TBleSensor {
                     this.data.energyPerMinute = data.readUInt8(offset);
                     offset += 1;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.HeartRatePresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.HeartRatePresent) && need(1)) {
                     this.data.heartrate = data.readUInt8(offset);
                     offset += 1;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.MetabolicEquivalentPresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.MetabolicEquivalentPresent) && need(1)) {
                     this.data.metabolicEquivalent = data.readUInt8(offset) / 10;
                     offset += 1;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.ElapsedTimePresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.ElapsedTimePresent) && need(2)) {
                     this.data.time = data.readUInt16LE(offset);
                     offset += 2;
                 }
-                if (flags & consts_js_2.IndoorBikeDataFlag.RemainingTimePresent) {
+                if ((flags & consts_js_2.IndoorBikeDataFlag.RemainingTimePresent) && need(2)) {
                     this.data.remainingTime = data.readUInt16LE(offset);
                 }
             }