Always refresh alpha-engine-lib in boot-pull (supports @main pin)#21
Merged
Conversation
The ecosystem is moving from @v0.1.x tag pinning to @main pinning on alpha-engine-lib. @main eliminates coordinated tag bumps across 5 repos every time lib changes, and hard-fail + tests catch breakage loudly — worth the tradeoff for a single-dev paper-trading project. But @main on a long-lived venv like ae-dashboard's doesn't work out of the box here: the existing conditional pip install only triggers when requirements.txt CHANGES. With @main pin, requirements.txt never changes, so lib stays frozen at whatever was installed last — the @main pin becomes a lie. This adds a targeted alpha-engine-lib upgrade after the main pip block. Scoped to a single package (cheap on the micro) and only runs in repos that actually list alpha-engine-lib in requirements.txt. Wheel is cached; expected runtime ~1-2s per repo. ae-trading's boot-pull.sh already runs `pip install -r requirements.txt` unconditionally (alpha-engine/infrastructure/boot-pull.sh:82) so no equivalent fix is needed there. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Ecosystem is switching from `@v0.1.x` tag pinning to `@main` pinning on alpha-engine-lib (eliminates coordinated tag bumps across 5 repos, hard-fail + lib CI catches breakage loudly).
Gotcha on ae-dashboard: the current conditional pip install only triggers when requirements.txt CHANGES. With `@main` pin, requirements.txt never changes after the initial switchover, so lib stays frozen at whatever was installed last. The `@main` pin becomes a lie.
Fix
Adds a targeted `pip install --upgrade alpha-engine-lib` step after the existing conditional. Scoped to the single package (cheap on a t3.micro), only runs in repos that list alpha-engine-lib in requirements.txt. Wheel is cached; expected runtime ~1-2s per repo.
Why not ae-trading
ae-trading's boot-pull.sh already runs `pip install -r requirements.txt` unconditionally per repo (`alpha-engine/infrastructure/boot-pull.sh:82`). Full refresh handles lib @main refresh for free. No change needed there.
Test plan
Pairs with
🤖 Generated with Claude Code