Remove ReasoningAgentTemplate and related logic

Deleted reasoning_agent.py and removed all references to ReasoningAgentTemplate in the agent factory and orchestration manager. The agent creation flow now exclusively uses FoundryAgentTemplate, and related comments and error messages have been updated accordingly. Also updated temperature setting in FoundryAgentTemplate to 1.0.

Author: Fr4nc3

src/backend/v4/magentic_agents/common/lifecycle.py

@@ -189,7 +189,7 @@ async def save_database_team_agent(self) -> None:
             await self.memory_store.add_team_agent(currentAgent)
 
         except Exception as ex:
-            self.logger.error("Failed to save ReasoningAgentTemplate: %s", ex)
+            self.logger.error("Failed to save save database: %s", ex)
 
 
     async def _prepare_mcp_tool(self) -> None:
@@ -247,35 +247,35 @@ def __init__(
 
 
 
-    async def open(self) -> "AzureAgentBase":
-        if self._stack is not None:
-            return self
-        self._stack = AsyncExitStack()
-
-        # Acquire credential
-        self.creds = DefaultAzureCredential()
-        if self._stack:
-            await self._stack.enter_async_context(self.creds)
-        # Create AgentsClient
-        self.client = AgentsClient(
-            endpoint=self.project_endpoint,
-            credential=self.creds,
-        )
-        if self._stack:
-            await self._stack.enter_async_context(self.client)
-        # Prepare MCP
-        await self._prepare_mcp_tool()
-
-        # Let subclass build agent client
-        await self._after_open()
-
-        # Register agent (best effort)
-        try:
-            agent_registry.register_agent(self)
-        except Exception:
-            pass
-
-        return self
+    # async def open(self) -> "AzureAgentBase":
+    #     if self._stack is not None:
+    #         return self
+    #     self._stack = AsyncExitStack()
+
+    #     # Acquire credential
+    #     self.creds = DefaultAzureCredential()
+    #     if self._stack:
+    #         await self._stack.enter_async_context(self.creds)
+    #     # Create AgentsClient
+    #     self.client = AgentsClient(
+    #         endpoint=self.project_endpoint,
+    #         credential=self.creds,
+    #     )
+    #     if self._stack:
+    #         await self._stack.enter_async_context(self.client)
+    #     # Prepare MCP
+    #     await self._prepare_mcp_tool()
+
+    #     # Let subclass build agent client
+    #     await self._after_open()
+
+    #     # Register agent (best effort)
+    #     try:
+    #         agent_registry.register_agent(self)
+    #     except Exception:
+    #         pass
+
+    #     return self
 
     async def close(self) -> None:
         """

src/backend/v4/magentic_agents/foundry_agent.py

@@ -254,7 +254,7 @@ async def _after_open(self) -> None:
                     description=self.agent_description,
                     tools=tools if tools else None,
                     tool_choice="auto" if tools else "none",
-                    temperature=0.7,
+                    temperature=1.0,
                     model_id=self.model_deployment_name,
                 )
 

src/backend/v4/magentic_agents/magentic_agent_factory.py

@@ -16,7 +16,6 @@
 # from v4.magentic_agents.models.agent_models import (BingConfig, MCPConfig,
 #                                                     SearchConfig)
 from v4.magentic_agents.proxy_agent import ProxyAgent
-from v4.magentic_agents.reasoning_agent import ReasoningAgentTemplate
 
 
 class UnsupportedModelError(Exception):
@@ -41,7 +40,7 @@ def __init__(self, team_service: Optional[TeamService] = None):
     #     with open(file_path, 'r') as f:
     #         data = json.load(f)
     #     return json.loads(json.dumps(data), object_hook=lambda d: SimpleNamespace(**d))
-    
+
     # Ensure only an explicit boolean True in the source sets this flag.
     def extract_use_reasoning(self, agent_obj):
         # Support both dict and attribute-style objects
@@ -59,7 +58,7 @@ async def create_agent_from_config(
         agent_obj: SimpleNamespace,
         team_config: TeamConfiguration,
         memory_store: DatabaseBase,
-    ) -> Union[FoundryAgentTemplate, ReasoningAgentTemplate, ProxyAgent]:
+    ) -> Union[FoundryAgentTemplate, ProxyAgent]:
         """
         Create an agent from configuration object.
 
@@ -91,16 +90,16 @@ async def create_agent_from_config(
             )
 
         # Determine which template to use
-            # Usage
+        # Usage
         use_reasoning = self.extract_use_reasoning(agent_obj)
 
-        # Validate reasoning template constraints
+        # Validate reasoning constraints
         if use_reasoning:
             if getattr(agent_obj, "use_bing", False) or getattr(
                 agent_obj, "coding_tools", False
             ):
                 raise InvalidConfigurationError(
-                    f"ReasoningAgentTemplate cannot use Bing search or coding tools. "
+                    f"Agent cannot use Bing search or coding tools. "
                     f"Agent '{agent_obj.name}' has use_bing={getattr(agent_obj, 'use_bing', False)}, "
                     f"coding_tools={getattr(agent_obj, 'coding_tools', False)}"
                 )
@@ -118,44 +117,30 @@ async def create_agent_from_config(
         # bing_config = BingConfig.from_env() if getattr(agent_obj, 'use_bing', False) else None
 
         self.logger.info(
-            f"Creating agent '{agent_obj.name}' with model '{deployment_name}' {index_name} "
-            f"(Template: {'Reasoning' if use_reasoning else 'Foundry'})"
+            "Creating agent '%s' with model '%s' %s (Template: %s)",
+            agent_obj.name,
+            deployment_name,
+            index_name,
+            "Reasoning" if use_reasoning else "Foundry",
         )
 
-        # Create appropriate agent
-        if use_reasoning:
-            # Get reasoning specific configuration
-            project_endpoint = config.AZURE_AI_PROJECT_ENDPOINT
-            agent = ReasoningAgentTemplate(
-                agent_name=agent_obj.name,
-                agent_description=getattr(agent_obj, "description", ""),
-                agent_instructions=getattr(agent_obj, "system_message", ""),
-                model_deployment_name=deployment_name,
-                project_endpoint=project_endpoint,  # type: ignore
-                search_config=search_config,
-                mcp_config=mcp_config,
-                team_service=self.team_service,
-                team_config=team_config,
-                memory_store=memory_store,
-            )
-        else:
-            agent = FoundryAgentTemplate(
-                agent_name=agent_obj.name,
-                agent_description=getattr(agent_obj, "description", ""),
-                agent_instructions=getattr(agent_obj, "system_message", ""),
-                model_deployment_name=deployment_name,
-                enable_code_interpreter=getattr(agent_obj, "coding_tools", False),
-                project_endpoint=config.AZURE_AI_PROJECT_ENDPOINT,
-                mcp_config=mcp_config,
-                search_config=search_config,
-                team_service=self.team_service,
-                team_config=team_config,
-                memory_store=memory_store,
-            )
+        agent = FoundryAgentTemplate(
+            agent_name=agent_obj.name,
+            agent_description=getattr(agent_obj, "description", ""),
+            agent_instructions=getattr(agent_obj, "system_message", ""),
+            model_deployment_name=deployment_name,
+            enable_code_interpreter=getattr(agent_obj, "coding_tools", False),
+            project_endpoint=config.AZURE_AI_PROJECT_ENDPOINT,
+            mcp_config=mcp_config,
+            search_config=search_config,
+            team_service=self.team_service,
+            team_config=team_config,
+            memory_store=memory_store,
+        )
 
         await agent.open()
         self.logger.info(
-            f"Successfully created and initialized agent '{agent_obj.name}'"
+            "Successfully created and initialized agent '%s'", agent_obj.name
         )
         return agent
 
@@ -184,7 +169,10 @@ async def get_agents(
             for i, agent_cfg in enumerate(team_config_input.agents, 1):
                 try:
                     self.logger.info(
-                        f"Creating agent {i}/{len(team_config_input.agents)}: {agent_cfg.name}"
+                        "Creating agent %d/%d: %s",
+                        i,
+                        len(team_config_input.agents),
+                        agent_cfg.name
                     )
 
                     agent = await self.create_agent_from_config(
@@ -194,7 +182,10 @@ async def get_agents(
                     self._agent_list.append(agent)  # Keep track for cleanup
 
                     self.logger.info(
-                        f"✅ Agent {i}/{len(team_config_input.agents)} created: {agent_cfg.name}"
+                        "✅ Agent %d/%d created: %s",
+                        i,
+                        len(team_config_input.agents),
+                        agent_cfg.name
                     )
 
                 except (UnsupportedModelError, InvalidConfigurationError) as e:
@@ -207,7 +198,10 @@ async def get_agents(
                     continue
 
             self.logger.info(
-                f"Successfully created {len(initalized_agents)}/{len(team_config_input.agents)} agents for team '{team_config_input.name}'"
+                "Successfully created %d/%d agents for team '%s'",
+                len(initalized_agents),
+                len(team_config_input.agents),
+                team_config_input.name
             )
             return initalized_agents