fix(proxy): preserve reset-window fallback ordering#779
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6f759e6451
ℹ️ 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".
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
Closing as duplicate of older PR #644. The branch points at the same fixed head, so review/merge should continue there. |
3 participants
Supersedes #644 because the repaired fork head is not receiving the required CI suite.
What changed:
Local verification:
uv run ruff check app/core/balancer/logic.py tests/unit/test_load_balancer.pyuv run ty check app/core/balancer/logic.py tests/unit/test_load_balancer.pyuv run pytest tests/unit/test_load_balancer.py::test_select_account_secondary_reset_fallback_uses_primary_subday_precision tests/unit/test_load_balancer.py::test_select_account_primary_reset_preference_uses_subday_precision tests/unit/test_load_balancer.py::test_select_account_prefers_lower_secondary_used_with_same_reset_bucket -qReplaces the currently blocked #644 head.