modelcontextprotocol/rust-sdk
Watch
1
0
Fork
You've already forked rust-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

fix: default CallToolResult content to empty vec on missing field (#752)

Browse source
This commit is contained in:
Dale Seo 2026-03-13 16:02:20 -04:00 • committed by GitHub
commit 44dfcf5550
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 42 additions and 33 deletions
Download patch file Download diff file Expand all files Collapse all files

28
crates/rmcp/tests/test_structured_output.rs
View file

@ -298,18 +298,20 @@ async fn test_empty_content_array_with_is_error() {
assert_eq!(result.is_error, Some(false));
}
#[tokio::test]
async fn test_missing_content_is_rejected() {
#[test]
fn test_missing_content_defaults_to_empty() {
let raw = json!({ "isError": false });
let result: Result<CallToolResult, _> = serde_json::from_value(raw);
assert!(result.is_err());
let result: CallToolResult = serde_json::from_value(raw).unwrap();
assert!(result.content.is_empty());
assert_eq!(result.is_error, Some(false));
}
#[tokio::test]
async fn test_missing_content_with_structured_content_is_rejected() {
#[test]
fn test_missing_content_with_structured_content_deserializes() {
let raw = json!({ "structuredContent": {"key": "value"}, "isError": false });
let result: Result<CallToolResult, _> = serde_json::from_value(raw);
assert!(result.is_err());
let result: CallToolResult = serde_json::from_value(raw).unwrap();
assert!(result.content.is_empty());
assert_eq!(result.structured_content.unwrap()["key"], "value");
}
#[tokio::test]
@ -333,3 +335,13 @@ async fn test_empty_content_roundtrip() {
let deserialized: CallToolResult = serde_json::from_value(v).unwrap();
assert_eq!(deserialized, result);
}
#[test]
fn test_call_tool_result_deserialize_without_content() {
let json = json!({
"structuredContent": {"message": "Hello"}
});
let result: CallToolResult = serde_json::from_value(json).unwrap();
assert!(result.content.is_empty());
assert!(result.structured_content.is_some());
}