-
Notifications
You must be signed in to change notification settings - Fork 474
feat(server): persist client info in sessions #313
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.
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
Conversation
Add SessionWithClientInfo interface and implementations to store and retrieve client information provided during initialization. This allows servers to access client implementation details throughout the session lifecycle.
WalkthroughThis change introduces the Changes
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (1.64.8)Error: you are using a configuration file for golangci-lint v2 with golangci-lint v1: please use golangci-lint v2 Note ⚡️ AI Code Reviews for VS Code, Cursor, WindsurfCodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback. Note ⚡️ Faster reviews with cachingCodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 16th. To opt out, configure 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (5)
🧰 Additional context used🧬 Code Graph Analysis (4)server/session.go (1)
server/server.go (2)
server/sse.go (2)
server/stdio.go (2)
🔇 Additional comments (15)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Hi @ezynda3 @pottekkat Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you 🚀
Since this is a non-breaking change, let's merge this to get the functionality in. Any modifications or improvements to the methods that aren't immediate can be addressed later.
@ezynda3 can you take a quick look?
Description
This PR adds the ability to persist client information in MCP sessions.
The main motivation is to enable tracking the origin of requests and which MCP Client triggered them. This makes it possible to use this data later on for analytics.
Type of Change
Checklist
Additional Information
This PR introduces a new interface SessionWithClientInfo that extends the ClientSession interface to provide the ability to store and retrieve client information.
Usage example
Summary by CodeRabbit