Skip to content

[codex] Add LongBridge HK low-vol dividend switch plan#121

Merged
Pigbibi merged 4 commits into
mainfrom
feature/live-enable-hk-low-vol-dividend
Jun 2, 2026
Merged

[codex] Add LongBridge HK low-vol dividend switch plan#121
Pigbibi merged 4 commits into
mainfrom
feature/live-enable-hk-low-vol-dividend

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Enable LongBridge switch-plan support for hk_low_vol_dividend_quality.
  • Update HK strategy dependency to the runtime/evidence-gated HK strategy commit.
  • Add factor snapshot artifact filename hints for snapshot-backed factor contracts.
  • Preserve dry-run-first behavior with required feature snapshot path and manifest placeholders.
  • Carry latest main runtime guard / heartbeat changes into the branch.

Validation

  • .venv/bin/python -m pytest -q tests/test_runtime_config_support.py tests/test_notifications.py tests/test_request_handling.py tests/test_runtime_strategy_adapters.py tests/test_signal_snapshot.py tests/test_decision_mapper.py -> 86 passed, 14 subtests passed
  • Switch plan smoke confirms hk_low_vol_dividend_quality True feature_snapshot <required> and factor snapshot artifact hints.

Dependency / merge order

Notes

  • This PR does not change production Cloud Run directly.
  • Real order enablement remains blocked until production factor snapshot, manifest, lineage, dry-run preview, notification log, and operator approval evidence pass validation.

@Pigbibi Pigbibi marked this pull request as ready for review June 2, 2026 20:01
@Pigbibi Pigbibi merged commit 20a55a4 into main Jun 2, 2026
1 check passed
@Pigbibi Pigbibi deleted the feature/live-enable-hk-low-vol-dividend branch June 2, 2026 20:01

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cb096ae401

ℹ️ 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".

Comment thread requirements.txt
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@v0.7.36
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@v0.7.50
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@v0.4.2
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@hk-low-vol-dividend-quality-20260603

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Don't advertise hk-verify dispatch for low-vol

With this dependency, hk_low_vol_dividend_quality becomes an enabled LongBridge profile, but the dry-run switch plan still points operators at sync-cloud-run-env.yml with target=hk-verify; that workflow's hk-verify setup hard-codes strategy_profile to hk_listed_global_etf_rotation, so following the generated plan for the new profile validates and syncs the wrong strategy instead of the low-vol dividend runtime. Add a profile-specific workflow override/target or suppress the hk-verify dispatch hint for this profile.

Useful? React with 👍 / 👎.

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