fix(rmcp-macros): use re-exported serde_json path in task_handler (#735)

kakarot-dev · DaleSeo · web-flow · commit be248980f25a · 2026-03-09T16:30:11.000-04:00
* fix(rmcp-macros): use re-exported serde_json path in task_handler Replace bare `::serde_json::` with `::rmcp::serde_json::` in task_handler.rs to prevent compilation errors in crates that don't directly depend on serde_json. Fixes #487 * Update crates/rmcp-macros/src/task_handler.rs --------- Co-authored-by: Dale Seo <5466341+DaleSeo@users.noreply.github.com>
diff --git a/crates/rmcp-macros/src/task_handler.rs b/crates/rmcp-macros/src/task_handler.rs
@@ -193,7 +193,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
                                     if let Some(tool) = boxed.as_any().downcast_ref::<rmcp::task_manager::ToolCallTaskResult>() {
                                         match &tool.result {
                                             Ok(call_tool) => {
-                                                let value = ::serde_json::to_value(call_tool).unwrap_or(::serde_json::Value::Null);
+                                                let value = ::rmcp::serde_json::to_value(call_tool).unwrap_or_default();
                                                 return Ok(rmcp::model::GetTaskPayloadResult::new(value));
                                             }
                                             Err(err) => return Err(McpError::internal_error(

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS`
`193`	`193`	`if let Some(tool) = boxed.as_any().downcast_ref::<rmcp::task_manager::ToolCallTaskResult>() {`
`194`	`194`	`match &tool.result {`
`195`	`195`	`Ok(call_tool) => {`
`196`		`- let value = ::serde_json::to_value(call_tool).unwrap_or(::serde_json::Value::Null);`
	`196`	`+ let value = ::rmcp::serde_json::to_value(call_tool).unwrap_or_default();`
`197`	`197`	`return Ok(rmcp::model::GetTaskPayloadResult::new(value));`
`198`	`198`	`}`
`199`	`199`	`Err(err) => return Err(McpError::internal_error(`