mirror of
https://github.com/tiennm99/litellm.git
synced 2026-08-23 08:24:15 +00:00
fix(mypy): fix llm_http_handler.py type issues (6 errors)
- Cast files param to Dict[str, Any] in multipart upload path - Add assert provider_config is not None for realtime handlers - Annotate params dict as Dict[str, Any] for vector store list handlers - Annotate request_body as Dict[str, Any] for vector store update handlers Co-authored-by: yuneng-jiang <yuneng-jiang@users.noreply.github.com>
This commit is contained in:
co-authored by
yuneng-jiang
parent
2b5be5c1ab
commit
b3a30a15c4
@@ -3036,10 +3036,11 @@ class BaseLLMHTTPHandler:
|
||||
elif isinstance(transformed_request, dict) and "file" in transformed_request:
|
||||
# Handle multipart form-data uploads (e.g., Anthropic Files API)
|
||||
# The dict contains tuples suitable for httpx's `files` parameter
|
||||
file_request = cast(Dict[str, Any], transformed_request)
|
||||
upload_response = sync_httpx_client.post(
|
||||
url=api_base,
|
||||
headers=headers,
|
||||
files=transformed_request,
|
||||
files=file_request,
|
||||
timeout=timeout,
|
||||
)
|
||||
else:
|
||||
@@ -4870,6 +4871,7 @@ class BaseLLMHTTPHandler:
|
||||
timeout=timeout,
|
||||
)
|
||||
except Exception as e:
|
||||
assert provider_config is not None
|
||||
raise self._handle_error(
|
||||
e=e,
|
||||
provider_config=provider_config,
|
||||
@@ -4954,6 +4956,7 @@ class BaseLLMHTTPHandler:
|
||||
timeout=timeout,
|
||||
)
|
||||
except Exception as e:
|
||||
assert provider_config is not None
|
||||
raise self._handle_error(
|
||||
e=e,
|
||||
provider_config=provider_config,
|
||||
@@ -8026,7 +8029,7 @@ class BaseLLMHTTPHandler:
|
||||
|
||||
url = api_base
|
||||
|
||||
params = {}
|
||||
params: Dict[str, Any] = {}
|
||||
if after is not None:
|
||||
params["after"] = after
|
||||
if before is not None:
|
||||
@@ -8108,7 +8111,7 @@ class BaseLLMHTTPHandler:
|
||||
|
||||
url = api_base
|
||||
|
||||
params = {}
|
||||
params: Dict[str, Any] = {}
|
||||
if after is not None:
|
||||
params["after"] = after
|
||||
if before is not None:
|
||||
@@ -8170,7 +8173,7 @@ class BaseLLMHTTPHandler:
|
||||
|
||||
url = f"{api_base}/{vector_store_id}"
|
||||
|
||||
request_body = dict(vector_store_update_optional_params)
|
||||
request_body: Dict[str, Any] = dict(vector_store_update_optional_params)
|
||||
|
||||
# Clean metadata to only include string values (OpenAI requirement)
|
||||
if "metadata" in request_body and request_body["metadata"] is not None:
|
||||
@@ -8253,7 +8256,7 @@ class BaseLLMHTTPHandler:
|
||||
|
||||
url = f"{api_base}/{vector_store_id}"
|
||||
|
||||
request_body = dict(vector_store_update_optional_params)
|
||||
request_body: Dict[str, Any] = dict(vector_store_update_optional_params)
|
||||
|
||||
# Clean metadata to only include string values (OpenAI requirement)
|
||||
if "metadata" in request_body and request_body["metadata"] is not None:
|
||||
|
||||
Reference in New Issue
Block a user