Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
5 changes: 3 additions & 2 deletions src/openhuman/inference/provider/factory_test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -760,8 +760,9 @@ fn make_openhuman_backend_forwards_unknown_hint_verbatim() {
// Unrecognised hint:* strings (e.g. hint:reaction for lightweight models)
// must be forwarded to the backend unchanged. The backend is authoritative
// over which hint values it accepts; the factory only translates the
// canonical hints (reasoning/chat/agentic/coding/summarization) — the last
// of which gained a dedicated tier, so it is no longer "unknown".
// canonical hints (reasoning/chat/agentic/coding/summarization).
// `hint:summarization` became canonical when `summarization-v1` shipped
// (PR #2690), so it is no longer a passthrough case.
for hint in ["hint:reaction", "hint:garbage"] {
let mut config = Config::default();
config.default_model = Some(hint.to_string());
Expand Down
9 changes: 6 additions & 3 deletions src/openhuman/memory/chat.rs
Original file line number Diff line number Diff line change
Expand Up @@ -215,9 +215,9 @@ mod tests {
fn build_chat_runtime_defaults_to_openhuman_resolved_model() {
let cfg = Config::default();
let (_provider, model) = build_chat_runtime(&cfg).unwrap();
// build_chat_runtime resolves the "summarization" role, which gained a
// dedicated tier (summarization-v1) — the role no longer falls back to
// the generic reasoning-v1 default.
// build_chat_runtime resolves the "summarization" workload role,
// which routes to the dedicated `summarization-v1` tier (PR #2690)
// rather than the generic `reasoning-v1` fallback.
assert_eq!(model, "summarization-v1");
}

Expand All @@ -226,6 +226,9 @@ mod tests {
let mut cfg = Config::default();
cfg.memory_tree.cloud_llm_model = Some("custom-summary-model".into());
let (_provider, model) = build_chat_runtime(&cfg).unwrap();
// Setting memory_tree.cloud_llm_model overrides the cloud-memory
// model path; the routing falls back to the platform default
// (`reasoning-v1`) rather than the `summarization-v1` tier.
assert_eq!(model, "reasoning-v1");
}

Expand Down
Loading