Skip to content

feat(insights): Adds web vitals seer suggestion issues to web vital meters#99483

Merged
edwardgou-sentry merged 2 commits into
masterfrom
egou/feat/wv-meters-seer-issues
Sep 16, 2025
Merged

feat(insights): Adds web vitals seer suggestion issues to web vital meters#99483
edwardgou-sentry merged 2 commits into
masterfrom
egou/feat/wv-meters-seer-issues

Conversation

@edwardgou-sentry

@edwardgou-sentry edwardgou-sentry commented Sep 15, 2025

Copy link
Copy Markdown
Contributor

Updates Web Vital meters to also query and link to Seer/Autofix Suggestion issues:
image

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 15, 2025
@codecov

codecov Bot commented Sep 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #99483      +/-   ##
==========================================
- Coverage   81.21%   81.21%   -0.01%     
==========================================
  Files        8585     8585              
  Lines      380186   380177       -9     
  Branches    24106    24103       -3     
==========================================
- Hits       308771   308763       -8     
+ Misses      71069    71068       -1     
  Partials      346      346              

Comment on lines +34 to +42
// For issue.type:web_vitals, we also want to filter for a specific web_vital tag.
// However, the issues api doesn't support OR conditions, so we can't apply a filter to the web_vital tag without it affecting all other filter conditions.
// To get around this, we instead filter out all other web_vital tags except the one we want to filter for.
if (webVital) {
mutableSearch.addFilterValue(
'!web_vital',
`[${ORDER.filter(v => v !== webVital).join(',')}]`
);
}

@edwardgou-sentry edwardgou-sentry Sep 15, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This might be a little confusing. Here's an example. We want to do:

(issue.type:[performance_render_blocking_asset_span,...] ) OR (issue.type:web_vitals web_vital:lcp)

The issue api doesn't support OR syntax though. So an equivalent hack without using OR would be:

issue.type:[web_vitals, performance_render_blocking_asset_span,...] !web_vital:[fcp,inp,cls,ttfb])

This works because no other issue type uses the web_vital tag

@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review September 15, 2025 16:03
@edwardgou-sentry edwardgou-sentry requested a review from a team September 15, 2025 16:03
@edwardgou-sentry edwardgou-sentry merged commit 1622d81 into master Sep 16, 2025
46 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/feat/wv-meters-seer-issues branch September 16, 2025 14:23
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 2, 2025
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