valknar/llmx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Persist search items (#3745)

Browse Source 
Let's record the search items because they are part of the history.
This commit is contained in:
Ahmed Ibrahim
2025-09-16 14:02:15 -04:00
committed by GitHub
parent 5e2c4f7e35
commit 244687303b
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
codex-rs/core/src/conversation_history.rs
View File

@@ -47,8 +47,9 @@ fn is_api_message(message: &ResponseItem) -> bool {
| ResponseItem::CustomToolCall { .. } | ResponseItem::CustomToolCall { .. }
| ResponseItem::CustomToolCallOutput { .. } | ResponseItem::CustomToolCallOutput { .. }
| ResponseItem::LocalShellCall { .. } | ResponseItem::LocalShellCall { .. }
| ResponseItem::Reasoning { .. } => true, | ResponseItem::Reasoning { .. }
ResponseItem::WebSearchCall { .. } | ResponseItem::Other => false, | ResponseItem::WebSearchCall { .. } => true,
ResponseItem::Other => false,
} }
} }

5
codex-rs/core/src/rollout/policy.rs
View File

@@ -25,8 +25,9 @@ pub(crate) fn should_persist_response_item(item: &ResponseItem) -> bool {
| ResponseItem::FunctionCall { .. } | ResponseItem::FunctionCall { .. }
| ResponseItem::FunctionCallOutput { .. } | ResponseItem::FunctionCallOutput { .. }
| ResponseItem::CustomToolCall { .. } | ResponseItem::CustomToolCall { .. }
| ResponseItem::CustomToolCallOutput { .. } => true, | ResponseItem::CustomToolCallOutput { .. }
ResponseItem::WebSearchCall { .. } | ResponseItem::Other => false, | ResponseItem::WebSearchCall { .. } => true,
ResponseItem::Other => false,
} }
} }