-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Mcp server instructions update fix #3699
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mcp server instructions update fix #3699
Conversation
|
|
@nexon33 do you happen to have discord? If so can you reach out to me? hrudolph is my username and I work for Roo Code. |
|
This looks good to me, checked the example on here and everything looks correct, tested with Sonnet 4 and was able to create a working MCP without any issues. |
* migrate didShowAnnouncement * changeset * remove comment * make variable public --------- Co-authored-by: Elephant Lumps <[email protected]>
Refactor weather server example implementation inside of prompt to use new MCP SDK features
Related GitHub Issue
Closes: #2032
Description
These changes fix the MCP prompt example so that it is back up to date.
Test Procedure
Type of Change
srcor test files.Pre-Submission Checklist
npm run lint).console.log) has been removed.npm test).mainbranch.npm run changesetif this PR includes user-facing changes or dependency updates.Additional Notes
This fix has fixed the actual prompt but is seperate from the deepseek r1 performance which doesn't always succeed to create the MCP server.
Important
Refactor MCP server example in
create-mcp-server.tsto use new SDK features and addzodfor validation.create-mcp-server.tsto use new MCP SDK features, simplifying server setup and tool/resource management.zodfor schema validation inget_forecasttool.zodand@modelcontextprotocol/sdkto dependencies increate-mcp-server.ts.This description was created by
for 5a76a17. You can customize this summary. It will automatically update as commits are pushed.