fix how to create planner agent

Author: Fr4nc3

src/backend/app_kernel.py

@@ -20,8 +20,7 @@
 # Local imports
 from middleware.health_check import HealthCheckMiddleware
 from auth.auth_utils import get_authenticated_user_details
-# Replace Config with our new AppConfig
-from app_config import config
+from config_kernel import Config
 from context.cosmos_memory_kernel import CosmosMemoryContext
 from models.messages_kernel import (
     HumanFeedback,
@@ -66,8 +65,7 @@
 # Initialize the FastAPI app
 app = FastAPI()
 
-# Use the frontend URL from our AppConfig instance
-frontend_url = config.FRONTEND_SITE_NAME
+frontend_url = Config.FRONTEND_SITE_NAME
 
 # Add this near the top of your app.py, after initializing the app
 app.add_middleware(

src/backend/kernel_agents/agent_base.py

@@ -301,7 +301,7 @@ def get_tools_from_config(cls, kernel: sk.Kernel, agent_type: str, config_path:
                 # Register the function with the kernel
                 kernel.add_function(plugin_name, kernel_func)
                 kernel_functions.append(kernel_func)
-                logging.info(f"Successfully created dynamic tool '{function_name}' for {agent_type}")
+                #logging.info(f"Successfully created dynamic tool '{function_name}' for {agent_type}")
             except Exception as e:
                 logging.error(f"Failed to create tool '{tool.get('name', 'unknown')}': {str(e)}")
 

src/backend/kernel_agents/agent_factory.py

@@ -6,6 +6,7 @@
 from semantic_kernel import Kernel
 from semantic_kernel.functions import KernelFunction
 from semantic_kernel.agents.azure_ai.azure_ai_agent import AzureAIAgent
+import inspect
 
 from models.agent_types import AgentType
 from kernel_agents.agent_base import BaseAgent
@@ -17,7 +18,6 @@
 from kernel_agents.human_agent import HumanAgent 
 from kernel_agents.marketing_agent import MarketingAgent
 from kernel_agents.generic_agent import GenericAgent
-from kernel_agents.planner_agent import PlannerAgent
 from kernel_agents.tech_support_agent import TechSupportAgent
 from kernel_agents.procurement_agent import ProcurementAgent
 from kernel_agents.product_agent import ProductAgent
@@ -41,7 +41,6 @@ class AgentFactory:
         AgentType.TECH_SUPPORT: TechSupportAgent,
         AgentType.GENERIC: GenericAgent,
         AgentType.HUMAN: HumanAgent,
-        AgentType.PLANNER: PlannerAgent,
         AgentType.GROUP_CHAT_MANAGER: GroupChatManager,
     }
 
@@ -54,7 +53,6 @@ class AgentFactory:
         AgentType.TECH_SUPPORT: "tech_support",
         AgentType.GENERIC: "generic",
         AgentType.HUMAN: "human",
-        AgentType.PLANNER: "planner",
         AgentType.GROUP_CHAT_MANAGER: "group_chat_manager",
     }
 
@@ -67,7 +65,6 @@ class AgentFactory:
         AgentType.TECH_SUPPORT: "You are a technical support expert helping with technical issues.",
         AgentType.GENERIC: "You are a helpful assistant ready to help with various tasks.",
         AgentType.HUMAN: "You are representing a human user in the conversation.",
-        AgentType.PLANNER: "You are a planner agent responsible for creating and managing plans.",
         AgentType.GROUP_CHAT_MANAGER: "You are a group chat manager coordinating the conversation between different agents.",
     }
 
@@ -128,7 +125,7 @@ async def create_agent(
         # Check if we already have an agent in the cache
         if session_id in cls._agent_cache and agent_type in cls._agent_cache[session_id]:
             return cls._agent_cache[session_id][agent_type]
-            
+
         # Get the agent class
         agent_class = cls._agent_classes.get(agent_type)
         if not agent_class:
@@ -177,27 +174,31 @@ async def create_agent(
 
         # Create the agent instance using the project-based pattern
         try:
-            agent = agent_class(
-                agent_name=agent_type_str,
-                kernel=kernel,
-                session_id=session_id,
-                user_id=user_id,
-                memory_store=memory_store,
-                tools=tools,
-                system_message=system_message,
-                client=client,
-                definition=definition,
+            # Filter kwargs to only those accepted by the agent's __init__
+            agent_init_params = inspect.signature(agent_class.__init__).parameters
+            valid_keys = set(agent_init_params.keys()) - {"self"}
+            filtered_kwargs = {k: v for k, v in {
+                "agent_name": agent_type_str,
+                "kernel": kernel,
+                "session_id": session_id,
+                "user_id": user_id,
+                "memory_store": memory_store,
+                "tools": tools,
+                "system_message": system_message,
+                "client": client,
+                "definition": definition,
                 **kwargs
-            )
+            }.items() if k in valid_keys}
+            agent = agent_class(**filtered_kwargs)
             logger.debug(f"[DEBUG] Agent object after instantiation: {agent}")
-            # Initialize the agent asynchronously
-            init_result = await agent.async_init()
-            logger.debug(f"[DEBUG] Result of agent.async_init(): {init_result}")
+            # Initialize the agent asynchronously if it has async_init
+            if hasattr(agent, 'async_init') and inspect.iscoroutinefunction(agent.async_init):
+                init_result = await agent.async_init()
+                logger.debug(f"[DEBUG] Result of agent.async_init(): {init_result}")
             # Register tools with Azure AI Agent for LLM function calls
-            if hasattr(agent._agent, 'add_function') and tools:
+            if hasattr(agent, '_agent') and hasattr(agent._agent, 'add_function') and tools:
                 for fn in tools:
                     agent._agent.add_function(fn)
-        
         except Exception as e:
             logger.error(
                 f"Error creating agent of type {agent_type} with parameters: {e}"

src/backend/kernel_agents/planner_agent.py

@@ -46,7 +46,6 @@ def __init__(
         session_id: str,
         user_id: str,
         memory_store: CosmosMemoryContext,
-        config_path: Optional[str] = None,
         available_agents: List[str] = None,
         agent_tools_list: List[str] = None
     ) -> None:
@@ -73,17 +72,15 @@ def __init__(
                                                     "TechSupportAgent", "GenericAgent"]
         self._agent_tools_list = agent_tools_list or []
 
-        # Load configuration
-        config = BaseAgent.load_tools_config("planner", config_path)
-        self._system_message = config.get(
-            "system_message", 
-            "You are a Planner agent responsible for creating and managing plans. You analyze tasks, break them down into steps, and assign them to the appropriate specialized agents."
-        )
+
+        self._system_message = "You are a Planner agent responsible for creating and managing plans. You analyze tasks, break them down into steps, and assign them to the appropriate specialized agents."
+        
 
         # Create the agent
-        self._agent = kernel.create_semantic_function(
+        self._agent = KernelFunction.from_prompt(
             function_name="PlannerFunction",
-            prompt=self._system_message,
+            plugin_name="planner_plugin",
+            prompt_template=self._system_message,
             description="Creates and manages execution plans"
         )