PylintIssue

Author: UtkarshMishra-Microsoft

src/backend/kernel_agents/planner_agent.py

@@ -150,10 +150,10 @@ async def create(
                 AgentType.TECH_SUPPORT.value,
                 AgentType.GENERIC.value,
             ]
-            
+
             agents_list = available_agents or default_available_agents
             agents_str = ", ".join(agents_list)
-            
+
             agent_tools_list = {
                 AgentType.HR: HrTools.generate_tools_json_doc(),
                 AgentType.MARKETING: MarketingTools.generate_tools_json_doc(),
@@ -162,7 +162,7 @@ async def create(
                 AgentType.TECH_SUPPORT: TechSupportTools.generate_tools_json_doc(),
                 AgentType.GENERIC: GenericTools.generate_tools_json_doc(),
             }
-            
+
             tools_str = str(agent_tools_list)
 
             # Create the Azure AI Agent using AppConfig with string instructions
@@ -419,12 +419,12 @@ async def _create_structured_plan(
                 logging.warning("Planner returned no steps; falling back to default 2-step plan.")
                 # Create default step data for roaming plan
                 from models.messages_kernel import AgentType
-                
+
                 class DefaultStep:
                     def __init__(self, action, agent):
                         self.action = action
                         self.agent = agent
-                
+
                 steps_data = [
                     DefaultStep(
                         action="Get information about available roaming packs and plans. Function: get_product_info",
@@ -435,7 +435,7 @@ def __init__(self, action, agent):
                         agent=AgentType.PRODUCT.value
                     )
                 ]
-                
+
                 # Update plan details for the default case
                 if not initial_goal:
                     initial_goal = "Enable roaming on mobile plan, starting next week."

src/backend/kernel_agents/product_agent.py

@@ -130,15 +130,16 @@ def plugins(self):
 
     # Explicitly inherit handle_action_request from the parent class
     # This is not technically necessary but makes the inheritance explicit
-    async def handle_action_request(self, action_request_json: str) -> str:
+    async def handle_action_request(self, action_request_json: str, **kwargs) -> str:
         """Handle an action request from another agent or the system.
 
         This method is inherited from BaseAgent but explicitly included here for clarity.
 
         Args:
             action_request_json: The action request as a JSON string
+            **kwargs: Additional keyword arguments (e.g., user_locale)
 
         Returns:
             A JSON string containing the action response
         """
-        return await super().handle_action_request(action_request_json)
+        return await super().handle_action_request(action_request_json, **kwargs)