Поток подписки недоступен через IXrplClient: события закомментированы, остаётся только публичное поле connection

[Platonenkov]

Наблюдение

IXrplClient позволяет отправить команду подписки, но не позволяет получать поток.

Есть:

• IXrplClient.Subscribe / Unsubscribe — Xrpl/Client/IXrplClient.cs:81,85.

Нет:

• события потока на интерфейсе закомментированы (Xrpl/Client/IXrplClient.cs:483-491):

//public event OnError OnError;
//public event OnConnected OnConnected;
//public event OnDisconnect OnDisconnect;
//public event OnLedgerClosed OnLedgerClosed;
//public event OnTransaction OnTransaction;

Работающие события есть только на Connection (Xrpl/Client/connection.cs:88,90), диспетчеризация — строки 3049 и 3069.

Почему это проблема

Единственный путь к потоку сегодня — публичное свойство на конкретном классе:

// Xrpl/Client/IXrplClient.cs:467
public Connection connection { get; set; }

Это лазейка, а не контракт:

• свойства нет на интерфейсе, поэтому код, работающий через IXrplClient, не может подписаться и не может быть покрыт тестом с подменённым клиентом;
• сеттер публичный — внешний код может подменить соединение целиком у живого клиента;
• имя в нижнем регистре выдаёт незапланированность.

Итог: подписка через SDK формально существует, но пользоваться ей можно только отказавшись от абстракции, которую сам SDK и предлагает.

Ожидаемое

Подписка на поток становится частью контракта IXrplClient: подписаться, получать сообщения, отписаться — не зная о существовании Connection.

Форма на усмотрение реализации — события, IAsyncEnumerable<T>, System.Threading.Channels или колбэки при подписке. Выбор стоит обосновать: он определяет и семантику доставки (см. отдельный issue про выполнение обработчика внутри приёмного цикла).

Свойство connection при этом разумно закрыть или как минимум лишить публичного сеттера — но это уже следствие, а не суть задачи.

[Platonenkov]

Реализовано в #108 (коммиты f0c4456, fa99460), смержено в dev.

Все 16 событий, которые поднимает Connection, объявлены на IXrplClient и проброшены к нему:

IXrplClient client = ...;
client.OnTransaction += r => { /* r.RawTransaction — байты, которые прислал узел */ };

Прежняя форма client.connection.OnX продолжает работать — поверхность добавлена, а не перенесена.

Три вещи из описания issue закрыты:

• подписка через интерфейс — есть;
• тестируемость с подставным клиентом — появилась: FeeTestClient в тестах теперь объявляет события, что и было невозможно, пока они жили только на Connection;
• публичный сеттер connection — убран, свойство стало get-only. С обработчиками, привязанными через события, подмена соединения оставила бы их все на старом объекте, и поток замолчал бы без признаков.

Одно уточнение к тексту issue: свойство connection на интерфейсе уже было (IXrplClient.cs:49) — не хватало именно событий и неизменяемости ссылки.

Реализация — проброс (add => connection.OnX += value), а не ретрансляция: клиент не хранит ни делегатов, ни своего списка подписчиков, ни подписки, которую никто не снимает. Это закреплено тестами, пересекающими поверхности (подписка через клиента → снятие через connection и наоборот), потому что снятие той же поверхностью ретрансляцию не отличает — проверено мутацией.