fix: Update vehicle awake state logic to treat as awake unless explicitly ASLEEP (#48)

Author: yoziru

include/vehicle.h

@@ -152,7 +152,7 @@ class Vehicle {
     bool is_connected_ = false;
     bool is_vcsec_authenticated_ = false;
     bool is_infotainment_authenticated_ = false;
-    bool is_vehicle_awake_ = false;  // Track vehicle sleep state from VCSEC status
+    bool is_vehicle_awake_ = false;   // From VCSEC sleep status (inverted: true unless explicitly ASLEEP)
 
 
     // Constants

src/vehicle.cpp

@@ -59,6 +59,10 @@ void Vehicle::set_connected(bool connected) {
         is_vcsec_authenticated_ = false;
         is_infotainment_authenticated_ = false;
 
+        // Reset vehicle awake state - we don't know the state after disconnect
+        // This prevents stale awake state from causing incorrect command handling
+        is_vehicle_awake_ = false;
+        
         // Clear command queue to prevent stale commands from blocking
         // New commands will be enqueued after reconnection
         while (!command_queue_.empty()) {
@@ -239,6 +243,7 @@ void Vehicle::initiate_vcsec_auth(std::shared_ptr<Command> command) {
 void Vehicle::initiate_infotainment_auth(std::shared_ptr<Command> command) {
     // FIRST: If vehicle is asleep and command doesn't require wake, skip immediately
     // This avoids unnecessary auth attempts for optional polls when vehicle is sleeping
+    // Note: is_vehicle_awake_ is true unless VCSEC explicitly reports ASLEEP
     if (!is_vehicle_awake_ && !command->requires_wake) {
         LOG_DEBUG("Vehicle is asleep and command doesn't require wake, skipping: %s", command->name.c_str());
         mark_command_completed(command);  // Mark as completed (no-op when asleep)
@@ -617,7 +622,8 @@ void Vehicle::handle_vcsec_message(const UniversalMessage_RoutableMessage& msg)
             log_vehicle_status(TESLA_LOG_TAG, &vcsec_msg.sub_message.vehicleStatus);
 
             // Track vehicle sleep state for command state machine
-            is_vehicle_awake_ = (vcsec_msg.sub_message.vehicleStatus.vehicleSleepStatus == VCSEC_VehicleSleepStatus_E_VEHICLE_SLEEP_STATUS_AWAKE);
+            // Assume awake unless explicitly asleep (charging vehicles may report UNKNOWN)
+            is_vehicle_awake_ = (vcsec_msg.sub_message.vehicleStatus.vehicleSleepStatus != VCSEC_VehicleSleepStatus_E_VEHICLE_SLEEP_STATUS_ASLEEP);
 
             if (vehicle_status_callback_) vehicle_status_callback_(vcsec_msg.sub_message.vehicleStatus);
 

tests/test_vehicle.cpp

@@ -89,6 +89,42 @@ TEST_F(VehicleTest, InfotainmentPollWithForceWakeSendsData) {
     EXPECT_GE(writes.size(), 1) << "Infotainment poll with force_wake should send data";
 }
 
+// ============================================================================
+// Vehicle Sleep State Tests - Regression test for charging vehicle polling bug
+// Issue: Infotainment polls were skipped when vehicle was charging because
+// VCSEC reports UNKNOWN status (not AWAKE) for charging vehicles.
+// Fix: Inverted logic to treat vehicle as awake unless explicitly ASLEEP.
+// ============================================================================
+
+TEST_F(VehicleTest, InfotainmentPollSkippedWhenAsleepByDefault) {
+    // Verify default behavior: vehicle starts in asleep state (no VCSEC status received)
+    // and infotainment polls without force_wake should be skipped
+    
+    bool poll_callback_called = false;
+    bool poll_success = false;
+    
+    vehicle_->send_command(
+        UniversalMessage_Domain_DOMAIN_INFOTAINMENT,
+        "Optional Poll",
+        [](Client* client, uint8_t* buff, size_t* len) {
+            return client->buildCarServerGetVehicleDataMessage(buff, len, CarServer_GetVehicleData_getChargeState_tag);
+        },
+        [&](bool success) {
+            poll_callback_called = true;
+            poll_success = success;
+        },
+        false  // requires_wake = false (optional poll)
+    );
+    vehicle_->loop();
+    
+    // Poll should be skipped (no BLE writes) but callback invoked with success
+    EXPECT_TRUE(poll_callback_called) << "Callback should be invoked for skipped poll";
+    EXPECT_TRUE(poll_success) << "Skipped poll should report success (no-op)";
+    
+    auto writes = mock_ble_->get_written_data();
+    EXPECT_EQ(writes.size(), 0) << "Poll should be skipped when vehicle is asleep";
+}
+
 TEST_F(VehicleTest, SetChargingAmpsSendsData) {
     vehicle_->set_charging_amps(16);
     vehicle_->loop();