From e65a6b523a80e47541cdf7ffbc85ebc84c8c611f Mon Sep 17 00:00:00 2001
From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Date: Wed, 12 Feb 2025 20:00:23 +0000
Subject: [PATCH] refactor: Improve feedback handling code quality

- Add type hints and docstrings
- Enhance error handling in feedback processing
- Standardize message formatting

Co-Authored-By: Joe Moura <joao@crewai.com>
---
 src/crewai/agents/crew_agent_executor.py | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/crewai/agents/crew_agent_executor.py b/src/crewai/agents/crew_agent_executor.py
index b4a246a63..845b77102 100644
--- a/src/crewai/agents/crew_agent_executor.py
+++ b/src/crewai/agents/crew_agent_executor.py
@@ -555,6 +555,9 @@ class CrewAgentExecutor(CrewAgentExecutorMixin):
 
         Returns:
             AgentFinish: The final answer after processing all feedback iterations
+            
+        Raises:
+            FeedbackProcessingError: If feedback processing fails
         """
         try:
             feedback = initial_feedback
@@ -576,11 +579,12 @@ class CrewAgentExecutor(CrewAgentExecutorMixin):
 
             return answer
         except Exception as e:
+            error_msg = f"Failed to process feedback: {str(e)}"
             self._printer.print(
-                content=f"Error processing feedback: {str(e)}",
+                content=error_msg,
                 color="red"
             )
-            raise
+            raise FeedbackProcessingError(error_msg, original_error=e)
 
     def _get_llm_feedback_response(self, feedback: str) -> Optional[str]:
         """Get LLM classification of whether feedback requires changes."""