Skip to content

feat: implement migration plan from low-level Server API to high-level McpServer API #386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jirispilka wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat: implement migration plan from low-level Server API to high-level McpServer API #386

jirispilka wants to merge 2 commits into from

Conversation

@jirispilka
Copy link
Collaborator

@jirispilka jirispilka commented Jan 8, 2026 •
edited
Loading

What:
Refactor src/mcp/server.ts from the deprecated low-level Server API to the recommended high-level McpServer API.

Why:

  • The low-level API is deprecated by the MCP SDK maintainers
  • The high-level API provides automatic tool management and notifications
  • Opportunity to simplify architecture and reduce code by ~500 lines

Impact:

  • All existing features preserved (dynamic tool loading, Actor-MCP proxy, notifications)
  • Cleaner, more maintainable codebase
  • Future-proof against SDK changes

IMPORTANT - We don't want to start with implementation. This is only an analysis and possible impact

@jirispilka jirispilka marked this pull request as ready for review January 8, 2026 13:38
@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Jan 8, 2026
@jirispilka jirispilka requested a review from MQ37 January 8, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MQ37 MQ37 Awaiting requested review from MQ37

At least 1 approving review is required to merge this pull request.

Assignees

@jirispilka jirispilka

Labels

t-ai Issues owned by the AI team.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jirispilka