Нет типа вложения share -> ошибка определения вложения OtherAttachmentPayload (файл)

[Pankovea]

Сейчас этот тип преобразовывается как OtherAttachmentPayload (то есть по описанию ожидается документ)

API: https://dev.max.ru/docs-api/objects/Message

type
string
payload
object ShareAttachmentPayload
Полезная нагрузка запроса ShareAttachmentRequest

title
string Nullable optional
Заголовок предпросмотра ссылки.

description
string Nullable optional
Описание предпросмотра ссылки

image_url
string Nullable optional
Изображение предпросмотра ссылки

Сырой словарь:

[
{'type': 'share',
'payload': {
    'url': 'https://max.ru/c/-73455...123/AZ2I....bkQ',
    'token': 'f9LHodD0cOLlDys2qELqwWmVgf8_..._o7MdDpRWRzvS8fXmecN2YHfWHu'},
    'title': 'MAX – быстрое и легкое приложение для общения и решения повседневных задач',
    'description': 'MAX позволяет отправлять любые виды сообщений и звонить даже на слабых устройствах и при низкой скорости интернета.',
    'image_url': 'https://i.oneme.ru/i?r=BTGBPUwtwgYUeoFhO7rESmr8PkKUFrFQzWAxDtP-JVoTo5BwxbPWZMl-nt3whnrS81A'}
]

На выходе получился объект OtherAttachmentPayload с двумя полями url и token по которым кажется что пришёл документ(файл)

[bish-x]

Разобрал сценарий на maxapi==0.9.18. Воспроизведение даёт не совсем то, что в описании:

from maxapi.types.message import MessageBody

body = MessageBody.model_validate({
    "mid": "mid.test", "seq": 1,
    "attachments": [{
        "type": "share",
        "payload": {"url": "https://max.ru/c/...", "token": "..."},
        "title": "MAX", "description": "desc",
        "image_url": "https://i.oneme.ru/i?r=..."
    }]
})
a = body.attachments[0]
print(type(a).__name__)           # Share    ← правильно
print(a.title, a.description, a.image_url)   # всё на месте
print(type(a.payload).__name__)   # OtherAttachmentPayload  ← вот это путает

Само вложение корректно определяется как Share (не File/что-то ещё), поля title / description / image_url доступны на уровне a — дискриминатор Annotated[..., Field(discriminator="type")] в maxapi/types/attachments/__init__.py работает.

Что действительно misleading — это a.payload: оно типизировано как OtherAttachmentPayload, потому что в Attachment.payload-union (maxapi/types/attachments/attachment.py:95-111) нет ShareAttachmentPayload, и {url, token} попадает в OtherAttachmentPayload, который по docstring'у «для файлов». Отсюда и ощущение «пришёл документ».

Предложение по фиксу:

1. Добавить в attachment.py:

   class ShareAttachmentPayload(BaseModel):
       """Payload для share-вложения: ссылка + токен."""
       url: str
       token: str

2. Внести в Attachment.payload-union перед OtherAttachmentPayload (OtherAttachmentPayload — суперсет по полям, порядок важен).
3. В maxapi/types/attachments/share.py переопределить payload: ShareAttachmentPayload | None = None, чтобы у Share.payload был строгий тип.
4. Регрессионный тест в tests/test_serialization_models.py.

Если верно понимаю ваш сценарий — готов прислать PR. Подтвердите, пожалуйста: проблема в том, что attachment.payload отдаётся как OtherAttachmentPayload, а не в определении самого attachment (он корректно Share)?

[Pankovea]

Да, я для определения типа вложения пользовался

        match att.payload:
            case PhotoAttachmentPayload(): ...
            case OtherAttachmentPayload(): ...
            case ButtonsPayload(): ...

OtherAttachmentPayload
Я пока для проверки, что это точно файл сделал так

if att.payload.url.startswith('https://fd.oneme.ru'): ...

Ну то есть надо добавить класс ShareAttachmentPayload в Attachment.payload-union (maxapi/types/attachments/attachment.py:95-111)

[bish-x]

@Pankovea спасибо, подтверждение получил. Открыл PR: #109.

Небольшая корректировка к моему первому фикс-плану (выяснилось при проверке):

• Добавить ShareAttachmentPayload в базовый Attachment.payload-union перед OtherAttachmentPayload нельзя — это перехватит payload у File / Audio / Video (там тоже {url, token}), то есть будет тот же superset-баг, только наоборот.
• Правильный минимальный фикс — переопределить payload именно в классе Share (payload: ShareAttachmentPayload | None = None). Base union не трогаем (в PR ShareAttachmentPayload всё же добавлен в конец union — чисто чтобы явное Attachment(type=SHARE, payload=ShareAttachmentPayload(...)) было типобезопасно, на поведение dict-десериализации это не влияет).

После мерджа вместо временного att.payload.url.startswith('https://fd.oneme.ru') можно будет использовать:

from maxapi.types import ShareAttachmentPayload

match att.payload:
    case ShareAttachmentPayload():
        ...
    case OtherAttachmentPayload():  # файл
        ...

Проверено на чистой установке maxapi==0.9.18: до фикса payload: OtherAttachmentPayload, после — ShareAttachmentPayload. File / Audio / Video в регрессионном тесте — без изменений.

[Pankovea]

Спасибо