agentjido / req_llm / 430f3843e68c4d889f6485cde1d78c0b42c05204-PR-105

50%

main: 49%

Build Type

Pull #105

github

Committed by Valian

Commit Message

refactor: use ReqLLM.ToolCall directly, remove normalization

Remove the normalization layer that converted tool calls to plain maps.
Since `ReqLLM.ToolCall` (based on `Req.ToolCall`) already exists as a
proper struct representation, there's no reason to introduce another
intermediate representation.

Changes:
- Remove `normalize_tool_call/1` from ReqLLM.Response
- Remove fallback logic for ContentPart-based tool calls (`:tool_result`
  content part type no longer exists in ContentPart)
- Update all tests to expect `ReqLLM.ToolCall` structs with
  `%{function: %{name: ..., arguments: ...}, id: ...}` structure
- Remove redundant pattern match in XAI provider

This simplifies the API and aligns with existing Req patterns.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Pull Request Pull Request #105: Refactor: use ReqLLM.ToolCall directly, remove normalization layer

Run Details

1 of 2 new or added lines in 2 files covered. (50.0%)

1 existing line in 1 file now uncovered.

3261 of 6520 relevant lines covered (50.02%)

198.49 hits per line