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

Serializing the eventmsg type to snake_case (#1709)

Browse Source 
This was an abrupt change on our clients. We need to serialize as
snake_case.
This commit is contained in:
aibrahim-oai
2025-07-28 10:26:27 -07:00
committed by GitHub
parent 5ebb7dd34c
commit 19bef7659f
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -280,7 +280,7 @@ pub struct Event {
/// Response event from the agent /// Response event from the agent
#[derive(Debug, Clone, Deserialize, Serialize, Display)] #[derive(Debug, Clone, Deserialize, Serialize, Display)]
#[serde(tag = "type", rename_all = "snake_case")] #[serde(tag = "type", rename_all = "snake_case")]
#[strum(serialize_all = "lowercase")] #[strum(serialize_all = "snake_case")]
pub enum EventMsg { pub enum EventMsg {
/// Error while executing a submission /// Error while executing a submission
Error(ErrorEvent), Error(ErrorEvent),

2
codex-rs/mcp-server/tests/common/mcp_process.rs
View File

@@ -294,7 +294,7 @@ impl McpProcess {
} }
} }
// New schema: method is the Display of EventMsg::SessionConfigured => "SessionConfigured" // New schema: method is the Display of EventMsg::SessionConfigured => "SessionConfigured"
if notification.method == "sessionconfigured" { if notification.method == "session_configured" {
if let Some(msg) = params.get("msg") { if let Some(msg) = params.get("msg") {
if let Some(session_id) = if let Some(session_id) =
msg.get("session_id").and_then(|v| v.as_str()) msg.get("session_id").and_then(|v| v.as_str())