Skip to content

Add IBKR SOXL plugin support#53

Merged
Pigbibi merged 1 commit into
mainfrom
soxl-20d-vol-plugin-support
May 10, 2026
Merged

Add IBKR SOXL plugin support#53
Pigbibi merged 1 commit into
mainfrom
soxl-20d-vol-plugin-support

Conversation

@Pigbibi
Copy link
Copy Markdown
Collaborator

@Pigbibi Pigbibi commented May 10, 2026

Summary

  • update QuantPlatformKit and UsEquityStrategies pins to the merged SOXX 20d volatility gate versions
  • add IBKR strategy plugin mount support and render plugin lines in existing Telegram notifications
  • sync IBKR_STRATEGY_PLUGIN_MOUNTS_JSON through the Cloud Run env workflow

Architecture

  • platform code only passes runtime inputs/plugin signals and notifications
  • SOXL allocation and volatility gate logic remains in UsEquityStrategies; shared indicator construction remains in QuantPlatformKit

Tests

  • .venv/bin/python -m pytest tests/test_runtime_strategy_adapters.py tests/test_runtime_composer.py tests/test_runtime_config_support.py::test_load_platform_runtime_settings_uses_minimal_group_config tests/test_runtime_config_support.py::test_load_platform_runtime_settings_reads_ibkr_strategy_plugin_mounts tests/test_notifications.py::test_build_translator_supports_chinese -q
  • bash tests/test_sync_cloud_run_env_workflow.sh
  • .venv/bin/python -m ruff check application/runtime_strategy_adapters.py application/runtime_composer.py application/runtime_dependencies.py application/rebalance_service.py notifications/renderers.py notifications/telegram.py main.py runtime_config_support.py tests/test_runtime_strategy_adapters.py tests/test_runtime_composer.py tests/test_runtime_config_support.py tests/test_notifications.py

@Pigbibi Pigbibi force-pushed the soxl-20d-vol-plugin-support branch from a9a9588 to 06e1295 Compare May 10, 2026 13:19
@Pigbibi Pigbibi merged commit 95ea3eb into main May 10, 2026
1 check passed
@Pigbibi Pigbibi deleted the soxl-20d-vol-plugin-support branch May 10, 2026 13:20
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a9a9588085

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread main.py

log_context = build_request_log_context()
report = build_execution_report(log_context)
strategy_plugin_signals, strategy_plugin_error = load_strategy_plugin_signals()
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Defer plugin signal loading until execution is allowed

handle_request now loads strategy plugin signals before both the run lock check and the market-open check, so overlapping or market-closed POSTs still execute plugin loading work. When IBKR_STRATEGY_PLUGIN_MOUNTS_JSON points to remote-backed plugin signals (for example GCS-based signal paths), skipped requests can still incur unnecessary I/O/latency and emit plugin-load diagnostics even though no strategy cycle will run. Move plugin loading inside the branch that has acquired the lock and passed the market-open gate.

Useful? React with 👍 / 👎.

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