fix(libsy): update escalation judge param defaults to benchmarked values - #201
Conversation
Signed-off-by: Lin Jia <linj@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
WalkthroughThe escalation judge now defaults to two consecutive escalation verdicts, larger transcript context limits, and a 30-second router timeout. Configuration documentation and tests were updated, while selected processor tests explicitly retain single-verdict behavior. ChangesEscalation judge defaults
Estimated code review effort: 2 (Simple) | ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
Update the Python-side escalation judge defaults to match the benchmarked configuration: escalate_confirmations 1→2, recent_turn_window 14→28, window_message_chars 300→500, max_request_chars 12k→18k, judge_timeout_s 5→30. This is part 1 of breaking up PR #172.
Summary by CodeRabbit
Improvements
Tests