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

Do not send reasoning item IDs (#3390)

Browse Source 
Response API doesn't require IDs on reasoning items anymore. 

Fixes: https://github.com/openai/codex/issues/3292
This commit is contained in:
pakrym-oai
2025-09-09 14:47:06 -07:00
committed by GitHub
parent 5eab4c7ab4
commit 5bcc9d8b77
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
codex-rs/protocol/src/models.rs
View File

@@ -49,7 +49,7 @@ pub enum ResponseItem {
content: Vec<ContentItem>, content: Vec<ContentItem>,
}, },
Reasoning { Reasoning {
#[serde(default)] #[serde(default, skip_serializing)]
id: String, id: String,
summary: Vec<ReasoningItemReasoningSummary>, summary: Vec<ReasoningItemReasoningSummary>,
#[serde(default, skip_serializing_if = "should_serialize_reasoning_content")] #[serde(default, skip_serializing_if = "should_serialize_reasoning_content")]