best-mcp: instructions via initialize, real version, workflow-first nudges - #9
Merged
Merged
Conversation
…ow nudges A live Copilot retry against a 0.9.4 server showed the agent still going catalogue -> raw schema without ever consulting workflows, and unable to say which best-mcp it was running. Root causes, all client-side: - SERVER_INSTRUCTIONS were only attached as a non-standard `_meta` on tools/list, which standard hosts (Copilot, Claude Desktop, Cursor) never read. They now ride the initialize result''s `instructions` field - the channel hosts actually consume. The `_meta` copy stays for hosts that surface it. - serverInfo.version was hardcoded ''1.0.0''; it now reads the real package.json version, so "which version am I running" is answerable. - The catalogue/schema tool descriptions now name the `workflows` cross-link and tell the model to read the recipe (get_workflows with workflow_id) before hand-assembling a multi-step sequence - tool descriptions being the one text every host feeds the model. Verified over stdio: initialize carries serverInfo with the package version and 4.4 KB of instructions including the workflows-first rule. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Client-side follow-up to the 0.9.4 workflows capability, after a live Copilot retry showed the agent STILL bypassing recipes:
_metaon tools/list, which Copilot et al. never read. Now in theinitializeresult''s standardinstructionsfield (the_metacopy remains).1.0.0; now read from package.json - "which best-mcp am I running" becomes answerable.get_command_catalogue/get_command_schema/get_query_cataloguenow name theworkflowscross-link and direct the model to read the recipe before hand-assembling multi-step choreography - tool descriptions are the one text every host feeds the model.Verified over stdio: initialize carries serverInfo + 4.4 KB instructions incl. the workflows-first rule.
tscclean.Release after merge:
LC_ALL=en_US.UTF-8 ./scripts/release-mcp.sh 2.3.1.🤖 Generated with Claude Code