Skip to content

fix: read IBKR reconciliation portfolio once - #421

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-live-continuity-reconcile-v2
Aug 30, 2026
Merged

fix: read IBKR reconciliation portfolio once#421
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-live-continuity-reconcile-v2

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • use the direct market-aware portfolio reader in /reconcile
  • avoid passing currency and cash-mode arguments through a wrapper that already binds them

Validation

  • PYTHONPATH=. PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 uv run --no-sync python -m pytest -q tests/test_broker_reconciliation.py tests/test_runtime_broker_adapters.py tests/test_deployed_runtime_target_admission.py
  • uv run --no-sync ruff check main.py application/broker_reconciliation.py tests/test_broker_reconciliation.py

The failed first probes completed before broker state collection and submitted no orders.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit fe95afe into main Aug 30, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-live-continuity-reconcile-v2 branch August 30, 2026 16:24
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