Adding kubernetes tools#23
Conversation
sajeerzeji
left a comment
There was a problem hiding this comment.
AI Review: Input Validation Gaps (Medium)
Several tools accept user input directly into shell commands without sanitization:
k8sDeleteResourceTool: args.resource, args.name
k8sDescribeTool: args.resource, args.name
k8sSwitchContextTool: args.context
Risk: Potential command injection if untrusted input is passed.
Fix: Add input validation regex or use array-based command construction more carefully.
|
AI Review: Suggested Improvements (Non-blocking), Maybe as another PR
|
Updated Kubernetes tools
Added structured label selector support for:
Added dryRun support for:
Added stronger argument validation on resource/name/context/namespace values. |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Explain how to reproduce the testing environment.
npm run testnpm run test:tools:fs)Checklist:
npm run lint)