aljaz/goose
1
0
Fork 0
mirror of https://github.com/aljazceru/goose.git synced 2025-12-17 22:24:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: handle JsonRPC error variants as responses (#2903)

Browse Source
This commit is contained in:
Jack Amadeo
2025-06-13 12:05:46 -04:00
committed by GitHub
parent 7506c050db
commit 1326db231e
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
crates/mcp-client/examples/integration_test.rs
View File

@@ -110,6 +110,12 @@ where
let collected_events_after = events.lock().await.len();
assert_eq!(collected_events_after - collected_eventes_before, n_steps);
let error_result = client
.call_tool("add", serde_json::json!({ "a": "foo", "b": "bar" }))
.await;
assert!(error_result.is_err());
println!("Error result: {error_result:#?}\n");
// List resources
let resources = client.list_resources(None).await?;
println!("Resources: {resources:#?}\n");

3
crates/mcp-client/src/client.rs
View File

@@ -135,7 +135,8 @@ where
Ok(message) => {
tracing::info!("Received message: {:?}", message);
match message {
JsonRpcMessage::Response(JsonRpcResponse { id: Some(id), .. }) => {
JsonRpcMessage::Response(JsonRpcResponse { id: Some(id), .. })
| JsonRpcMessage::Error(JsonRpcError { id: Some(id), .. }) => {
service_ptr.respond(&id.to_string(), Ok(message)).await;
}
_ => {