Skip to content

feat(mcp): [TKC-6706] add the Insights board tools - #8414

Open
vsukhin wants to merge 5 commits into
feat/mcp-boards/3-formattersfrom
feat/mcp-boards/4-tools
Open

vsukhin wants to merge 5 commits into
feat/mcp-boards/3-formattersfrom
feat/mcp-boards/4-tools

Conversation

@vsukhin

@vsukhin vsukhin commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

list_boards, get_board, create_board, update_board, add_board_report, update_board_report, remove_board_report, delete_board and render_board. Board updates are conditional on the version they read and are rebuilt from a fresh read, by ID, after a conflict; render_board queries only the reports the layout shows, each with its own debug state. Boards need a user session: the tools report API tokens with a sign-in hint.

Pull request description

Checklist (choose whats happened)

  • breaking change! (describe)
  • tested locally
  • tested on cluster
  • added new dependencies
  • updated the docs
  • added a test

Breaking changes

Changes

Fixes

@vsukhin
vsukhin added this pull request to stack #8409 September 25, 2026 18:57
@testkubebot

testkubebot Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

✅ Testkube GitHub Integration

Review based on commit 7e02b21.

All tests and quality gates passed.


Phase Status
Test Workflow Execution ✅ Passed
Quality Gate ✅ Passed

7 workflows executed

✅ lint-go passed in 7m47s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:27:37 UTC

✅ lint-proto passed in 22s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:20:12 UTC

✅ integration-tests passed in 11m3s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:30:53 UTC

✅ unit-tests passed in 10m8s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:29:58 UTC

✅ verify-crds passed in 3m17s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:23:08 UTC

✅ verify-protobuf passed in 24s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:20:15 UTC

✅ lint-pr passed in 23s
🚀 29. Sep. 2026 - 13:19:50 UTC / 🏁 29. Sep. 2026 - 13:20:13 UTC


Manage this Integration

@vsukhin

vsukhin commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

@greptile-apps

greptile-apps Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Medium risk] Adds new board management tools to the insights system.

The board behavior appears sound, but the explicit repository documentation requirement should be satisfied before merging.

Findings

  1. P2 Board tool guidance is missing ▶

Summary

Adds MCP tools to manage and render Insights boards, with conditional-update retries and per-report debug isolation. The remaining issue is that the repository-required guidance for the new key file has not been updated.

Reviews (2) · Last reviewed commit: "fix(mcp): validate, not normalize, a rep..."

Comment thread pkg/mcp/tools/boards.go
vsukhin and others added 2 commits September 28, 2026 14:04
list_boards, get_board, create_board, update_board, add_board_report,
update_board_report, remove_board_report, delete_board and render_board.
Board updates are conditional on the version they read and are rebuilt
from a fresh read, by ID, after a conflict; render_board queries only
the reports the layout shows, each with its own debug state. Boards
need a user session: the tools report API tokens with a sign-in hint.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
update_board_report ran NormalizeReport over the report's stored params,
so an unrelated edit could fill in creation defaults and change the
report's window or segmentation. It now uses ValidateReport, and starts
from the kind's defaults only when the kind changes or with
replaceParams, as its description says.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vsukhin
vsukhin force-pushed the feat/mcp-boards/4-tools branch from 92069ef to 096ca60 Compare September 28, 2026 14:14
@vsukhin vsukhin changed the title feat(mcp): add the Insights board tools feat(mcp): [TKC-6706] add the Insights board tools Sep 28, 2026
@vsukhin
vsukhin marked this pull request as ready for review September 28, 2026 14:56
@vsukhin
vsukhin requested a review from a team as a code owner September 28, 2026 14:56
@vsukhin
vsukhin requested a review from devcatalin September 28, 2026 14:56
Comment thread pkg/mcp/tools/boards.go
vsukhin and others added 3 commits September 28, 2026 15:05
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Name statusDetailsType and statusDetailsReason among the filters, the
executions groupBy values and the time-series segments, as the dashboard
offers them since kubeshop/testkube-cloud-api#5880.

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

This branch has not been deployed

No deployments
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.

1 participant