Skip to content

Remove KubeArchive CLI fallback instructions from tool docstrings#41

Merged
geored merged 1 commit into
mainfrom
remove-kubearchive-cli-instructions
May 21, 2026
Merged

Remove KubeArchive CLI fallback instructions from tool docstrings#41
geored merged 1 commit into
mainfrom
remove-kubearchive-cli-instructions

Conversation

@geored

@geored geored commented May 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Removed inline KubeArchive CLI instructions (kubectl ka logs ...) from get_pipelinerun_logs, analyze_failed_pipeline, and automated_triage_rca_report_generator docstrings
  • Updated references to point to the dedicated query_kubearchive MCP tool instead
  • These instructions were redundant after the query_kubearchive tool was added in Add query_kubearchive tool #40

Test plan

  • Verify MCP server starts without errors
  • Confirm query_kubearchive tool is available and functional
  • Check that get_pipelinerun_logs, analyze_failed_pipeline, and automated_triage_rca_report_generator tool descriptions no longer show CLI instructions

🤖 Generated with Claude Code

Now that the dedicated query_kubearchive tool exists, the inline
instructions pointing users to kubectl ka CLI commands are redundant.
Updated get_pipelinerun_logs, analyze_failed_pipeline, and
automated_triage_rca_report_generator to reference the tool instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

@olegbet olegbet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mftb mftb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@geored geored merged commit 277fdc7 into main May 21, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants