agentjido / req_llm / aff7a726aa33c1bf8da2655255148c48247fdaae-PR-221

50%

main: 49%

Build Type

Pull #221

github

Committed by neilberkman

Commit Message

Fix tool_budget_for pattern match regression from LLMDB integration

The test helper tool_budget_for/1 was using an obsolete pattern match
that expected {:ok, {provider, id, model}} but ReqLLM.model/1 now returns
{:ok, model} after the LLMDB integration in v1.1.0.

This caused the pattern match to fail and always return the default 150
tokens instead of calculating the correct budget based on model.limits.

Additionally fixed the field name from model.limit (singular) to
model.limits (plural) to match the LLMDB.Model struct.

The regression primarily affected reasoning-enabled models like Gemini
2.5 Pro, where 150 tokens was insufficient for the model's extended
reasoning process, causing tool call tests to fail with MAX_TOKENS
errors.

Pull Request Pull Request #221: Fix tool_budget_for pattern match regression from LLMDB integration

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

3 existing lines in 1 file now uncovered.

3588 of 7156 relevant lines covered (50.14%)

58.78 hits per line