Skip to content

[codex] Publish Intrade trading condition snapshots - #150

Merged
LimiNode merged 1 commit into
mainfrom
feat/intrade-trading-condition-updates
Sep 3, 2026
Merged

[codex] Publish Intrade trading condition snapshots#150
LimiNode merged 1 commit into
mainfrom
feat/intrade-trading-condition-updates

Conversation

@LimiNode

@LimiNode LimiNode commented Aug 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • add an Intrade Bar TradingConditionManager that publishes supported symbol/option condition snapshots through the existing platform callback
  • refresh market/session, amount, open-trade, and sprint-duration conditions on account events and time changes
  • share one canonical Intrade symbol list across validation, FX websocket support, and condition generation
  • document why Intrade payout remains absent until condition scope can identify amount and duration
  • cover direct platform callbacks and TradingConditionHub routing with Intrade-focused tests

Contract

  • condition snapshots start after account type and currency are known
  • only scopes whose values changed are emitted
  • old account scopes receive tradable=false when account identity changes
  • exact payout remains an AccountInfoRequest query because it depends on concrete trade parameters

Checks

  • intrade_bar_api_response_test.exe --gtest_brief=1: 84 passed
  • trading_condition_hub_test.exe --gtest_brief=1: 8 passed
  • header_only_odr_test.exe --gtest_brief=1: 4 passed
  • trade_manager_test.exe --gtest_brief=1: 21 passed
  • market_data_subscription_contract_test.exe --gtest_brief=1: 21 passed
  • git diff --cached --check

Follow-up

  • On full platform shutdown, publish terminal trading-condition updates with tradable=false before clearing manager state, so a long-lived TradingConditionHub cannot retain stale tradable snapshots; cover this with a regression test.

@LimiNode
LimiNode force-pushed the feat/intrade-trading-condition-updates branch from 6018fa1 to 80682d3 Compare September 3, 2026 00:20
@LimiNode
LimiNode marked this pull request as ready for review September 3, 2026 00:38
@LimiNode
LimiNode merged commit 23c31c6 into main Sep 3, 2026
2 checks passed
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