[thread-store] clarify empty source filters

Author: adamvy-oai

codex-rs/app-server/src/thread_views_tests.rs

@@ -83,6 +83,7 @@ fn stored_thread_projection_applies_fallbacks() {
         reasoning_effort: None,
         created_at,
         updated_at,
+        recency_at: updated_at,
         archived_at: None,
         cwd: cwd.clone(),
         cli_version: "0.0.0".to_string(),
@@ -112,6 +113,7 @@ fn stored_thread_projection_applies_fallbacks() {
             model_provider: "fallback-provider".to_string(),
             created_at: created_at.timestamp(),
             updated_at: updated_at.timestamp(),
+            recency_at: Some(updated_at.timestamp()),
             status: ThreadStatus::NotLoaded,
             path: None,
             cwd: AbsolutePathBuf::from_absolute_path(cwd).expect("absolute path"),

codex-rs/thread-store/src/types.rs

@@ -202,7 +202,7 @@ pub struct ListThreadsParams {
     pub sort_key: ThreadSortKey,
     /// Sort direction requested by the caller.
     pub sort_direction: SortDirection,
-    /// Allowed session sources. Empty means implementation default.
+    /// Allowed session sources. Empty means no source filter.
     pub allowed_sources: Vec<SessionSource>,
     /// Optional model provider filter. `None` means implementation default, while an empty vector
     /// means all providers.
@@ -231,7 +231,7 @@ pub struct SearchThreadsParams {
     pub sort_key: ThreadSortKey,
     /// Sort direction requested by the caller.
     pub sort_direction: SortDirection,
-    /// Allowed session sources. Empty means implementation default.
+    /// Allowed session sources. Empty means no source filter.
     pub allowed_sources: Vec<SessionSource>,
     /// Whether archived threads should be searched instead of active threads.
     pub archived: bool,