remove duration_ms

Author: khvn26

mcp/src/flagsmith_mcp/events.py

@@ -1,5 +1,3 @@
-import time
-
 import mcp.types as mt
 import structlog
 from fastmcp.server.dependencies import get_context
@@ -46,7 +44,6 @@ async def on_call_tool(
         client_info = get_client_info()
         client_name = client_info.name if client_info else ""
         client_version = client_info.version if client_info else ""
-        start = time.perf_counter()
         try:
             result = await call_next(context)
         except Exception:
@@ -56,7 +53,6 @@ async def on_call_tool(
                 flagsmith__client__name=client_name,
                 flagsmith__client__version=client_version,
                 status="error",
-                duration_ms=(time.perf_counter() - start) * 1000,
             )
             raise
         logger.info(
@@ -65,6 +61,5 @@ async def on_call_tool(
             flagsmith__client__name=client_name,
             flagsmith__client__version=client_version,
             status="success",
-            duration_ms=(time.perf_counter() - start) * 1000,
         )
         return result

mcp/tests/integration/test_events.py

@@ -35,7 +35,6 @@ async def test_events__successful_tool_call__emits_tool_called(
 
     # Then
     [event] = [e for e in log.events if e["event"] == "tool.called"]
-    assert event.pop("duration_ms") >= 0
     assert event == {
         "event": "tool.called",
         "level": "info",
@@ -60,7 +59,6 @@ async def test_events__failing_tool_call__emits_tool_called_with_error_status(
 
     # Then
     [event] = [e for e in log.events if e["event"] == "tool.called"]
-    assert event.pop("duration_ms") >= 0
     assert event == {
         "event": "tool.called",
         "level": "info",