chore(7107): Include a Sentry Explore Link in the Announcement#41481
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
✨ Files requiring CODEOWNER review ✨👨🔧 @MetaMask/extension-platform (7 files, +704 -103)
👨🔧 @HowardBraham (7 files, +704 -103)
👨🔧 @itsyoboieltr (7 files, +704 -103)
|
Builds ready [c5713a1]
⚡ Performance Benchmarks (Total: 🟢 2 pass · 🟡 5 warn · 🔴 0 fail)
Bundle size diffs
|
a19e172 to
79ae6e7
Compare
79ae6e7 to
393b767
Compare
393b767 to
afec841
Compare
afec841 to
0cfde3a
Compare
Builds ready [0cfde3a]
⚡ Performance Benchmarks (Total: 🟢 7 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
0cfde3a to
13ea7c9
Compare
Builds ready [13ea7c9]
⚡ Performance Benchmarks (Total: 🟢 7 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
13ea7c9 to
e8cf8cf
Compare
Builds ready [eaf4008]
⚡ Performance Benchmarks (Total: 🟢 7 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [d3bcbde]
⚡ Performance Benchmarks (Total: 🟢 7 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 52c4210. Configure here.
52c4210 to
24fbed6
Compare
Builds ready [c1b5181] [reused from 3e28494]
⚡ Performance Benchmarks (Total: 🟢 2 pass · 🟡 7 warn · 🔴 0 fail)
Bundle size diffs
|
Builds ready [47db788] [reused from 3e28494]
⚡ Performance Benchmarks (Total: 🟢 7 pass · 🟡 12 warn · 🔴 0 fail)
Bundle size diffs
|
Builds ready [fb05692] [reused from 5d9df13]
⚡ Performance Benchmarks (Total: 🟢 6 pass · 🟡 9 warn · 🔴 0 fail)
Bundle size diffs
|
|




Description
Add a direct link to Sentry Logs Explorer in the PR comment, pre-filtered to:
Changelog
CHANGELOG entry: null
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/7107
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Low risk: changes are limited to CI workflow env wiring and PR-comment HTML/link generation; main impact is on observability links and benchmark reporting formatting.
Overview
Adds a Sentry Logs Explorer link to the performance benchmark announcement output, generating a pre-filtered URL from
SENTRY_DSN_PERFORMANCE(main +release/*, optional message filters) and showing it per benchmark row and in regression details.Updates benchmark report formatting by standardizing artifact links as
[CI log], improving multi-timer breakdown rendering, and propagatingbenchmarkTypefor correct Sentry message matching.Wires
SENTRY_DSN_PERFORMANCEthrough the prerelease publish and benchmark workflows (usingenvinif:guards), and adds unit tests for DSN parsing/URL generation plus adjusted expectations for updated link/benchmark naming.Reviewed by Cursor Bugbot for commit fb05692. Bugbot is set up for automated code reviews on this repo. Configure here.