Skip to content

fix: opt in one additional daily account facts publisher - #566

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-additional-facts-daily-20261002
Oct 2, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
codex/ibkr-additional-facts-daily-20261002

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Allow the existing daily account-facts publisher to include one additional protected target only when the repository opt-in flag is enabled.
  • Keep the default scheduled target unchanged and manual publication confined to the selected target.
  • Resolve protected target configuration before cloud authentication and prevent one publication failure from canceling the other target.
  • Exercise isolated configuration and publication for all existing additional target choices, including GitHub Actions case-insensitive flag comparison.

Verification

  • 96 focused tests passed.
  • actionlint, Ruff, diff checks, and added-line privacy checks passed.
  • Independent review passed after correcting the flag comparison test semantics.

This change does not create or enable a cloud scheduler, read a broker account, or authorize trading. The additional daily flag remains unset; cloud task activation and natural update verification are separate.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 139ba16 into main Oct 2, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-additional-facts-daily-20261002 branch October 2, 2026 00:15
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