-
Notifications
You must be signed in to change notification settings - Fork 34
Add MCP tools management commands matching Claude CLI syntax #828
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
Co-authored-by: pelikhan <[email protected]>
pelikhan
left a comment
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.
@copilot add a step in the integration workflow that runs mcp list dev
|
|
||
| // Helper functions | ||
|
|
||
| func getWorkflowPath(workflowID string) (string, error) { |
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.
@copilot there must be already other helpers for resolving a workflow file. Find those helpers and reuse them.
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.
Replaced the custom getWorkflowPath function with resolveWorkflowPath that uses the existing readWorkflowFile helper for consistent workflow path resolution throughout the codebase. Commit: a1c55cc
| } | ||
|
|
||
| // AddMCPToolWithFlags adds an MCP tool using flags from the cobra command | ||
| func AddMCPToolWithFlags(workflowID, toolName string, toolConfigArgs []string, cmd *cobra.Command, verbose bool) error { |
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.
@copilot move to mcp_add.go file
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.
Moved NewMCPAddCommand and AddMCPToolWithFlags functions to the new mcp_add.go file for better code organization. Commit: a1c55cc
| } | ||
|
|
||
| // NewMCPListCommand creates the mcp list command | ||
| func NewMCPListCommand() *cobra.Command { |
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.
@copilot mode to mcp_list.go list
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.
Moved NewMCPListCommand and ListMCPTools functions to the new mcp_list.go file for better code organization. Commit: a1c55cc
| } | ||
|
|
||
| // NewMCPAddCommand creates the mcp add command | ||
| func NewMCPAddCommand() *cobra.Command { |
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.
@copilot move to mcp_add.go file
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.
Moved AddMCPToolWithFlags function to the new mcp_add.go file for better code organization. Commit: a1c55cc
| } | ||
|
|
||
| // ListMCPTools lists MCP tools in a workflow | ||
| func ListMCPTools(workflowID string, showAllowedTools bool, outputFormat string, verbose bool) error { |
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.
@copilot move to mcp_list.go file
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.
Moved ListMCPTools function to the new mcp_list.go file for better code organization. Commit: a1c55cc
Added a "Test MCP list command" step in the integration workflow that runs Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.