fix: show inferred scores for sticky session logs - #2598
Conversation
Sticky-session routing now scores providers with the weighted algorithm and pins the result, so the logged provider scores are real values. Stop hiding them in the activity log by removing "session-sticky" from the score-bypassed selection reasons. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe PR updates the activity log detail view to display provider scores for ChangesProvider Score Visibility
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Problem
The activity-log "Routing Info" panel hides the per-provider scores for
sticky-session requests. As of #2596, sticky-session routing scores
providers with the normal weighted algorithm and only pins the result
for the session — so the log already carries the real inferred scores
under
selectionReason: "session-sticky". The dashboard still listedsession-stickyinSCORE_BYPASSED_SELECTION_REASONS, a leftover fromwhen sticky picks used rendezvous hashing and emitted hardcoded
0placeholders, so those real scores were never displayed.
Fix
Remove
session-stickyfromSCORE_BYPASSED_SELECTION_REASONSso theinferred weighted scores that informed the sticky routing decision show
up in the log. The existing all-zero fallback still hides them in the
degenerate case where scoring couldn't run (e.g. no metrics available).
Testing
pnpm formatpnpm --filter ui build+tsc --noEmit(clean)🤖 Generated with Claude Code
Summary by CodeRabbit