fix: remove double Option wrapping for propertyKeys

[CertainLach]

Polkadot.js неверно понимает что от него требуется в этом случае. Хоть RPC и работает через JSON, параметры кодируются через модель данных scale, а соответственно Option<Option<T>> не то же самое что и Option<T>, как это происходит с T?? vs T? в typescript

[CertainLach]

Возможно имеет смысл в релизнутые типы бекпортнуть, чтобы пользователям проблем не доставлять? В polkadot apps это точно вызывает непонятную ошибку, в js вроде как всё работает, но не факт

[CertainLach]

Этот фикс правильный, однако Даниил говорит что с ним у веб команды какие-то проблемы, нужно разбираться.

[CertainLach]

Разобрались, isOptional ставит пустое значение для типа, т.е пустой массив
С Даниилом обсудили - нужно трактовать пустой список полей как получение всех полей, потому что другие варианты в подобной модели данных выглядят плохо, а текущий код вовсе несовместим с некоторыми сценариями применения polkadot.js
@uandysmith последнее слово за тобой, требуюмое изменение в ноде - однострочник

[uandysmith]

Я пока не совсем понимаю контекст, возможно стоит голосом обсудить завтра.
Этот код, где это исправление, мы сами пишем? Его yarn polkadotTypes не затрет потом? Бекпортировать можно, просто затагаем как 942.57.1, у нас же можно патчи для типов.

[CertainLach]

Сами пишем, бекпорт не выйдет, нужно в ноде чуть поведение поменять.
Завтра обговорим.