fix: mutate track data dict in place and guard tool_calls with isinstance check

jsonbailey · claude · jsonbailey · commit 91cd3005b6ca · 2026-03-26T12:55:46.000-05:00
Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/packages/ai-providers/server-ai-langchain/src/ldai_langchain/langchain_helper.py b/packages/ai-providers/server-ai-langchain/src/ldai_langchain/langchain_helper.py
@@ -125,7 +125,7 @@ def get_tool_calls_from_response(response: Any) -> List[str]:
     :return: List of tool names in order, or empty list if none
     """
     names: List[str] = []
-    if hasattr(response, 'tool_calls') and response.tool_calls:
+    if hasattr(response, 'tool_calls') and isinstance(response.tool_calls, list):
         for tc in response.tool_calls:
             n = tc.get('name')
             if n:
diff --git a/packages/sdk/server-ai/src/ldai/tracker.py b/packages/sdk/server-ai/src/ldai/tracker.py
@@ -113,7 +113,7 @@ def __get_track_data(self, graph_key: Optional[str] = None) -> dict:
             "providerName": self._provider_name,
         }
         if graph_key is not None:
-            return {**data, "graphKey": graph_key}
+            data['graphKey'] = graph_key
         return data
 
     def track_duration(self, duration: int, *, graph_key: Optional[str] = None) -> None:

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ def __get_track_data(self, graph_key: Optional[str] = None) -> dict:`
`113`	`113`	`"providerName": self._provider_name,`
`114`	`114`	`}`
`115`	`115`	`if graph_key is not None:`
`116`		`- return {**data, "graphKey": graph_key}`
	`116`	`+ data['graphKey'] = graph_key`
`117`	`117`	`return data`
`118`	`118`
`119`	`119`	`def track_duration(self, duration: int, *, graph_key: Optional[str] = None) -> None:`