-
Notifications
You must be signed in to change notification settings - Fork 3
MCP Core implementation - Initialization, Tools/List, Tools/Call (Missing Health monitor and tool cancellation) #13
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
base: mcp_server
Are you sure you want to change the base?
Conversation
This commit removes an unncessary config for the qemu target Signed-off-by: David Piskula <david.piskula@nxp.com>
Added thread pool and message queue initialization and enabled logging Signed-off-by: David Piskula <david.piskula@nxp.com>
Basic steps towards lifecycle handling, starting with initialization Signed-off-by: David Piskula <david.piskula@nxp.com>
Added tool registration, improved type definitions and tools/list Signed-off-by: David Piskula <david.piskula@nxp.com>
Refactored comments and code Signed-off-by: David Piskula <david.piskula@nxp.com>
9ebba85 to
5eef5aa
Compare
Added unit tests for implemented features Signed-off-by: David Piskula <david.piskula@nxp.com>
5eef5aa to
a7587e4
Compare
Implemented the tools/call method Signed-off-by: David Piskula <david.piskula@nxp.com>
Implemented tool response processing and testing Signed-off-by: David Piskula <david.piskula@nxp.com>
Added server error responses Signed-off-by: David Piskula <david.piskula@nxp.com>
Defined an enum for error codes Signed-off-by: David Piskula <david.piskula@nxp.com>
Refactored code, made response workers optional Signed-off-by: David Piskula <david.piskula@nxp.com>
Additional error handling for client initialization Signed-off-by: David Piskula <david.piskula@nxp.com>
|
Hi @dpiskula-nxp. |
Hello @butok, this is a development PR from a dev sub-branch (mcp-core) into our dev main-branch (mcp-server). The PR into the main upstream branch will be opened properly when ready. Since we are developing a library and need to collaborate by developing different parts in paralel, I wanted to split development into these sub-branches and merge them into the main dev branch through our own PRs. I realize this is not commonly done on the nxp repository but I don't see any other way of doing it except for developing on an internal repository outside GitHub, which would just complicate things unnecessarily. |
Intermediary PR containing a partial working implementation of the MCP core with the following features:
Core Infrastructure
Client Management
Protocol Support
Tool Management
Message Processing
Testing
Currently Missing/TODO