apunkt/plano
1
0
Fork 0
mirror of https://github.com/katanemo/plano.git synced 2026-06-14 15:15:15 +02:00
Code Issues Projects Releases Packages Wiki Activity

Add the system role into messages array (#967)

Browse source 
* add teh system role into messages array

* ci: trigger workflows

* dont normalize for anthropic

---------

Co-authored-by: Spherrrical <malikmusa1323@gmail.com>
This commit is contained in:
shivani kumar 2026-06-12 17:25:22 -04:00 • committed by GitHub
parent 2e38f7fa09
commit ecf864df25
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 8 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
crates/hermesllm/src/apis/anthropic.rs
View file

@ -128,6 +128,7 @@ pub struct MessagesRequest {
pub enum MessagesRole {
User,
Assistant,
System,
}
/// Cache control types for content blocks
@ -632,6 +633,7 @@ impl MessagesRole {
match self {
MessagesRole::User => "user",
MessagesRole::Assistant => "assistant",
MessagesRole::System => "system",
}
}
}

6
crates/hermesllm/src/transforms/request/from_anthropic.rs
View file

@ -223,6 +223,7 @@ impl From<MessagesRole> for Role {
match val {
MessagesRole::User => Role::User,
MessagesRole::Assistant => Role::Assistant,
MessagesRole::System => Role::System,
}
}
}
@ -340,6 +341,11 @@ impl TryFrom<MessagesMessage> for BedrockMessage {
let role = match message.role {
MessagesRole::User => ConversationRole::User,
MessagesRole::Assistant => ConversationRole::Assistant,
MessagesRole::System => {
return Err(TransformError::UnsupportedConversion(
"System messages must be set via the system prompt, not messages".to_string(),
));
}
};
let mut content_blocks = Vec::new();