fix: parse model.parameters.tools as list

[jsonbailey]

Summary

• model.parameters.tools wire format is a list of LDTool objects; root-level tools is a dict keyed by tool name. Previously _resolve_tools checked for a dict in both paths, so the model params fallback always returned None.
• Fixes _resolve_tools to check isinstance(tools_list, list) for the model params path and iterate it to build the tools dict.
• Removes the _parse_tools and _parse_tools_from_list helpers, inlining the logic. No warnings are emitted — data from the LD API is trusted to be our shape, matching the JS SDK approach.
• Updates test fixtures and adds/renames tests to cover list format, dict format rejection, and silent skipping of nameless entries.

Test plan

• uv run pytest tests/test_tools.py — all 11 tools tests pass
• uv run pytest — all 154 tests pass
• uv run mypy src/ldai — no type errors

🤖 Generated with Claude Code

---

Note

Low Risk
Low risk: targeted change to tools parsing that affects only how AI config variations are interpreted; main risk is behavior change for previously accepted/ignored malformed tool shapes.

Overview
Fixes tools parsing for AI configs by updating _resolve_tools in client.py to treat root-level tools as a dict (unchanged behavior) while correctly parsing model.parameters.tools as a list of tool objects.

Adds/updates tests to reflect the list wire format, assert that dict-shaped model.parameters.tools is rejected (returns None), and verify invalid list entries are skipped while valid tools are still parsed.

^{Reviewed by Cursor Bugbot for commit db79f74. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 13e2820. Configure here.}