Skip to content

feat!: make rust-mcp-sdk the sole dependency #43

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

Merged
merged 5 commits into from
May 28, 2025

Conversation

hashemix
Copy link
Member

📌 Summary

Make rust-mcp-sdk the sole dependency—no need to add rust-mcp-schema separately anymore.

This PR updates the rust-mcp-sdk ecosystem to make rust-mcp-sdk the sole dependency required for users, eliminating the need to separately include rust-mcp-schema.

Macro generation logic in rust_mcp_macros also updated to adapt to the context of rust-mcp-sdk usage while maintaining compatibility for standalone rust_mcp_macros users with a new sdk cargo feature.

✨ Changes Made

  • Added an sdk feature flag in rust_mcp_macros's Cargo.toml. Modified rust_mcp_macros to conditionally adapt its generated code based on the sdk feature, ensuring seamless integration with rust-mcp-sdk while keeping it compatible with rust-mcp-schema.
  • Ensured rust-mcp-sdk re-exports rust-mcp-schema as rust_mcp_sdk::schema for ergonomic access to schema types (e.g., rust_mcp_sdk::schema::Tool);
  • Updated documentation in READMEs to clarify dependency requirements and usage.
  • Removed rust-mcp-schema dependency from examples and updated them to use re-exported rust_mcp_sdk::schema instead

@hashemix hashemix merged commit d1973ca into main May 28, 2025
3 checks passed
@hashemix hashemix mentioned this pull request May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant