copilot comments

Author: Jesus Terrazas

libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/utility.py

@@ -8,25 +8,25 @@
 """
 
 from __future__ import annotations
-from importlib.metadata import version
 
 import platform
 import uuid
+from importlib.metadata import PackageNotFoundError, version
 from typing import Any, Optional
 
 import jwt
 
 
 class Utility:
-    _cached_version = None
-
     """
     Utility class providing common runtime operations for Agent 365.
 
     This class contains static methods for token processing, agent identity resolution,
     and other utility functions used across the Agent 365 runtime.
     """
 
+    _cached_version = None
+
     @staticmethod
     def get_app_id_from_token(token: Optional[str]) -> str:
         """
@@ -101,7 +101,7 @@ def get_user_agent_header(orchestrator: str = "") -> str:
         if Utility._cached_version is None:
             try:
                 Utility._cached_version = version("microsoft-agents-a365-runtime")
-            except Exception:
+            except PackageNotFoundError:
                 Utility._cached_version = "unknown"
 
         orchestrator_part = f"; {orchestrator}" if orchestrator else ""

tests/runtime/test_utility.py

@@ -3,10 +3,10 @@
 
 """Unit tests for Utility class."""
 
-import uuid
-from unittest.mock import Mock
 import platform
 import re
+import uuid
+from unittest.mock import Mock
 
 import jwt
 import pytest
@@ -135,7 +135,7 @@ def test_get_user_agent_header_default():
 
     result = Utility.get_user_agent_header()
 
-    # Regex for Agent365SDK/version (OS; Python/version)
+    # Regex for Agent365SDK/version (OS; Python version)
     pattern = rf"^Agent365SDK/.+ \({os_type}; Python {py_version}\)$"
     assert re.match(pattern, result)
 
@@ -148,6 +148,6 @@ def test_get_user_agent_header_with_orchestrator():
 
     result = Utility.get_user_agent_header(orchestrator)
 
-    # Regex for Agent365SDK/version (OS; Python/version; TestOrchestrator)
+    # Regex for Agent365SDK/version (OS; Python version; TestOrchestrator)
     pattern = rf"^Agent365SDK/.+ \({os_type}; Python {py_version}; {orchestrator}\)$"
     assert re.match(pattern, result)