Bug
The All field in DeleteInput (pkg/mcp/tools_delete.go) is typed as *string:
All *string `json:"all,omitempty" jsonschema:"Delete all related resources like Pipelines, Secrets (true/false)"`
The --all flag on the underlying func delete CLI is a boolean flag. The current implementation:
- Presents a string field in the JSON schema to the AI agent instead of a boolean
- Requires the AI to send
"all": "true" (string) instead of "all": true (boolean)
- Passes the flag as
--all true (two separate args via appendStringFlag) instead of --all
If the underlying cobra command's --all flag is a BoolVar, then --all true may be interpreted as --all followed by a positional argument "true", causing unexpected behavior.
Impact
The --all flag may not work correctly. The JSON schema also misleads AI agents about the expected type.
Fix
Change All from *string to *bool and use appendBoolFlag instead of appendStringFlag.
Bug
The
Allfield inDeleteInput(pkg/mcp/tools_delete.go) is typed as*string:The
--allflag on the underlyingfunc deleteCLI is a boolean flag. The current implementation:"all": "true"(string) instead of"all": true(boolean)--all true(two separate args viaappendStringFlag) instead of--allIf the underlying cobra command's
--allflag is aBoolVar, then--all truemay be interpreted as--allfollowed by a positional argument"true", causing unexpected behavior.Impact
The
--allflag may not work correctly. The JSON schema also misleads AI agents about the expected type.Fix
Change
Allfrom*stringto*booland useappendBoolFlaginstead ofappendStringFlag.