From 47a102446a3a04dde484cad94ca7e5e1cc9ec1e5 Mon Sep 17 00:00:00 2001 From: Krishna Date: Thu, 29 Jan 2026 16:39:01 +0530 Subject: [PATCH 1/3] Update request handling text for clarity --- src/google/adk/models/lite_llm.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/google/adk/models/lite_llm.py b/src/google/adk/models/lite_llm.py index 79182d7b0a..a7b50130ef 100644 --- a/src/google/adk/models/lite_llm.py +++ b/src/google/adk/models/lite_llm.py @@ -490,7 +490,7 @@ def _append_fallback_user_content_if_missing( content.parts = [] content.parts.append( types.Part.from_text( - text="Handle the requests as specified in the System Instruction." + text="Handle the incoming request according to the provided requirements." ) ) return @@ -500,8 +500,7 @@ def _append_fallback_user_content_if_missing( parts=[ types.Part.from_text( text=( - "Handle the requests as specified in the System" - " Instruction." + "Handle the incoming request according to the provided requirements." ) ), ], From c1eea2d39baee301e6901c1ca05869756e50d3d3 Mon Sep 17 00:00:00 2001 From: Krishna Date: Thu, 29 Jan 2026 16:41:24 +0530 Subject: [PATCH 2/3] Update test assertion message content --- tests/unittests/models/test_litellm.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/unittests/models/test_litellm.py b/tests/unittests/models/test_litellm.py index 2ebbc5dfe8..598cb0356b 100644 --- a/tests/unittests/models/test_litellm.py +++ b/tests/unittests/models/test_litellm.py @@ -979,14 +979,14 @@ async def test_generate_content_async_adds_fallback_user_message( ] assert any( message.get("content") - == "Handle the requests as specified in the System Instruction." + == "Handle the incoming request according to the provided requirements." for message in user_messages ) assert ( sum(1 for content in llm_request.contents if content.role == "user") == 1 ) assert llm_request.contents[-1].parts[0].text == ( - "Handle the requests as specified in the System Instruction." + "Handle the incoming request according to the provided requirements." ) From ef7e376ca9f58e8d0482fe1aa1d51e86fb35f656 Mon Sep 17 00:00:00 2001 From: Krishna Date: Thu, 29 Jan 2026 16:54:19 +0530 Subject: [PATCH 3/3] Update comments for finish reason mapping --- src/google/adk/models/lite_llm.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/google/adk/models/lite_llm.py b/src/google/adk/models/lite_llm.py index a7b50130ef..9d7ddc74b7 100644 --- a/src/google/adk/models/lite_llm.py +++ b/src/google/adk/models/lite_llm.py @@ -98,6 +98,9 @@ # 1. FinishReason.TOOL_CALL enum does not exist (as of google-genai 0.8.0) # 2. Tool calls represent normal completion (model stopped to invoke tools) # 3. Gemini native responses use STOP for tool calls (see lite_llm.py:910) +# 4. Mapping tool-related finish reasons to STOP preserves backward +# compatibility with existing ADK consumers that assume STOP +# indicates a successful, non-error completion. _FINISH_REASON_MAPPING = { "length": types.FinishReason.MAX_TOKENS, "stop": types.FinishReason.STOP,