From 9eb283420aaf3a9fab80947fadd94e208acd030d Mon Sep 17 00:00:00 2001 From: Thaddeus Hughes Date: Thu, 12 Mar 2026 21:38:16 -0500 Subject: [PATCH] Update TODO.md --- TODO.md | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/TODO.md b/TODO.md index d05495e..3846495 100644 --- a/TODO.md +++ b/TODO.md @@ -54,10 +54,12 @@ - [clauded] Add terse comments to FSM state transitions in `control_fsm.c` (focus on "why", not "what") +20. - [clauded] Fix compile warnings — unused vars (uart_comms.c, rf_433.c), const-correctness (log_write signatures), fallthrough annotation (control_fsm.c) +21. - [clauded] NVS is required: WiFi blob stores RF cal data (CONFIG_ESP_WIFI_NVS_ENABLED), Bluedroid stores bonding/GATT cache unconditionally, bt_hid.c stores last-connected BDA. Cannot remove nvs_flash_init(). +22. - [clauded] NVS vs custom params: NVS serves WiFi/BT internals + BDA storage; custom flash partition serves app params with CRC32 protection. Different purposes, no consolidation needed. +23. - [clauded] BUG FIX: `FSM_CMD_START` fallthrough was overwriting `this_move_dist = MIN(...)` with unconditional `DRIVE_DIST` — replaced fallthrough with goto to shared start logic so leash limit is preserved -20. - [ ] Extract pure logic (e-fuse thermal model, param serialization, sensor debounce) into host-testable modules with Unity/CMock -21. - [ ] UART integration test framework: Python runner + ESP-side test commands -22. - [clauded] Fix compile warnings — unused vars (uart_comms.c, rf_433.c), const-correctness (log_write signatures), fallthrough annotation (control_fsm.c) -25. - [clauded] BUG FIX: `FSM_CMD_START` fallthrough was overwriting `this_move_dist = MIN(...)` with unconditional `DRIVE_DIST` — replaced fallthrough with goto to shared start logic so leash limit is preserved -23. - [clauded] NVS is required: WiFi blob stores RF cal data (CONFIG_ESP_WIFI_NVS_ENABLED), Bluedroid stores bonding/GATT cache unconditionally, bt_hid.c stores last-connected BDA. Cannot remove nvs_flash_init(). -24. - [clauded] NVS vs custom params: NVS serves WiFi/BT internals + BDA storage; custom flash partition serves app params with CRC32 protection. Different purposes, no consolidation needed. \ No newline at end of file +24. - [ ] Do general bug-scan with claude. Especially think through the FSM logic. +25. - [ ] Extract pure logic (e-fuse thermal model, param serialization, sensor debounce) into host-testable modules with Unity/CMock? +26. - [ ] UART integration test framework: Python runner + ESP-side test commands +27. - [ ] Bug: WiFi won't want to connect to STA except at first boot \ No newline at end of file