Skip to content

Fix IBKR managed symbols in reports - #29

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-ibkr-report-managed-symbols
Apr 17, 2026
Merged

Fix IBKR managed symbols in reports#29
Pigbibi merged 1 commit into
mainfrom
codex/fix-ibkr-report-managed-symbols

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • prefer configured strategy managed_symbols when building runtime reports
  • avoid falling back to only the safe-haven symbol when ranking_pool is empty
  • add regression coverage for SOXL/SOXX runtime reports

Tests

  • git diff --check
  • .venv/bin/python -m pytest tests/test_request_handling.py::test_execution_report_prefers_configured_managed_symbols_without_ranking_pool tests/test_request_handling.py::test_handle_request_persists_machine_readable_report -q
  • .venv/bin/python -m pytest -q

@Pigbibi
Pigbibi merged commit d262da9 into main Apr 17, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/fix-ibkr-report-managed-symbols branch April 17, 2026 18:31
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