fix(events): accept numeric reaction message ids - #98
Conversation
|
Лучше строго |
|
И так же там по моему лишние мои коммиты. Там должен быть только один ваш коммит |
4bff5bc to
68f828f
Compare
изме
почему "они" если ты вроде как основной разработчик, нет?) |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
"Они" в данном случае разработчики макса. Я говорил про их протокол и апи |
пон |
|
Тесты не прошли. Исправьте пожалуйста |
|
Проверил. Эти падения не внесены PR #98: целевая ветка В CI PR новый
Добавлять исправления несвязанных фильтров в этот PR не стоит: это расширит его область. Сначала нужно исправить |
Описание
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— passeduv run ruff format --check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.pyuv run ruff check src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.pyuv run pre-commit run --files src/pymax/types/events/reaction.py tests/dispatch/test_dispatcher.py