docs: remove primitives from Server Card contents (fixes #10)#19
Open
SamMorrowDrums wants to merge 1 commit into
Open
docs: remove primitives from Server Card contents (fixes #10)#19SamMorrowDrums wants to merge 1 commit into
SamMorrowDrums wants to merge 1 commit into
Conversation
The 'A Server Card includes:' list in docs/discovery.md mentioned 'Capabilities — Tools, resources, and prompts the server offers', which contradicts schema.ts and SEP-2127: Server Cards do not enumerate primitives. Primitives remain subject to runtime listing via the protocol's standard list operations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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.
Fixes #10.
The "A Server Card includes:" list in
docs/discovery.mdlisted:This contradicts both:
schema.ts(lines 12–26), which explicitly states Server Cards "do not enumerate primitives (tools, resources, prompts) — those remain subject to runtime listing via the protocol's standard list operations."As agreed by @SamMorrowDrums and @tadasant in the issue discussion, this PR removes the contradictory bullet. The intro paragraph above the list also referred to Server Cards describing a server's "identity, capabilities, and connection details" — that has been updated to "identity and connection details", and a short clarifying sentence has been added stating that Server Cards do not enumerate primitives, mirroring the language in
schema.ts.No changes to
mediaTypestrings — that's #9, handled separately.