chore: replace @platformatic/mcp patch with @getlarge/fastify-mcp fork#5
Merged
chore: replace @platformatic/mcp patch with @getlarge/fastify-mcp fork#5
Conversation
- Add npm override to redirect @platformatic/mcp to @getlarge/fastify-mcp - Add @getlarge/fastify-mcp as direct dependency for proper resolution - Remove patch file (no longer needed) - Update package-lock.json The fork includes: - OIDC discovery support (PR #97) - Resource subscription handlers (PR #98) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…file This prevents CI failures where `npm ci` complains about missing packages like pg, ioredis, and @aws-sdk/client-s3 which are optionalDependencies in the mcp-server workspace. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Instead of using npm override (which caused lock file sync issues), directly import from @getlarge/fastify-mcp in all source files. Changes: - Replace all @platformatic/mcp imports with @getlarge/fastify-mcp - Update mcp-server/package.json dependency - Remove override from root package.json - Regenerate package-lock.json Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The fork uses singular method names: - mcpSetResourceSubscribeHandler (not mcpSetResourcesSubscribeHandler) - mcpSetResourceUnsubscribeHandler (not mcpSetResourcesUnsubscribeHandler) Also removed duplicate FastifyInstance augmentation since the types are already provided by @getlarge/fastify-mcp. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.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.
Summary
@getlarge/fastify-mcpfork@platformatic/mcp@getlarge/fastify-mcpas direct dependency for proper resolutionWhy
The fork at getlarge/fastify-mcp includes features needed for this project:
resources/subscribeandresources/unsubscribeMCP methodsOnce these PRs are merged upstream, we can remove the override and switch back to
@platformatic/mcp.Test plan
🤖 Generated with Claude Code