fix planner instructions, and api endpoint

Author: Fr4nc3

.gitignore

@@ -458,3 +458,4 @@ __pycache__/
 *.whl
 !autogen_core-0.3.dev0-py3-none-any.whl
 .azure
+.github/copilot-instructions.md

src/backend/app_kernel.py

@@ -83,7 +83,7 @@
 logging.info("Added health check middleware")
 
 
-@app.post("/input_task")
+@app.post("/api/input_task")
 async def input_task_endpoint(input_task: InputTask, request: Request):
     """
     Receive the initial input task from the user.
@@ -175,7 +175,7 @@ async def input_task_endpoint(input_task: InputTask, request: Request):
         raise HTTPException(status_code=400, detail="Error creating plan")
 
 
-@app.post("/human_feedback")
+@app.post("/api/human_feedback")
 async def human_feedback_endpoint(human_feedback: HumanFeedback, request: Request):
 
     """
@@ -268,7 +268,7 @@ async def human_feedback_endpoint(human_feedback: HumanFeedback, request: Reques
     }
 
 
-@app.post("/human_clarification_on_plan")
+@app.post("/api/human_clarification_on_plan")
 async def human_clarification_endpoint(
     human_clarification: HumanClarification, request: Request
 ):
@@ -349,7 +349,7 @@ async def human_clarification_endpoint(
     }
 
 
-@app.post("/approve_step_or_steps")
+@app.post("/api/approve_step_or_steps")
 async def approve_step_endpoint(
     human_feedback: HumanFeedback, request: Request
 ) -> Dict[str, str]:
@@ -453,7 +453,7 @@ async def approve_step_endpoint(
         return {"status": "All steps approved"}
 
 
-@app.get("/plans", response_model=List[PlanWithSteps])
+@app.get("/api/plans", response_model=List[PlanWithSteps])
 async def get_plans(
     request: Request, session_id: Optional[str] = Query(None)
 ) -> List[PlanWithSteps]:
@@ -553,7 +553,7 @@ async def get_plans(
     return list_of_plans_with_steps
 
 
-@app.get("/steps/{plan_id}", response_model=List[Step])
+@app.get("/api/steps/{plan_id}", response_model=List[Step])
 async def get_steps_by_plan(plan_id: str, request: Request) -> List[Step]:
     """
     Retrieve steps for a specific plan.
@@ -616,7 +616,7 @@ async def get_steps_by_plan(plan_id: str, request: Request) -> List[Step]:
     return steps
 
 
-@app.get("/agent_messages/{session_id}", response_model=List[AgentMessage])
+@app.get("/api/agent_messages/{session_id}", response_model=List[AgentMessage])
 async def get_agent_messages(session_id: str, request: Request) -> List[AgentMessage]:
     """
     Retrieve agent messages for a specific session.
@@ -680,7 +680,7 @@ async def get_agent_messages(session_id: str, request: Request) -> List[AgentMes
     return agent_messages
 
 
-@app.delete("/messages")
+@app.delete("/api/messages")
 async def delete_all_messages(request: Request) -> Dict[str, str]:
     """
     Delete all messages across sessions.
@@ -723,7 +723,7 @@ async def delete_all_messages(request: Request) -> Dict[str, str]:
     return {"status": "All messages deleted"}
 
 
-@app.get("/messages")
+@app.get("/api/messages")
 async def get_all_messages(request: Request):
     """
     Retrieve all messages across sessions.
@@ -804,39 +804,6 @@ async def get_agent_tools():
     return []
 
 
-# Initialize the application when it starts
-@app.on_event("startup")
-async def startup_event():
-    """Initialize the application on startup.
-    
-    This function runs when the FastAPI application starts up.
-    It sets up the agent types and tool loaders so the first request is faster.
-    """
-    # Log startup
-    logging.info("Application starting up. Initializing agent factory...")
-    
-    try:
-        # Create a temporary session and user ID to pre-initialize agents
-        # This ensures tools are loaded into the factory on startup
-        temp_session_id = "startup-session"
-        temp_user_id = "startup-user"
-        
-        # Create a test agent to initialize the tool loading system
-        # This will pre-load tool configurations into memory
-        test_agent = await AgentFactory.create_agent(
-            agent_type=AgentType.GENERIC,
-            session_id=temp_session_id,
-            user_id=temp_user_id
-        )
-        
-        # Clean up initialization resources
-        AgentFactory.clear_cache(temp_session_id)
-        logging.info("Agent factory successfully initialized")
-        
-    except Exception as e:
-        # Don't fail startup, but log the error
-        logging.error(f"Error initializing agent factory: {e}")
-
 
 # Run the app
 if __name__ == "__main__":

src/backend/kernel_agents/planner_agent.py

@@ -147,11 +147,17 @@ async def async_init(self) -> None:
         """
         try:
             logging.info("Initializing PlannerAgent from async init azure AI Agent")
-            # Create the Azure AI Agent using AppConfig
+            
+            # Generate instructions as a string rather than returning an object
+            # We'll use a blank input since this is just for initialization
+            instruction_args = self._generate_instruction("")
+            instructions = self._get_template().format(**instruction_args)
+            
+            # Create the Azure AI Agent using AppConfig with string instructions
             self._azure_ai_agent = await config.create_azure_ai_agent(
                 kernel=self._kernel,
                 agent_name="PlannerAgent",
-                instructions=self._generate_instruction(""),
+                instructions=instructions,  # Pass the formatted string, not an object
                 temperature=0.0
             )
             logging.info("Successfully created Azure AI Agent for PlannerAgent")
@@ -311,8 +317,11 @@ async def _create_structured_plan(self, input_task: InputTask) -> Tuple[Plan, Li
             logging.debug(f"Input: {input_task}")
             logging.debug(f"Available agents: {self._available_agents}")
 
-            instruction = self._generate_instruction(input_task.description)
+            # Get template variables as a dictionary
+            args = self._generate_instruction(input_task.description)
 
+            # Format the template with the arguments
+            instruction = self._get_template().format(**args)
             logging.info(f"Generated instruction: {instruction}")   
             # Log the input task for debugging
             logging.info(f"Creating plan for task: '{input_task.description}'")
@@ -651,14 +660,14 @@ async def _create_fallback_plan_from_text(self, input_task: InputTask, text_cont
 
         return plan, steps
 
-    def _generate_instruction(self, objective: str) -> str:
+    def _generate_instruction(self, objective: str) -> any:
         """Generate instruction for the LLM to create a plan.
         
         Args:
             objective: The user's objective
             
         Returns:
-            Instruction string for the LLM
+            Dictionary containing the variables to populate the template
         """
         # Create a list of available agents
         agents_str = ", ".join(self._available_agents)
@@ -766,11 +775,16 @@ def _generate_instruction(self, objective: str) -> str:
         # Convert the tools list to a string representation
         tools_str = str(tools_list)
 
-        return args
+        # Return a dictionary with template variables
+        return {
+            "objective": objective,
+            "agents_str": agents_str,
+            "tools_str": tools_str,
+        }
 
     def _get_template(self):
         """Generate the instruction template for the LLM."""  
-        # Build the instruction, avoiding backslashes in f-string expressions  
+        # Build the instruction with proper format placeholders for .format() method
 
         instruction_template = """
         You are the Planner, an AI orchestrator that manages a group of AI agents to accomplish tasks.
@@ -782,13 +796,13 @@ def _get_template(self):
                 These actions are passed to the specific agent. Make sure the action contains all the information required for the agent to execute the task.
                 
                 Your objective is:
-                {{$objective}}
+                {objective}
 
                 The agents you have access to are:
-                {{$agents_str}}
+                {agents_str}
 
                 These agents have access to the following functions:
-                {{$tools_str}}
+                {tools_str}
 
                 IMPORTANT AGENT SELECTION GUIDANCE:
                 - HrAgent: ALWAYS use for ALL employee-related tasks like onboarding, hiring, benefits, payroll, training, employee records, ID cards, mentoring, background checks, etc.
@@ -816,7 +830,7 @@ def _get_template(self):
 
                 Limit the plan to 6 steps or less.
 
-                Choose from {{$agents_str}} ONLY for planning your steps.
+                Choose from {agents_str} ONLY for planning your steps.
 
                 """        
         return instruction_template

src/frontend/frontend_server.py

@@ -29,6 +29,7 @@
 def get_config():
     backend_url = html.escape(os.getenv("BACKEND_API_URL", "http://localhost:8000"))
     auth_enabled = html.escape(os.getenv("AUTH_ENABLED", "True"))
+    backend_url = backend_url + "/api"
     return f'''
         const BACKEND_API_URL = "{backend_url}";
         const AUTH_ENABLED = "{auth_enabled}";