Skip to content

fix(ci): stage coherent QPK pin propagation - #333

Merged
Pigbibi merged 1 commit into
mainfrom
fix/qpk-pin-propagation
Aug 23, 2026
Merged

fix(ci): stage coherent QPK pin propagation#333
Pigbibi merged 1 commit into
mainfrom
fix/qpk-pin-propagation

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • advance QPK_PIN to 68fcad8c68ab without rewriting the still-coherent aggregate bundle
  • split propagation into strategy-first and broker/QMT consumer phases
  • fail closed until every strategy main pins the staged QPK commit
  • reconcile broker/QMT QPK and strategy pins together, including QmtPlatform
  • validate each downstream dependency closure with uv pip install --dry-run

Root cause

The previous workflow rewrote only the QPK line in qsl-pins.txt, while the pinned strategy commits still declared QPK 3acab1923a97. Pip correctly rejected that mixed bundle with ResolutionImpossible, so QPK_PIN could never land and downstream strategy PRs could never start.

Verification

  • 14 passed (test_update_qpk_pin_workflow.py, test_qpk_pin_consistency.py)
  • Ruff passed
  • actionlint 1.7.12 passed
  • git diff --check passed
  • QPK 68fcad8c68ab install dry-run passed
  • CN strategy staged pin dry-run passed
  • consumer phase correctly PARKED while strategy mains still pin 3acab1923a97

No live authority, trading logic, Binance, or OCI changes.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi enabled auto-merge (squash) August 23, 2026 18:40
@Pigbibi
Pigbibi merged commit e3fa3e8 into main Aug 23, 2026
1 check passed
@Pigbibi
Pigbibi deleted the fix/qpk-pin-propagation branch August 23, 2026 18:41
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