Skip to content

fix(events): accept numeric reaction message ids - #98

Open
ShiroKSH wants to merge 1 commit into
MaxApiTeam:dev/2.5.0from
ShiroKSH:fix/reaction-message-id-type
Open

fix(events): accept numeric reaction message ids#98
ShiroKSH wants to merge 1 commit into
MaxApiTeam:dev/2.5.0from
ShiroKSH:fix/reaction-message-id-type

Conversation

@ShiroKSH

@ShiroKSHShiroKSH commented Sep 1, 2026

Copy link
Copy Markdown

Описание

MAX может прислать messageId события NOTIF_MSG_REACTIONS_CHANGED числом. ReactionUpdateEvent принимал только строку, поэтому Pydantic отклонял payload и dispatcher терял событие.

Модель теперь строго принимает int в соответствии с текущим wire-format MAX. Регрессионный тест проводит числовой ID через настоящий dispatcher и проверяет доставленное обработчику событие.

Изменение ограничено разбором payload. Поведение доставки событий со стороны сервера, описанное в #97, не меняется.

Тип изменений

  • Исправление бага
  • Новая функциональность
  • Улучшение документации
  • Рефакторинг

Связанные задачи / Issue

Fixes#96

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

  • uv run pytest tests/dispatch/test_dispatcher.py::test_dispatcher_maps_reaction_update_event — passed
  • uv run ruff format --check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py
  • uv run ruff check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py
  • uv run pre-commit run --files src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py

@ink-developer
ink-developer changed the base branch from main to dev/2.5.0September 1, 2026 17:04
@ink-developer

Copy link
Copy Markdown
Collaborator

Лучше строго int, по моему они полностью на него перешли

@ink-developer

Copy link
Copy Markdown
Collaborator

И так же там по моему лишние мои коммиты. Там должен быть только один ваш коммит

@ShiroKSH
ShiroKSHforce-pushed the fix/reaction-message-id-type branch from 4bff5bc to 68f828fCompareSeptember 1, 2026 17:13
@ShiroKSH

Copy link
Copy Markdown
Author

И так же там по моему лишние мои коммиты. Там должен быть только один ваш коммит

изме

Лучше строго int, по моему они полностью на него перешли

почему "они" если ты вроде как основной разработчик, нет?)

@coderabbitai

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 3c528d21-4284-4f42-be26-1c1a0c7019b7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ink-developer

Copy link
Copy Markdown
Collaborator

почему "они" если ты вроде как основной разработчик, нет?)

"Они" в данном случае разработчики макса. Я говорил про их протокол и апи

@ShiroKSH

Copy link
Copy Markdown
Author

почему "они" если ты вроде как основной разработчик, нет?)

"Они" в данном случае разработчики макса. Я говорил про их протокол и апи

пон

@ink-developer

Copy link
Copy Markdown
Collaborator

Тесты не прошли. Исправьте пожалуйста

@ShiroKSH

Copy link
Copy Markdown
Author

Проверил. Эти падения не внесены PR #98: целевая ветка dev/2.5.0 уже падает на своём текущем коммите c926510 — в её CI-запуске те же 5 ошибок Ruff и те же 6 падений тестов фильтров.

В CI PR новый test_dispatcher_maps_reaction_update_event проходит на Python 3.10–3.14. Локально дополнительно проверил:

  • uv run pytest tests/dispatch/test_dispatcher.py::test_dispatcher_maps_reaction_update_event — passed;
  • uv run ruff format --check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py — passed;
  • uv run ruff check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py — passed.

Добавлять исправления несвязанных фильтров в этот PR не стоит: это расширит его область. Сначала нужно исправить dev/2.5.0, после чего CI PR можно перезапустить.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Баг с реакциями

2 participants

@ShiroKSH@ink-developer