Improve agent client initialization and error logging

Added checks for agent_id presence before returning or creating AzureAIAgentClient instances. Enhanced logging for client creation and error handling, and prevented saving database team agent when agent_id is None.

Author: Fr4nc3

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

@@ -139,13 +139,14 @@ def get_chat_client(self, chat_client) -> AzureAIAgentClient:
         """Return the underlying ChatClientProtocol (AzureAIAgentClient)."""
         if chat_client:
             return chat_client
-        if self._agent:
+        if self._agent and self._agent.chat_client and self._agent.chat_client.agent_id is not None:
             return self._agent.chat_client  # type: ignore
         chat_client = AzureAIAgentClient(
                         project_endpoint=self.project_endpoint,
                         model_deployment_name=self.model_deployment_name,
                         async_credential=self.creds,
                     )
+        self.logger.info("Created new AzureAIAgentClient for  get chat client", extra={"agent_id": chat_client.agent_id})
         return chat_client
 
     async def get_database_team_agent(self) -> Optional[AzureAIAgentClient]:
@@ -158,21 +159,25 @@ async def get_database_team_agent(self) -> Optional[AzureAIAgentClient]:
             )
             if currentAgent and currentAgent.agent_foundry_id:
                 agent = await self.client.get_agent(agent_id=currentAgent.agent_foundry_id)
-                if agent:
+                if agent and agent.agent_id is not None:
                     chat_client = AzureAIAgentClient(
                         project_endpoint=self.project_endpoint,
-                        agent_id=currentAgent.agent_foundry_id,
+                        agent_id=agent.agent_id,
                         model_deployment_name=self.model_deployment_name,
                         async_credential=self.creds,
                     )
 
         except Exception as ex:  # Consider narrowing this to specific exceptions if possible
-            self.logger.error("Failed to initialize ReasoningAgentTemplate: %s", ex)
+            self.logger.error("Failed to initialize Get database team agent: %s", ex)
         return chat_client
 
     async def save_database_team_agent(self) -> None:
         """Save current team agent to database."""
         try:
+            if self._agent.chat_client.agent_id is None:
+                self.logger.error("Cannot save database team agent: agent_id is None")
+                return  
+            
             currentAgent = CurrentTeamAgent(
                 team_id=self.team_config.team_id,
                 team_name=self.team_config.name,