feat!: sync longbridge/openapi 4.4.0–4.5.0 and remove symbol↔counter_id conversions (#562) - #7
Open
hogan-yuan wants to merge 5 commits into
Conversation
Brings over everything the upstream repo shipped in 4.4.0 through 4.5.0 (brand-renamed longbridge -> longport): - US market APIs (14 interfaces, us_ token + x-dc-region routing) - AI Agent module (AgentContext: workspaces/agents/public_agents, blocking + SSE-streamed conversations, typed events, references, interrupts/interactions) - Paper trading mode, attached orders (TP/SL), client_request_id, all_executions (disabled pending API), OutsideRTH.OptionPreMarket - OrderTag GTC -> Gtc; charge_detail now optional; musl target removed - HTTP error responses preserve status/trace-id/headers/body The 4.3.6/4.3.7 stability fixes previously released here (shared reqwest::Client, reconnect-loop shutdown, Java JNI hardening) are part of the same upstream range and remain unchanged.
…api #562) - Request params counter_id/counter_ids/underlying_counter_id -> symbol/symbols, sending the user-facing symbol as-is; comparison_counter_ids -> comparison_symbols; us_query_orders body counter_ids -> symbols - Responses read the server's symbol field directly; US assets stock entries read full_symbol, crypto entries read symbol; removed utils/counter.rs and the embedded US-ETF/IX/WT directories - DELETE requests send parameters via query string with no body (alert ids, sharelist symbols); alert/sharelist write methods now return ()/void - AlertItem.value_map typed (AlertValueMap); CorpActionLive.status normalized to string; RankCategoriesResponse typed; next_params/executed_timestamp typed; industry_rank/industry_peers expose symbol - Every optional response field now tolerates an explicit JSON null (~700 fields); AI Agent streamed events tolerate null outputs/lists - Java: AlertContext.update wired up; six FundamentalContext JNI bindings implemented - fundamental ratings endpoint (GET /v1/quote/ratings) not yet open upstream
The longport-local Makefile.toml adds a create-cmake-build-dir dependency to every cmake task (this repo's CI runs 'cargo make cmake' on a fresh checkout where cmake.build does not exist). The 4.4.0-4.5.0 sync clobbered it with the upstream Makefile.toml; restore the local version.
…ccepts comparison_symbols The gateway does not yet accept the new comparison_symbols parameter for GET /v1/quote/compare/valuation and answers 500 for any non-empty peer list. As a stopgap, valuation_comparison converts the peer symbols to counter-ids locally and keeps sending the legacy comparison_counter_ids parameter. The public API is unchanged (still takes user symbols) and the response already reads symbol directly. Marked with a TODO to drop the local conversion once the gateway supports comparison_symbols.
…ow accepts it) The gateway now supports the comparison_symbols parameter on GET /v1/quote/compare/valuation as a standard HTTP array (repeated keys), not a JSON array string. Drop the stopgap that converted peer symbols to counter-ids and sent the legacy comparison_counter_ids parameter; pass the user symbols straight through as Option<Vec<String>>, which the qs serializer emits as repeated keys. Removes the symbol_to_counter_id shim. Verified on staging. Note: the gateway fix is on staging first; production still answers this format with code 13 until its rollout lands.
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
1 participant
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.
Two-commit sync bringing this repo to content parity with longbridge/openapi (brand-renamed
longbridge→longport, URLs →longportapp.com; CN URLs, package coords and workspace version untouched).Commit 1 — sync 4.4.0 → 4.5.0
us_token) + x-dc-region routingAgentContext: workspaces / agents / public_agents, blocking + SSE conversations, typed events, references, interrupts) —Config; attached orders (TP/SL);client_request_id;all_executions(disabled pending API);OutsideRTH.OptionPreMarket;OrderTag.GTC→Gtc;charge_detailoptional; musl target removedCommit 2 — counter_id → symbol removal (longbridge/openapi#562)
counter_id/counter_ids/underlying_counter_id→symbol/symbolscarrying the user-facing symbol as-is;comparison_symbols;us_query_ordersbodysymbolssymboldirectly (US assets: stockfull_symbol, cryptosymbol);utils/counter.rs+ embedded ETF/IX/WT directories removed()/voidAlertValueMap,CorpActionLive.status,RankCategoriesResponse,next_params,executed_timestamp;industry_rank/industry_peersexposesymbolnull; agent streamed events toleratenulloutputs/listsAlertContext.updatewired up; sixFundamentalContextJNI bindings implementedVerification
cargo check --workspace --all-featuresandcargo clippy --workspace --all-featurespass after each commit (remaining warnings are identical to upstream main, e.g. the intentionally disabledall_executions);cargo +nightly fmt --allclean🤖 Generated with Claude Code