feat: Implement MCP List Tools #9
Merged
Conversation
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
Signed-off-by: Eden Reich <[email protected]>
Signed-off-by: Eden Reich <[email protected]>
Signed-off-by: Eden Reich <[email protected]>
Signed-off-by: Eden Reich <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
This PR introduces MCP tools support by adding a new function, list_tools, along with its corresponding response structures and tests. Additionally, the PR refactors the development environment configuration, updates the devcontainer setup, and enables GitHub Copilot with an authentication provider.
- Added new MCP tool definitions and the list_tools API call with corresponding tests.
- Updated provider handling by adding Deepseek and its associated conversion logic.
- Modified devcontainer files: removed launch.json and .zshrc, updated Dockerfile and devcontainer.json for enhanced tooling and configuration.
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/lib.rs | Introduces MCP tool structures, list_tools function, and tests, including Deepseek provider support. |
| README.md | Updates documentation to include MCP tool listing details. |
| .github/copilot-instructions.md | Adds custom instructions for GitHub Copilot integration. |
| .devcontainer/launch.json | Removed launch configuration for debugging. |
| .devcontainer/devcontainer.json | Updates features, GitHub Copilot settings, and MCP server configuration. |
| .devcontainer/Dockerfile | Upgrades TASK_VERSION, installs GitHub CLI, configures shell completions, and clones powerlevel10k. |
| .devcontainer/.zshrc | Removed custom Zsh configuration file. |
Comments suppressed due to low confidence (1)
.devcontainer/.zshrc:1
- [nitpick] The removal of .devcontainer/.zshrc should be reviewed to ensure that all necessary shell customizations are now provided elsewhere (e.g., via Dockerfile modifications). Verify that this change does not affect developers relying on custom Zsh configurations.
# Entire file removed
| USER vscode | ||
|
|
||
| # Use Powerlevel10k theme | ||
| RUN git clone --depth=1 https://github.com/romkatv/powerlevel10k.git /home/vscode/.powerlevel10k && \ |
There was a problem hiding this comment.
[nitpick] Consider pinning the powerlevel10k repository to a specific commit or tag to prevent potential breakage from upstream changes. This would improve the long-term stability of the development environment.
Suggested change
| RUN git clone --depth=1 https://github.com/romkatv/powerlevel10k.git /home/vscode/.powerlevel10k && \ | |
| RUN git clone https://github.com/romkatv/powerlevel10k.git /home/vscode/.powerlevel10k && \ | |
| cd /home/vscode/.powerlevel10k && \ | |
| git checkout 1.18.0 && \ |
Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Eden Reich <[email protected]>
github-actions bot
pushed a commit
that referenced
this pull request
May 26, 2025
## [0.10.0](0.9.2...0.10.0) (2025-05-26) ### ✨ Features * Implement MCP List Tools ([#9](#9)) ([1f07d95](1f07d95))
|
🎉 This PR is included in version 0.10.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
This Pull Request (PR) introduces a new function called
list_toolsto list available MCP tools. It also includes a refactor of the development environment setup and enables GitHub Copilot with an authentication provider.Changes