feat: add groupByColumnsOnLeft to MCP dashboard table tile schema#2198
feat: add groupByColumnsOnLeft to MCP dashboard table tile schema#2198alex-fedotyev wants to merge 2 commits intomainfrom
Conversation
Surfaces the table-tile display option that PR #2149 added to the external API to MCP clients. The conversion path already round-trips the field; this advertises it on the MCP table tile schema and extends the existing multi-tile MCP test to assert the value survives save -> fetch. Follow-up to #2149.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 3de9c9a The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
🔵 Tier 2 — Low RiskSmall, isolated change with no API route or data model modifications. Why this tier:
Review process: AI review + quick human skim (target: 5–15 min). Reviewer validates AI assessment and checks for domain-specific concerns. Stats
|
PR Review✅ No critical issues found. The change is minimal and correct —
|
E2E Test Results✅ All tests passed • 157 passed • 3 skipped • 1199s
Tests ran across 4 shards in parallel. |
|
Good catch. Added |
Summary
Follow-up to #2149. Drew added
groupByColumnsOnLeftto theexternal-API table chart config and to the conversion path in #2149,
but the MCP
hyperdx_save_dashboardtool did not advertise the fieldon its table-tile schema. As a result, MCP clients had no way to
discover or set the option, even though the underlying save / fetch
plumbing already supports it.
This change adds the field to
mcpTableTileSchema.configand extendsthe existing multi-tile MCP test to assert a table tile with
groupByColumnsOnLeft: trueround-trips throughhyperdx_save_dashboard.Test plan
yarn tsc --noEmitclean inpackages/apiyarn lintclean inpackages/apishould create a dashboard with multiple tile typesrow now also assertsgroupByColumnsOnLefton the table tile round-tripsgroupByColumnsOnLeftround-trip still pass (no change to that surface; verified locally)