fix: allow description field to be absent from Tool#159
Merged
mattt merged 3 commits intomodelcontextprotocol:mainfrom Sep 7, 2025
Merged
Conversation
Contributor
Author
|
see issue #158 |
* description field is not a requried field of Tool
mattt
force-pushed
the
fix/description_is_not_a_required_field
branch
from
4a42963 to
187c7d7
Compare
mattt
reviewed
Sep 7, 2025
mattt
reviewed
Sep 7, 2025
mattt
approved these changes
Sep 7, 2025
Velly69
added a commit
to MacPaw/mcp-swift-sdk
that referenced
this pull request
Mar 2, 2026
* Enable building with the Swift Linux SDK (modelcontextprotocol#151) * Update README to document support for Alpine Linux (modelcontextprotocol#153) * Adopt strict concurrency for project (modelcontextprotocol#157) * Adopt strict concurrency for project * Fix one warning related to that * Add version-specific manifest for Swift 6.0 --------- Co-authored-by: Alexey Ivashko <a_ivashko@apple.com> Co-authored-by: Mattt Zmuda <mattt@me.com> * Make `Tool.description` optional (modelcontextprotocol#160) * Make Tool.description optional * Make description parameter in Tool initializer optional * fix: allow description field to be absent from `Tool` (modelcontextprotocol#159) * fix: allow description field to be absent from `Tool` * description field is not a requried field of Tool * Apply suggestions from code review * Update Tests/MCPTests/ToolTests.swift --------- Co-authored-by: Mattt <mattt@me.com> * Add Mac Catalyst to target dependencies for EventSource. (modelcontextprotocol#163) * chore: update licensing to Apache 2.0 for new contributions (modelcontextprotocol#177) * Add SECURITY.md with GitHub Security Advisories guidance (modelcontextprotocol#200) * V0.11.x (modelcontextprotocol#199) * Support 2025-06-18 Spec (modelcontextprotocol#167) * Support OpenAI _meta for Tool * Update init * Add title and outputSchema * Concurreny issue * General fields * Add Generate Fields to Server elements * 4 tab spacing * Update Version * Elicitation * Format * More tests and inits * Elicitation in README * Fix orphaned test * Add title to more structs * Public init * Fix meta validation and make tests more generic * Format and make Result properties public again * Revert more visibility changes * feat: 2025-11-25 phase 1, includes Version, Icon, Progress (modelcontextprotocol#181) * chore: fixed unit tests * feat: added icon and progess base structs * feat: added progress notification and meta to callTool * feat: added tests * feat: simplified RequestMeta encoding/decoding * tests: added unit test to track progress * chore: rebased v0.11.x * feat: implemented cancellation * feat: added overloaded callTool method * fix: encoding Response _meta fix * chore: added old version to the versions list * feat: 2025-11-25 phase 2; added Logging and Completions (modelcontextprotocol#196) * feat: added Logging and Completions * feat: updated EmbeddedResource for the latest spec * chore: changed encoding of meta in Prompts to encodeIfPresent * feat: compliance tests + http server transport (modelcontextprotocol#197) * feat: http server transport * feat: added http server transport * feat: removed unused files * feat: added conformance tests * feat: improved conformace testing * chore: removed redundant files * feat/2025-11-25: sampling, elicitation, roots (modelcontextprotocol#198) * feat: Roots implementation * feat: added unit testing * feat: improved unit tests and fixed NetworkTransport crash * feat: implemented sampling and elicitation * feat: improved sampling tests * feat: improved elicitation tests * feat: improved confirmance tests * feat: improved Prompts unit tests * feat: removed test code * feat: updated conformance tests * chore: removed redundant commas * chore: removed 6.0.3 swift version from ci * chore: improved unit tests a little bit * chore: update README.md * chore: add server progress tracking example to README.md * chore: updated README.md * chore: updated transports table formatting * chore: updated MockNetworkConnection --------- Co-authored-by: Austin Evans <austinjevans@me.com> Co-authored-by: yehorsobko <yehorsobko@macpaw.com> --------- Co-authored-by: Tobias <t089@users.noreply.github.com> Co-authored-by: Mattt <mattt@me.com> Co-authored-by: Alexey Ivashko <mcivashko@gmail.com> Co-authored-by: Alexey Ivashko <a_ivashko@apple.com> Co-authored-by: Yifan Liu <liuyf1117@hotmail.com> Co-authored-by: Semen Kudriavtsev <sam@kudr.me> Co-authored-by: adam jones <domdomegg+git@gmail.com> Co-authored-by: Den Delimarsky <53200638+localden@users.noreply.github.com> Co-authored-by: Maksym Mova <maks.mova@gmail.com> Co-authored-by: Austin Evans <austinjevans@me.com> Co-authored-by: yehorsobko <yehorsobko@macpaw.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
2 participants
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.
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context