Skip to content

feat(mcp): [TKC-6706] add formatters for boards and rendered reports - #8413

Open
vsukhin wants to merge 3 commits into
feat/mcp-boards/2-corefrom
feat/mcp-boards/3-formatters
Open

vsukhin wants to merge 3 commits into
feat/mcp-boards/2-corefrom
feat/mcp-boards/3-formatters

Conversation

@vsukhin

@vsukhin vsukhin commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

Compact views of a board list, a board with its reports in dashboard order, and each report kind's rendered data. The insight series summary is factored out so time-series reports reuse it.

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:56
@testkubebot

testkubebot Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

✅ Testkube GitHub Integration

Review based on commit e696664.

All tests and quality gates passed.


Phase Status
Test Workflow Execution ✅ Passed
Quality Gate ✅ Passed

7 workflows executed

✅ lint-go passed in 9m9s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:29:02 UTC

✅ lint-proto passed in 39s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:20:33 UTC

✅ integration-tests passed in 10m5s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:29:59 UTC

✅ unit-tests passed in 10m12s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:30:06 UTC

✅ verify-crds passed in 4m3s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:23:57 UTC

✅ verify-protobuf passed in 35s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:20:29 UTC

✅ lint-pr passed in 28s
🚀 29. Sep. 2026 - 13:19:53 UTC / 🏁 29. Sep. 2026 - 13:20:22 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 formatters for board and report data display.

The PR should not merge until an edit cannot leave a time-series report unrenderable.

Summary

This PR adds compact board and rendered-report formatters. The changes since the previous review separate validation of existing reports from creation defaults and tighten label-filter validation.

  • A time-series report edit can now store an empty measure, leaving that report unrenderable.

Reviews (2) · Last reviewed commit: "feat(mcp): add formatters for boards and..."

Comment thread pkg/mcp/formatters/boards.go
Compact views of a board list, a board with its reports in dashboard
order, and each report kind's rendered data. The insight series summary
is factored out so time-series reports reuse it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vsukhin
vsukhin force-pushed the feat/mcp-boards/3-formatters branch from d4f0ce4 to 8481b87 Compare September 28, 2026 14:14
@vsukhin vsukhin changed the title feat(mcp): add formatters for boards and rendered reports feat(mcp): [TKC-6706] add formatters for boards and rendered reports Sep 28, 2026
@vsukhin
vsukhin marked this pull request as ready for review September 28, 2026 14:54
@vsukhin
vsukhin requested a review from a team as a code owner September 28, 2026 14:54
@vsukhin
vsukhin requested a review from buarki September 28, 2026 14:54

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