Address code review feedback

- Remove unused _wall_clock_start_ms variable
- Fix documentation to accurately describe TimeInput accepted formats
- Remove unused time import

Co-authored-by: nikhilNava <211831449+nikhilNava@users.noreply.github.com>

Author: Copilot

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/execute_tool_scope.py

@@ -43,11 +43,12 @@ def start(
             request: Optional request details for additional context
             parent_id: Optional parent Activity ID used to link this span to an upstream
                 operation
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime). Useful when
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime. Useful when
                 recording a tool call after execution has already completed.
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime). When provided,
-                the span will use this timestamp when disposed instead of the current
-                wall-clock time.
+            end_time: Optional explicit end time in the same formats as start_time. When
+                provided, the span will use this timestamp when disposed instead of the
+                current wall-clock time.
 
         Returns:
             A new ExecuteToolScope instance
@@ -75,11 +76,12 @@ def __init__(
             request: Optional request details for additional context
             parent_id: Optional parent Activity ID used to link this span to an upstream
                 operation
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime). Useful when
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime. Useful when
                 recording a tool call after execution has already completed.
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime). When provided,
-                the span will use this timestamp when disposed instead of the current
-                wall-clock time.
+            end_time: Optional explicit end time in the same formats as start_time. When
+                provided, the span will use this timestamp when disposed instead of the
+                current wall-clock time.
         """
         super().__init__(
             kind="Internal",

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/inference_scope.py

@@ -47,8 +47,9 @@ def start(
             request: Optional request details for additional context
             parent_id: Optional parent Activity ID used to link this span to an upstream
                 operation
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime)
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime)
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime.
+            end_time: Optional explicit end time in the same formats as start_time.
 
         Returns:
             A new InferenceScope instance
@@ -76,8 +77,9 @@ def __init__(
             request: Optional request details for additional context
             parent_id: Optional parent Activity ID used to link this span to an upstream
                 operation
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime)
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime)
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime.
+            end_time: Optional explicit end time in the same formats as start_time.
         """
 
         super().__init__(

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/invoke_agent_scope.py

@@ -62,8 +62,9 @@ def start(
             request: Optional request details for additional context
             caller_agent_details: Optional details of the caller agent
             caller_details: Optional details of the non-agentic caller
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime)
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime)
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime.
+            end_time: Optional explicit end time in the same formats as start_time.
 
         Returns:
             A new InvokeAgentScope instance
@@ -96,8 +97,9 @@ def __init__(
             request: Optional request details for additional context
             caller_agent_details: Optional details of the caller agent
             caller_details: Optional details of the non-agentic caller
-            start_time: Optional explicit start time (ms epoch, Date, or HrTime)
-            end_time: Optional explicit end time (ms epoch, Date, or HrTime)
+            start_time: Optional explicit start time. Accepts int (nanoseconds since epoch),
+                float (seconds since epoch), tuple[int, int] (HrTime), or datetime.
+            end_time: Optional explicit end time in the same formats as start_time.
         """
         activity_name = INVOKE_AGENT_OPERATION_NAME
         if invoke_agent_details.details.agent_name:

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/opentelemetry_scope.py

@@ -5,7 +5,6 @@
 
 import logging
 import os
-import time
 from datetime import datetime
 from threading import Lock
 from typing import TYPE_CHECKING, Any
@@ -111,7 +110,6 @@ def __init__(
                 instead of the current wall-clock time.
         """
         self._span: Span | None = None
-        self._wall_clock_start_ms = time.time() * 1000  # milliseconds
         self._custom_start_time: TimeInput = start_time
         self._custom_end_time: TimeInput = end_time
         self._has_ended = False