Skip to content

fix(reporting): isolate account facts ingress diagnostics - #555

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-ingress-diagnostic-20260930
Sep 30, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
codex/ibkr-ingress-diagnostic-20260930

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep account-facts publishing disabled by default; add an explicitly selected ingress-diagnostic mode to the existing heartbeat workflow.
  • Isolate that mode from heartbeat, GCP/WIF, GCS reports, runtime-evidence publishing, brokers and daily digests.
  • Send exactly one fixed empty JSON body to the existing account-facts endpoint with the existing protected token, a 15-second timeout, no redirects and no retries.
  • Report only allowlisted HTTP categories/error codes; only HTTP 400 plus invalid_account_facts_history verifies authentication and rejection before storage. Other outcomes remain unverified/unknown.
  • Preserve the existing live publisher identity guards and 15-minute freshness boundary.

Validation

  • 29 targeted mock tests passed.
  • actionlint and git diff --check passed.
  • Independent incremental review passed.
  • Synthetic receiver validation returned HTTP 400 / invalid_account_facts_history with zero KV writes and zero Durable Object accesses.

Operation scope

User explicitly approved publication/adoption of these three diagnostic files and one empty-body diagnostic execution. No broker sampling, real report replay, funds storage, credential update, Cloud Run deployment or Scheduler change is included. Historical publish_failed root cause remains unknown until further evidence; this diagnostic cannot establish actual funds sync success.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 0155390 into main Sep 30, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-ingress-diagnostic-20260930 branch September 30, 2026 09:07
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