Skip to content

Conversation

@corvid-agent
Copy link
Contributor

Summary

Fixes #1425

@modelcontextprotocol/node depends on @hono/node-server (declared in dependencies), which has hono as a peer dependency. However, hono was not declared as a peer dependency in @modelcontextprotocol/node's package.json. This causes warnings and errors with strict package managers like Yarn that enforce peer dependency resolution.

This PR adds "hono": "catalog:runtimeServerOnly" to the peerDependencies section of packages/middleware/node/package.json, matching the pattern already used by @modelcontextprotocol/hono which correctly declares hono as a peer dependency.

Changes

  • Added hono as a peer dependency in packages/middleware/node/package.json using the existing catalog:runtimeServerOnly version specifier

Test plan

  • Verified that @modelcontextprotocol/hono already uses the same "hono": "catalog:runtimeServerOnly" peer dependency pattern
  • Install @modelcontextprotocol/node with Yarn in strict mode and confirm no missing peer dependency warnings for hono

🤖 Generated with Claude Code

Fixes modelcontextprotocol#1425

@hono/node-server requires hono as a peer dependency, but it was not
declared in @modelcontextprotocol/node's package.json. This caused
warnings/errors with strict package managers like Yarn.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@corvid-agent corvid-agent requested a review from a team as a code owner February 9, 2026 20:31
@changeset-bot
Copy link

changeset-bot bot commented Feb 9, 2026

🦋 Changeset detected

Latest commit: 4921907

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@modelcontextprotocol/node Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 9, 2026

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/client@1504

@modelcontextprotocol/server

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/server@1504

@modelcontextprotocol/express

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/express@1504

@modelcontextprotocol/hono

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/hono@1504

@modelcontextprotocol/node

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/node@1504

commit: 4921907

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

Missing dependency hono

2 participants