fix: stop_execution and send_coordinate wait for idle before returning

- stop_actions() now waits for hardware to reach idle state (30s timeout)
- send_coordinate endpoint waits for idle after move (60s timeout)
- Clear stop_requested flag before idle check to allow check_idle_async to work

This ensures API responses only return after the machine has physically
stopped moving, consistent with move_to_center and move_to_perimeter.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

.gitignore

@@ -30,4 +30,5 @@ static/custom/*
 !static/custom/.gitkeep
 .claude/
 static/dist/
-.planning
+.planning
+.coverage

main.py

@@ -2150,6 +2150,12 @@ async def send_coordinate(request: CoordinateRequest):
     try:
         logger.debug(f"Sending coordinate: theta={request.theta}, rho={request.rho}")
         await pattern_manager.move_polar(request.theta, request.rho)
+
+        # Wait for machine to reach idle before returning
+        idle = await connection_manager.check_idle_async(timeout=60)
+        if not idle:
+            logger.warning("Machine did not reach idle after send_coordinate")
+
         return {"success": True}
     except Exception as e:
         logger.error(f"Failed to send coordinate: {str(e)}")

modules/core/pattern_manager.py

@@ -1647,6 +1647,17 @@ async def stop_actions(clear_playlist = True, wait_for_lock = True):
             state.current_playing_file = None
             state.execution_progress = None
 
+        # Clear stop_requested now that the pattern has stopped - this allows
+        # check_idle_async to work (it exits early if stop_requested is True)
+        state.stop_requested = False
+
+        # Wait for hardware to reach idle state before returning
+        # This ensures the machine has physically stopped moving
+        if not timed_out:
+            idle = await connection_manager.check_idle_async(timeout=30.0)
+            if not idle:
+                logger.warning("Machine did not reach idle after stop")
+
         # Call async function directly since we're in async context
         await connection_manager.update_machine_position()
         return not timed_out