fix: добавлен retry при edit для ошибки attachment.file.not.processed

[2-3savage]

Проблема

При использовании EditMessage с предварительно загруженным через upload_media файлом (AttachmentUpload), API MAX иногда возвращает HTTP 200 с телом:

{
  "success": false,
  "message": "Key: errors.process.attachment.file.not.processed"
}

Ожидаемое поведение — возврат HTTP 400 с кодом attachment.not.ready, который должен быть перехвачен MaxApiError и отправлен в retry-механизм с задержкой 2 секунды.

Однако фактически:

• Retry-механизм не срабатывает, так как отсутствует ожидаемый код ошибки
• raw не содержит поле code с attachment.not.ready, только сообщение Key: errors.process.attachment.file.not.processed

Важно отметить, что данная проблема воспроизводится только в EditMessage. В SendMessage API корректно возвращает HTTP 400 с правильным кодом ошибки.

Фикс

Добавлена обработка ошибки attachment.file.not.processed в BaseConnection.request()

Теперь:

Ответ с success=False перехватывается

Сообщение attachment.file.not.processed маппится в код attachment.not.ready

Выбрасывается MaxApiError с корректным кодом

Retry-механизм в EditMessage.fetch() перехватывает ошибку и делает повторную попытку через 2 секунды и т.д.

TODO

Удалить костыль, когда MAX API начнёт возвращать корректный HTTP-статус для этой ошибки

[2-3savage]

Всем привет!

Хотелось бы побыстрее увидеть в проде, т.к. буквально сегодня с этим столкнулся и очень нужен фикс 😊

Либо можно написать разрабам апишки и показать им, что выдает 200 при ошибке.

[Olegt0rr]

Висячие таски не надо делать, посмотри как в других местах реализован сбор

[2-3savage]

Привет! Спасибо за ревью.

Я посмотрел, как это реализовано в других местах проекта, например, при обработке ошибок if not response.ok. Там используется точно такой же паттерн с asyncio.create_task.

if not response.ok:
    raw = await response.json()
    if bot.dispatcher:
        asyncio.create_task(
            bot.dispatcher.handle_raw_response(
                UpdateType.RAW_API_RESPONSE, raw
            )
        )
    raise MaxApiError(code=response.status, raw=raw)

Мне удалить asyncio.create_task и сделать асинхронный вызов await bot.dispatcher.handle_raw_response(...), или я что-то упускаю?

[Copilot]

Pull request overview

Normalizes a malformed HTTP 200 attachment error so EditMessage can trigger its existing retry mechanism.

Changes:

• Detects unsuccessful attachment-processing responses.
• Maps them to attachment.not.ready.
• Raises MaxApiError and dispatches the raw response.

Suppressed comments (1)

maxapi/connection/base.py:235

• MaxApiError.raw normally contains the complete API response (see the HTTP-error path above), but this replacement drops success, message, and any future diagnostic fields. Preserve the original body while adding the normalized code so callers and logs can still inspect the actual server error.

                raise MaxApiError(code=400, raw={"code": "attachment.not.ready"})

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Olegt0rr]

Сработать могло на "attachment.file.not.processed", а ты в ошибку пишешь "attachment.not.ready". Лучше сохранять оригинальное сообщение - иначе это может ввести в заблуждение