admin/openhuman
1
0
Fork 0
mirror of https://github.com/tinyhumansai/openhuman.git synced 2026-07-27 21:08:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(memory): accept time_window_days alias in query_global (#2350)

Browse Source 
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Cyrus Gray <144336577+graycyrus@users.noreply.github.com>
This commit is contained in:
Liohtml
2026-05-22 00:12:15 +05:30
committed by GitHub
co-authored by Claude Opus 4.6 Cyrus Gray
parent f02543b80a
commit a20f75bc4b
1 changed files with 16 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/openhuman/memory/tree/retrieval/rpc.rs
+16 -7
View File
@@ -82,6 +82,11 @@ pub async fn query_source_rpc(
// ── query_global ──────────────────────────────────────────────────────
/// Request body for `memory_tree_query_global`.
///
/// The consolidated `memory_tree` tool schema advertises `time_window_days`
/// (consistent with `query_source` / `query_topic`), while the standalone
/// tool uses `window_days`. Accept both via serde alias so callers using
/// either field name succeed.
#[derive(Clone, Debug, Serialize, Deserialize)]
pub struct QueryGlobalRequest {
#[serde(alias = "window_days")]
@@ -425,15 +430,19 @@ mod tests {
}
#[test]
fn query_global_request_accepts_legacy_window_days_alias() {
let req: QueryGlobalRequest = serde_json::from_value(serde_json::json!({
"window_days": 7
}))
.expect("legacy window_days alias should deserialize");
fn query_global_request_accepts_both_field_names() {
// The consolidated memory_tree tool schema uses "time_window_days"
// while the standalone tool uses "window_days". Both must deserialize.
let from_alias: QueryGlobalRequest =
serde_json::from_value(serde_json::json!({"window_days": 7}))
.expect("legacy window_days alias should deserialize");
assert_eq!(from_alias.time_window_days, 7);
assert_eq!(req.time_window_days, 7);
let from_primary: QueryGlobalRequest =
serde_json::from_value(serde_json::json!({"time_window_days": 30}))
.expect("primary time_window_days should deserialize");
assert_eq!(from_primary.time_window_days, 30);
}
// ── query_topic_rpc ───────────────────────────────────────────────
#[tokio::test]