Upgrade crypto live pool artifact namespace#84
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1876345f93
ℹ️ 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".
| DOWNLOAD_TOP_LIQUID: ${{ vars.DOWNLOAD_TOP_LIQUID || '90' }} | ||
| FIRESTORE_COLLECTION: ${{ vars.FIRESTORE_COLLECTION || 'strategy' }} | ||
| FIRESTORE_DOCUMENT: ${{ vars.FIRESTORE_DOCUMENT || 'CRYPTO_LEADER_ROTATION_LIVE_POOL' }} | ||
| FIRESTORE_DOCUMENT: ${{ vars.FIRESTORE_DOCUMENT || 'CRYPTO_LIVE_POOL_ROTATION_LIVE_POOL' }} |
There was a problem hiding this comment.
Preserve the existing Firestore pointer during migration
When the scheduled monthly workflow runs in an environment that has not explicitly set FIRESTORE_DOCUMENT, it will now publish only to strategy/CRYPTO_LIVE_POOL_ROTATION_LIVE_POOL; the previous v1 contract documented strategy/CRYPTO_LEADER_ROTATION_LIVE_POOL as the compatibility pointer for downstream readers. Without a dual-write, alias, or staged migration, those existing readers will keep seeing the old document and stop receiving monthly pool updates after this workflow publishes.
Useful? React with 👍 / 👎.
Summary:
Tests: