Rename crypto live profile namespace#29
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a75b7d1e9d
ℹ️ 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".
| attribute_name="crypto_leader_rotation_entrypoint", | ||
| attribute_name="crypto_live_pool_rotation_entrypoint", | ||
| ), | ||
| required_inputs=CRYPTO_CANONICAL_REQUIRED_INPUTS, |
There was a problem hiding this comment.
Keep the previous profile as an alias
Existing deployments or platform configs that still pass the former public profile crypto_leader_rotation now stop resolving after this rename because the new catalog metadata leaves aliases empty. Since this package already routes profile lookups through the catalog/adapter resolution helpers and profile strings are persisted outside the codebase, keep the old profile as an alias for crypto_live_pool_rotation so callers can migrate without breaking runtime adapter and entrypoint lookup.
Useful? React with 👍 / 👎.
Summary:
Tests: