Skip to content

fix: pass protected project configuration to facts publisher - #561

Merged
Pigbibi merged 1 commit into
mainfrom
codex/ibkr-facts-project-env-20261001
Oct 1, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
codex/ibkr-facts-project-env-20261001

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Export the existing protected project Secret under the environment name the account-facts publisher consumes.
  • Preserve the GCP authentication setting and scheduled workflow behavior.
  • Add a regression that constructs an isolated subprocess environment from the actual publisher job configuration, with synthetic secrets and mocked report reads and publication.

Verification

  • Publisher tests: 37 passed.
  • Workflow dependency tests: 3 passed.
  • The new subprocess regression was reproduced before the fix and passed after it; independently rerun by the primary assistant.
  • Ruff, actionlint, Python compilation, and whitespace checks passed.

Real account data, protected values, and broker operations are excluded from the regression. Production publication is validated separately after adoption.

@Pigbibi
Pigbibi merged commit c0a9868 into main Oct 1, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/ibkr-facts-project-env-20261001 branch October 1, 2026 11:00
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