Merge agent/fix-output-bugs: fix mashed lists and tool-call leak

lumen_formatting.py

@@ -152,10 +152,10 @@ def _normalize_bullet_markers(text: str) -> str:
 # 2+ разделителя (т.е. уже 3 пункта: прод 26.09.2026 показал, что ровно
 # трёхпунктовые списки — самый частый случай, а с порогом 3 они не ловились).
 # Короткие "чай • кофе" (1 разделитель) и короткие абзацы — обычная проза, не трогаем.
-# Прод 05.10.2026: модель склеила тот же список без пробелов вокруг "•" —
-# сепаратор ловим regex'ом с необязательными пробелами, плюс "·" тем же путём.
+# Прод 07.10.2026: строки "Telegram: • ..." (~140 символов) не ловились, порог 200→120.
+# Сепаратор с необязательными пробелами ловит склейку без пробелов (прод 05.10.2026).
 _INLINE_BULLETS_MIN_SEPS = 2
-_INLINE_BULLETS_MIN_LEN = 200
+_INLINE_BULLETS_MIN_LEN = 120
 _INLINE_BULLET_SEP_RE = re.compile(r"\s*[•·]\s*")
 _INLINE_BULLET_START_RE = re.compile(r"\s*[•·]")
 

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):
     # Повтор после битого вызова сам упал (а не просто не влез в бюджет) или

tests/test_lumen_formatting.py

@@ -351,9 +351,14 @@ def test_split_inline_bullets_ignores_short_prose():
         "для длины строки и смыслом · третий пункт с подробностями для длины строки и смыслом · четвёртый пункт.",
         4, "Итоги такие: ",
     ),
+    # Прод 07.10.2026: строки платформ (~150 символов) не ловились порогом 200.
+    (
+        "Telegram: • Поищи канал по названию и нажми вступить для проверки длины строки • Часто приватные каналы шлют приглашения в личку",
+        3, "Telegram:",
+    ),
 ])
 def test_split_inline_bullets_splits_prod_answers(text, n_lines, intro):
-    assert len(text) >= 200
+    assert len(text) >= 120
     lines = lumen_formatting._split_inline_bullets(text).split("\n")
     assert len(lines) == n_lines
     assert "" not in lines