scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-17 04:34:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 172

Merge pull request #38644 from BerriAI/litellm_techdebt_20260828

Browse source 
chore(techdebt): type new signatures and drop slop comments from the last 24h
This commit is contained in:
Mateo Wang 2026-08-28 12:19:38 -07:00 • committed by GitHub
commit 936e07b0a5
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
7 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
basedpyright-code-budget.json
View file

@ -57,7 +57,7 @@
"limit": 5659 "limit": 5659
}, },
"reportMissingTypeArgument": { "reportMissingTypeArgument": {
"limit": 15484 "limit": 15482
}, },
"reportMissingTypeStubs": { "reportMissingTypeStubs": {
"limit": 40 "limit": 40
@ -105,13 +105,13 @@
"limit": 109 "limit": 109
}, },
"reportUnknownMemberType": { "reportUnknownMemberType": {
"limit": 38782 "limit": 38779
}, },
"reportUnknownParameterType": { "reportUnknownParameterType": {
"limit": 19829 "limit": 19827
}, },
"reportUnknownVariableType": { "reportUnknownVariableType": {
"limit": 30349 "limit": 30348
}, },
"reportUnnecessaryCast": { "reportUnnecessaryCast": {
"limit": 117 "limit": 117

2
litellm-proxy-extras/litellm_proxy_extras/utils.py
View file

@ -470,7 +470,7 @@ class ProxyExtrasDBManager:
ProxyExtrasDBManager._mark_migrations_applied(migrations_dir) ProxyExtrasDBManager._mark_migrations_applied(migrations_dir)
@staticmethod @staticmethod
def _mark_migrations_applied(migrations_dir: str): def _mark_migrations_applied(migrations_dir: str) -> None:
migration_names = ProxyExtrasDBManager._get_migration_names(migrations_dir) migration_names = ProxyExtrasDBManager._get_migration_names(migrations_dir)
logger.info(f"Resolving {len(migration_names)} migrations") logger.info(f"Resolving {len(migration_names)} migrations")
for migration_name in migration_names: for migration_name in migration_names:

2
litellm/integrations/SlackAlerting/slack_alerting.py
View file

@ -1447,10 +1447,8 @@ Model Info:
from datetime import datetime from datetime import datetime
# Get the current timestamp
current_time: Final = datetime.now().strftime("%H:%M:%S") current_time: Final = datetime.now().strftime("%H:%M:%S")
_proxy_base_url: Final = os.getenv("PROXY_BASE_URL", None) _proxy_base_url: Final = os.getenv("PROXY_BASE_URL", None)
# Use .name if it's an enum, otherwise use as is
alert_type_name: Final = getattr(alert_type, "name", alert_type) alert_type_name: Final = getattr(alert_type, "name", alert_type)
alert_type_formatted: Final = f"Alert type: `{alert_type_name}`" alert_type_formatted: Final = f"Alert type: `{alert_type_name}`"
if alert_type == "daily_reports" or alert_type == "new_model_added": if alert_type == "daily_reports" or alert_type == "new_model_added":

4
litellm/litellm_core_utils/streaming_handler.py
View file

@ -819,7 +819,9 @@ class CustomStreamWrapper:
except Exception as e: except Exception as e:
raise e raise e
def model_response_creator(self, chunk: dict | None = None, hidden_params: Mapping[str, object] | None = None): def model_response_creator(
self, chunk: dict | None = None, hidden_params: Mapping[str, object] | None = None
) -> ModelResponseStream:
_model: Final = self._cached_model_name _model: Final = self._cached_model_name
_logging_obj_llm_provider: Final = self._cached_logging_llm_provider _logging_obj_llm_provider: Final = self._cached_logging_llm_provider

1
litellm/proxy/_lazy_openapi_snapshot.py
View file

@ -126,7 +126,6 @@ def _feature_fragment(app: "FastAPI", feat: "LazyFeature", used_operation_ids: s
full: Final = get_openapi(title=app.title, version=app.version, routes=feat_routes) full: Final = get_openapi(title=app.title, version=app.version, routes=feat_routes)
paths: Final = full.get("paths", {}) paths: Final = full.get("paths", {})
_normalize_operation_ids(paths) _normalize_operation_ids(paths)
# Group all of a feature's routes under one tag.
for path_ops in paths.values(): for path_ops in paths.values():
for method, op in path_ops.items(): for method, op in path_ops.items():
if isinstance(op, dict): if isinstance(op, dict):

4
litellm/proxy/guardrails/guardrail_hooks/crowdstrike_aidr/crowdstrike_aidr.py
View file

@ -385,7 +385,7 @@ class CrowdStrikeAIDRHandler(CustomGuardrail):
return [_extract_text_from_message(msg) for msg in tail] return [_extract_text_from_message(msg) for msg in tail]
async def _call_or_fail_open( async def _call_or_fail_open(
self, payload: dict[str, Any], hook_name: str, request_data: dict self, payload: dict[str, Any], hook_name: str, request_data: dict[str, object]
) -> _GuardChatCompletionsResult: ) -> _GuardChatCompletionsResult:
start_time: Final = time.time() start_time: Final = time.time()
try: try:
@ -421,7 +421,7 @@ class CrowdStrikeAIDRHandler(CustomGuardrail):
structured_messages: list[AllMessageValues], structured_messages: list[AllMessageValues],
guard_output: _GuardInput, guard_output: _GuardInput,
sent_indices: tuple[int, ...], sent_indices: tuple[int, ...],
request_data: dict, request_data: dict[str, object],
) -> list[AllMessageValues] | None: ) -> list[AllMessageValues] | None:
if effective_skip_system_message_for_guardrail(self) or effective_skip_tool_message_for_guardrail(self): if effective_skip_system_message_for_guardrail(self) or effective_skip_tool_message_for_guardrail(self):
request_messages: Final = request_data.get("messages") request_messages: Final = request_data.get("messages")

2
ruff-strict-budget.json
View file

@ -9,7 +9,7 @@
"limit": 827 "limit": 827
}, },
"ANN201": { "ANN201": {
"limit": 2012 "limit": 2011
}, },
"ANN202": { "ANN202": {
"limit": 847 "limit": 847