fix: исправлена сериализация format и text в SendMessage/EditMessage by bish-x · Pull Request #91 · love-apples/maxapi

bish-x · 2026-04-08T12:50:06Z

Описание

1. `format.value` → `format` в SendMessage и EditMessage

Файлы: methods/send_message.py, methods/edit_message.py

self.format.value вызывал AttributeError при передаче строки вместо TextFormat enum. Поскольку TextFormat наследует StrEnum, enum-значение можно использовать напрямую — оно сериализуется как строка автоматически.

Связано с #89 — данный PR расширяет фикс на edit_message.py, который не был затронут в PR #89.

2. `text: null` больше не отправляется безусловно

Файл: methods/send_message.py

Ранее json["text"] = self.text выполнялось безусловно, даже при text=None. В edit_message.py аналогичный код был обёрнут в if self.text is not None:. Приведено к единообразию — при отправке сообщения только с вложениями text не включается в тело запроса.

3. Устранён двойной `DeprecationWarning` для `parse_mode`

Файлы: bot.py

При вызове bot.send_message(parse_mode=X) предупреждение выводилось дважды: в resolve_format() и в SendMessage.__init__(). Причина — parse_mode передавался в SendMessage после того как format уже был resolved. Теперь передаётся parse_mode=None.

Тестирование

Все существующие тесты проходят
ruff check / ruff format — без замечаний

- format.value → format: убран .value, так как TextFormat наследует StrEnum и сериализуется как строка автоматически (send_message.py, edit_message.py) - text: null не отправляется когда text is None (send_message.py) - устранён двойной DeprecationWarning для parse_mode в bot.send_message и bot.edit_message — передаётся parse_mode=None после resolve_format

Olegt0rr · 2026-04-08T15:02:18Z

            json["notify"] = self.notify
        if self.format is not None:
-            json["format"] = self.format.value
+            json["format"] = self.format


Замечание такое же, как к упомянутому ранее PR: нужны тесты с прямой передачей текста в качестве значения (не enum, а именно текста)

Регрессионные тесты для SendMessage и EditMessage, проверяющие что format="html" (строка) корректно сериализуется в JSON без AttributeError. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

PR направлен на исправление сериализации параметров форматирования/текста при отправке и редактировании сообщений, а также на устранение дублирующего DeprecationWarning при использовании parse_mode.

Changes:

SendMessage/EditMessage: format сериализуется без .value, text=None больше не отправляется в JSON.
Bot.send_message/Bot.edit_message: устранено двойное предупреждение для parse_mode за счёт передачи parse_mode=None в методы после resolve_format.
Дополнительно внесены изменения в обработку update/context и логирование (не отражены в описании PR).

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
tests/test_format_parameter.py	Добавлены регрессионные тесты на поддержку строкового `format`.
maxapi/methods/send_message.py	Условное добавление `text` в JSON и сериализация `format` без `.value`.
maxapi/methods/edit_message.py	Сериализация `format` без `.value`.
maxapi/bot.py	Изменена прокидка `parse_mode` (для предупреждений), добавлен `__repr__`.
maxapi/types/updates/user_added.py	`get_ids()` теперь возвращает user_id добавленного пользователя.
maxapi/types/updates/user_removed.py	`get_ids()` теперь возвращает user_id удалённого пользователя.
maxapi/utils/updates.py	Добавлен logger (пока не используется).
maxapi/dispatcher.py	Добавлено debug-логирование при LRU-вытеснении контекстов.
maxapi/context/context.py	`MemoryContext.get_data()` возвращает копию данных контекста.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-09T11:17:31Z

 from __future__ import annotations

+import logging
 from typing import TYPE_CHECKING


В модуле добавлен logger = logging.getLogger(__name__), но он нигде не используется. Лучше убрать его (и импорт logging), либо начать реально использовать для логирования внутри enrich_event()/_resolve_*, чтобы не оставлять мёртвый код.

Copilot · 2026-04-09T11:17:32Z

                пользователя.
        """

-        return self.chat_id, self.admin_id
+        return self.chat_id, self.user.user_id


Изменение get_ids() на (chat_id, removed_user_id) меняет ключ контекста в Dispatcher.handle() для события USER_REMOVED. При этом в maxapi/utils/updates.py::_resolve_from_user() для UserRemoved отправителем считается admin_id (если он есть), и раньше get_ids() был согласован с этим. Сейчас from_user (админ) и context.user_id (удалённый пользователь) будут расходиться, что может ломать state/context-логику обработчиков. Нужна единая семантика: либо возвращать admin_id когда он задан, либо обновить _resolve_from_user()/документацию так, чтобы from_user и get_ids() соответствовали одному и тому же пользователю.

Copilot · 2026-04-09T11:17:32Z

+        if self.text is not None:
+            json["text"] = self.text


Поведение изменилось: при text=None поле text больше не включается в JSON (и это важно для отправки «только вложения»). В текущих тестах файла уже проверяется сериализация format, но нет теста на то, что ключ text отсутствует при text=None. Стоит добавить регрессионный тест, чтобы это поведение не сломалось в будущем.

Copilot · 2026-04-09T11:17:32Z

+    def __repr__(self) -> str:
+        return "Bot(token='***')"
+


Описание PR сфокусировано на сериализации format/text и устранении двойного DeprecationWarning, но в diff также есть несвязанные изменения (например, Bot.__repr__, копирование данных контекста, логирование LRU-вытеснения, изменение get_ids() у update-моделей). Лучше отразить эти изменения в описании или вынести их в отдельный PR, чтобы ревью и релиз-ноты были прозрачнее.

- str(self.format) handles plain string without AttributeError - text=None no longer included in JSON body (SendMessage) - removed unused logger import from utils/updates.py - added tests: format as string, text=None regression Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

codecov · 2026-04-09T11:52:57Z

Codecov Report

❌ Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
maxapi/methods/edit_message.py	83.33%	1 Missing ⚠️
maxapi/methods/send_message.py	85.71%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

…t_ids Covers 3 lines missing from Codecov patch coverage report on PR love-apples#91: - Bot.__repr__ — verifies token is masked as '***' - UserAdded.get_ids() — verifies returns (chat_id, user.user_id) - UserRemoved.get_ids() — verifies returns (chat_id, user.user_id) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Возвращён self.format.value в send_message и edit_message (как на main) - В __init__ добавлена конвертация строки в TextFormat/ParseMode enum для обратной совместимости — пользователь может передать "html" вместо TextFormat.HTML - Добавлен ClassVar[dict] для class-level _USER в test_types.py (RUF012) - Обновлены тесты под новое поведение

Olegt0rr requested changes Apr 8, 2026

View reviewed changes

Olegt0rr requested a review from Copilot April 9, 2026 11:10

Copilot started reviewing on behalf of Olegt0rr April 9, 2026 11:11 View session

Copilot AI reviewed Apr 9, 2026

View reviewed changes

bish-x and others added 2 commits April 9, 2026 19:06

love-apples merged commit 0e8fefe into love-apples:main Apr 11, 2026
13 checks passed

bish-x mentioned this pull request Apr 12, 2026

Update send_message.py #89

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: исправлена сериализация format и text в SendMessage/EditMessage#91

fix: исправлена сериализация format и text в SendMessage/EditMessage#91
love-apples merged 5 commits intolove-apples:mainfrom
bish-x:fix/message-format-fixes

bish-x commented Apr 8, 2026

Uh oh!

Olegt0rr Apr 8, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Uh oh!

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

Copilot AI Apr 9, 2026

Uh oh!

codecov Bot commented Apr 9, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

bish-x commented Apr 8, 2026

Описание

1. format.value → format в SendMessage и EditMessage

2. text: null больше не отправляется безусловно

3. Устранён двойной DeprecationWarning для parse_mode

Тестирование

Uh oh!

Olegt0rr Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

Uh oh!

codecov Bot commented Apr 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

1. `format.value` → `format` в SendMessage и EditMessage

2. `text: null` больше не отправляется безусловно

3. Устранён двойной `DeprecationWarning` для `parse_mode`

codecov Bot commented Apr 9, 2026 •

edited

Loading