Fix: Preserve direction-change commands when stopping motors

Author: protocol113

src/components/EscDshotDirection/EscDshotDirectionComponent.js

@@ -419,7 +419,8 @@ class EscDshotDirectionComponent {
             return;
         }
 
-        // SPACEBAR RELEASE: Stop motors immediately
+        // SPACEBAR RELEASE: Stop motors (preserves pending direction changes)
+        // Motors stop immediately unless direction change is in progress (~100ms delay)
         if (event.code === "Space") {
             event.preventDefault();
             event.stopPropagation();
@@ -475,11 +476,11 @@ class EscDshotDirectionComponent {
             this._deactivateWizardMotorButtons();
         }
 
-        // Stop motors (this adds stop commands to the queue)
+        // Stop motors - adds stop commands to end of queue (preserves pending direction changes)
         this._motorDriver.stopAllMotorsNow();
 
-        // Deactivate motor driver - this tells queue to stop AFTER processing current commands
-        // This is critical - it allows direction change + save commands to complete
+        // Deactivate motor driver - prevents new commands and stops queue when empty
+        // Queue will drain naturally, completing any pending direction change + save commands
         this._motorDriver.deactivate();
 
         // Clear wizard flag after motor driver deactivation

src/components/EscDshotDirection/EscDshotDirectionMotorDriver.js

@@ -46,7 +46,17 @@ class EscDshotDirectionMotorDriver {
     }
 
     stopAllMotorsNow() {
-        this._EscDshotCommandQueue.clear();
+        // Adds stop command to END of queue without clearing pending commands
+        // This preserves critical direction-change + save commands that must complete
+        //
+        // Expected behavior:
+        // - Normal case: Motors stop immediately (no pending commands)
+        // - Edge case: If direction change is in progress when called, queue contains:
+        //   [stop, pause, direction+save, resume, STOP] - motor may briefly spin for
+        //   ~100ms (one queue interval) between resume and final stop
+        //
+        // This trade-off ensures direction changes are never lost while maintaining
+        // prompt motor stopping for safety
         this._spinAllMotors(this._motorStopValue);
     }