Move agent utility functions to utils_agents.py

Refactored generate_assistant_id and get_database_team_agent_id from utils_af.py to a new utils_agents.py for better separation of concerns. Updated imports in lifecycle.py and orchestration_manager.py to use the new module.

Author: Fr4nc3

src/backend/common/utils/utils_af.py

@@ -217,37 +217,3 @@ async def rai_validate_team_config(
     except Exception as e:
         logging.error("Error validating team configuration content: %s", e)
         return False, "Unable to validate team configuration content. Please try again."
-
-
-def generate_assistant_id(prefix: str = "asst_", length: int = 24) -> str:
-    """
-    Generate a unique ID like 'asst_jRgR5t2U7o8nUPkNGv5HWOgV'.
-
-    - prefix: leading string (defaults to 'asst_')
-    - length: number of random characters after the prefix
-    """
-    # URL-safe characters similar to what OpenAI-style IDs use
-    alphabet = string.ascii_letters + string.digits  # a-zA-Z0-9
-
-    # cryptographically strong randomness
-    random_part = "".join(secrets.choice(alphabet) for _ in range(length))
-    return f"{prefix}{random_part}"
-
-
-async def get_database_team_agent_id(
-    memory_store: DatabaseBase, team_config: TeamConfiguration, agent_name: str
-) -> Optional[str]:
-    """Retrieve existing team agent from database, if any."""
-    agent_id = None
-    try:
-        currentAgent = await memory_store.get_team_agent(
-            team_id=team_config.team_id, agent_name=agent_name
-        )
-        if currentAgent and currentAgent.agent_foundry_id:
-            agent_id = currentAgent.agent_foundry_id
-
-    except (
-        Exception
-    ) as ex:  # Consider narrowing this to specific exceptions if possible
-        logging.error("Failed to initialize Get database team agent: %s", ex)
-    return agent_id

src/backend/common/utils/utils_agents.py

@@ -0,0 +1,42 @@
+
+import logging
+import secrets
+import string
+from typing import Optional
+
+from common.database.database_base import DatabaseBase
+from common.models.messages_af import TeamConfiguration
+
+
+def generate_assistant_id(prefix: str = "asst_", length: int = 24) -> str:
+    """
+    Generate a unique ID like 'asst_jRgR5t2U7o8nUPkNGv5HWOgV'.
+
+    - prefix: leading string (defaults to 'asst_')
+    - length: number of random characters after the prefix
+    """
+    # URL-safe characters similar to what OpenAI-style IDs use
+    alphabet = string.ascii_letters + string.digits  # a-zA-Z0-9
+
+    # cryptographically strong randomness
+    random_part = "".join(secrets.choice(alphabet) for _ in range(length))
+    return f"{prefix}{random_part}"
+
+
+async def get_database_team_agent_id(
+    memory_store: DatabaseBase, team_config: TeamConfiguration, agent_name: str
+) -> Optional[str]:
+    """Retrieve existing team agent from database, if any."""
+    agent_id = None
+    try:
+        currentAgent = await memory_store.get_team_agent(
+            team_id=team_config.team_id, agent_name=agent_name
+        )
+        if currentAgent and currentAgent.agent_foundry_id:
+            agent_id = currentAgent.agent_foundry_id
+
+    except (
+        Exception
+    ) as ex:  # Consider narrowing this to specific exceptions if possible
+        logging.error("Failed to initialize Get database team agent: %s", ex)
+    return agent_id

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

@@ -18,7 +18,7 @@
 from azure.identity.aio import DefaultAzureCredential
 from common.database.database_base import DatabaseBase
 from common.models.messages_af import CurrentTeamAgent, TeamConfiguration
-from common.utils.utils_af import (
+from common.utils.utils_agents import (
     generate_assistant_id,
     get_database_team_agent_id,
 )

src/backend/v4/orchestration/orchestration_manager.py

@@ -22,7 +22,7 @@
 from common.models.messages_af import TeamConfiguration
 
 from common.database.database_base import DatabaseBase
-from common.utils.utils_af import (
+from common.utils.utils_agents import (
     generate_assistant_id,
     get_database_team_agent_id,
 )