chore: deprecate @zuplo/mcp in favor of @modelcontextprotocol/sdk#71
Open
ntotten wants to merge 2 commits into
Open
chore: deprecate @zuplo/mcp in favor of @modelcontextprotocol/sdk#71ntotten wants to merge 2 commits into
ntotten wants to merge 2 commits into
Conversation
Add a deprecation warning callout to the README and a `deprecated` field in package.json pointing users to the official MCP TypeScript SDK (@modelcontextprotocol/sdk). The official SDK now covers these use cases and is the recommended path forward for new projects and existing users. Note: `npm deprecate @zuplo/mcp "..."` must still be run against the registry to surface the warning at install time. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Deprecates @zuplo/mcp and directs users to the official Model Context Protocol TypeScript SDK (@modelcontextprotocol/sdk), aligning repository docs and package metadata with the project’s end-of-maintenance status.
Changes:
- Add a prominent deprecation warning callout at the top of the README and reword the package description in past tense.
- Update
package.jsondescription to indicate deprecation and add a deprecation/migration message.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| README.md | Adds a deprecation warning banner and updates wording to reflect that the SDK is no longer maintained. |
| package.json | Updates package description and adds a deprecation/migration message in metadata. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "version": "0.1.0", | ||
| "description": "A fetch API based, remote server first, TypeScript SDK for MCP.", | ||
| "description": "DEPRECATED — please use the official @modelcontextprotocol/sdk instead.", | ||
| "deprecated": "@zuplo/mcp is deprecated and no longer maintained. Please migrate to the official MCP TypeScript SDK: @modelcontextprotocol/sdk (https://github.com/modelcontextprotocol/typescript-sdk).", |
The `deprecated` field is not a standard npm package.json field and is ignored by npm when determining whether to show install-time deprecation warnings — those come from registry metadata set via `npm deprecate`. Keeping it here risks misleading readers into thinking the package is already deprecated on the registry. The README warning plus the manual `npm deprecate` step cover this properly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Member
Author
|
Addressed Copilot's review in 7fcc17e — removed the non-standard |
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
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.
Summary
Marks
@zuplo/mcpas deprecated and points users to the official MCP TypeScript SDK,@modelcontextprotocol/sdk.Changes
> [!WARNING]deprecation callout at the top with a link to the official SDK, and rewords the description to past tense.descriptionand adds adeprecatedfield with the migration message.Why
The official
@modelcontextprotocol/sdknow covers the use cases this package was built for. New projects should adopt it directly, and existing users should migrate.Follow-up (manual)
To make
npm install @zuplo/mcpactually surface a warning, the package must also be deprecated on the npm registry:npm deprecate @zuplo/mcp "@zuplo/mcp is deprecated and no longer maintained. Please migrate to the official MCP TypeScript SDK: @modelcontextprotocol/sdk (https://github.com/modelcontextprotocol/typescript-sdk)."This can only be done by a publisher with credentials to the npm package — it can't happen in CI from this PR.
🤖 Generated with Claude Code