Removed the commented code

Author: Dhruvkumar-Microsoft

src/backend/v3/api/router.py

@@ -396,9 +396,6 @@ async def plan_approval(
                 orchestration_config
                 and human_feedback.m_plan_id in orchestration_config.approvals
             ):
-                # orchestration_config.approvals[human_feedback.m_plan_id] = (
-                #     human_feedback.approved
-                # )
                 orchestration_config.set_approval_result(
                     human_feedback.m_plan_id, human_feedback.approved
                 )
@@ -531,9 +528,6 @@ async def user_clarification(
             orchestration_config
             and human_feedback.request_id in orchestration_config.clarifications
         ):
-            # orchestration_config.clarifications[human_feedback.request_id] = (
-            #     human_feedback.answer
-            # )
             # Use the new event-driven method to set clarification result
             orchestration_config.set_clarification_result(
                 human_feedback.request_id, human_feedback.answer

src/backend/v3/magentic_agents/proxy_agent.py

@@ -270,17 +270,6 @@ async def invoke_stream(
     async def _wait_for_user_clarification(
         self, request_id: str
     ) -> Optional[UserClarificationResponse]:
-        # """Wait for user clarification response."""
-        # # To do: implement timeout and error handling
-        # if request_id not in orchestration_config.clarifications:
-        #     orchestration_config.clarifications[request_id] = None
-        # while orchestration_config.clarifications[request_id] is None:
-        #     await asyncio.sleep(0.2)
-        # return UserClarificationResponse(
-        #     request_id=request_id,
-        #     answer=orchestration_config.clarifications[request_id],
-        # )
-
         """
         Wait for user clarification response using event-driven pattern with timeout handling.
 

src/backend/v3/orchestration/human_approval_manager.py

@@ -206,16 +206,6 @@ async def create_progress_ledger(
     async def _wait_for_user_approval(
         self, m_plan_id: Optional[str] = None
     ) -> Optional[messages.PlanApprovalResponse]:
-        # """Wait for user approval response."""
-
-        # # To do: implement timeout and error handling
-        # if m_plan_id not in orchestration_config.approvals:
-        #     orchestration_config.approvals[m_plan_id] = None
-        # while orchestration_config.approvals[m_plan_id] is None:
-        #     await asyncio.sleep(0.2)
-        # return messages.PlanApprovalResponse(
-        #     approved=orchestration_config.approvals[m_plan_id], m_plan_id=m_plan_id
-        # )
         """
         Wait for user approval response using event-driven pattern with timeout handling.
 

src/backend/v3/orchestration/orchestration_manager.py

@@ -119,7 +119,6 @@ async def run_orchestration(self, user_id, input_task) -> None:
         """Run the orchestration with user input loop."""
 
         job_id = str(uuid.uuid4())
-        # orchestration_config.approvals[job_id] = None
 
         # Use the new event-driven method to set approval as pending
         orchestration_config.set_approval_pending(job_id)