refactor: remove warn parameter from _parse_tools

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: jsonbailey

packages/sdk/server-ai/src/ldai/client.py

@@ -53,15 +53,14 @@
 _DISABLED_JUDGE_DEFAULT = AIJudgeConfigDefault.disabled()
 
 
-def _parse_tools(tools_data: Optional[Dict[str, Any]], *, warn: bool = True) -> Optional[Dict[str, LDTool]]:
+def _parse_tools(tools_data: Optional[Dict[str, Any]]) -> Optional[Dict[str, LDTool]]:
     """Parse the root-level tools map from a flag variation dict."""
     if not isinstance(tools_data, dict):
         return None
     result: Dict[str, LDTool] = {}
     for tool_name, tool_dict in tools_data.items():
         if not isinstance(tool_dict, dict):
-            if warn:
-                log.warning('Skipping tool "%s": expected a dict, got %s', tool_name, type(tool_dict).__name__)
+            log.warning('Skipping tool "%s": expected a dict, got %s', tool_name, type(tool_dict).__name__)
             continue
         result[tool_name] = LDTool(
             name=tool_dict.get('name', tool_name),
@@ -87,7 +86,7 @@ def _resolve_tools(variation: Dict[str, Any]) -> Optional[Dict[str, LDTool]]:
     if not isinstance(tools_data, dict):
         return None
 
-    return _parse_tools(tools_data, warn=False)
+    return _parse_tools(tools_data)
 
 
 class LDAIClient: