Repository navigation
Keep pooled affinity through the extended cache interval - #5265
Open
bb-slop-cop[bot] wants to merge 1 commit into
Open
bb-slop-cop[bot] wants to merge 1 commit into
bb-slop-cop[bot] wants to merge 1 commit into
Conversation
This branch has not been deployed
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
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.
🚨 SLOP COP 🚨 ·
new-issue-autopilotHuman comments
What was wrong
The Account Pooler enables the one-hour Claude prompt-cache setting for subscription routing but expires live and persisted session-account affinity after 30 idle minutes. When another conversation advances the pool cursor, a recovered account's original session can resume on a different account during the cache interval, including after a plugin reload. Verified reproduction and root-cause report.
What changed
Extend the existing idle affinity TTL to 65 minutes: one hour plus a five-minute margin. Add routing regressions at 45 and 61 idle minutes, with and without a full plugin reload. Advance existing Claude/Codex and parent-affinity expiry checks to the new retention window. Eligibility, account failover, cursor behavior, and affinity capacity remain unchanged. No dependency, schema, stored-data, generated-file, public API, protocol, CLI, or release change.
Changed-line count: 79 total text lines (74 additions + 5 deletions) across 2 files in the Account Pooler subsystem.
How you verified
origin/maincommitd474d2932dea14b5ccdfa305760c547c03a4d251: all four cases expect the first account but receive the second. The same agent repeated this failure in a second clean checkout with fresh fixture storage.pnpm exec turbo run test --filter=bb-plugin-account-pool -- --testNamePattern='retains Claude session affinity': four cases pass after the fix.pnpm exec turbo run test typecheck lint --filter=bb-plugin-account-pool: all 353 tests pass; typecheck and lint succeed. Two existing lint warnings also occur on unchanged production code.git diff --checkand formatting checks pass; the diff contains no binary files.Fixes #5262