feat: update command arguments and add new client configurations for Trae and Trae CN #35
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
…Trae and Trae CN Signed-off-by: CFC4N <[email protected]>
dosubot
bot
added
the
size:XS
There was a problem hiding this comment.
Pull Request Overview
This PR updates server initialization and client configuration to support enhanced compatibility and functionality.
- Updated the default server arguments in NewMCPServerConfig to always enable all modes.
- Added new configuration paths for Trae and Trae CN for both macOS and Windows client configurations.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| client/client.go | Updated the default Args to ["-m", "all"] in the server config. |
| client/client_config.go | Added new client configuration entries for Trae and Trae CN on macOS. |
| client/client_config_windows.go | Added new client configuration entries for Trae and Trae CN on Windows. |
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.
This pull request introduces updates to the client configuration and server initialization logic to enhance functionality and improve compatibility across different platforms. The most important changes include modifying default server arguments and adding new client configuration paths for both macOS and Windows environments.
Server configuration updates:
Argsin theNewMCPServerConfigfunction to include"-m"and"all", replacing the previously unused empty array. This change ensures that all modes are enabled by default when initializing the server. (client/client.go, client/client.goL54-R54)Client configuration updates:
TraeandTrae CNin theclientListsmap for macOS, pointing tomcp.jsonfiles. This addition provides support for these clients with a simplified configuration file. (client/client_config.go, client/client_config.goR34-R35)TraeandTrae CNin theclientListsmap for Windows, ensuring cross-platform compatibility with the samemcp.jsonfiles. (client/client_config_windows.go, client/client_config_windows.goR32-R33)