From ff2da724642742c92a583a7a7fc6a3c7dc038661 Mon Sep 17 00:00:00 2001
From: Vasilisa Parshikova <vasilisa.parshikova@sap.com>
Date: Tue, 24 Mar 2026 13:50:27 +0400
Subject: [PATCH] (sap) fix after bot review

---
 litellm/llms/sap/credentials.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/litellm/llms/sap/credentials.py b/litellm/llms/sap/credentials.py
index 2484ec33e0c..4db296b5cf5 100644
--- a/litellm/llms/sap/credentials.py
+++ b/litellm/llms/sap/credentials.py
@@ -233,7 +233,7 @@ def _parse_service_key_once(
 
 def _resolve_credential_from_service_key(
     service_key: Optional[Union[str, dict]], cv: CredentialsValue
-):
+)-> Optional[str]:
     if service_key is None:
         return None
     val = _str_or_none(
@@ -284,7 +284,7 @@ def fetch_credentials(
         Source(
             "service key",
             lambda cv: _resolve_credential_from_service_key(service_key, cv),
-        ),  # type: ignore[arg-type]
+        ),
         Source(
             "environment variables",
             lambda cv: _str_or_none(os.environ.get(f"AICORE_{cv.name.upper()}")),
@@ -332,7 +332,7 @@ def validate_credentials(
     key_str: Optional[str] = None,
     cert_file_path: Optional[str] = None,
     key_file_path: Optional[str] = None,
-):
+)-> None:
     """
         Validate SAP AI Core credentials for completeness and consistency.