From 836680bc110bb3ed82b0b403dba9d261c51e698d Mon Sep 17 00:00:00 2001 From: theo-michel Date: Sat, 8 Aug 2026 07:28:47 +0000 Subject: [PATCH] fix(skills): restore _send_feedback alias for pre-#542 workspace skills #542 renamed Skill._send_feedback to feedback(); existing workspace skills still call the old name and crash with AttributeError. Keep a thin deprecated alias so user code keeps working. --- ros2_ws/src/brain/brain_client/brain_client/skills/types.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/ros2_ws/src/brain/brain_client/brain_client/skills/types.py b/ros2_ws/src/brain/brain_client/brain_client/skills/types.py index 90eb8d644..948596481 100644 --- a/ros2_ws/src/brain/brain_client/brain_client/skills/types.py +++ b/ros2_ws/src/brain/brain_client/brain_client/skills/types.py @@ -1103,6 +1103,10 @@ def feedback(self, message: str, image_b64: str | None = None) -> None: except Exception as e: self.logger.error(f"Error sending feedback for skill {self.name}: {e}") + def _send_feedback(self, message: str, image_b64: str | None = None) -> None: + """Deprecated pre-#542 name for feedback(), kept for existing workspace skills.""" + self.feedback(message, image_b64) + def set_feedback_callback(self, callback: Callable[[str, str | None], None]) -> None: self._feedback_callback = callback self.logger.debug(f"Feedback callback set for skill {self.name}.")