Skip to content

fix: expose safe reconciliation rejection reason - #478

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-reconcile-rejection-20260905
Sep 5, 2026
Merged

fix: expose safe reconciliation rejection reason#478
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-reconcile-rejection-20260905

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Scope

  • Classify the existing HTTP 400 scheduler-identity rejection as missing or invalid, without logging any header value.
  • Preserve the current rejection, authentication, broker, report, and execution boundaries. No new schema, dependency, workflow, or hash.

Evidence

  • Three regression cases failed on the unchanged implementation because rejection was silent.
  • Focused request/reconciliation/report/workflow tests: 69 passed. Ruff and diff check passed.
  • Tested with the actual pinned QPK and strategy dependencies in an existing isolated Python 3.12 environment.

Limit

This fixes a diagnostic blind spot only. The production HTTP 400 root cause is not yet proven; no deployment, retry, or trading activation is included in this PR.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 76e548e into main Sep 5, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-reconcile-rejection-20260905 branch September 5, 2026 06:34
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