Instruct clients to prefer MCP #161
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Clients may be influenced by things like instructions for running tests in `README.md` and then decided to run `dart test` in a shell instead of using the `run_tests` MCP tool. If we are providing a MCP version of a tool it may be tailored to the LLM client use case and so should be preferred. Add a server level instruction.
|
Some clients seems to use MCP more reliably than others, this instruction makes it more consistent. |
PR HealthChangelog Entry ✔️
Changes to files need to be accounted for in their respective changelogs. |
|
Love this. I was running into this today. Thank you! |
kenzieschmoll
approved these changes
Jun 6, 2025
jakemac53
reviewed
Jun 6, 2025
| # Version 0.1.0 (Dart SDK 3.8.0) | ||
| # 0.1.1-wip | ||
|
|
||
| * Instruct clients to prefer MCP tools over running tools in the shell. |
There was a problem hiding this comment.
This can actually move into the below version, I plan to do a roll soon so it will make it into the 3.8 SDK.
The versions here are weird since its not actually published but I did want some sort of changelog.
copybara-service bot
pushed a commit
to dart-lang/sdk
that referenced
this pull request
Jun 9, 2025
Revisions updated by `dart tools/rev_sdk_deps.dart`. ai (https://github.com/dart-lang/ai/compare/1d9d60c..f2b48c6): f2b48c6 2025-06-09 Nate Bosch Retain compatibility with 3.7 SDKs (dart-lang/ai#163) 52adf08 2025-06-06 Jacob MacDonald add homepage, repository, and documentation links to the pub result (dart-lang/ai#155) fa1c2be 2025-06-06 Nate Bosch Always use the failures-only reporter for testing (dart-lang/ai#160) 66a152f 2025-06-06 Nate Bosch Instruct clients to prefer MCP (dart-lang/ai#161) 55ad850 2025-06-05 Jacob MacDonald Add a retroactive changelog (dart-lang/ai#157) b08a610 2025-06-05 Jacob MacDonald Update instructions, add cursor install link (dart-lang/ai#159) web (https://github.com/dart-lang/web/compare/f1becf0..de6b3e4): de6b3e4 2025-06-06 Srujan Gaddam Add missing copyrights and delete empty files (dart-lang/web#371) 74a33ba 2025-06-06 Kevin Moore Add in a missing library directive, missing new line (dart-lang/web#370) 4d24eb5 2025-06-06 nikeokoronkwo [web_generator] Setting up `web_generator` for Dart JS Interop Gen (dart-lang/web#368) webdev (https://github.com/dart-lang/webdev/compare/55941b0..661dafd): 661dafd4 2025-06-08 Srujan Gaddam Defer execution of main until resume for hot restart with DDC library bundle format (dart-lang/webdev#2623) 01a3b9d7 2025-06-06 Nicholas Shahan Remove skip from chrome proxy service test Change-Id: If3d4326d0bacf47a4d95520dbd0aac5fb58de439 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/433363 Commit-Queue: Devon Carew <[email protected]> Reviewed-by: Konstantin Shcheglov <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Clients may be influenced by things like instructions for running
tests in
README.mdand then decided to rundart testin a shellinstead of using the
run_testsMCP tool. If we are providing a MCPversion of a tool it may be tailored to the LLM client use case and so
should be preferred. Add a server level instruction.