fix(messages): classify 2188 as a data advisory, not an error

[ms32035]

Fixes #764.

TWS sends 2188 (Up-to-the-second historical data requires additional subscription for the API.) on the error callback and then serves the historical data anyway — the account merely cannot receive the up-to-the-second tail.

Because 2188 sits outside WARNING_CODE_RANGE (2100..=2169) and was not in DATA_ADVISORY_CODES, classify_error routed it as RoutedItem::Error. historical_data() inspects only the first frame on its subscription, so it returned Err before HistoricalData arrived and the caller got no bars at all for every instrument whose real-time feed the account does not hold.

Change

Add 2188 to DATA_ADVISORY_CODES, whose doc comment already describes this case: an advisory on a request that then proceeds. RoutedItem::Notice is skipped by into_legacy(), so the fetch reads the following HistoricalData frame and returns the bars. Both the sync and async paths go through the same classifier, so the one constant covers both.

Tests

Regression tests on both fetch paths push a 2188 frame ahead of HistoricalData and assert the bars still arrive. Both fail on main without the classification change:

test market_data::historical::sync::tests::test_historical_data_up_to_the_second_advisory_is_skipped ... FAILED
test market_data::historical::async::tests::test_historical_data_up_to_the_second_advisory_is_skipped ... FAILED

Verified against a live gateway: a bulk daily-bar gather over a US equity universe on an account without NYSE Network A returned no bars for the affected symbols before the change, and the full history after it.

Checks

cargo fmt --check
cargo test                                    # async
cargo test --no-default-features --features sync
cargo test --all-features
cargo clippy --all-features --all-targets

All clean.

[wboayue]

Thanks @ms32035 — excellent diagnosis in #764, and the live before/after verification plus regression tests on both fetch paths made this easy to confirm. I rebased onto main (resolving the overlap with #759), added a changelog entry, and updated the one remaining doc site (Notice::is_data_advisory) to include 2188. Merging.