Add live-gated default option overlay configs#154
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7c28aa305e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| self.assertEqual(decision.diagnostics["option_growth_overlay_start_usd"], 250000.0) | ||
| self.assertIs(decision.diagnostics["option_growth_overlay_active"], False) | ||
| self.assertEqual(decision.diagnostics["option_growth_overlay_skip_reason"], "below_start_usd") | ||
| self.assertNotIn("option_growth_overlay_enabled", decision.diagnostics) |
There was a problem hiding this comment.
Update stale option-overlay entrypoint assertions
With this commit the entrypoint manifests now include **option_overlay_default_config(...), and evaluate_tqqq_growth_income passes that non-empty config into build_option_overlay_diagnostics, which emits option_growth_overlay_enabled and related research-gate fields even though it does not create order intents. In a full dependency environment this assertion is therefore false for the default TQQQ entrypoint path; the same stale expectation appears in the nearby default-config and SOXL entrypoint checks, so CI will fail until these tests assert the live-gated diagnostics instead of absence.
Useful? React with 👍 / 👎.
Summary
Tests
Note: tests.test_entrypoints was not run locally because this VPS Python environment does not have pandas installed; CI installs project dependencies.