Rename battery_failure_level to battery_fail_level

Refactored all occurrences of the battery telemetry field from battery_failure_level to battery_fail_level in structs, BMS logic, BLE service, and debug code for consistency. No functional changes were made.

Author: zjwhitehead

inc/sp140/structs.h

@@ -100,7 +100,7 @@ typedef struct {
   float lowest_temperature;     // Lowest temperature reading (°C)
   float energy_cycle;          // Energy per cycle (kWh)
   uint32_t battery_cycle;      // Battery cycle count
-  uint8_t battery_failure_level;  // Battery failure status
+  uint8_t battery_fail_level;  // Battery failure status
   float voltage_differential;   // Highest cell minus lowest cell voltage (V)
   unsigned long lastUpdateMs;   // Timestamp of last telemetry update
   bool is_charging;
@@ -149,7 +149,7 @@ typedef struct {
   float highest_temperature;    // Highest temperature (°C)
   float lowest_temperature;     // Lowest temperature (°C)
   float voltage_differential;   // Cell voltage differential (V)
-  uint8_t battery_failure_level;// Battery failure status
+  uint8_t battery_fail_level;   // Battery failure status
   uint8_t is_charge_mos;        // Charge MOSFET state (0/1)
   uint8_t is_discharge_mos;     // Discharge MOSFET state (0/1)
   uint8_t is_charging;          // Charging state (0/1)

src/sp140/ble/bms_service.cpp

@@ -140,7 +140,7 @@ void updateBMSPackedTelemetry(const STR_BMS_TELEMETRY_140& telemetry, uint8_t bm
   packet.highest_temperature = telemetry.highest_temperature;
   packet.lowest_temperature = telemetry.lowest_temperature;
   packet.voltage_differential = telemetry.voltage_differential;
-  packet.battery_failure_level = telemetry.battery_failure_level;
+  packet.battery_fail_level = telemetry.battery_fail_level;
   packet.is_charge_mos = telemetry.is_charge_mos ? 1 : 0;
   packet.is_discharge_mos = telemetry.is_discharge_mos ? 1 : 0;
   packet.is_charging = telemetry.is_charging ? 1 : 0;
@@ -219,7 +219,7 @@ void updateBMSTelemetry(const STR_BMS_TELEMETRY_140& telemetry) {
   }
 
   // Handle state-change characteristics - only notify on change
-  setAndNotifyOnChange(pBMSFailureLevel, telemetry.battery_failure_level, lastFailureLevel);
+  setAndNotifyOnChange(pBMSFailureLevel, telemetry.battery_fail_level, lastFailureLevel);
   setAndNotifyOnChange(pBMSChargeMos, static_cast<uint8_t>(telemetry.is_charge_mos ? 1 : 0), lastChargeMos);
   setAndNotifyOnChange(pBMSDischargeMos, static_cast<uint8_t>(telemetry.is_discharge_mos ? 1 : 0), lastDischargeMos);
 

src/sp140/bms.cpp

@@ -58,7 +58,7 @@ void updateBMSData() {
   // Battery statistics
   bmsTelemetryData.battery_cycle = bms_can->getBatteryCycle();
   bmsTelemetryData.energy_cycle = bms_can->getEnergyCycle();
-  bmsTelemetryData.battery_failure_level = bms_can->getBatteryFailureLevel();
+  bmsTelemetryData.battery_fail_level = bms_can->getBatteryFailureLevel();
 
   // Battery status
   bmsTelemetryData.is_charging = bms_can->isBatteryCharging();

src/sp140/debug.cpp

@@ -64,7 +64,7 @@ void generateFakeTelemetry(STR_ESC_TELEMETRY_140& escTelemetry,
     bmsTelemetry.lowest_temperature = escTelemetry.mcu_temp;
     bmsTelemetry.energy_cycle = 3.5;
     bmsTelemetry.battery_cycle = 42;
-    bmsTelemetry.battery_failure_level = 0;
+    bmsTelemetry.battery_fail_level = 0;
     bmsTelemetry.voltage_differential = 0.2;
     bmsTelemetry.lastUpdateMs = now;
 

src/sp140/system_monitors.cpp

@@ -13,7 +13,7 @@ extern bool bmpPresent;
 // Cached CPU temperature to avoid "tsens: Do not configure the temp sensor when it's running!" error
 static float cachedCpuTemp = 0.0f;
 static unsigned long lastCpuTempRead = 0;
-static const unsigned long CPU_TEMP_READ_INTERVAL = 1000; // Read every 1 second
+static const unsigned long CPU_TEMP_READ_INTERVAL = 1000;  // Read every 1 second
 
 float getCachedCpuTemperature() {
   unsigned long now = millis();