scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-09 03:18:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 108

test(anthropic): type the Responses tool fixtures instead of suppressing

Browse source 
The two new `translate_tools_to_responses_api` calls carried
`# type: ignore[arg-type]`, which CLAUDE.md bans as LIT009: pyrightconfig.json
sets enableTypeIgnoreComments to false, so the comment silently does nothing and
the reportArgumentType error stands. Annotating the fixtures as
list[AllAnthropicToolsValues] makes both calls check clean with no suppression
at all.
This commit is contained in:
Scott Wilson 2026-08-14 17:56:44 -04:00
parent 4c49d03732
commit a62798de63
1 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
tests/test_litellm/llms/anthropic/experimental_pass_through/responses_adapters/test_responses_adapters_transformation.py
View file

@ -21,7 +21,10 @@ from litellm.constants import (
from litellm.llms.anthropic.experimental_pass_through.responses_adapters.transformation import (
LiteLLMAnthropicToResponsesAPIAdapter,
)
from litellm.types.llms.anthropic import AnthropicMessagesRequest
from litellm.types.llms.anthropic import (
AllAnthropicToolsValues,
AnthropicMessagesRequest,
)
from litellm.types.llms.openai import ResponseAPIUsage
@ -618,7 +621,7 @@ class TestTranslateToolsToResponsesAPI:
def test_tool_with_optional_properties_stays_non_strict(self):
"""Regression: an unset Anthropic `strict` must not become the Responses strict default,
which would rewrite `required` to include every optional property."""
tools = [
tools: List[AllAnthropicToolsValues] = [
{
"name": "search",
"input_schema": {
@ -633,14 +636,14 @@ class TestTranslateToolsToResponsesAPI:
}
]
result = _ADAPTER.translate_tools_to_responses_api(tools) # type: ignore[arg-type]
result = _ADAPTER.translate_tools_to_responses_api(tools)
assert result[0]["strict"] is False
assert result[0]["parameters"]["required"] == ["query"]
def test_tool_forwards_explicit_strict_true(self):
"""An explicit Anthropic `strict: True` still reaches Responses as True."""
tools = [
tools: List[AllAnthropicToolsValues] = [
{
"name": "search",
"strict": True,
@ -653,7 +656,7 @@ class TestTranslateToolsToResponsesAPI:
}
]
result = _ADAPTER.translate_tools_to_responses_api(tools) # type: ignore[arg-type]
result = _ADAPTER.translate_tools_to_responses_api(tools)
assert result == [
{