Skip to content

Commit a0bc651

Browse files
jsonbaileyclaude
andcommitted
refactor: remove redundant None check before isinstance in _resolve_tools
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent ca83775 commit a0bc651

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,9 @@ def _resolve_tools(variation: Dict[str, Any]) -> Optional[Dict[str, LDTool]]:
8383
if not isinstance(parameters, dict):
8484
return None
8585
tools_data = parameters.get('tools')
86-
if tools_data is None:
87-
return None
8886
if not isinstance(tools_data, dict):
89-
log.warning('Skipping model.parameters.tools: expected a dict, got %s', type(tools_data).__name__)
87+
if tools_data is not None:
88+
log.warning('Skipping model.parameters.tools: expected a dict, got %s', type(tools_data).__name__)
9089
return None
9190

9291
return _parse_tools(tools_data)

0 commit comments

Comments
 (0)