Skip to content

refactor(libsy): separate task context from judge instructions #899

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
nachiketb-nvidia wants to merge 1 commit into from
Closed

refactor(libsy): separate task context from judge instructions #899

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions crates/libsy/src/algorithms/llm_class.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -186,8 +186,9 @@ struct TaskInput {
recent_turn_window: Option<usize>,
}

impl ClassifierInput for TaskInput {
fn build_messages(&self, _state: &State, request: &Request) -> Vec<Message> {
impl TaskInput {
/// Selects conversation content without adding judge instructions.
fn messages(&self, request: &Request) -> Vec<Message> {
// The default preserves the whole-task anchor and latest user update. A
// configured window widens that to the surrounding conversation.
let mut messages = match self.recent_turn_window {
Expand All @@ -204,6 +205,13 @@ impl ClassifierInput for TaskInput {
.retain(|block| !matches!(block, ContentBlock::Reasoning { .. }));
}
messages.retain(|message| !message.content.is_empty());
messages
}
}

impl ClassifierInput for TaskInput {
fn build_messages(&self, _state: &State, request: &Request) -> Vec<Message> {
let mut messages = self.messages(request);
// Only the windowed path carries assistant turns and tool traffic for the judge
// to be distracted by. The default path is user task messages only — the anchor
// and the latest follow-up — so there is nothing there to outrank.
Expand Down
Loading