(low) Классификация таймаута LLM по имени класса вместо isinstance (хрупко) #1397

New issue

Closed

opened 2026-06-14 18:11:26 +00:00 by bot-backend · 0 comments

bot-backend commented

2026-06-14 18:11:26 +00:00

Collaborator

Описание

В _call_with_retries тип ошибки определяется строковым сравнением имени класса: e.__class__.__name__ == 'LLMTimeoutError' (строка 247). Работает для текущего напрямую поднимаемого LLMTimeoutError, но молча сломается, если появится подкласс LLMTimeoutError (не пройдёт строковое равенство → будет помечен как 'provider_error' вместо 'timeout') или класс переименуют без обновления литерала.

Корень

LLMTimeoutError не импортирован в client.py (строки 29-36 импортируют из .provider только LLMProvider, LLMProviderError, LLMRateLimitedError, OpenAIProvider, ProviderResponse, ToolCall). Строковое сравнение — обходной путь из-за пропущенного импорта, а не намеренный паттерн (класс экспортируется в provider.__all__ и в __init__.py, то есть доступен).

Evidence

client.py:247:

reason = "timeout" if e.__class__.__name__ == "LLMTimeoutError" else "provider_error"

Suggested fix

Добавить импорт LLMTimeoutError из .provider и заменить на проверку типа: reason = "timeout" if isinstance(e, LLMTimeoutError) else "provider_error" — покроет и будущие подклассы.

Severity-обоснование

low: сейчас бескрылый — единственный raise-сайт (provider.py) поднимает ровно LLMTimeoutError (не подкласс), субклассов в кодовой базе нет, поэтому сравнение работает корректно. Последствие при гипотетическом переименовании/подклассировании — неверный reason в LLMResult.fallback (влияет только на логи/метрики, не на доступность сервиса).

week ревью 1 · multi-agent аудит 2026-06-14 · severity=low · category=error-handling

Затронутые файлы:

/Users/anton/Птица/gendesign/backend/app/services/llm/client.py

## Описание В `_call_with_retries` тип ошибки определяется строковым сравнением имени класса: `e.__class__.__name__ == 'LLMTimeoutError'` (строка 247). Работает для текущего напрямую поднимаемого `LLMTimeoutError`, но молча сломается, если появится подкласс `LLMTimeoutError` (не пройдёт строковое равенство → будет помечен как 'provider_error' вместо 'timeout') или класс переименуют без обновления литерала. ## Корень `LLMTimeoutError` не импортирован в `client.py` (строки 29-36 импортируют из `.provider` только `LLMProvider`, `LLMProviderError`, `LLMRateLimitedError`, `OpenAIProvider`, `ProviderResponse`, `ToolCall`). Строковое сравнение — обходной путь из-за пропущенного импорта, а не намеренный паттерн (класс экспортируется в `provider.__all__` и в `__init__.py`, то есть доступен). ## Evidence `client.py:247`: ```python reason = "timeout" if e.__class__.__name__ == "LLMTimeoutError" else "provider_error" ``` ## Suggested fix Добавить импорт `LLMTimeoutError` из `.provider` и заменить на проверку типа: `reason = "timeout" if isinstance(e, LLMTimeoutError) else "provider_error"` — покроет и будущие подклассы. ## Severity-обоснование **low**: сейчас бескрылый — единственный raise-сайт (`provider.py`) поднимает ровно `LLMTimeoutError` (не подкласс), субклассов в кодовой базе нет, поэтому сравнение работает корректно. Последствие при гипотетическом переименовании/подклассировании — неверный `reason` в `LLMResult.fallback` (влияет только на логи/метрики, не на доступность сервиса). --- _week ревью 1 · multi-agent аудит 2026-06-14 · severity=**low** · category=`error-handling`_ Затронутые файлы: - `/Users/anton/Птица/gendesign/backend/app/services/llm/client.py`