Skip to content

Comments

Update dart_mcp to spec version 2025-11-25#357

Merged
jakemac53 merged 12 commits intodart-lang:mainfrom
jakemac53:update-to-2025-11-05
Feb 25, 2026
Merged

Update dart_mcp to spec version 2025-11-25#357
jakemac53 merged 12 commits intodart-lang:mainfrom
jakemac53:update-to-2025-11-05

Conversation

@jakemac53
Copy link
Contributor

@jakemac53 jakemac53 commented Feb 23, 2026

Note: This does not implement Task support, which is still experimental. It would be a fair bit of implementation work and should be done in a separate PR if we choose to implement it. But I think we should wait until it is non-experimental anyways.

This ended up being fairly complex, but it's not as bad as it looks, as much of the changes are to examples and tests.

See the CHANGELOG.md for a full description of the changes, some highlights:

  • Adds Icon support
  • Adds support for URL elicitation, including handling UrlElicitationRequired errors in tool calls.
    • Added examples showing all the possible interactions here.
  • There were also some changes to how enums are represented, and multi-select enums are now supported.
    • Includes validation for these.
  • I added a skill for updating to the latest spec.
    • Note however that use of this skill MUST be overseen, the LLMs do love to hallucinate spec changes.
  • Adds ToolChoice support for sampling

@github-actions
Copy link

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

required String elicitationId,
Meta? meta,
}) => ElicitationCompleteNotification.fromMap({
'elicitationId': elicitationId,
Copy link
Contributor

Choose a reason for hiding this comment

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

are we using raw strings here because AI tooling is authoring these specs? Perhaps we could the agent via skills to use consts instead of raw strings to adhere to best practices?

Copy link
Contributor Author

@jakemac53 jakemac53 Feb 24, 2026

Choose a reason for hiding this comment

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

Probably if we just did a single cleanup to replace these all with constants, then in the future it would use constants.

It is a mixture of AI/Hand coded stuff (but even the hand coded stuff, I just go with the existing pattern here which has been raw strings). But it would be good to avoid any typos by using the constants.

@jakemac53 jakemac53 merged commit 44b22f4 into dart-lang:main Feb 25, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants