Skip to content

Conversation

@alexhancock
Copy link
Collaborator

Fixes #626

Capabilities for tasks take the following form.

Client

{
  "capabilities": {
    "tasks": {
      "list": {},
      "cancel": {},
      "requests": {
        "sampling": {
          "createMessage": {}
        },
        "elicitation": {
          "create": {}
        }
      }
    }
  }
}

Server

{
  "capabilities": {
    "tasks": {
      "list": {},
      "cancel": {},
      "requests": {
        "tools": {
          "call": {}
        }
      }
    }
  }
}

We were previously using booleans to model these instead of objects, causing errors during initialization between servers and clients that expressed these capabilities

@github-actions github-actions bot added T-test Testing related changes T-config Configuration file changes T-core Core library changes T-model Model/data structure changes labels Jan 23, 2026
@alexhancock alexhancock force-pushed the alexhancock/task-capabilities-format branch from 40145f3 to 38aaf98 Compare January 23, 2026 17:18
@github-actions github-actions bot added the T-documentation Documentation improvements label Jan 23, 2026
@alexhancock alexhancock force-pushed the alexhancock/task-capabilities-format branch from 38aaf98 to e20f6df Compare January 23, 2026 17:20
@github-actions github-actions bot removed the T-documentation Documentation improvements label Jan 23, 2026
Copy link
Collaborator

@jamadeo jamadeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Works on my test with goose.


// Verify serialization matches expected format
let json = serde_json::to_value(&tasks).unwrap();
assert_eq!(json["list"], serde_json::json!({}));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are these really testing all that much?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not terribly. the serialization one has value but these I agree don't add much. merging for not to not wait for another CI cycle but I will remove them on a future change.

@alexhancock alexhancock merged commit 613eafb into main Jan 23, 2026
11 checks passed
@github-actions github-actions bot mentioned this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-config Configuration file changes T-core Core library changes T-model Model/data structure changes T-test Testing related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

in 0.13.0, TasksCapability structure has incorrect types

3 participants