feat: add dapr mcpservers cli cmd#1628
Conversation
Signed-off-by: Samantha Coyle <[email protected]>
There was a problem hiding this comment.
Pull request overview
Adds a new Kubernetes-mode CLI command (dapr mcpservers -k) to list Dapr MCPServer CRD resources, aligning output/flags with existing components and configurations listing patterns.
Changes:
- Introduces Kubernetes client helpers to list/print MCPServer resources with table and JSON/YAML output.
- Adds a new
dapr mcpserversCobra command with--name,--namespace,--all-namespaces, and-oflags. - Adds unit tests for listing/filtering behavior and endpoint-derived fields (transport/URL).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
pkg/kubernetes/mcpservers.go |
Implements listing, filtering, table output, and JSON/YAML detail output for MCPServers. |
pkg/kubernetes/mcpservers_test.go |
Adds tests for write path, transport detection, and URL derivation. |
cmd/mcpservers.go |
Adds the new dapr mcpservers CLI command and flags wiring. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Samantha Coyle <[email protected]>
Signed-off-by: Samantha Coyle <[email protected]>
|
@holopin-bot @sicoyle cli-badge Thank you! Here's a digital badge as a small token of appreciation. |
|
Congratulations @sicoyle, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmqbf0lef0003kz04av0c79x2 This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Description
Summary
dapr mcpservers -kcommand to list MCPServer CRD resources in Kubernetes--name,--namespace,--all-namespaces, and-o json/yamlflagsdapr components -kanddapr configurations -kThis will fail until my upstream PR here is merged with the clientset changes required for this:
dapr/dapr#9742
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: