Skip to content

feat(dashboards): Support dashboard editing via Seer chat session#112489

Merged
edwardgou-sentry merged 5 commits intomasterfrom
edwardgou-sentry/feat/dashboard-edit-seer-session
Apr 8, 2026
Merged

feat(dashboards): Support dashboard editing via Seer chat session#112489
edwardgou-sentry merged 5 commits intomasterfrom
edwardgou-sentry/feat/dashboard-edit-seer-session

Conversation

@edwardgou-sentry
Copy link
Copy Markdown
Contributor

@edwardgou-sentry edwardgou-sentry commented Apr 8, 2026

Adds chat panel to dashboards edit mode:

  • Update useSeerDashboardSession to accept an optional dashboard prop. When provided without a seerRunId, the first sendFollowUpMessage call starts a new Seer session via the dashboard generate endpoint, passing the current dashboard as editing context.
  • Add DashboardEditSeerChat component that wires the Seer chat panel into the dashboard detail edit view behind the dashboards-ai-generate-edit feature flag.
  • Extract startDashboardEditSession helper for the generate endpoint call.

Update useSeerDashboardSession to accept an optional dashboard prop. When
provided without a seerRunId, the first sendFollowUpMessage call starts a
new Seer session via the dashboard generate endpoint with the current
dashboard as editing context. Add DashboardEditSeerChat component and
wire it into the dashboard detail edit view.
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 8, 2026
@edwardgou-sentry
Copy link
Copy Markdown
Contributor Author

@cursor review

Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

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 3 potential issues.

Autofix Details

Bugbot Autofix prepared fixes for all 3 issues found in the latest run.

  • ✅ Fixed: Test asserts wrong prompt string, will always fail
    • Changed test assertion from 'Add a latency widget' to 'Add me another widget' to match the actual message sent in the test.
  • ✅ Fixed: Child component checks wrong feature flags for edit
    • Updated DashboardEditSeerChat to check 'dashboards-ai-generate-edit' flag instead of 'dashboards-edit' AND 'dashboards-ai-generate' to match parent component's feature gate.
  • ✅ Fixed: Widget error tracking is dead code, map never populated
    • Removed unused widgetErrorsMap ref and related code, passing empty array for widgetErrors since the component lacks WidgetErrorProvider context to populate it.

Create PR

Or push these changes by commenting:

@cursor push cd4b740d49

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

@edwardgou-sentry
Copy link
Copy Markdown
Contributor Author

@cursor review

Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 5b414ea. Configure here.

@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review April 8, 2026 18:43
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner April 8, 2026 18:43
@edwardgou-sentry edwardgou-sentry merged commit ca0fcd2 into master Apr 8, 2026
65 checks passed
@edwardgou-sentry edwardgou-sentry deleted the edwardgou-sentry/feat/dashboard-edit-seer-session branch April 8, 2026 20:04
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