chore(readme): clarify default enabled toolsets in documentation #469
Conversation
Signed-off-by: Marc Nuri <marc@marcnuri.com>
README.md
Outdated
| | Toolset | Description | | ||
| |---------|----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | ||
| | config | View and manage the current local Kubernetes configuration (kubeconfig) | | ||
| | core | Most common tools for Kubernetes management (Pods, Generic Resources, Events, etc.) | | ||
| | helm | Tools for managing Helm charts and releases | | ||
| | kiali | Most common tools for managing Kiali, check the [Kiali integration documentation](https://github.com/containers/kubernetes-mcp-server/blob/main/docs/KIALI_INTEGRATION.md) for more details. | |
There was a problem hiding this comment.
@manusa can we add a column to this table about enabled/disabled by default?
There was a problem hiding this comment.
At the moment this is summarized in line 207 (managed manually).
By next version, I think we'll only enable core and config by default.
Do you still think it makes sense to add the column? (i.e. at some point there might be 10 rows and only two of them will have the check)
There was a problem hiding this comment.
Yeah I get your point
My thinking was that it can be clearer to understand what is enabled while looking at the table (that has the descriptions), than going between the comment above and the table.
Maybe we can add to the descriptions for the enabled by default toolsets "Enabled by default" ? That way there's no extra column but it's also easy to tell what is enabled just from the table
There was a problem hiding this comment.
Let me try different alternatives and see what looks better.
There was a problem hiding this comment.
I let Claude work freely on this one. It seems it did a good job, please recheck
Signed-off-by: Marc Nuri <marc@marcnuri.com>
2 participants
Follows-up on #425