Skip to content

ref(assisted-query): Remove old ai_query analytics events and analyticsSource prop#112083

Merged
aliu39 merged 17 commits into
masterfrom
andrewliu/cleanup-ai-query-analytics
Apr 2, 2026
Merged

ref(assisted-query): Remove old ai_query analytics events and analyticsSource prop#112083
aliu39 merged 17 commits into
masterfrom
andrewliu/cleanup-ai-query-analytics

Conversation

@aliu39
Copy link
Copy Markdown
Member

@aliu39 aliu39 commented Apr 2, 2026

Removes all *.ai_query* analytics event definitions, their trackAnalytics call sites, and the analyticsSource prop that existed solely to fire them. These are migrated to ai_query.* with an area prop in https://github.com/getsentry/sentry/pull/112029/changes

aliu39 added 7 commits March 31, 2026 20:32
…prop

Remove all *.ai_query* event definitions, their trackAnalytics call
sites, and the analyticsSource prop that was used solely to fire them.

The events removed span trace explorer, logs, issues, and errors areas.
The analyticsSource prop on AskSeerComboBox and AskSeerPollingComboBox
had no other purpose and is removed from the interface entirely.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 2, 2026
Restore the feedback event and its call site in AskSeerFeedback — this
one needs separate migration work before removal.
@aliu39 aliu39 changed the title ref(analytics): Remove ai_query analytics events and analyticsSource prop ref(assisted-query): Remove old ai_query analytics events and analyticsSource prop Apr 2, 2026
Keep all ai_query.* events (applied, error, interface, rejected, submitted)
that take an area param — these are the canonical generic events that
should remain. Only the area-specific prefixed variants (trace.explorer.*,
logs.*, errors.*, issue.list.*) and the analyticsSource-dynamic ones are
being removed.
Base automatically changed from andrewliu/generic-ai-query-analytics to master April 2, 2026 17:42
@aliu39 aliu39 marked this pull request as ready for review April 2, 2026 20:11
@aliu39 aliu39 requested review from a team as code owners April 2, 2026 20:11
Comment thread static/app/views/issueList/issueListSeerComboBox.tsx
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 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.

Comment thread static/app/components/searchQueryBuilder/askSeer/askSeerOption.tsx
@aliu39 aliu39 marked this pull request as draft April 2, 2026 20:23
@aliu39 aliu39 marked this pull request as ready for review April 2, 2026 20:32
Comment thread static/app/components/searchQueryBuilder/askSeerCombobox/askSeerComboBox.tsx Outdated
@aliu39 aliu39 enabled auto-merge (squash) April 2, 2026 20:44
@aliu39 aliu39 disabled auto-merge April 2, 2026 20:44
@aliu39 aliu39 merged commit 1b357ce into master Apr 2, 2026
82 of 84 checks passed
@aliu39 aliu39 deleted the andrewliu/cleanup-ai-query-analytics branch April 2, 2026 22:05
aliu39 added a commit that referenced this pull request Apr 2, 2026
… analyticsSource prop" (#112184)

Reverts #112083
Keeping this for a few weeks for smooth migration from old dashboard
(owned by Dhrumil) to a new one
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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