Skip to content

feat(market): add trade status enum#546

Merged
huacnlee merged 2 commits into
mainfrom
feat/market-trade-status
Jun 16, 2026
Merged

feat(market): add trade status enum#546
huacnlee merged 2 commits into
mainfrom
feat/market-trade-status

Conversation

@huacnlee

Copy link
Copy Markdown
Member

Summary

  • Add longbridge::market::TradeStatus for market status codes with engine-compatible variants, normalization, labels, and names.
  • Type /v1/quote/market-status Rust fields as TradeStatus while preserving integer values for C, Java, Node.js, and Python wrappers.
  • Add the serde_repr workspace dependency and changelog entry.

Test Plan

  • cargo +nightly fmt --all
  • cargo test -p longbridge market_trade_status
  • cargo check --workspace
  • cargo clippy --all --all-features
  • git diff --check

Co-authored-by: Codex <codex@openai.com>
@huacnlee huacnlee merged commit 1dac793 into main Jun 16, 2026
56 checks passed
@huacnlee huacnlee deleted the feat/market-trade-status branch June 16, 2026 08:15
huacnlee added a commit to longbridge/longbridge-mcp that referenced this pull request Jun 16, 2026
## Summary
- Use `longbridge::market::TradeStatus` as the single source for market
status display labels.
- Update market status docs, output schema text, and zh-CN/zh-HK tool
descriptions to match the OpenAPI labels.
- Add coverage for representative status codes including engine
supplemental values.

Depends on longbridge/openapi#546.

## Test Plan
- `cargo test tools::market::tests --config
'patch."https://github.com/longbridge/openapi.git".longbridge.path="/Users/jason/work/openapi/rust"'`
- `cargo clippy --all-features --all-targets --config
'patch."https://github.com/longbridge/openapi.git".longbridge.path="/Users/jason/work/openapi/rust"'`
- `cargo test --config
'patch."https://github.com/longbridge/openapi.git".longbridge.path="/Users/jason/work/openapi/rust"'`
- `git diff --check`

---------

Co-authored-by: Codex <codex@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant