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

fix: update OutgoingMessageSender::send_response() to take Serialize (#2263)

Browse Source 
This makes `send_response()` easier to work with.
This commit is contained in:
Michael Bolin
2025-08-13 14:29:13 -07:00
committed by GitHub
parent d4533a0bb3
commit 37fc4185ef
5 changed files with 29 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
codex-rs/mcp-server/src/codex_tool_runner.rs
View File

@@ -61,7 +61,7 @@ pub async fn run_codex_tool_session(
is_error: Some(true),
structured_content: None,
};
outgoing.send_response(id.clone(), result.into()).await;
outgoing.send_response(id.clone(), result).await;
return;
}
};
@@ -235,9 +235,7 @@ async fn run_codex_tool_session_inner(
is_error: None,
structured_content: None,
};
outgoing
.send_response(request_id.clone(), result.into())
.await;
outgoing.send_response(request_id.clone(), result).await;
// unregister the id so we don't keep it in the map
running_requests_id_to_codex_uuid
.lock()
@@ -296,9 +294,7 @@ async fn run_codex_tool_session_inner(
// structured way.
structured_content: None,
};
outgoing
.send_response(request_id.clone(), result.into())
.await;
outgoing.send_response(request_id.clone(), result).await;
break;
}
}