You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Question: version in create_initialization_options is the pkg_ver for "mcp" #87
shows the use of create_initialization_options to set the initialization options. However this function is setting the version to 'pkg_ver("mcp")' which seems to indicate that it is the protocol version that is being indicated.
In the sqlite example, the version is manually set to 0.1.0, so it won't use the protocol version. Is this a mistake?
Is the version meant to record the version of whatever the actual server program is? or is it mean to be the protocol version.
This version is pure informational and for debugging purposes. The protocol version is exchanged differently and hardcoded into the current version of the SDK.
If it's for user visible debugging, perhaps the create_initialization_options could take the version as a parameter? it could be set to the pkg_ver of the server, that's more useful in the Claude Desktop UX than the the mcp version, which is relatively static. it's a small change, but i can submit the PR if that's of interest.
Activity
dsp-ant commentedon Dec 4, 2024
This version is pure informational and for debugging purposes. The protocol version is exchanged differently and hardcoded into the current version of the SDK.
restlessronin commentedon Dec 4, 2024
If it's for user visible debugging, perhaps the create_initialization_options could take the version as a parameter? it could be set to the pkg_ver of the server, that's more useful in the Claude Desktop UX than the the mcp version, which is relatively static. it's a small change, but i can submit the PR if that's of interest.
dsp-ant commentedon Dec 4, 2024
I think that would be an improvement. If you are so kind and create a PR @restlessronin
restlessronin commentedon Dec 5, 2024
@dsp-ant here it is: #90