Skip to content

chore: auto-update QPK_PIN and pin manifests#250

Open
Pigbibi wants to merge 1 commit into
mainfrom
auto/qpk-pin-update
Open

chore: auto-update QPK_PIN and pin manifests#250
Pigbibi wants to merge 1 commit into
mainfrom
auto/qpk-pin-update

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Automated update of QPK_PIN and QSL Git SHA pin manifests.

Compatibility checks passed ✅ — all downstream-critical QPK modules verified importable.

Updated SHAs:

  • QPK: a06a99f74566ce60ae91fc39317cdfc16558e325
  • Strategy repos: latest HEAD from each

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

🚫 Merge blocked: active blocking history requires independent source-of-truth clearance

⚖️ Codex Review Arbitration

🚫 block: Trusted blocking history remains active because the cumulative PR diff does not demonstrably fix the prior packaging defect. In the current diff, src/quant_platform_kit.egg-info/SOURCES.txt still adds package/test entries including src/quant_platform_kit/strategy_lifecycle/capabilities.py, legacy_profile_index.py, qpk_vnext_n1.py, tests/test_backtest_capabilities.py, tests/test_legacy_profile_index.py, tests/test_qpk_vnext_n1.py, and tests/test_strategy_review_validator.py, but the cumulative PR diff contains no corresponding file additions for any of those paths. Since SOURCES.txt is the sdist manifest contract, this still describes contents not actually merged by the PR, so the prior high-severity release/packaging break is not shown fixed.


Review by Codex PR Review bot • PR

@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: 31d353a91a

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

src/quant_platform_kit/strategy_lifecycle/audit_log.py
src/quant_platform_kit/strategy_lifecycle/backtest_orchestrator.py
src/quant_platform_kit/strategy_lifecycle/backtest_scheduler.py
src/quant_platform_kit/strategy_lifecycle/capabilities.py

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 Keep generated egg-info out of pin-only updates

In the context where this auto-update is merged to main, this non-pin file change means the push is no longer covered by .github/workflows/update-qpk-pin.yml's paths-ignore list, which only ignores QPK_PIN, qsl-pins.txt, constraints.txt, docs, and Markdown files. That will fire the same pin workflow again and create another pin PR for the pin commit itself instead of stopping at the advertised feature SHA f3e719e; it can also cause duplicate downstream pin-sync PRs. Either drop the regenerated SOURCES.txt change from this pin-only commit or update the generated metadata in the feature commit being pinned.

Useful? React with 👍 / 👎.

Comment thread constraints.txt
Comment on lines +5 to +6
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@604842139331316f55486cfe33f316bf3e9698e5
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@45ab0dce847f8f728c68e9c67d15b661bb6eb80d

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Align strategy repo pins before publishing constraints

When consumers use this manifest as pip constraints, line 5 constrains QPK to f3e719e while the newly pinned strategy commits still declare their own dependency on QPK 651c9ac in their pyproject.toml metadata (checked UsEquityStrategies@604842, HkEquityStrategies@45ab0d, CnEquityStrategies@a16978, and CryptoStrategies@2aac3a). python -m pip install --help defines -c as “Constrain versions using the given constraints file”, so installing any of these strategy packages under constraints.txt will ask the resolver for two different direct URLs for quant-platform-kit and fail until those downstream commits are repinned first.

Useful? React with 👍 / 👎.

@Pigbibi Pigbibi force-pushed the auto/qpk-pin-update branch from 31d353a to f3087ac Compare July 14, 2026 02:51
@Pigbibi Pigbibi force-pushed the auto/qpk-pin-update branch from f3087ac to 0cf083b Compare July 14, 2026 20:01
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