Skip to content

feat(explore): Apply Seer-suggested interval to Explore charts#117196

Merged
DominikB2014 merged 4 commits into
masterfrom
dominikbuszowiecki/browse-546-update-ui-to-use-interval
Jun 10, 2026
Merged

feat(explore): Apply Seer-suggested interval to Explore charts#117196
DominikB2014 merged 4 commits into
masterfrom
dominikbuszowiecki/browse-546-update-ui-to-use-interval

Conversation

@DominikB2014

@DominikB2014 DominikB2014 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The assisted query agent now optionally returns an interval per visualization. The Explore search bar preview surfaces that interval, and applying a Seer suggestion sets it on the chart via the interval URL param. This covers the spans, logs, and metrics tabs.

Interval is hoisted to a single chart-level value

Seer returns the interval nested per-visualization, but the Explore chart uses one interval shared across all y-axes. We hoist the first interval Seer provides to a chart-level field (getRawSeerInterval / mapSeerResponseItem), surface it in the search bar preview (QueryTokens), and pass it into the chart. useChartInterval already validates the URL value against the allowed options for the active time range, so an interval that isn't valid for the selected range is silently ignored rather than breaking the chart.

Only overrides when Seer suggests one

When Seer doesn't return an interval, the user's current interval is left untouched.

Refs BROWSE-546

@linear-code

linear-code Bot commented Jun 9, 2026

Copy link
Copy Markdown

BROWSE-546

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 9, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review June 10, 2026 13:13
@DominikB2014 DominikB2014 requested review from a team as code owners June 10, 2026 13:13
@DominikB2014 DominikB2014 removed request for a team June 10, 2026 13:13
@DominikB2014 DominikB2014 marked this pull request as draft June 10, 2026 14:02
@DominikB2014 DominikB2014 changed the title feat(explore): Apply Seer-suggested interval to the chart feat(explore): Apply Seer-suggested interval to the spans and logs charts Jun 10, 2026
@DominikB2014 DominikB2014 changed the title feat(explore): Apply Seer-suggested interval to the spans and logs charts feat(explore): Apply Seer-suggested interval to Explore charts Jun 10, 2026
The assisted query agent now optionally returns an interval per
visualization. Thread it through so the Explore search bar preview shows
it and the chart adopts it via the interval URL param.

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 45b54bb9611d03520bde62bc1ca7f62e4e16567f. Configure here.

Hoist the per-visualization interval Seer returns to a single chart-level
field and surface it in the search bar preview, then apply it to the logs
chart via the interval URL param (only overriding when Seer suggests one).
Apply the chart-level interval Seer returns to the metrics tab via the
interval URL param, only overriding when Seer suggests one.
@DominikB2014 DominikB2014 force-pushed the dominikbuszowiecki/browse-546-update-ui-to-use-interval branch from 45b54bb to 46ec180 Compare June 10, 2026 14:20
getRawSeerInterval picked the first visualization with any interval, but
mapSeerResponseItem only keeps visualizations with y-axes. Restrict the
interval to plotted visualizations so we don't surface an interval from a
dropped, axis-less entry.
@DominikB2014 DominikB2014 marked this pull request as ready for review June 10, 2026 14:24
@DominikB2014 DominikB2014 enabled auto-merge (squash) June 10, 2026 14:30
@DominikB2014 DominikB2014 merged commit 7307904 into master Jun 10, 2026
76 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/browse-546-update-ui-to-use-interval branch June 10, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants