GeneralBots/generalbots
1
0
Fork 0
Code Activity Actions 5

fix: normalize episodic/compact roles to system in all LLM providers
All checks were successful
BotServer CI/CD / build (push) Successful in 4m1s
Details

Browse source
This commit is contained in:
Rodrigo Rodriguez (Pragmatismo) 2026-04-14 13:47:18 -03:00
parent d6527a438b
commit 32f8a10825
2 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/llm/glm.rs
View file

@ -207,8 +207,13 @@ impl LLMProvider for GLMClient {
let role = m.get("role")?.as_str()?; let role = m.get("role")?.as_str()?;
let content = m.get("content")?.as_str()?; let content = m.get("content")?.as_str()?;
let sanitized = Self::sanitize_utf8(content); let sanitized = Self::sanitize_utf8(content);
let normalized_role = match role {
"user" | "assistant" | "system" | "tool" => role,
"episodic" | "compact" => "system",
_ => "user",
};
Some(GLMMessage { Some(GLMMessage {
role: role.to_string(), role: normalized_role.to_string(),
content: Some(sanitized), content: Some(sanitized),
tool_calls: None, tool_calls: None,
}) })

7
src/llm/kimi.rs
View file

@ -211,8 +211,13 @@ impl LLMProvider for KimiClient {
let role = m.get("role")?.as_str()?; let role = m.get("role")?.as_str()?;
let content = m.get("content")?.as_str()?; let content = m.get("content")?.as_str()?;
let sanitized = Self::sanitize_utf8(content); let sanitized = Self::sanitize_utf8(content);
let normalized_role = match role {
"user" | "assistant" | "system" | "tool" => role,
"episodic" | "compact" => "system",
_ => "user",
};
Some(KimiMessage { Some(KimiMessage {
role: role.to_string(), role: normalized_role.to_string(),
content: Some(sanitized), content: Some(sanitized),
tool_calls: None, tool_calls: None,
}) })