fix(binance): preserve Unicode and numeric symbol identity - #289
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Chinese Binance symbols could reach the trading page but fail the ASCII-only depth validators, leaving the depth overlay stuck on connecting. Share the Unicode letter/number/underscore contract across the route, depth book, session, native snapshot, and native stream while retaining strict wire casing and complete identifier matching.
Open-order evidence now recognizes Chinese and single-character assets such as
4USDTandWUSDT, preserves complete symbol-cell boundaries and Unicode whitespace, and keeps numeric prefixes distinct. This prevents other contracts from being truncated into the current symbol or disappearing from cancellation scope. Balance parsing and CoinMarketCap multiplier mapping also accept Chinese asset names without stripping numeric-only names.Regenerate Orderbook 2.7.205, Trading Data 1.1.15, CoinMarketCap Data 0.1.17, Strategy27 0.6.3, and Strategy29 0.5.3. Existing install/update URLs are unchanged.
Validation:
git diff --checkpassed.Browser tests use simulated exchange data and test-owned orders; they do not exercise a live trading account.