Retry tool-only Gemini replies instead of leaking tool JSON

lumen_routes.py

@@ -695,12 +695,13 @@ async def _extract_gemini_answer_text(resp: Any, *, model_id: str, call_contents
                              tool_calls.append(f"response:{json.dumps(bot._json_prune_defaults(getattr(fn_resp, 'response', None)), ensure_ascii=False)}")
                          except Exception:
                              tool_calls.append("response")
-        if not ans and tool_calls:
-            ans = "[Tool call: " + "; ".join(tool_calls) + "]"
-        elif not ans and reasons:
+        if not ans and (tool_calls or reasons):
             retry_failed = False
-            if any("MALFORMED_FUNCTION_CALL" in r for r in reasons):
-                # Модель сломала свой вызов инструмента — повторяем без инструментов.
+            if tool_calls or any("MALFORMED_FUNCTION_CALL" in r for r in reasons):
+                # Ответ без текста: битый вызов или голый tool_call — пользователю
+                # JSON инструментов не показываем (прод 07.10.2026), повторяем без них.
+                if tool_calls:
+                    log.warning("[gemini] Model %s returned only tool calls without text, retrying without tools.", model_id)
                 try:
                     retry_gconfig = gconfig.model_copy(update={"tools": None}) if gconfig is not None else None
                     # Async-клиент, а не to_thread: wait_for реально отменяет зависший
@@ -737,7 +738,7 @@ async def _extract_gemini_answer_text(resp: Any, *, model_id: str, call_contents
                     # модель, а не показывать заглушку блокировки как готовый ответ
                     # (иначе цепочка останавливалась на первой же битой модели).
                     retry_failed = True
-            if not ans and not retry_failed:
+            if not ans and not retry_failed and reasons:
                 ans = f"[Ответ заблокирован или пуст. Причина: {', '.join(reasons)}]"
     # Пустая строка без блокировки — не "Empty response": ask_gemini пробует следующую модель.
     return ans.strip()

tests/test_bot_routes.py

@@ -376,6 +376,35 @@ def test_extract_gemini_answer_reports_non_malformed_block_without_retry_state()
     assert "SAFETY" in ans
 
 
+def test_extract_gemini_answer_retries_without_tools_on_tool_only_response():
+    # Прод 07.10.2026: модель вернула только function_call без текста, юзер видел "[Tool call: ...]".
+    from types import SimpleNamespace
+    fn_part = SimpleNamespace(
+        text="",
+        function_call=SimpleNamespace(name="google_search", args={"query": "test"}),
+        function_response=None,
+    )
+    content = SimpleNamespace(parts=[fn_part])
+    resp = _FakeGeminiResponse(text="", candidates=[_FakeCandidate(finish_reason="STOP", content=content)])
+
+    async def retry_ok(*, model, contents, config=None):
+        assert config is None or getattr(config, "tools", None) is None
+        return _FakeGeminiResponse(text="Ответ без инструментов")
+
+    fake_client = MagicMock()
+    fake_client.aio.models.generate_content = retry_ok
+    original_client = bot.client
+    bot.client = fake_client
+    try:
+        ans = asyncio.run(bot._extract_gemini_answer_text(
+            resp, model_id="gemini-3.8-flash", call_contents=[], gconfig=None,
+        ))
+        assert ans == "Ответ без инструментов"
+        assert "Tool call" not in ans
+    finally:
+        bot.client = original_client
+
+
 @pytest.mark.parametrize("retry_outcome", ["raises", "empty"])
 def test_extract_gemini_answer_failed_retry_goes_to_next_model(retry_outcome):
     # Повтор после битого вызова сам упал (а не просто не влез в бюджет) или