wboayue / rust-ibapi / 25413798864

92%

Build Type

push

github

Committed by web-flow

Commit Message

fix(decoders): proto-aware dispatch + TickTypes::MarketDataType (#508) (#516)

Root cause: TWS sends protobuf-format responses for most incoming
messages on server_version >= 201, but many decoders only ran the
text path. They EOF on field 2 because the protobuf ResponseMessage
carries only [msg_id_str] in fields (payload lives in raw_bytes).

Issue #508 was the visible symptom for TickTypes::decode (msg 58
fell into the catch-all Err(NotImplemented) and killed the
subscription). The same root cause silently broke ~11 other decoders.

Wires message.decode_proto_or_text(proto, text) across:
- TickTypes (+ new MarketDataType variant exposing the active feed)
- MarketDepthExchanges
- accounts: managed_accounts, position{,_multi}, account_value,
  account_portfolio_value, account_summary, account_multi_value,
  pnl{,_single}, family_codes
- contracts: contract_details, contract_descriptions, market_rule,
  option_chain
- historical_data: data, data_end, data_update, schedule, ticks_*,
  histogram_data, head_timestamp
- orders: open_order, order_status, execution_data,
  commission_report, completed_order, next_valid_id
- news: providers, bulletin, historical, article
- scanners: parameters, data

Adds 14 proto decoders for types previously missing a wrapper.
Drops #[allow(dead_code)] from the now-live ones.

Hardens TickTypes::decode catch-all from Err(NotImplemented) to
Err(UnexpectedResponse) so future unknown message types are
skip-classified rather than terminating the subscription.

Verified: live IB Gateway integration suite green except two
pre-existing unrelated failures (US-market-closed guard,
cancel_bracket_order code-202 notice classification).

Closes #508

Coverage Stats

840 of 906 new or added lines in 14 files covered. (92.72%)

5 existing lines in 3 files now uncovered.

17535 of 19928 relevant lines covered (87.99%)

26.52 hits per line